Skip to content

protocol: query a shared Delta table version - #105

Draft
littlegrasscao wants to merge 1 commit into
server/split2/14-protocol-tablesfrom
server/split2/15-protocol-get-table
Draft

littlegrasscao wants to merge 1 commit into
server/split2/14-protocol-tablesfrom
server/split2/15-protocol-get-table

Conversation

@littlegrasscao

@littlegrasscao littlegrasscao commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add Delta Sharing GET /shares/{share}/schemas/{schema}/tables/{table}/version to OpenSharingProtocolService
  • return an empty body with the required Delta-Table-Version response header
  • core ProtocolTables.version finds the table among the share's included tables or the catalog children of a shared schema, resolving it as the share owner
  • core DeltaKernel vends short-lived read credentials through CatalogConnector.getStorageCredentials as the share owner, configures Hadoop for AWS/R2, Azure, or GCP, and reads the table's _delta_log
  • return the latest version when startingTimestamp is absent; for an ISO-8601 UTC startingTimestamp, return the earliest version at or after it
  • report invalid/too-late timestamps as INVALID_PARAMETER_VALUE, and missing or ungranted objects as RESOURCE_DOES_NOT_EXIST
  • local catalog: ENV credential mode for cloud integration tests; add runtime commons-lang3, which hadoop-aws needs and Spring no longer brings in
  • CI passes Azure and GCS test credentials
  • test support: TestServer.start(Properties) starts a server with overrides, and ProtocolApiSupport.restartServer replaces the default server with one on another catalog, closing it rather than leaking it

Testing

  • mvn clean install: 144 tests passed
  • ProtocolTableVersionTest reads real Delta logs written to a temp directory: latest, before the first commit, exact, between commits, after the latest, shared-schema children, invalid timestamp, missing table, ungranted share
  • ProtocolTableVersionIntegrationTest ran against the AWS, Azure, and GCS test tables (each case is skipped without its credentials)

Stacked on #103.

@littlegrasscao littlegrasscao changed the title server: get a table in a granted share server: query a shared Delta table version Sep 24, 2026
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 1f668fc to 6c13c98 Compare September 25, 2026 22:09
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 6c13c98 to 13de9d3 Compare September 25, 2026 22:20
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 13de9d3 to 09d0c47 Compare September 25, 2026 22:28
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from abf9448 to 57d140e Compare September 25, 2026 22:33
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 57d140e to 0c68e96 Compare September 25, 2026 22:38
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 75456bb to 48f3b07 Compare September 26, 2026 00:09
@littlegrasscao
littlegrasscao marked this pull request as ready for review September 26, 2026 00:13
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 48f3b07 to 3d63d9f Compare September 26, 2026 00:15
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch 2 times, most recently from 3b67287 to 27acbd6 Compare September 29, 2026 18:35
@littlegrasscao
littlegrasscao marked this pull request as draft October 7, 2026 01:57
@littlegrasscao littlegrasscao changed the title server: query a shared Delta table version protocol: query a shared Delta table version Oct 9, 2026
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch 2 times, most recently from 092b39e to 364eb1b Compare October 14, 2026 01:26
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 364eb1b to cff2805 Compare October 14, 2026 02:07
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from cff2805 to c1d2dc5 Compare October 14, 2026 19:25
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from c1d2dc5 to 0046254 Compare October 15, 2026 16:46
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 0046254 to 71948cd Compare October 15, 2026 18:14
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 71948cd to 68acb7c Compare October 15, 2026 18:29
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 68acb7c to e7934c1 Compare October 15, 2026 23:35
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from e7934c1 to c1becfb Compare October 15, 2026 23:38
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from c1becfb to c11cae9 Compare October 15, 2026 23:54
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from c11cae9 to 5ed39a7 Compare October 16, 2026 00:17
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 5ed39a7 to b967602 Compare October 16, 2026 00:41
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from b967602 to a2d2ffb Compare October 19, 2026 17:13
@zhu-tom
zhu-tom added this pull request to stack #116 October 19, 2026 19:06
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from a2d2ffb to 6aa86cc Compare October 19, 2026 21:44
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 6aa86cc to 8f63f82 Compare October 19, 2026 22:34
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 8f63f82 to ac497bc Compare October 19, 2026 22:37
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch 2 times, most recently from a262802 to 5a6744f Compare October 21, 2026 17:01
@littlegrasscao
littlegrasscao force-pushed the server/split2/15-protocol-get-table branch from 5a6744f to f3a49e4 Compare October 21, 2026 19:11
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