Airblader commented on PR #19712: URL: https://github.com/apache/flink/pull/19712#issuecomment-1137172279
> one can easily reuse and integrate them for the future feature/page implementation And what plans are there to use that in the future? I don't see the point in introducing such complex architecture so that maybe at some point in the future it could be used. As of right now, it is unnecessarily complex. If there are specific plans of what to do in the future, we should be looking at that and deciding based on that whether this architecture is justified or not. There are several disadvantages to the change introduced in this PR. There's much more complexity, and inputs passed to the components are type-unsafe now. Even if we needed such an architecture, we might need to invest more work to make it a more stable solution. -- 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]
