zjffdu commented on issue #8144: [FLINK-12159]. Enable YarnMiniCluster integration test under non-secure mode URL: https://github.com/apache/flink/pull/8144#issuecomment-486142784 Agree that we need to deprecate/refactor `YarnTestBase` which is too complicated (1000 LOC just for unit test). > Next, we change the YarnTestBase#createYarnClusterDescriptor to add the yarn-site.xml file as a file to be shipped to the cluster. Sorry, I don't quite understand this. Do you mean we remove `IN_TESTS` and always get yarn-site.xml via Thread.currentThread().getContextClassLoader().getResource('yarn-site.xml') ?
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
