gravitino icon indicating copy to clipboard operation
gravitino copied to clipboard

[#8311] feat(iceberg-rest-catalog):Support scan planning endpoint for Iceberg REST server

Open sunxiaojian opened this issue 1 month ago • 5 comments

What changes were proposed in this pull request?

Support scan planning endpoint for Iceberg REST server

Why are the changes needed?

Fix: #(8311)

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

org.apache.gravitino.iceberg.service.rest.TestPlanTableScan

sunxiaojian avatar Oct 31 '25 03:10 sunxiaojian

Waiting me fix it.

sunxiaojian avatar Oct 31 '25 11:10 sunxiaojian

Waiting me fix it.

Done, Upgraded Iceberg to version 1.10.0 and upgraded the dependent Hadoop to Hadoop 3.

sunxiaojian avatar Nov 02 '25 15:11 sunxiaojian

Thanks @sunxiaojian , it's an amazing feature, the overall architecture looks good to me, would you like to split this PR into three part to make each PR more simple to review?

  1. upgrade Iceberg version
  2. support scan planning interface
  3. scan planning cache support

FANNG1 avatar Nov 07 '25 02:11 FANNG1

Thanks @sunxiaojian , it's an amazing feature, the overall architecture looks good to me, would you like to split this PR into three part to make each PR more simple to review?

  1. upgrade Iceberg version
  2. support scan planning interface
  3. scan planning cache support

ok, I'll handle it.

sunxiaojian avatar Nov 07 '25 08:11 sunxiaojian

Thanks @sunxiaojian , it's an amazing feature, the overall architecture looks good to me, would you like to split this PR into three part to make each PR more simple to review?

  1. upgrade Iceberg version
  2. support scan planning interface
  3. scan planning cache support

ok, I'll handle it.

  1. upgrade Iceberg version #9049
  2. support scan planning interface #9050
  3. Current PR.

Because the PRs are interdependent, they need to be merged in order. After each PR is merged, the remaining PRs need to merge in the main branch.

sunxiaojian avatar Nov 09 '25 14:11 sunxiaojian

@FANNG1 I have already rebased, PTAL.

sunxiaojian avatar Nov 27 '25 10:11 sunxiaojian

@FANNG1 fixed.

sunxiaojian avatar Nov 28 '25 06:11 sunxiaojian