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

[#1328] fix(trino-connector): Fix the failed 'drop catalog' test on Trino image gravitino-ci-trino:0.1.3. #1329

Merged
merged 3 commits into from
Jan 5, 2024

Conversation

diqiu50
Copy link
Contributor

@diqiu50 diqiu50 commented Jan 4, 2024

What changes were proposed in this pull request?

Fix the failed 'drop catalog' test on Trino image gravitino-ci-trino:0.1.3.
Trino image gravitino-ci-trino:0.1.2 use the gravitino-trino-connector-0.3.0-SNAPSHOT.jar, it does not support drop catalog, so the tester has made it compatible with this.
Trino image gravitino-ci-trino:0.1.3 use the gravitino-trino-connector-0.4.0-SNAPSHOT.jar, it support drop catalog.
the test did not pass.

Why are the changes needed?

Fix: #1328

Does this PR introduce any user-facing change?

NO

How was this patch tested?

NO

@diqiu50 diqiu50 requested a review from yuqi1129 January 4, 2024 08:21
@jerryshao jerryshao added need backport Issues that need to backport to another branch branch-0.3 labels Jan 4, 2024
@@ -1001,7 +1001,7 @@ void testDropCatalogAndCreateAgain() throws InterruptedException {
createdMetalake.dropCatalog(NameIdentifier.of(metalakeName, catalogName));
// We need to test we can't load this catalog any more by Trino.
success = checkTrinoHasRemoved(sql, 30);
Assertions.assertFalse(success, "Trino should not load the catalog any more: " + sql);
Assertions.assertTrue(success, "Trino should not load the catalog any more: " + sql);
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add some description about why the Trino image 0.1.3 brings these changes?

@diqiu50 diqiu50 self-assigned this Jan 5, 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 26731a9 into apache:main Jan 5, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 5, 2024
…rino image gravitino-ci-trino:0.1.3. (#1329)

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

Fix the failed 'drop catalog' test on Trino image
gravitino-ci-trino:0.1.3.
Trino image gravitino-ci-trino:0.1.2 use the
gravitino-trino-connector-0.3.0-SNAPSHOT.jar, it does not support drop
catalog, so the tester has made it compatible with this.
Trino image gravitino-ci-trino:0.1.3 use the
gravitino-trino-connector-0.4.0-SNAPSHOT.jar, it support drop catalog.
 the test did not pass. 

### Why are the changes needed?

Fix: #1328

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

NO

### How was this patch tested?

NO
@diqiu50 diqiu50 deleted the fix-1328 branch January 5, 2024 09:05
diqiu50 added a commit to diqiu50/gravitino that referenced this pull request Jan 8, 2024
…t on Trino image gravitino-ci-trino:0.1.3. (apache#1329)

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

Fix the failed 'drop catalog' test on Trino image
gravitino-ci-trino:0.1.3.
Trino image gravitino-ci-trino:0.1.2 use the
gravitino-trino-connector-0.3.0-SNAPSHOT.jar, it does not support drop
catalog, so the tester has made it compatible with this.
Trino image gravitino-ci-trino:0.1.3 use the
gravitino-trino-connector-0.4.0-SNAPSHOT.jar, it support drop catalog.
 the test did not pass. 

### Why are the changes needed?

Fix: apache#1328

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

NO

### How was this patch tested?

NO
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-0.3 need backport Issues that need to backport to another branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug report] The 'Drop catalog' test failed on the Docker image for gravitino-ci-trino:0.1.3.
3 participants