-
Notifications
You must be signed in to change notification settings - Fork 304
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[#614] feat(docker): Gravitino Trino Docker image #702
Conversation
Code Coverage Report
|
Please change the tag name as proposed in #680. |
Overall LGTM, could others please also take a look. |
@yuqi1129 I fixed Trino JDBC port issue. |
dev/docker/README.md
Outdated
This Docker image is used to support Gravitino integration testing. | ||
It includes Hadoop-2.x and Hive-2.x, you can use this Docker image to test the Gravitino catalog-hive module. | ||
# Gravitino Docker images | ||
This Docker image is designed to facilitate Gravitino integration testing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the trailing whitespace.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not fixed, don't forget to do it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DONE
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Will merge if no more comments. |
What changes were proposed in this pull request?
Why are the changes needed?
Gravitino uses Trino for integration tests.
Trino has more than 40 plugins, Which makes the Trino server start up very slowly, usually taking 2~3 minutes.
We need to rebuild the Gravitino Trino Docker image and remove unused plugins.
Fix: #614
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
I build Trino Docker image success. https://hub.docker.com/repository/docker/datastrato/gravitino-ci-trino/general