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

[#5573] Add test cases for decimal, array, map, row, format in Trino CI for hive catalog #5562

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[#5573] Add test cases for decimal, array, map, row, format in Trino CI for hive catalog #5562

wants to merge 3 commits into from

Conversation

danhuawang
Copy link
Contributor

@danhuawang danhuawang commented Nov 13, 2024 •
edited
Loading

What changes were proposed in this pull request?

  1. Add some cases for complex data type
  2. Add some cases for different storage format

Why are the changes needed?

#5573

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

Test locally.

diqiu50
diqiu50 reviewed Nov 13, 2024
View reviewed changes
Copy link
Contributor

@diqiu50 diqiu50 left a comment

Choose a reason for hiding this comment

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

It is recommended to use "gt_" as the prefix for schema names, so that residual data can be automatically cleaned up during test startup.

@danhuawang
Copy link
Contributor Author

It is recommended to use "gt_" as the prefix for schema names, so that residual data can be automatically cleaned up during test startup.

updated

@jerryshao
Copy link
Contributor

@danhuawang do you have an issue tracking this PR, if not please create an issue, if so please update the PR title, thanks.

@danhuawang danhuawang changed the title [Test] Add test cases for decimal, array, map, row, format in Trino CI for hive catalog [#5573] Add test cases for decimal, array, map, row, format in Trino CI for hive catalog Nov 14, 2024
@danhuawang
Copy link
Contributor Author

@danhuawang do you have an issue tracking this PR, if not please create an issue, if so please update the PR title, thanks.

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@diqiu50 diqiu50 Awaiting requested review from diqiu50

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@danhuawang @jerryshao @diqiu50