Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-02-12 Thread via GitHub
nabaskes commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2654798498 @amitgilad3 thank you so much, that was so so fast ! -- 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: support rewrite on specified target branch [iceberg]

2025-02-12 Thread via GitHub
amitgilad3 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2654746253 @nabaskes - https://github.com/zinking/iceberg/pull/5/ this is an attempt to finish the pr -- This is an automated message from the Apache Git Service. To respond to the message

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-02-12 Thread via GitHub
nabaskes commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2654687692 Is the outstanding items here just a matter of doing a rebase? If so, I can take that up. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-01-31 Thread via GitHub
nabaskes commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2627592390 Thank you so much! -- 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 commen

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-01-31 Thread via GitHub
amitgilad3 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2626969579 Ill try to finish the pr this weekend, last time i tried we had merge conflicts -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-01-30 Thread via GitHub
nabaskes commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2625474565 Is this work permanently abandoned? Is there another good way to solve this, or is this pattern just not really supported? -- This is an automated message from the Apache Git Service.

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-01-29 Thread via GitHub
github-actions[bot] closed pull request #8797: Spark: support rewrite on specified target branch URL: https://github.com/apache/iceberg/pull/8797 -- 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

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-01-29 Thread via GitHub
github-actions[bot] commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2623222633 This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If y

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2025-01-15 Thread via GitHub
github-actions[bot] commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2594191922 This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-11-04 Thread via GitHub
zinking commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2456291421 @amitgilad3 I have some issue doing the rebase with my current computer. so you probably just copy the branch and doing it on your own repo. -- This is an automated message from the Ap

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-11-04 Thread via GitHub
jackye1995 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2455701262 I think you might want to rebase the commit against latest main branch -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-31 Thread via GitHub
amitgilad3 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2449684887 Hey @jackye1995 - i fixed all your comments but for some reason one test fails and i am not able to reproduce it locally , any chance to assist with this ? -- This is an automated

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-18 Thread via GitHub
amitgilad3 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2422284309 Tnx for looking into this @jackye1995 , Will fix all the comment -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2417996073 also @amitgilad3 @zinking -- 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

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2417995218 looks like this did not get much attention, @amit-cloudinary I did another pass, let me know if you can make the updates -- This is an automated message from the Apache Git Service.

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1803785511 ## api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java: ## @@ -171,6 +171,16 @@ default RewriteDataFiles zOrder(String... columns) { */ Rewrite

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r180389 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -134,6 +138,17 @@ private RewriteDataFiles checkAndApplyF

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1803775302 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -134,6 +138,17 @@ private RewriteDataFiles checkAndApplyF

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r180389 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -134,6 +138,17 @@ private RewriteDataFiles checkAndApplyF

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1803777014 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/Spark3Util.java: ## @@ -254,6 +258,15 @@ private static void apply(UpdateSchema pendingUpdate, TableChang

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1803775302 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -134,6 +138,17 @@ private RewriteDataFiles checkAndApplyF

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-16 Thread via GitHub
jackye1995 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1803766282 ## api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java: ## @@ -171,6 +171,16 @@ default RewriteDataFiles zOrder(String... columns) { */ Rewrite

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-10-09 Thread via GitHub
github-actions[bot] commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2403653646 This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-09-02 Thread via GitHub
amitgilad3 commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2325561162 Hey @szehon-ho @nastra @nk1506 - i fixed you suggestions , can you please review again. thanks -- This is an automated message from the Apache Git Service. To respond to the mes

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-08-02 Thread via GitHub
amitgilad3 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1702257014 ## api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java: ## @@ -171,6 +171,17 @@ default RewriteDataFiles zOrder(String... columns) { */ Rewrite

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-08-02 Thread via GitHub
amitgilad3 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1702205353 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -134,6 +138,21 @@ private RewriteDataFiles checkAndApplyF

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-27 Thread via GitHub
amit-cloudinary commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2132978069 > > @zinking - if i can assist with this please let me know (would love to contribute if possible) > > feel free to pick it up here. Ill give it a try :) ,thanks --

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-27 Thread via GitHub
zinking commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2132920372 > @zinking - if i can assist with this please let me know (would love to contribute if possible) feel free to pick it up here. -- This is an automated message from the Apache Gi

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-27 Thread via GitHub
amit-cloudinary commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2132899272 @zinking - if i can assist with this please let me know (would love to contribute if possible) -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-22 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1610904400 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,33 @@ public RewriteDataFilesSparkAction filter(

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-22 Thread via GitHub
szehon-ho commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1610874491 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -85,6 +87,66 @@ public void testRewriteData

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-22 Thread via GitHub
szehon-ho commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1610849033 ## api/src/main/java/org/apache/iceberg/actions/RewriteDataFiles.java: ## @@ -171,6 +171,17 @@ default RewriteDataFiles zOrder(String... columns) { */ RewriteD

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-22 Thread via GitHub
szehon-ho commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1610846416 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/Spark3Util.java: ## @@ -254,6 +258,15 @@ private static void apply(UpdateSchema pendingUpdate, TableChange

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-20 Thread via GitHub
zinking commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2121627469 ping @nastra for another review. -- 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 spec

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-05-20 Thread via GitHub
amit-cloudinary commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-2120206876 any updates on this ?? is this something thats going to be merged soon -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-02-27 Thread via GitHub
zinking commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-1968104833 let me update the PR, and @nastra can have 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

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2024-02-27 Thread via GitHub
edgarRd commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-1967195464 Just checking on the state of this PR, @zinking is there any thing pending to move this forward? @ajantha-bhat @nastra any remaining concerns? With branches and [merge-on-read deletes n

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-12 Thread via GitHub
ajantha-bhat commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1356348310 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -109,6 +110,8 @@ public InternalRow[] call(InternalRow

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1355083626 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,31 @@ public RewriteDataFilesSparkAction filter(

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1355078213 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -109,6 +110,8 @@ public InternalRow[] call(InternalRow args)

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
rakesh-das08 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354860039 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -109,6 +110,8 @@ public InternalRow[] call(InternalRow

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
rakesh-das08 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354860039 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## @@ -109,6 +110,8 @@ public InternalRow[] call(InternalRow

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
ajantha-bhat commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354746204 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,31 @@ public RewriteDataFilesSparkAction fi

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-1757376846 #8762 associating the original issue here. -- 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

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354654198 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,53 @@ public void testRewriteDataFi

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354653413 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,29 @@ public RewriteDataFilesSparkAction filter(

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354616589 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,53 @@ public void testRewriteDataFil

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354615972 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,53 @@ public void testRewriteDataFil

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354511567 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,53 @@ public void testRewriteDataFi

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354506575 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,53 @@ public void testRewriteDataFi

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-1757203967 > > can you also please update the respective docs? > > do you mean the branch & tagging doc? anywhere else? I mean in the Spark procedures section: https://github.com/apache/

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354492487 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,29 @@ public RewriteDataFilesSparkAction filter(E

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354489402 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,29 @@ public RewriteDataFilesSparkAction filter(E

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354490434 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,29 @@ public RewriteDataFilesSparkAction filter(E

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354486142 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,53 @@ public void testRewriteDataFil

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354480965 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,53 @@ public void testRewriteDataFil

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354478071 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## Review Comment: it's usually ok to fix something for the lat

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-1757165507 > can you also please update the respective docs? do you mean the branch & tagging doc? anywhere else? -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r135605 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/procedures/RewriteDataFilesProcedure.java: ## Review Comment: I prefer it in following ups -- This i

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354405286 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## Review Comment: is that necessary, it's alr

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nk1506 commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354331345 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## Review Comment: I think we need to add this for other spark

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on PR #8797: URL: https://github.com/apache/iceberg/pull/8797#issuecomment-1757029634 can you also please update the respective docs? -- 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] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354320039 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RewriteDataFilesSparkAction.java: ## @@ -146,13 +148,29 @@ public RewriteDataFilesSparkAction filter(E

Re: [PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
nastra commented on code in PR #8797: URL: https://github.com/apache/iceberg/pull/8797#discussion_r1354318589 ## spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestRewriteDataFilesProcedure.java: ## @@ -87,6 +89,51 @@ public void testRewriteDataFil

[PR] Spark: support rewrite on specified target branch [iceberg]

2023-10-11 Thread via GitHub
zinking opened a new pull request, #8797: URL: https://github.com/apache/iceberg/pull/8797 ## why the change currently, rewrite only happens on main branch. with the branch functionality it is also useful to conduct rewrite related tests on branches ## what is changed