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

[Improvement] Add check for metadata object full name #5070

Closed
jerqi opened this issue Oct 9, 2024 · 0 comments · Fixed by #5075
Closed

[Improvement] Add check for metadata object full name #5070

jerqi opened this issue Oct 9, 2024 · 0 comments · Fixed by #5075
Assignees
@jerqi
Labels
0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 improvement Improvements on everything

Comments

@jerqi
Copy link
Collaborator

jerqi commented Oct 9, 2024

What would you like to be improved?

Add check for metadata object full name. For example, a fullName catalog1.schema1 can't be a table. So we should check it.

How should we improve?

Add check for metadata object full name. For example, a fullName catalog1.schema1 can't be a table. So we should check it.
@jerqi jerqi added the improvement Improvements on everything label Oct 9, 2024
jerqi added a commit to qqqttt123/gravitino that referenced this issue Oct 9, 2024
@jerqi
[apache#5070] improvement(core): Add check for full name of the metad…
ca25609 
…ata object
@jerqi jerqi mentioned this issue Oct 9, 2024
Merged
jerqi added a commit to qqqttt123/gravitino that referenced this issue Oct 9, 2024
@jerqi
[apache#5070] improvement(core): Add check for full name of the metad…
aef1706 
…ata object
@jerqi jerqi self-assigned this Oct 10, 2024
@jerqi jerqi added the 0.7.0 Release v0.7.0 label Oct 10, 2024
@xunliu xunliu closed this as completed in 630cf67 Oct 10, 2024
github-actions bot pushed a commit that referenced this issue Oct 10, 2024
@jerqi @web-flow
[#5070] improvement(core): Add check for the full name of the metadat…
1d5a31a 
…a object (#5075)

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

Add check for full name of the metadata object.

### Why are the changes needed?

Fix: #5070 

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

No.

### How was this patch tested?

Add UTs.
@github-actions github-actions bot mentioned this issue Oct 10, 2024
Merged
@jerqi jerqi added the 0.6.1 Release v0.6.1 label Oct 10, 2024
jerryshao pushed a commit that referenced this issue Oct 11, 2024
@github-actions @jerqi
[#5070] improvement(core): Add check for the full name of the metadat…
c5c103d 
…a object (#5091)

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

Add check for full name of the metadata object.

### Why are the changes needed?

Fix: #5070 

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

No.

### How was this patch tested?

Add UTs.

Co-authored-by: roryqi <[email protected]>
jerqi added a commit to qqqttt123/gravitino that referenced this issue Oct 11, 2024
@jerqi
[apache#5070]
8e8ebf5
@jerqi jerqi mentioned this issue Oct 11, 2024
Merged
jerqi added a commit to qqqttt123/gravitino that referenced this issue Oct 11, 2024
@jerqi
[apache#5070]
91c5869
jerqi added a commit to qqqttt123/gravitino that referenced this issue Oct 11, 2024
@jerqi
[apache#5070]
9f5becc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jerqi jerqi

Labels
0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 improvement Improvements on everything
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#5070] improvement(core): Add check for the full name of the metadata object qqqttt123/gravitino
1 participant
@jerqi