zentol commented on pull request #17036: URL: https://github.com/apache/flink/pull/17036#issuecomment-907761770
I'm already looking into this topic as part of FLINK-23986. I will close the PR because I have already solved all these problems. Overall you were on a good track. I'd argue that using reflection against Scala classes isn't really clean. It makes the code hard to write, maintain and extend. Instead there should be a form of FieldAccessFactory extension in the Scala API modules that is instead loaded via reflection. Furthermore, we can also remove the Scala dependency from flink-cep. Finally, the removal of suffixes cascades into _way more_ modules than shown here. -- 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]
