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

Optimize varchar/char mapping between Gravitino Iceberg catalog and Trino #2569

Closed
Tracked by #2356
yuqi1129 opened this issue Mar 18, 2024 · 0 comments · Fixed by #2579
Closed
Tracked by #2356

Optimize varchar/char mapping between Gravitino Iceberg catalog and Trino #2569

yuqi1129 opened this issue Mar 18, 2024 · 0 comments · Fixed by #2579

Comments

@yuqi1129
Copy link
Contributor

yuqi1129 commented Mar 18, 2024

There exists type mapping issues for data type varchar/type between Gravitino and Trino, we need to refine it. for more please see
#2035
docs.google.com/document/d/1xNkl4Y2t8Bfb8UOOdGBhyQr5DDreAOLmK8FOWiIDiQM/edit

This issue focuses on the Iceberg catalog and Trino.

yuqi1129 added a commit that referenced this issue Mar 19, 2024
… catalog between Gravitino and Trino (#2579)

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

Changing the char/varchar conversion for Iceberg catalog between
Gravitino and Trino.

### Why are the changes needed?

We need to make type conversion more exactly.

Fix: #2569 

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

N/A.

### How was this patch tested?

UT and ITs
coolderli pushed a commit to coolderli/gravitino that referenced this issue Apr 2, 2024
…ceberg catalog between Gravitino and Trino (apache#2579)

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

Changing the char/varchar conversion for Iceberg catalog between
Gravitino and Trino.

### Why are the changes needed?

We need to make type conversion more exactly.

Fix: apache#2569 

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

N/A.

### How was this patch tested?

UT and ITs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant