rohangarg commented on a change in pull request #11241:
URL: https://github.com/apache/druid/pull/11241#discussion_r643770224
##########
File path:
processing/src/main/java/org/apache/druid/query/aggregation/ExpressionLambdaAggregatorFactory.java
##########
@@ -65,7 +65,7 @@
// minimum permitted agg size is 10 bytes so it is at least large enough to
hold primitive numerics (long, double)
// | expression type byte | is_null byte | primitive value (8 bytes) |
private static final int MIN_SIZE_BYTES = 10;
- private static final HumanReadableBytes DEFAULT_MAX_SIZE_BYTES = new
HumanReadableBytes(1L << 10);
+ public static final HumanReadableBytes DEFAULT_MAX_SIZE_BYTES = new
HumanReadableBytes(1L << 10);
Review comment:
looks like an extra change - can't find its usage
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
