Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-25 Thread via GitHub
findepi commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2439395678 thank you @tomtongue for your work on this! -- 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

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-16 Thread via GitHub
tomtongue commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2417064056 Thanks so much for the review! @nastra @findepi -- 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

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-16 Thread via GitHub
nastra merged PR #11262: URL: https://github.com/apache/iceberg/pull/11262 -- 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 unsubscribe, e-mail: issues-unsubscr...@iceberg.ap

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-16 Thread via GitHub
nastra commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2415996921 @findepi would you like to take another look? -- 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

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-16 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1802491767 ## flink/v1.18/flink/src/test/java/org/apache/iceberg/flink/sink/TestFlinkIcebergSinkV2Branch.java: ## @@ -46,12 +46,19 @@ public class TestFlinkIcebergSinkV2Branch

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-16 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1802488056 ## flink/v1.19/flink/src/test/java/org/apache/iceberg/flink/sink/TestFlinkIcebergSinkV2Branch.java: ## @@ -46,12 +46,19 @@ public class TestFlinkIcebergSinkV2Branch ex

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-16 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1802487746 ## flink/v1.18/flink/src/test/java/org/apache/iceberg/flink/sink/TestFlinkIcebergSinkV2Branch.java: ## @@ -46,12 +46,19 @@ public class TestFlinkIcebergSinkV2Branch ex

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-16 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1802483572 ## flink/v1.18/flink/src/test/java/org/apache/iceberg/flink/sink/TestFlinkIcebergSinkV2Branch.java: ## @@ -46,12 +46,19 @@ public class TestFlinkIcebergSinkV2Branch ex

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800721063 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/ExtensionsTestBase.java: ## @@ -43,6 +43,8 @@ public static void startMetastoreAndS

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2414157721 @nastra Reflected your review. Could you review new changes when you have a chance? -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1801225247 ## flink/v1.20/flink/src/test/java/org/apache/iceberg/flink/sink/TestFlinkIcebergSinkV2Branch.java: ## @@ -46,12 +46,19 @@ public class TestFlinkIcebergSinkV2Branch

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800718584 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameter

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800744929 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameter

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800732927 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameterVal

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800718584 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameter

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800721063 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/ExtensionsTestBase.java: ## @@ -43,6 +43,8 @@ public static void startMetastoreAndS

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800718584 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameter

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800718584 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameter

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800706912 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/ExtensionsTestBase.java: ## @@ -43,6 +43,8 @@ public static void startMetastoreAndSpar

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800706543 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameter

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-15 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1800693298 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -225,7 +222,11 @@ private Stream createContextForParameters( Stream parameterVal

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-14 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1799596701 ## spark/v3.5/build.gradle: ## @@ -303,6 +310,11 @@ project(":iceberg-spark:iceberg-spark-runtime-${sparkMajorVersion}_${scalaVersio description = "Test Spark

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-14 Thread via GitHub
tomtongue commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2410285393 Thank you for taking a look at this! (Sorry I just sent the commit) Let me reflect your comment. -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-14 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1798898667 ## api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.java: ## @@ -73,13 +73,10 @@ public Stream provideTestTemplateInvocationContex // Search metho

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-14 Thread via GitHub
nastra commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1798896293 ## spark/v3.5/build.gradle: ## @@ -303,6 +310,11 @@ project(":iceberg-spark:iceberg-spark-runtime-${sparkMajorVersion}_${scalaVersio description = "Test Spark3 R

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-14 Thread via GitHub
nastra commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2410267744 > Do you think it's a good idea if I change ParamerizedTest as you suggested, and also add the legacy mode flag to Spark? I took a closer look to see why this happens. It turns out

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-11 Thread via GitHub
tomtongue commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2408149842 @nastra Thanks for looking into this. When I tried the change, the `@Parameters` annotation in an inherited classe is basically used, so this change can solve the error (for the `@Par

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-11 Thread via GitHub
nastra commented on PR #11262: URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2407180732 Rather than changing stuff everywhere, I'd probably consider changing how we look up parameter providers: ``` --- a/api/src/test/java/org/apache/iceberg/ParameterizedTestExtension.j

Re: [PR] Bump junit from 5.10.1 to 5.11.1 [iceberg]

2024-10-10 Thread via GitHub
tomtongue commented on code in PR #11262: URL: https://github.com/apache/iceberg/pull/11262#discussion_r1794954223 ## flink/v1.18/build.gradle: ## @@ -255,6 +260,11 @@ project(":iceberg-flink:iceberg-flink-runtime-${flinkMajorVersion}") { description = "Test Flink Runtime