Support second and third parameter of read_object are not constant

[JackieTien97]

This pull request enhances the handling of the READ_OBJECT function in the query engine by adding support for dynamic offset and length parameters, improving test coverage, and making related codebase adjustments. The main changes include introducing new column transformers to support dynamic arguments, updating the query planner, and extending integration tests to verify the new functionality.

Enhancements to READ_OBJECT function support:

• Added ReadObject2ColumnTransformer and ReadObject3ColumnTransformer classes to handle READ_OBJECT with dynamic offset and length parameters, enabling the function to accept column-based arguments instead of just literals. [1] [2]
• Updated ColumnTransformerBuilder to utilize the new transformers when READ_OBJECT is called with column arguments, and refactored the logic to distinguish between literal and column inputs for offset and length. [1] [2] [3]

Query planner and type handling improvements:

• Modified the operator tree generator to support linear fill for columns of type OBJECT, ensuring consistent behavior with other supported types.

Test and codebase updates:

• Expanded integration tests in IoTDBObjectQueryIT.java to cover various usages of READ_OBJECT with dynamic parameters and linear fill, verifying correct results for the new scenarios. [1] [2]
• Renamed the test class IoTDBObjectQueryIT2.java to IoTDBObjectQuery2IT.java for consistency with naming conventions.

Refactoring and API improvements:

• Refactored the ReadObjectColumnTransformer to expose its transform and readObject methods as static, enabling reuse by the new transformer classes and supporting the extended API. [1] [2]

[Copilot]

Pull request overview

This pull request enhances the READ_OBJECT function to support dynamic offset and length parameters from columns, rather than only accepting constant literal values. This improvement increases flexibility by allowing runtime-determined values for these parameters.

• Added ReadObject2ColumnTransformer and ReadObject3ColumnTransformer classes to handle dynamic parameter scenarios
• Refactored ReadObjectColumnTransformer to expose static methods for code reuse across transformers
• Updated ColumnTransformerBuilder to detect and route to appropriate transformer based on parameter types (literal vs column)
• Extended OperatorTreeGenerator to support linear fill for OBJECT data type

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
ReadObjectColumnTransformer.java	Refactored transform and readObject methods to be static with expanded parameters to support reuse by new transformer classes
ReadObject2ColumnTransformer.java	New binary transformer handling READ_OBJECT with dynamic offset parameter (defaults length to -1 for reading to end)
ReadObject3ColumnTransformer.java	New ternary transformer handling READ_OBJECT with both dynamic offset and length parameters
ColumnTransformerBuilder.java	Updated to distinguish between literal and column-based arguments for READ_OBJECT and instantiate appropriate transformer
OperatorTreeGenerator.java	Added OBJECT type to linear fill identity handling alongside TEXT, STRING, and BLOB types
IoTDBObjectQueryIT.java	Expanded test coverage with additional columns (l1, l2) and new test cases for dynamic parameters and linear fill functionality
IoTDBObjectQuery2IT.java	Renamed test class from IoTDBObjectQueryIT2 to IoTDBObjectQuery2IT for naming consistency

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

❌ Patch coverage is 5.35714% with 53 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.34%. Comparing base (8e99486) to head (1db8673).
⚠️ Report is 5 commits behind head on master.

Files with missing lines	Patch %	Lines
...g/column/ternary/ReadObject3ColumnTransformer.java	0.00%	21 Missing ⚠️
...execution/relational/ColumnTransformerBuilder.java	0.00%	16 Missing ⚠️
...ag/column/binary/ReadObject2ColumnTransformer.java	0.00%	14 Missing ⚠️
...lumn/unary/scalar/ReadObjectColumnTransformer.java	60.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #16967      +/-   ##
============================================
+ Coverage     39.32%   39.34%   +0.01%     
  Complexity      212      212              
============================================
  Files          5053     5055       +2     
  Lines        337216   337314      +98     
  Branches      42923    42940      +17     
============================================
+ Hits         132603   132708     +105     
+ Misses       204613   204606       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.