[GitHub] TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test

2018-09-23 Thread GitBox
TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test URL: https://github.com/apache/flink/pull/6716#issuecomment-423867059 ... close as it would be covered by FLINK-10392 This is an automated message

[GitHub] TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test

2018-09-20 Thread GitBox
TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test URL: https://github.com/apache/flink/pull/6716#issuecomment-423129508 @tillrohrmann Currently, `YARNHighAvailabilityITCase` is only a test for legacy mode. If we want to maintain `YARNHighAvailabilityITCase`

[GitHub] TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test

2018-09-19 Thread GitBox
TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test URL: https://github.com/apache/flink/pull/6716#issuecomment-422914196 FYI, Travis fails on legacy test `JobManagerFailsTest`. This is an automated

[GitHub] TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test

2018-09-19 Thread GitBox
TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test URL: https://github.com/apache/flink/pull/6716#issuecomment-422799861 Ok, it is just `TestingYarnClusterDescriptor.TestJarFinder` which is a util class, we can move it out of `TestingYarnClusterDescriptor`

[GitHub] TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test

2018-09-19 Thread GitBox
TisonKun commented on issue #6716: [hotfix] [yarn-test] Clean up inactive test URL: https://github.com/apache/flink/pull/6716#issuecomment-422799453 @tillrohrmann sure but I notice that it relies on `TestingYarnClusterDescriptor` which inherited from `LegacyYarnClusterDescriptor`. I see