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