mcvsubbu commented on issue #10996: URL: https://github.com/apache/pinot/issues/10996#issuecomment-1613615894
> > Proposal accepted to sunset stream level consumption. if it makes sense for some unpartitioned stream later, they can implement/resurrect things as needed (better start fresh implementation, since a lot has changed) > > Thanks for all the context, @mcvsubbu ! And I agree that a lot has changed and if necessary, we can redesign for any new unpartitioned stream. > > Would the path forward be to mark the existing stream level APIs as deprecated for the upcoming release (`0.14.0`) and actually, remove the code in `0.15.0` ? Happy to open a g doc to facilitate more discussion on this topic. The path you proposed, will certainly work. I may go even more aggressive. Ask in pinot-dev email as well as chat channels (since not many much activity in email lists) whether anyone is using HLC. My guess is that it will be a silence. In which case, we can actually remove code in he next release. Another idea: - Disable all HLC integration tests right away. - Add some minor code so that use of HLC throws exception in controller and server (and even broker/minion if possible) during upgrade (important that this exception be thrown during the upgrade so that admins have a chance). If too many voices come up, then retract the exception code, and follow your steps. - If nobody complains after a month after the release, make n announcement and start removing code -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
