Re: [PR] Spark 3.4: Fix issue when partitioning by UUID [iceberg]

2023-10-16 Thread via GitHub
nastra commented on code in PR #8250: URL: https://github.com/apache/iceberg/pull/8250#discussion_r1360824697 ## spark/v3.4/spark/src/main/java/org/apache/iceberg/spark/source/InternalRowWrapper.java: ## @@ -71,8 +77,12 @@ public void set(int pos, T value) { row.update(pos

Re: [PR] Spark 3.4: Fix issue when partitioning by UUID [iceberg]

2023-10-14 Thread via GitHub
singhpk234 commented on code in PR #8250: URL: https://github.com/apache/iceberg/pull/8250#discussion_r1359495723 ## spark/v3.3/spark/src/main/java/org/apache/iceberg/spark/source/InternalRowWrapper.java: ## @@ -71,8 +77,12 @@ public void set(int pos, T value) { row.update