gravitino icon indicating copy to clipboard operation
gravitino copied to clipboard
verified publisher icon apache

[ #9047] feat(iceberg):Support submit table scan plan

Open sunxiaojian opened this issue 1 month ago • 2 comments

What changes were proposed in this pull request?

Support submit table scan plan.

Why are the changes needed?

Fix: #(9047)

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

sunxiaojian avatar Nov 09 '25 12:11 sunxiaojian

@FANNG1 PTAL

sunxiaojian avatar Nov 12 '25 12:11 sunxiaojian

@FANNG1 PTAL

could you rebase the PR?

FANNG1 avatar Nov 13 '25 14:11 FANNG1

@FANNG1 PTAL

could you rebase the PR?

@FANNG1 Sorry, I just saw it, I've already rebased.

sunxiaojian avatar Nov 17 '25 14:11 sunxiaojian

@FANNG1 PTAL

could you rebase the PR?

@FANNG1 Sorry, I just saw it, I've already rebased.

Fixed UT.

sunxiaojian avatar Nov 20 '25 03:11 sunxiaojian

@FANNG1 PTAL, thanks

sunxiaojian avatar Nov 24 '25 03:11 sunxiaojian

@FANNG1 fixed all.

sunxiaojian avatar Nov 26 '25 06:11 sunxiaojian

LGTM except minor comment

FANNG1 avatar Nov 26 '25 07:11 FANNG1

LGTM except minor comment

@FANNG1 It is now compatible with IncrementalAppendScan. Please review again.

sunxiaojian avatar Nov 26 '25 10:11 sunxiaojian

@sunxiaojian merged to main, thanks for contribution!

FANNG1 avatar Nov 27 '25 06:11 FANNG1