slachiewicz opened a new pull request, #57: URL: https://github.com/apache/flink-connector-shared-utils/pull/57
## What is the purpose of the change 1. **Remove deprecated `stCarolas/setup-maven@v5`**: Eliminates Node.js 20 deprecation warnings on GitHub Actions runners. `actions/setup-java@v5` already configures Java, utilizes runner preinstalled Maven, and manages `cache: maven`. 2. **Fix cache save condition for Flink binary**: Prevents `Unable to reserve cache with key` warnings on cache hits by adding `steps.restore-cache-flink.outputs.cache-hit != 'true'`. 3. **Prevent 50-minute hangs on remote artifact resolution**: Adds explicit Wagon and Aether network timeouts to `MVN_CONNECTION_OPTIONS`. ## Brief change log - Removed `stCarolas/setup-maven@v5` and redundant version extraction from `.github/workflows/ci.yml` and `.github/workflows/python_ci.yml` - Added `steps.restore-cache-flink.outputs.cache-hit != 'true'` check to `Cache Flink binary` step - Added Wagon and Aether timeout properties to `MVN_CONNECTION_OPTIONS` -- 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]
