shyjsarah opened a new pull request, #7378:
URL: https://github.com/apache/paimon/pull/7378

   <!-- Please specify the module before the PR name: [core] ... or [flink] ... 
-->
   
   ### 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.
   
   <!-- What is the purpose of the change -->
   
   ### Tests
   
   <!-- List UT and IT cases to verify this change -->
   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
   
   <!-- Does this change affect API or storage format -->
   
   ### Documentation
   
   <!-- Does this change introduce a new feature -->
   
   ### Generative AI tooling
   
   <!--
   If generative AI tooling has been used in the process of authoring this 
patch, please include the
   phrase: 'Generated-by: ' followed by the name of the tool and its version.
   If no, write 'No'.
   Please refer to the [ASF Generative Tooling 
Guidance](https://www.apache.org/legal/generative-tooling.html) for details.
   -->
   Generated-by: Aone Copilot
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to