mayankshriv commented on pull request #5316: URL: https://github.com/apache/incubator-pinot/pull/5316#issuecomment-622154639
> Curious in general - should we support modifying some of these theta sketch parameters per union/intersection/diff or possibly a table-wide parameter to tune. > > I might want to change the value of nominal entries per OR/AND in the same query right? I am planning to add a `ThetaSketchParams` argument to the aggregation function (second argument, which can be empty), that will be of form `"param1=val1;param2=val2..."`. This would be a backward compatible way to add more params in future. ---------------------------------------------------------------- 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]
