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

[Bug report] Varchar type of PG catalog is not handle correctly by Gravitino #2035

Closed
yuqi1129 opened this issue Feb 4, 2024 · 0 comments · Fixed by #2358
Closed

[Bug report] Varchar type of PG catalog is not handle correctly by Gravitino #2035

yuqi1129 opened this issue Feb 4, 2024 · 0 comments · Fixed by #2358
Assignees
@yuqi1129
Labels
bug Something isn't working
Milestone
Gravitino 0.5.0

Comments

@yuqi1129
Copy link
Contributor

yuqi1129 commented Feb 4, 2024

Version

main branch

Describe what's wrong

image image

Error message and/or stacktrace

image

How to reproduce

The main branch.

Additional context

No response
@yuqi1129 yuqi1129 added the bug Something isn't working label Feb 4, 2024
@yuqi1129 yuqi1129 mentioned this issue Feb 5, 2024
Closed
@jerryshao jerryshao added this to the Gravitino 0.4.0 milestone Feb 5, 2024
This was referenced Feb 26, 2024
Closed
Merged
yuqi1129 added a commit that referenced this issue Mar 11, 2024
@yuqi1129
[#2357] improvement(trino): Fix varchar type mapping between Gravitin…
77fcc13 
…o JDBC catalog and Trino (#2358)

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

Modify the type mapping for VARCHAR and CHAR to match Gravitino and
Trino.

### Why are the changes needed?

please see issue #2356

Fix: #2035
Fix: #2357

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

N/A. 

### How was this patch tested?

New UTs.
This was referenced Mar 18, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuqi1129 yuqi1129

Labels
bug Something isn't working
Projects
None yet
Milestone
Gravitino 0.5.0
2 participants
@jerryshao @yuqi1129