Jackie-Jiang commented on PR #12164: URL: https://github.com/apache/pinot/pull/12164#issuecomment-1863655307
IIUC, this PR has introduced the mapping from aggregation used in the query to aggregation used in star-tree index to store the pre-aggregated values. IMO, we can make this mapping completely transparent to the users. When loading the star-tree, we can lookup this map and load star-tree with the stored type; when serving queries, we can lookup this map again and try to match the stored type with the loaded star-tree. We don't need to introduce these 2 types of aggregation to the config. -- 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]
