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

[Bug report] Can't create a table that exists in other schema #2115

Closed
yuqi1129 opened this issue Feb 6, 2024 · 0 comments · Fixed by #2116
Closed

[Bug report] Can't create a table that exists in other schema #2115

yuqi1129 opened this issue Feb 6, 2024 · 0 comments · Fixed by #2116
Assignees
Labels
bug Something isn't working

Comments

@yuqi1129
Copy link
Contributor

yuqi1129 commented Feb 6, 2024

Version

main branch

Describe what's wrong

image

db1 contains a table same and db_ contains no table, then we create a table same under db_ will get error.

Error message and/or stacktrace

please see above.

How to reproduce

please see above.

Additional context

No response

@yuqi1129 yuqi1129 added the bug Something isn't working label Feb 6, 2024
@jerryshao jerryshao added this to the Gravitino 0.5.0 milestone Feb 20, 2024
@jerryshao jerryshao assigned yuqi1129 and unassigned Clearvive Mar 4, 2024
mchades pushed a commit that referenced this issue Apr 23, 2024
…same name prefix. (#2116)

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

Add check logic about schema name when loading table meta from driver. 

### Why are the changes needed?

Some drivers , such as PG drivers, contain schema name information, we
need to filter it. Some drivers like MySQL don't like it, we do not need
to check it.
check it. 

Fix: #2115 

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

N/A.

### How was this patch tested?

Add ITs: `testCreateSameTableInDifferentSchema`
github-actions bot pushed a commit that referenced this issue Apr 23, 2024
…same name prefix. (#2116)

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

Add check logic about schema name when loading table meta from driver. 

### Why are the changes needed?

Some drivers , such as PG drivers, contain schema name information, we
need to filter it. Some drivers like MySQL don't like it, we do not need
to check it.
check it. 

Fix: #2115 

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

N/A.

### How was this patch tested?

Add ITs: `testCreateSameTableInDifferentSchema`
yuqi1129 added a commit that referenced this issue Apr 23, 2024
…same name prefix. (#3134)

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

Add check logic about schema name when loading table meta from driver. 

### Why are the changes needed?

Some drivers , such as PG drivers, contain schema name information, we
need to filter it. Some drivers like MySQL don't like it, we do not need
to check it.
check it. 

Fix: #2115 

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

N/A.

### How was this patch tested?

Add ITs: `testCreateSameTableInDifferentSchema`

Co-authored-by: Qi Yu <[email protected]>
diqiu50 pushed a commit to diqiu50/gravitino that referenced this issue Jun 13, 2024
…h the same name prefix. (apache#2116)

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

Add check logic about schema name when loading table meta from driver. 

### Why are the changes needed?

Some drivers , such as PG drivers, contain schema name information, we
need to filter it. Some drivers like MySQL don't like it, we do not need
to check it.
check it. 

Fix: apache#2115 

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

N/A.

### How was this patch tested?

Add ITs: `testCreateSameTableInDifferentSchema`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants