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