Skip to content

protocol: list tables in granted schemas - #103

Draft
littlegrasscao wants to merge 3 commits into
server/split2/13-protocol-schemasfrom
server/split2/14-protocol-tables
Draft

littlegrasscao wants to merge 3 commits into
server/split2/13-protocol-schemasfrom
server/split2/14-protocol-tables

Conversation

@littlegrasscao

@littlegrasscao littlegrasscao commented Sep 22, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add GET /shares/{share}/schemas/{schema}/tables and GET /shares/{share}/all-tables to OpenSharingProtocolService
  • core ProtocolTables lists the share's included tables plus the catalog children of each schema shared as a whole, ordered by schema and then name
  • every catalog call (resolveAsset, listChildren) is made as the share owner, so the catalog decides whether the owner may still read each table; included tables are resolved only when they land on the requested page
  • use the catalog table id and accessModes url,dir for managed and external Delta tables
  • report a missing or ungranted share, or unknown schema, as RESOURCE_DOES_NOT_EXIST
  • reuse bounded maxResults / opaque pageToken pagination, where maxResults=0 returns no items with a token

Testing

  • mvn clean install: 138 tests passed
  • ProtocolTablesTest: lists the granted tables in a schema, the catalog children of a shared schema, and every table in a share, and pages tables in a schema, where maxResults=0 returns no tables but a token

Stacked on #102.

@littlegrasscao
littlegrasscao marked this pull request as ready for review September 24, 2026 17:51
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 246e86e to 791bee3 Compare September 25, 2026 22:08
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 791bee3 to 033e2a0 Compare September 25, 2026 22:20
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 033e2a0 to 7179238 Compare September 25, 2026 22:28
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 7179238 to cf89674 Compare September 25, 2026 22:33
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from cf89674 to 6358939 Compare September 25, 2026 22:38
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch 2 times, most recently from de64a36 to a663142 Compare September 26, 2026 00:09
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from a663142 to 3103787 Compare September 26, 2026 00:14
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 3103787 to 68d31dc Compare September 26, 2026 00:16
@littlegrasscao
littlegrasscao marked this pull request as draft October 7, 2026 01:57
@littlegrasscao littlegrasscao changed the title server: list tables in granted schemas protocol: list tables in granted schemas Oct 9, 2026
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 80db9e8 to 1f84f1a Compare October 14, 2026 01:16
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 1f84f1a to b6cd2b2 Compare October 14, 2026 19:25
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from b6cd2b2 to 63c1f8d Compare October 15, 2026 16:46
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 63c1f8d to 1ef89db Compare October 15, 2026 18:13
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 1ef89db to a497f64 Compare October 15, 2026 18:29
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from a497f64 to 37ea464 Compare October 15, 2026 23:35
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 37ea464 to 38a5799 Compare October 15, 2026 23:38
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 38a5799 to ffa76a9 Compare October 15, 2026 23:54
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from ffa76a9 to 8f52855 Compare October 16, 2026 00:17
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 8f52855 to 58e6afe Compare October 16, 2026 00:41
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from 58e6afe to cb0619e 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/14-protocol-tables branch from cb0619e to e4ea5dd Compare October 19, 2026 21:44
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from e4ea5dd to c14704d Compare October 19, 2026 22:34
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from c14704d to b517f78 Compare October 19, 2026 22:37
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch 2 times, most recently from ee9d3b0 to ae5c0fd Compare October 21, 2026 17:01
@littlegrasscao
littlegrasscao force-pushed the server/split2/14-protocol-tables branch from ae5c0fd to b1918ad Compare October 21, 2026 19:10
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