yihua opened a new pull request #4997:
URL: https://github.com/apache/hudi/pull/4997
## What is the purpose of the pull request
The latest master of Presto relies on hudi-presto-bundle, while current
Presto releases rely on hudi-common and hudi-hadoop-mr 0.9.0, which are pulled
in plugin/hive-hadoop2/ in the Docker setup, making it hard to test the latest
changes in Hudi due to class conflict.
As a temporary workaround, this PR removes `hudi-common-0.9.0.jar` and
`hudi-hadoop-mr-0.9.0.jar` in `plugin/hive-hadoop2/` to avoid conflicts with
latest hudi-presto-bundle.jar, which is copied from the build. This PR also
upgrades Presto to 0.271, so that the new presto docker image is pushed to
`apachehudi/hudi-hadoop_2.8.4-prestobase_0.271:latest`. In this way, if there
is any issue with the changes in this PR, we can roll this back without any
side effect.
## Brief change log
- Removes `hudi-common-0.9.0.jar` and `hudi-hadoop-mr-0.9.0.jar` within
`plugin/hive-hadoop2/` in `docker/hoodie/hadoop/prestobase/Dockerfile`
- Upgrades Presto from 0.268 to 0.271 in Docker setup
- Pushes new Presto Docker image to
`apachehudi/hudi-hadoop_2.8.4-prestobase_0.271:latest`
## Verify this pull request
This pull request is already covered by existing integration tests.
## Committer checklist
- [ ] Has a corresponding JIRA in PR title & commit
- [ ] Commit message is descriptive of the change
- [ ] CI is green
- [ ] Necessary doc changes done or have another open PR
- [ ] For large changes, please consider breaking it into sub-tasks under
an umbrella JIRA.
--
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]
