{"push_id":2852479194,"size":2,"distinct_size":2,"ref":"refs/heads/groupby-ordering","head":"2230fd471291fa185c6658e7c47a57c659593783","before":"f040590554a97f8a2214a5374b8626d7ce9d61a2","commits":[{"sha":"3828f38457581b738b6e8d1b38a894100f2a0de6","author":{"name":"Radu Berinde","email":"8cd32adbfdfb986a6f0d684f7a16030a20d97029@cockroachlabs.com"},"message":"sql: fix bugs in streaming groupby distsql planning\n\nFixing two distsql planning bugs related to streaming group-by:\n - `orderedGroupCols` contains groupNode input columns, not indexes\n   into `groupCols`.\n - props.ordering columns refer to output columns, not input columns.\n\nWith the heuristic planner, the grouping cols are always `@1-@x` so\nthese bugs only showed up with the optimizer.\n\nRelease note: None\n\nfix2","distinct":true,"url":"https://api.github.com/repos/RaduBerinde/cockroach/commits/3828f38457581b738b6e8d1b38a894100f2a0de6"},{"sha":"2230fd471291fa185c6658e7c47a57c659593783","author":{"name":"Radu Berinde","email":"8cd32adbfdfb986a6f0d684f7a16030a20d97029@cockroachlabs.com"},"message":"opt: make use of streaming group-by\n\nThe execution path supports telling aggregation operations that there\nis an ordering on a subset of the grouping columns, which causes the\naggregation to be performed in a \"streaming\" fashion: we accumulate\ndata from rows until the set of values on those ordered columns\nchange, at which point we emit the results and start fresh. This has\ntwo benefits: it uses less memory; and it preserves the ordering on\nthose grouping columns.\n\nThis change implements passing ordering requirements on grouping\ncolumns through `GroupBy`.\n\nFixes #29161.\n\nRelease note: None","distinct":true,"url":"https://api.github.com/repos/RaduBerinde/cockroach/commits/2230fd471291fa185c6658e7c47a57c659593783"}]}
{"actor":{"display_login":"RaduBerinde"}}