jon-wei commented on a change in pull request #6950: bloom filter sql aggregator
URL: https://github.com/apache/incubator-druid/pull/6950#discussion_r252911745
 
 

 ##########
 File path: docs/content/configuration/index.md
 ##########
 @@ -1312,6 +1312,7 @@ The Druid SQL server is configured through the following 
properties on the broke
 |`druid.sql.planner.useFallback`|Whether to evaluate operations on the broker 
when they cannot be expressed as Druid queries. This option is not recommended 
for production since it can generate unscalable query plans. If false, SQL 
queries that cannot be translated to Druid queries will fail.|false|
 |`druid.sql.planner.requireTimeCondition`|Whether to require SQL to have 
filter conditions on __time column so that all generated native queries will 
have user specified intervals. If true, all queries wihout filter condition on 
__time column will fail|false|
 |`druid.sql.planner.sqlTimeZone`|Sets the default time zone for the server, 
which will affect how time functions and timestamp literals behave. Should be a 
time zone name like "America/Los_Angeles" or offset like "-08:00".|UTC|
+|`druid.sql.planner.serializeComplexValues`|Whether to serialize "complex" 
output values, false will return the class name instead of the serialized 
value.|true|
 
 Review comment:
   Commenting for posterity, I think it's fine to set this to true although it 
changes the old behavior. 
   
   Previously the only complex columns usable in SQL queries were 
ApproxHistogram and the cardinality/hyperunique aggs, but APPROX_QUANTILE and 
APPROX_COUNT_DISTINCT would apply a postagg on those and return a numeric 
value, so a serialized form of the complex column would never be returned 
previously.

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

Reply via email to