mcvsubbu commented on issue #7156: URL: https://github.com/apache/incubator-pinot/issues/7156#issuecomment-880933728
Sure, please do. Just make sure that we accept _both_ old and new configurations, and prefer the new one if both or specified. Be sure to change the documentation as well to say that these new configs have been introduced and the old ones deprecated. [Sigh!] we have no way of deprecating configurations to make them easily visible for admin (other than release notes), so please tag your PR with release noes, and mention what needs to go in there. Thanks for your contribution. -- 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
