Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Improvement] Create docker-compose containers for testing Trino queries. #919

Closed
diqiu50 opened this issue Dec 1, 2023 · 1 comment
Closed

[Improvement] Create docker-compose containers for testing Trino queries. #919

diqiu50 opened this issue Dec 1, 2023 · 1 comment
Assignees
@xunliu @diqiu50
Labels
improvement Improvements on everything
Milestone
Gravitino 0.4.0

Comments

@diqiu50
Copy link
Contributor

diqiu50 commented Dec 1, 2023

What would you like to be improved?

The required containers for the setup include: hdfs, hive, trino, gravition, mysql, iceberg, prostosql.

How should we improve?

No response
@diqiu50 diqiu50 added the improvement Improvements on everything label Dec 1, 2023
@jerryshao jerryshao added this to the Gravitino 0.3.0 milestone Dec 4, 2023
@xunliu
Copy link
Member

xunliu commented Dec 5, 2023

Please add more detailed information.

  1. What's the version of MySQL, PostgreSQL?
  2. What's your use Docker image of MySQL, PostgreSQL?

@diqiu50 diqiu50 self-assigned this Dec 25, 2023
@diqiu50 diqiu50 changed the title [Improvement] Create containers for testing Trino queries. [Improvement] Create docker-compose containers for testing Trino queries. Dec 25, 2023
@diqiu50 diqiu50 mentioned this issue Dec 26, 2023
Merged
jerryshao added a commit that referenced this issue Jan 8, 2024
@diqiu50 @Clearvive
@qqqttt123 @jerryshao @brandboat @hiirrxnn
[#1247] Improvement (trino-connector) Make Trino QueryIT test cases r…
b8c8179 
…unning in ci. (#1262)

### What changes were proposed in this pull request?

Make Trino QueryIT test cases running in ci.
To run Trino Query IT, a complete test environment including Hive,
Iceberg, PostgreSQL, and MySQL dependencies is required. We need to set
up this test environment and execute the test cases automatically.

### Why are the changes needed?

Fix: #1247 #919

### Does this PR introduce _any_ user-facing change?

NO

### How was this patch tested?

NO

---------

Co-authored-by: Clearvive <[email protected]>
Co-authored-by: Clearvive <[email protected]>
Co-authored-by: qqqttt123 <[email protected]>
Co-authored-by: Heng Qin <[email protected]>
Co-authored-by: Jerry Shao <[email protected]>
Co-authored-by: Kuan-Po Tseng <[email protected]>
Co-authored-by: hiirrxnn <[email protected]>
@diqiu50 diqiu50 closed this as completed Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xunliu xunliu

@diqiu50 diqiu50

Labels
improvement Improvements on everything
Projects
None yet
Milestone
Gravitino 0.4.0
Development

No branches or pull requests
3 participants
@jerryshao @xunliu @diqiu50