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

Check table existence in Pinot connector #8859

Merged
merged 1 commit into from
Aug 13, 2021
Merged

Conversation

ebyhr
Copy link
Member

@ebyhr ebyhr commented Aug 11, 2021

Fixes #8834 Fixes #8307

Confirmed added two tests failed before this change.

Copy link
Member

@hashhar hashhar left a comment

Choose a reason for hiding this comment

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

LGTM % a test to cover interaction with Pinot <-> Trino table name translation.

@ebyhr ebyhr merged commit 2663e49 into trinodb:master Aug 13, 2021
@ebyhr ebyhr deleted the ebi/pinot-table branch August 13, 2021 14:31
@ebyhr ebyhr mentioned this pull request Aug 13, 2021
10 tasks
@ebyhr ebyhr added this to the 361 milestone Aug 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants