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.

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

[#3524] feat(trino-connector): Load catalog by Trino dynamic catalog feature #3664

Merged
merged 4 commits into from
May 31, 2024

Conversation

diqiu50
Copy link
Contributor

@diqiu50 diqiu50 commented May 30, 2024

What changes were proposed in this pull request?

Load catalog by Trino dynamic catalog feature

Why are the changes needed?

Fix: #3524

diqiu50 added 4 commits May 30, 2024 21:26
… by using Trino CREATE CATALOG command (apache#3540)

1. Upgrade code to Trino-435
2. Support create Gravitino catalog by CREATE CATALOG statement
3. Disable testers of  Trino Connector

Fix: apache#3526

NO

Exists UT
…r like hive/iceberg/mysql/pg/memory (apache#3588)

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

Support create Internal connector like hive/iceberg/mysql/pg/memory

### Why are the changes needed?

Fix: apache#3587

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

NO

### How was this patch tested?

NO
…talog to Trino by Trino SQL command (apache#3602)

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

1、Support automatically loading catalog to Trino by using CREATE CATALOG
command
2、Support drop catalog
3、Support update catalog

### Why are the changes needed?

Fix: apache#3601

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

NO

### How was this patch tested?

NO
…ion on Gravitino catalog (apache#3631)

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

1. Support insert and select operation on Gravitino catalog
2. Support query optimize
3. Upgrade CI docker environment  

### Why are the changes needed?

Fix: apache#3630

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

NO

### How was this patch tested?

UT IT
@diqiu50 diqiu50 self-assigned this May 30, 2024
@diqiu50 diqiu50 requested a review from yuqi1129 May 30, 2024 14:21
@diqiu50 diqiu50 closed this May 30, 2024
@diqiu50 diqiu50 reopened this May 30, 2024
Copy link
Contributor

@yuqi1129 yuqi1129 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jerryshao jerryshao merged commit d0699e7 into apache:main May 31, 2024
68 of 69 checks passed
jerryshao pushed a commit that referenced this pull request May 31, 2024
@diqiu50 diqiu50 deleted the dynamic_catalog2 branch June 5, 2024 03:44
diqiu50 added a commit to diqiu50/gravitino that referenced this pull request Jun 13, 2024
…talog feature (apache#3664)

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

Load catalog by Trino dynamic catalog feature

### Why are the changes needed?

Fix: apache#3524
diqiu50 added a commit to diqiu50/gravitino that referenced this pull request Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[EPIC] Gravitino Trino connector loads the catalog using Trino's dynamic catalog feature
3 participants