mosche commented on PR #24380: URL: https://github.com/apache/beam/pull/24380#issuecomment-1339179680
@aromanenko-dev Generally I would agree. Though, Beam runners are not the typical library used in user code. Currently there's no way users can do anything meaningful with NamedAggregators, it's simply a broken piece of code left over for whatever reasons. Why would any user import any such related class? It's not buggy, it's dead... you cannot do anything with it. Btw, the primary reason not to use open source, is the poor quality of so much open source code out there. The deprecation strategy is a bad trap (unless these deprecations are removed in a disciplined manner) deteriorating quality quickly over time. And Beam is just a perfect example here if you look at all the deprecations in Beam's core. We have to be very careful about our limited resources and energy. IMHO going through the deprecation detour is wasted energy in this case. -- 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]
