{"push_id":1118740706,"size":1,"distinct_size":1,"ref":"refs/heads/rollup_fix_optimizer","head":"936ec12c744fac4f9cb5bd31c52594b0aba74469","before":"dafa33280ae89b379b69f98bdb34a89d9ad30e20","commits":[{"sha":"936ec12c744fac4f9cb5bd31c52594b0aba74469","author":{"email":"dd14f1821c8a37c41cd853d5ede068f7ccfcd319@fb.com","name":"Raghav Sethi"},"message":"Fix grouping on aggregation arguments\n\n- Modify the GroupIdNode and GroupIdOperator to pass aggregation\narguments through.\n- Change PruneUnreferencedOutputs optimizer to only\npass through columns that are also aggregation arguments.\n\nThe expected behavior is that arguments to the aggregations reference\nvalues in the source relation rather than the outputs of the GroupId\noperation (which can produce nulls for these columns). Prior to this\nchange, aggregation arguments and grouping columns referenced the same\nsymbol, producing incorrect results.","distinct":true,"url":"https://api.github.com/repos/raghavsethi/presto/commits/936ec12c744fac4f9cb5bd31c52594b0aba74469"}]}