gpdb icon indicating copy to clipboard operation
gpdb copied to clipboard

Refactor some codes about guc gp_eager_two_phase_agg

Open charliettxx opened this issue 1 year ago • 6 comments

In previous PR, we introduced GUC gp_eager_two_phase_agg to force planner generating multi-stage agg path in Planner, but applying disable costs may not a good choice, we should force planner generate only multi-stage agg paths instead of single-stage agg paths. So after this patch, single-stage agg path will not exist in our planner path.

Here are some reminders before you submit the pull request

  • [x] Add tests for the change
  • [ ] Document changes
  • [ ] Communicate in the mailing list if needed
  • [ ] Pass make installcheck
  • [ ] Review a PR in return to support the community

charliettxx avatar Dec 19 '23 03:12 charliettxx