NickBurkard commented on pull request #14544: URL: https://github.com/apache/flink/pull/14544#issuecomment-929111051
Thanks for the feedback @MartijnVisser & @zentol. I addressed each point, although I was hesitant to start removing methods from `JavaScalaConversionUtil` as its comment suggested, since its methods were used by many different classes. I instead opted to simplify the methods since Flink always uses at least Scala 2.12. If there's any other changes that need to be made let me know. -- 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]
