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

   ### Describe the issue this Pull Request addresses
   
   The Flink-related GitHub Actions workflow had Maven arguments that did not 
consistently pass the intended Flink profile. In the Hadoop MR and Java client 
CI job, `FLINK_PROFILE` was passed as a literal Maven property name instead of 
expanding the matrix value, which could make the job rely on default profile 
behavior rather than the configured matrix profile.
   
   The workflow also repeated the same `parquet.version` argument in two Flink 
build commands. This PR cleans up those CI command lines so the configured 
Flink and Parquet versions are applied clearly and consistently.
   
   ### Summary and Changelog
   
   - Expands `FLINK_PROFILE` correctly in the 
`test-hudi-hadoop-mr-and-hudi-java-client` build and unit test Maven commands.
   - Removes duplicate `-Dparquet.version="$FLINK_PARQUET_VERSION"` arguments 
from Flink bundle validation and Java 17 Flink build commands.
   
   ### Impact
   
   No product runtime behavior change; this only updates GitHub Actions Maven 
invocations.
   
   ### Risk Level
   
   low. The change is limited to `.github/workflows/bot.yml` command-line 
argument cleanup. The main risk is CI command regression, mitigated by the 
narrow scope and by preserving the existing Maven goals, modules, and version 
variables.
   
   ### Documentation Update
   
   none
   
   ### 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