[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-20 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1359433691 Double checked and works. -- 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

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-17 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1356122504 I'm having vacation till end of the year but had a look to help here :) I've tested that the following works locally: ``` @BeforeClass public static void

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-16 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1355096159 Well, I agree to sort things out well but then it would be good to start from master. The question is why Hadoop3 tests are failing there w/o any changes. This can be challenging

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-16 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1354770239 @MartijnVisser I would say we have similar issue locally w/o this change and if you see the same + considering the fact that the tests are passing on CI then I would merge it.

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-16 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1354767847 I've executed the same commands on the latest master and fails in a similar way: ``` [ERROR] Errors: [ERROR] YARNHighAvailabilityITCase.testClusterClientRetrieval »

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-14 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1351056930 Sad to say not passed on my side: ``` [INFO] Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 141.416 s - in org.apache.flink.yarn.YARNHighAvailabilityITCase

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-14 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1350884310 Started. Get back to you when executed... -- 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

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-08 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1342922281 BTW, nice catch related the directory diff  -- 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

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-08 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1342921503 +1 on Hadoop 3.x update if it's not horror complex. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-08 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1342640340 Something is really not good in this area, I've tried to exec pure `mvn verify` but it failed the following way: ``` [ERROR] Rule 0:

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-12-08 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1342636206 Just reached this task. Sadly I've the same result on my side: ``` [ERROR] Tests run: 27, Failures: 9, Errors: 2, Skipped: 0 ``` In general I'm not sure whether this

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-11-28 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1329228786 Cool :) -- 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

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-11-22 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1323705086 @MartijnVisser could you rebase it to the latest master and I can have a look too? -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-11-18 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1319721388 Wait a moment, the dirs are not the same :) The directory which is created: ``` /__w/1/s/flink-yarn-tests/../target/flink-yarn-tests-capacityscheduler ``` The

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-11-18 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1319714655 > Thanks, but that code is executed. Yeah, now I see it. Interestingly when I remove that directory locally then the tests are passing. While I was hanging around this

[GitHub] [flink] gaborgsomogyi commented on pull request #21128: [FLINK-29710] Bump minimum supported Hadoop version to 2.10.2

2022-11-17 Thread GitBox
gaborgsomogyi commented on PR #21128: URL: https://github.com/apache/flink/pull/21128#issuecomment-1318699210 @MartijnVisser I was digging why the YARN tests are failing and I think I've found one issue :) Namely I have the feeling that the