Skip to content

feat(rest): add scan plan endpoints to REST catalog client#614

Open
gsandeep1241 wants to merge 9 commits intoapache:mainfrom
gsandeep1241:sandeepg-scan-plan-endpoint-for-rest-catalog-client-2
Open

feat(rest): add scan plan endpoints to REST catalog client#614
gsandeep1241 wants to merge 9 commits intoapache:mainfrom
gsandeep1241:sandeepg-scan-plan-endpoint-for-rest-catalog-client-2

Conversation

@gsandeep1241
Copy link
Copy Markdown

@gsandeep1241 gsandeep1241 commented Apr 5, 2026

  • Introduced 4 endpoints: planning table scan, fetching a scan result, cancelling a scan plan and fetching scan planning tasks (endpoint.h)
  • Introduced corresponding request and response objects (types.h)
  • Implemented handlers (catalog.h). The handlers construct a request object and return a response object
    • Handlers use newly introduced helper methods to serialize and deserialize request response json objects
  • Added unit tests for all the above helper methods and handler methods
  • Added integration tests for the handlers

Sandeep Gottimukkala added 2 commits April 4, 2026 22:57
- Add PlanTableScan, FetchPlanningResult, CancelPlanning, FetchScanTasks
  methods to RestCatalog, wiring them to the corresponding REST endpoints
- Add scan plan endpoint definitions and resource path helpers
- Add ScanPlanErrorHandler with proper 404/406 error dispatch
  (NoSuchPlanIdException, NoSuchPlanTaskException, etc.)
- Add PlanTableScanRequest/FetchScanTasksRequest serialization helpers
- Add kNoSuchPlanId and kNoSuchPlanTask ErrorKind values
- Add DataFileFromJson and FileScanTasksFromJson overloads with
  partition spec and schema support
- Add endpoint and integration tests for all 4 scan plan operations
@gsandeep1241 gsandeep1241 force-pushed the sandeepg-scan-plan-endpoint-for-rest-catalog-client-2 branch from 2b6bc41 to 4fde2a5 Compare April 5, 2026 20:44
@gsandeep1241 gsandeep1241 force-pushed the sandeepg-scan-plan-endpoint-for-rest-catalog-client-2 branch from 4fde2a5 to ab6c8d4 Compare April 5, 2026 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant