Skip to content

Core: Fix Scan Plan API resource paths #14120

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
amogh-jahagirdar merged 2 commits into from
Sep 20, 2025
Merged

Core: Fix Scan Plan API resource paths #14120

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
30dd83a
Core: Fix Scan Plan API resource paths
sfc-gh-prsingh Sep 19, 2025
8eb7775
Address review feedback
sfc-gh-prsingh Sep 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions .palantir/revapi.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1365,6 +1365,18 @@ acceptedBreaks:
justification: "Removing deprecations for 1.10.0"
"1.10.0":
org.apache.iceberg:iceberg-core:
- code: "java.field.constantValueChanged"
old: "field org.apache.iceberg.rest.ResourcePaths.V1_TABLE_SCAN_PLAN"
new: "field org.apache.iceberg.rest.ResourcePaths.V1_TABLE_SCAN_PLAN"
justification: "Plan API is table scoped and path constant value should include namespace. No actual breakage because it never worked before with incorrect value."
- code: "java.field.constantValueChanged"
old: "field org.apache.iceberg.rest.ResourcePaths.V1_TABLE_SCAN_PLAN_SUBMIT"
new: "field org.apache.iceberg.rest.ResourcePaths.V1_TABLE_SCAN_PLAN_SUBMIT"
justification: "Plan API is table scoped and path constant value should include namespace. No actual breakage because it never worked before with incorrect value."
- code: "java.field.constantValueChanged"
old: "field org.apache.iceberg.rest.ResourcePaths.V1_TABLE_SCAN_PLAN_TASKS"
new: "field org.apache.iceberg.rest.ResourcePaths.V1_TABLE_SCAN_PLAN_TASKS"
justification: "Plan API is table scoped and path constant value should include namespace. No actual breakage because it never worked before with incorrect value."
- code: "java.class.removed"
old: "class org.apache.iceberg.PartitionStatsUtil"
justification: "Removing deprecated code for 1.11.0"
Expand Down
9 changes: 6 additions & 3 deletions core/src/main/java/org/apache/iceberg/rest/ResourcePaths.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,12 @@ public class ResourcePaths {
public static final String V1_TABLE_METRICS =
"/v1/{prefix}/namespaces/{namespace}/tables/{table}/metrics";
public static final String V1_TABLE_RENAME = "/v1/{prefix}/tables/rename";
public static final String V1_TABLE_SCAN_PLAN_SUBMIT = "/v1/{prefix}/tables/{table}/plan";
public static final String V1_TABLE_SCAN_PLAN = "/v1/{prefix}/tables/{table}/plan/{plan-id}";
public static final String V1_TABLE_SCAN_PLAN_TASKS = "/v1/{prefix}/tables/{table}/tasks";
public static final String V1_TABLE_SCAN_PLAN_SUBMIT =
"/v1/{prefix}/namespaces/{namespace}/tables/{table}/plan";
public static final String V1_TABLE_SCAN_PLAN =
"/v1/{prefix}/namespaces/{namespace}/tables/{table}/plan/{plan-id}";
public static final String V1_TABLE_SCAN_PLAN_TASKS =
"/v1/{prefix}/namespaces/{namespace}/tables/{table}/tasks";
public static final String V1_TRANSACTIONS_COMMIT = "/v1/{prefix}/transactions/commit";
public static final String V1_VIEWS = "/v1/{prefix}/namespaces/{namespace}/views";
public static final String V1_VIEW = "/v1/{prefix}/namespaces/{namespace}/views/{view}";
Expand Down