abhioncbr commented on PR #13671: URL: https://github.com/apache/pinot/pull/13671#issuecomment-2241791704
> I'd suggest keeping the backward compatible handling code because there is no guarantee all users migrated to the new config. We may modify the tests to only use the new field That makes sense. Are we following any timeline for deleting it (say, in version 1.2.0 or 1.3.0) and communicating this to users? Keeping it forever in deprecated mode doesn't make sense. -- 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
