jiaqizho commented on code in PR #1192: URL: https://github.com/apache/cloudberry/pull/1192#discussion_r2206133535
########## src/backend/utils/misc/guc_gp.c: ########## @@ -4488,6 +4489,17 @@ struct config_int ConfigureNamesInt_gp[] = NULL, NULL, NULL }, + { + {"optimizer_agg_pds_strategy", PGC_USERSET, DEVELOPER_OPTIONS, + gettext_noop("Set the strategy of agg required distribution."), + NULL, + GUC_NOT_IN_SAMPLE + }, + &optimizer_agg_pds_strategy, + OPTIMIZER_AGG_PDS_ALL_KEY, OPTIMIZER_AGG_PDS_ALL_KEY, OPTIMIZER_AGG_PDS_MINIMAL_LEN_KEY, Review Comment: good idea, i will add this logic in GUC `optimizer_agg_pds_strategy ` for one of strategy -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org For additional commands, e-mail: commits-h...@cloudberry.apache.org