KKcorps opened a new pull request, #10384: URL: https://github.com/apache/pinot/pull/10384
Currently you need to add a new variable in the execution stats aggregator if you add a new key to `Datablock.MetadataKey` . With this change, all the keys coming in response will automatically be aggregated. The PR is the first step of a much larger change to send only `MetadataKey` enum with its integer id in the blocks instead of its String name to reduce the payload size in V2 engine. V2 engine involves a lot of shuffles so this reduction in metadata block size will improve query times but a small margin. I haven't changed the `BrokerResponseNative` class in this due to maintain backward compatibility. The plan is that the class should only contain a Map<MetadataKey, AggregatedStats> instead of each individual key. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
