cryptoe commented on code in PR #14476: URL: https://github.com/apache/druid/pull/14476#discussion_r1242128159
########## extensions-core/multi-stage-query/src/main/java/org/apache/druid/msq/util/MultiStageQueryContext.java: ########## @@ -87,6 +90,8 @@ public class MultiStageQueryContext public static final String CTX_DURABLE_SHUFFLE_STORAGE = "durableShuffleStorage"; private static final boolean DEFAULT_DURABLE_SHUFFLE_STORAGE = false; + public static final String CTX_LIMIT_SELECT_RESULT = "limitSelectResult"; Review Comment: Lets rename this to `selectDestination="report"` and then not limit the results to maintain backward compatibility. I am working on another PR which can use this limiting where` selectDestination="Report+s3"`. -- 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...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
