roryqi commented on code in PR #11635:
URL: https://github.com/apache/gravitino/pull/11635#discussion_r3409752233
##########
iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/CatalogWrapperForREST.java:
##########
@@ -462,6 +464,42 @@ public PlanTableScanResponse planTableScan(
}
}
+ /**
+ * Fetch the result of a previously submitted async scan plan.
+ *
+ * <p>The current implementation performs synchronous scan planning only
(always returns {@code
+ * COMPLETED} from {@link #planTableScan}), so no plan IDs are retained.
This method always throws
+ * {@link NoSuchPlanIdException}.
+ *
+ * @param tableIdentifier The table identifier.
+ * @param planId The plan identifier to look up.
+ * @return never returns normally.
+ * @throws NoSuchPlanIdException always, because synchronous planning does
not retain plan IDs.
+ */
+ public FetchPlanningResultResponse fetchPlanningResult(
+ TableIdentifier tableIdentifier, String planId) {
+ throw new NoSuchPlanIdException(
Review Comment:
We don't override this method.
--
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 comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
