FANNG1 opened a new pull request, #10517:
URL: https://github.com/apache/gravitino/pull/10517

   ### What changes were proposed in this pull request?
   
   This PR extracts the first step of the Flink multi-version refactor and 
limits the scope to the framework split plus Flink 1.18 support.
   
   The main changes are:
   - split the original single Flink connector module into `flink-common`, 
`flink-1.18`, and `flink-runtime-1.18`
   - move shared connector logic and shared test bases into `flink-common`
   - add versioned Flink 1.18 catalog/factory entry classes and Flink 1.18 
runtime packaging
   - introduce typed catalog compatibility hooks for the versioned layout
   - run Flink integration coverage from the Flink 1.18 module instead of the 
old shared module
   - keep the scope of this PR limited to Flink 1.18 as the baseline lane
   
   This PR intentionally does not include Flink 1.19 or 1.20 support. Those 
will follow in a separate PR.
   
   ### Why are the changes needed?
   
   The previous implementation mixed the framework refactor and all supported 
Flink minors into one large change set, which made review difficult.
   
   This PR reduces the review scope by first establishing the versioned 
framework with a single baseline Flink lane, Flink 1.18. That lets reviewers 
focus on:
   - whether the new module split is correct
   - whether the shared/common and versioned entry boundaries are reasonable
   - whether the Flink 1.18 baseline remains functional after the refactor
   
   Fix: #9710
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes.
   
   The Flink connector build layout changes from a single Flink module to:
   - `flink-common`
   - `flink-1.18`
   - `flink-runtime-1.18`
   
   The user-facing SQL behavior is unchanged for Flink 1.18, but the connector 
packaging and internal implementation are now version-scoped to support later 
minor-version expansion.
   
   ### How was this patch tested?
   
   The following validations were run:
   
   - `./gradlew :common:jar :flink-connector:flink-1.18:test -PskipITs 
:flink-connector:flink-runtime-1.18:test`
   - `./gradlew :flink-connector:flink-1.18:test -PskipTests 
-PtestMode=embedded -PskipDockerTests=false --tests 
"org.apache.gravitino.flink.connector.integration.test.catalog.GravitinoCatalogManagerIT118"
 --console=plain`
   


-- 
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