Skip to content

[core] Support options in ObjectTable#7378

Open
shyjsarah wants to merge 1 commit intoapache:masterfrom
shyjsarah:dev-object-options
Open

[core] Support options in ObjectTable#7378
shyjsarah wants to merge 1 commit intoapache:masterfrom
shyjsarah:dev-object-options

Conversation

@shyjsarah
Copy link
Contributor

Purpose

Previously, ObjectTable did not support storing and passing options — its options() method always returned an empty map, and copy() ignored dynamic options entirely. This was inconsistent with other table types like FormatTable, LanceTable, and IcebergTable, which all properly support options.

This PR adds options support to ObjectTable, aligning it with other table type implementations.

Tests

Modified RESTCatalogTest#testObjectTable to verify:

  • Custom options set during table creation are preserved and accessible via options()
  • copy() correctly merges dynamic options with existing options
  • copy() can override existing option values
  • copy() does not modify the original table's options

API and Format

Documentation

Generative AI tooling

Generated-by: Aone Copilot

@shyjsarah shyjsarah changed the title [WIP][core] Support options in ObjectTable [core] Support options in ObjectTable Mar 9, 2026
@shyjsarah shyjsarah closed this Mar 9, 2026
@shyjsarah shyjsarah reopened this Mar 9, 2026
@shyjsarah shyjsarah force-pushed the dev-object-options branch from 95ae177 to 395def1 Compare March 10, 2026 02:14
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