rkhachatryan commented on code in PR #24869:
URL: https://github.com/apache/flink/pull/24869#discussion_r1619465130


##########
flink-core/src/main/java/org/apache/flink/configuration/AlgorithmOptions.java:
##########
@@ -60,4 +61,56 @@ public class AlgorithmOptions {
                             "Whether to use the LargeRecordHandler when 
spilling. If a record will not fit into the sorting"
                                     + " buffer. The record will be spilled on 
disk and the sorting will continue with only the key."
                                     + " The record itself will be read 
afterwards when merging.");
+
+    public static final ConfigOption<MemorySize> GLOBAL_AGG_BUFFER_SIZE =

Review Comment:
   @twalthr I remember we had some discussions around the right place for these 
options.
   Do you have any alternatives in mind?
   How about org.apache.flink.configuration.ExecutionOptions?
   



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to