Skip to content

[trino] Add initial connector build and plugin skeleton - #4207

Open
yangshangqing95 wants to merge 1 commit into
apache:mainfrom
yangshangqing95:trino/init
Open

[trino] Add initial connector build and plugin skeleton#4207
yangshangqing95 wants to merge 1 commit into
apache:mainfrom
yangshangqing95:trino/init

Conversation

@yangshangqing95

Copy link
Copy Markdown

Summary

This PR introduces the initial build and plugin foundation for the native Fluss Trino connector proposed in #4197.

The change includes:

  • an isolated fluss-trino Maven build targeting Trino 483 and JDK 25;
  • a dedicated Maven Wrapper using Maven 3.9.16;
  • a dedicated CI workflow for the Trino connector;
  • the initial FlussPlugin, FlussConnectorFactory, and FlussConnector
    skeleton;
  • build environment checks for the supported Java and Maven versions;
  • Checkstyle and Spotless checks aligned with the Fluss Java code style;
  • Apache RAT license checks;
  • a basic test verifying registration of the fluss connector factory.

fluss-trino is intentionally kept outside the default Fluss Maven reactor. As a result, introducing the connector does not change the JDK or Maven requirements of the existing Fluss build, nor does this PR change the runtime behavior of existing Fluss modules.

The connector is not functionally complete in this PR. Metadata discovery and data reading will be implemented incrementally in follow-up PRs.

Keeping the initial build and plugin foundation as a separate change keeps the scope small and reviewable, and avoids introducing the complete connector as one large change that would be harder to review, maintain, and evolve based on
community feedback.

Test Plan

  • Build and run all connector checks with JDK 25:

    cd fluss-trino
    JAVA_HOME=$(/usr/libexec/java_home -v 25) ./mvnw -B clean verify

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