apache / apache/gravitino

[Subtask] Add Trino 482-483 version-segment module

Open
#12,638 0 comments 0 reactions 0 assignees View on GitHub
subtask
Dominant language
Java
Stars
3.2k
Forks
935
Avg merge
1d 15h
Merged PRs (30d)
315

Description

### Describe the subtask

Add `trino-connector-482-483` (JDK 25 toolchain), depending on the shared
version seam (#). Trino 482 and 483 are identical in wrapped surface
(verified: same compile errors), so one module covers both.

Trino 482 reworks the data path (on top of 481):
- `ConnectorSplitManager.getSplits`: `DynamicFilter` -> `Set`.
- `ConnectorSplitSource.getNextBatch` signature change.
- Page source & page sink provider signatures (`ConnectorTableCredentials`,
`ConnectorPageSinkId`).
- `GravitinoConstraint`: `predicate()` removed.
- `JsonCodec` / `TypeSignatureDeserializer` symbols removed.
This is the highest-risk piece (hot data path); seam the affected shared
classes carefully and keep 435–481 green.

Includes: module wiring, unit test (TestGravitinoConnector482), and
docs/compat-matrix updates.

### Parent issue

https://github.com/apache/gravitino/issues/12632

Contributor guide

Open the contributing guide

Research direction

Start with the existing 435–481 version modules and the shared version seam referenced by parent issue #12632, then trace the affected connector split, page source, page sink, GravitinoConstraint, and JSON-related classes. Use TestGravitinoConnector482 as the validation entry point. Done means the 482–483 module is wired, its unit test passes, docs/compat-matrix is updated, and versions 435–481 remain green.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
databases
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.