You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/06/23 22:08:41 UTC

[GitHub] [druid] maytasm commented on a change in pull request #10063: Make return type consistent for HyperUniquesAggregator and HyperUniquesVectorAggregator when no value found

maytasm commented on a change in pull request #10063:
URL: https://github.com/apache/druid/pull/10063#discussion_r444536063



##########
File path: processing/src/main/java/org/apache/druid/query/QueryContexts.java
##########
@@ -59,7 +59,7 @@
   public static final boolean DEFAULT_USE_CACHE = true;
   public static final boolean DEFAULT_POPULATE_RESULTLEVEL_CACHE = true;
   public static final boolean DEFAULT_USE_RESULTLEVEL_CACHE = true;
-  public static final Vectorize DEFAULT_VECTORIZE = Vectorize.FALSE;
+  public static final Vectorize DEFAULT_VECTORIZE = Vectorize.TRUE;

Review comment:
       Just to see if anything else fail. Will make this change in a separate PR with relevant doc change




----------------------------------------------------------------
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org