fhueske commented on issue #7012: [FLINK-10769][Table & SQL] port InMemoryExternalCatalog to java URL: https://github.com/apache/flink/pull/7012#issuecomment-436183200 Hi @bowenli86, sure no worries. I'm also in favor of moving the code base towards Java, but as I said I don't like a fragmented code base where tightly connected classes are in mixed languages. I just wanted to raise awareness that we don't want to implement *all* new (or heavily modified) classes in Java. Some parts of the `flink-table` code base (esp. most of the existing functionality) should remain implemented in Scala for now until we agree on a coordinated effort to move towards Java. For certain parts like the ones in this PR, Java is fine.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
