justinborromeo commented on a change in pull request #7024: Time Ordering Option on Small-Result-Set Scan Queries URL: https://github.com/apache/incubator-druid/pull/7024#discussion_r257413300
########## File path: processing/src/main/java/org/apache/druid/query/scan/ScanQuery.java ########## @@ -40,13 +40,18 @@ public static final String RESULT_FORMAT_COMPACTED_LIST = "compactedList"; public static final String RESULT_FORMAT_VALUE_VECTOR = "valueVector"; + public static final String TIME_ORDER_ASCENDING = "ascending"; + public static final String TIME_ORDER_DESCENDING = "descending"; + public static final String TIME_ORDER_NONE = "none"; Review comment: Does it make sense to make result format an enum too? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
