eskabetxe opened a new issue, #12633:
URL: https://github.com/apache/gravitino/issues/12633
### Describe the subtask
Trino 480+ change wrapped SPI methods that live in SHARED connector code
(compiled by every version-segment module), so the per-module wrapper
classes can't handle them alone. Introduce a version seam so the shared
`trino-connector` source compiles across Trino 435–483.
Scope:
- Adapter-level accessor for the `ColumnMetadata` comment (returns String
on <=479, Optional on 480+), used by `CatalogConnectorMetadataAdapter`,
the Hive/Iceberg/MySQL adapters, and `GravitinoTable`.
- Move `finishTableExecute` and `SchemaFunctionName` usage out of shared
`GravitinoMetadata` into the per-version `GravitinoMetadataNNN` wrappers.
- Replace the accumulating reflective `BlockEncodingManager` probe in
shared `JsonCodec` with a per-module `BlockEncodingSerde` factory seam:
each module carries only its range's construction logic (reflection
stays — it's a trino-main internal class — and within-version
OSS-vs-Starburst probing stays).
Regression gate: modules 435–479 must still compile and pass their tests
after the refactor.
### Parent issue
12630
--
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]
