Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-24 Thread via GitHub
dramaticlly commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1928991587 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-24 Thread via GitHub
dramaticlly commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1928992930 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-24 Thread via GitHub
dramaticlly commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1928991587 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-23 Thread via GitHub
dramaticlly commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1928176085 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-23 Thread via GitHub
amogh-jahagirdar commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1927795585 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-23 Thread via GitHub
szehon-ho commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1927465677 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteTablePathProcedure.java: ## @@ -0,0 +1,133 @@ +/* + * Licensed to the Apache Software F

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-23 Thread via GitHub
szehon-ho commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1927411883 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java: ## @@ -219,8 +219,10 @@ private String validateVersion(TableMet

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-23 Thread via GitHub
nastra commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1926969493 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java: ## @@ -219,8 +219,10 @@ private String validateVersion(TableMetada

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-23 Thread via GitHub
nastra commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1926969493 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteTablePathSparkAction.java: ## @@ -219,8 +219,10 @@ private String validateVersion(TableMetada

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-22 Thread via GitHub
szehon-ho commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1925702789 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteTablePathProcedure.java: ## @@ -0,0 +1,133 @@ +/* + * Licensed to the Apache Software F

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-20 Thread via GitHub
dramaticlly commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1923052475 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-20 Thread via GitHub
nastra commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1922479965 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the Apach

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-20 Thread via GitHub
nastra commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1922477983 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the Apach

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-20 Thread via GitHub
nastra commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1922473129 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to the Apach

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-13 Thread via GitHub
dramaticlly commented on PR #11931: URL: https://github.com/apache/iceberg/pull/11931#issuecomment-257095 > @dramaticlly would you mind updating doc as well? Yes @manuzhang , I plan to have a followup PR with spark procedure documentation once this is merged. -- This is an aut

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-13 Thread via GitHub
manuzhang commented on PR #11931: URL: https://github.com/apache/iceberg/pull/11931#issuecomment-252806 @dramaticlly would you mind updating doc as well? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-09 Thread via GitHub
karuppayya commented on code in PR #11931: URL: https://github.com/apache/iceberg/pull/11931#discussion_r1909323927 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteTablePathProcedure.java: ## @@ -0,0 +1,141 @@ +/* + * Licensed to the A

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-08 Thread via GitHub
dramaticlly closed pull request #11931: Spark 3.5: Procedure to rewrite table path URL: https://github.com/apache/iceberg/pull/11931 -- 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 comm

Re: [PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-08 Thread via GitHub
dramaticlly commented on PR #11931: URL: https://github.com/apache/iceberg/pull/11931#issuecomment-2579105024 FYI @szehon-ho @flyrain @karuppayya @anuragmantri if you want to take a look -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[PR] Spark 3.5: Procedure to rewrite table path [iceberg]

2025-01-08 Thread via GitHub
dramaticlly opened a new pull request, #11931: URL: https://github.com/apache/iceberg/pull/11931 Add spark procedure for rewrite table path -- 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 sp