[GitHub] [pinot] noon-stripe commented on pull request #8462: Generalized Ingestion Pre-Aggregation

[GitBox <gi...@apache.org>]

noon-stripe commented on PR #8462:
URL: https://github.com/apache/pinot/pull/8462#issuecomment-1089073481

   @snleee Great comment! I agree that we should work towards depreciating the existing feature and move all codepaths to use the new version of the feature. 
   
   I'm going to comment on the issue to add the depreciation as "future work," but I like that the previous behavior is exactly the same as before for now. One important difference is that the DataType is preserved in the prior version. There are use cases where this might be important and I'd like to think more about how to best address this. 
   
   Let me know if this all sounds good to you. Thanks!


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org