gpdb
gpdb copied to clipboard
Refactor some codes about guc gp_eager_two_phase_agg
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