danny0405 opened a new pull request, #18920:
URL: https://github.com/apache/hudi/pull/18920

   ### Describe the issue this Pull Request addresses
   
   Hudi still carried build, CI, release, documentation, and adapter-module 
support for Flink 1.17. Since Flink 1.17 support is being dropped, keeping the 
old profile and `hudi-flink1.17-bundle` paths would continue to advertise and 
validate an unsupported runtime.
   
   This PR removes the Flink 1.17-specific module and build/release wiring. It 
also removes the now-redundant `TestHoodieCatalogs` test shim, because the 
remaining supported Flink versions all support the advanced alter table syntax.
   
   ### Summary and Changelog
   
   Drops Flink 1.17 as a supported Flink target and makes Flink 1.18 the oldest 
remaining Flink 1.x profile.
   
   #### Commit 1: chore: drop support for Flink 1.17 (`bf2cc6f52eab`)
   - Removed the root Maven `flink1.17` profile, `flink1.17.version`, and 
`hudi-flink-datasource/hudi-flink1.17.x` module.
   - Removed `hudi-flink1.17-bundle` from release staging validation and deploy 
scripts.
   - Removed Flink 1.17 entries from GitHub test and bundle-validation matrices.
   - Deleted the Flink 1.17 bundle-validation base image script.
   - Updated README Flink build instructions to remove `-Dflink1.17`.
   - Removed all remaining `TestHoodieCatalogs` shims and simplified 
`ITTestSchemaEvolutionBySQL` to run unconditionally.
   - Retained the null-metric test path while making its comments 
version-neutral.
   
   ### Impact
   
   This is a compatibility change for users building or consuming Hudi with 
Flink 1.17: the `flink1.17` Maven profile and `hudi-flink1.17-bundle` are 
removed. Users should move to `-Dflink1.18` or a newer Flink profile.
   
   There is no storage format change, no public Java API change, and no 
expected performance impact. The main affected areas are Flink build profiles, 
version adapter modules, release automation, CI validation, and Flink SQL 
schema-evolution tests.
   
   ### Risk Level
   
   medium
   
   The risk is medium because this intentionally removes a supported 
build/runtime target and touches Maven profile selection, CI matrices, release 
scripts, and Flink adapter modules. The change is mostly deletion and profile 
cleanup, which reduces ongoing maintenance surface.
   
   Validation performed:
   - `mvn test-compile -Dflink1.18 -pl hudi-flink-datasource/hudi-flink -am 
-DskipTests`
   - `rg -n "TestHoodieCatalogs|supportAdvancedAlterTableSyntax|EnabledIf" 
hudi-flink-datasource`
   - `git diff --check`
   
   ### Documentation Update
   
   README build instructions were updated to remove Flink 1.17 from the 
supported Flink profile table and example commands. No additional docs are 
needed because this removes an old build target rather than adding new behavior.
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Enough context is provided in the sections above
   - [ ] Adequate tests were added if applicable
   


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