davecromberge opened a new pull request, #12778: URL: https://github.com/apache/pinot/pull/12778
Summary --------- The merge operation results in a null reference error when the set operations is not set properly for Tuple Sketches. Description ------------ When more than one server returns an intermediate result to the broker, the custom object accumulators need to perform a merge. When some of the state on the object accumulator is not re-initialised, this can lead to unpredictable behaviour. The serialisation on these custom objects (for sketches) is somewhat cumbersome. Ideally, we would encode the fields in binary together with the underlying sketch payload. I came across a test case where sketches were fetched directly from the servers and I'm not sure as to whether there are users in the community who expect results to conform directly to a sketch for these server responses. Finally, I was not able to find a good way to test multiple server responses with an aggregation function - please let me know if there is a good example. Tags ----- `bugfix` . -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
