[GitHub] [flink] kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir…

kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir… URL: https://github.com/apache/flink/pull/11322#issuecomment-603130060 The more I think about it, the more I lean towards leaving the `setupLocalResource()` as it is and add a method that

[GitHub] [flink] kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir…

kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir… URL: https://github.com/apache/flink/pull/11322#issuecomment-603127358 I am asking to rebase because the AZURE run was fixed and I think it also runs the S3 tests, so you can see if there

[GitHub] [flink] kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir…

kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir… URL: https://github.com/apache/flink/pull/11322#issuecomment-603127011 Hi @jiasheng55, can you rebase on the current master and push here? There seem to be more test failures https://api

[GitHub] [flink] kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir…

kl0u commented on issue #11322: [FLINK-16376][yarn] Use consistent method to get Yarn application dir… URL: https://github.com/apache/flink/pull/11322#issuecomment-599391211 Hi @jiasheng55 . The problem is that to reproduce it you need S3 credentials. I will have another look at the PR but