Skip to content

Core, OpenAPI: Add RCK coverage for variant columns - #17500

Open
nssalian wants to merge 1 commit into
apache:mainfrom
nssalian:variant-rck
Open

Core, OpenAPI: Add RCK coverage for variant columns#17500
nssalian wants to merge 1 commit into
apache:mainfrom
nssalian:variant-rck

Conversation

@nssalian

@nssalian nssalian commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Rationale for the change

#17256 added the variant type to the REST catalog spec. This follows up that change with an RCK test.

Changes

  • CatalogTests: adds a supportsVariant() flag (default false); testCreateTableWithVariantColumn (variant at top level, list element, and map value; asserts round-trip + format version 3); testCreateV2TableWithVariantColumnFails (variant is v3-only).
  • TestRESTCatalog: enables supportsVariant() so the tests run against the REST reference implementation.
  • RCK (RESTCompatibilityKitCatalogTests / RESTCompatibilityKitSuite): adds opt-in rck.supports-variant property (default false);
  • Documentation in open-api/README.md

@nssalian
nssalian requested review from amogh-jahagirdar and laskoviymishka and removed request for amogh-jahagirdar August 3, 2026 21:47
@nssalian
nssalian marked this pull request as ready for review August 3, 2026 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant