AndreaBozzo commented on PR #487:
URL: https://github.com/apache/hudi-rs/pull/487#issuecomment-3697632283

   Hi @xushiyan, sorry for the delay, holidays got in the way, I did my best to 
get this done before Christmas but couldn't make it.
   
   ## Changes in this commit
   
   Simplified the CI approach as you suggested:
   
   1. **Pre-built Docker image published**: 
[`andreab92/hudi-demo-runner:latest`](https://hub.docker.com/r/andreab92/hudi-demo-runner)
      - Tags: `rust-1.87-py3.13`, `latest`
      - SHA: `a4dd53edc8d11499efb6634a7152742bdc60404e1789dcb4c74df8263ea2cd96`
   
   2. **Removed `prepare-integration-build` job** - no more 7min overhead
   
   3. **Integration tests now**:
      - Pull image directly from Docker Hub (~30s)
      - Use GitHub Actions cache for Rust/Python build artifacts
      - Run in parallel immediately
   
   4. **Reverted** `compose.yaml` network section and `ci_run.sh` to original 
versions
   
   ## After merge
   
   You can republish the image under Hudi's official Docker Hub:
   ```bash
   docker pull andreab92/hudi-demo-runner:latest
   docker tag andreab92/hudi-demo-runner:latest apache/hudi-demo-runner:latest
   docker push apache/hudi-demo-runner:latest
   ```
   
   Then update CI to use `apache/hudi-demo-runner:latest`.
   
   Let me know if anything needs adjustment!


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