jackjlli commented on PR #8602: URL: https://github.com/apache/pinot/pull/8602#issuecomment-1116445461
> Will this remain backward compatible? A recent discussion in slack made me think about this one: https://apache-pinot.slack.com/archives/CDRCA57FC/p1651496807383319 @npawar After merging this change, the code will fail at compile stage if the newer release is cut, since there is no more import paths like `shaded.org.apache.pinot.*`. We can mark this PR to be backward incompatible. -- 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
