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

[#4599][#4809] feat(core): Add tag support for column #5186

Merged
merged 9 commits into from
Oct 25, 2024

Conversation

jerryshao
Copy link
Contributor

@jerryshao jerryshao commented Oct 21, 2024

What changes were proposed in this pull request?

This PR proposes to add tag support for columns.

Why are the changes needed?

With this PR, user could add and remove tags in the column level.

Fix: #4599
Fix: #4809

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Add new UT and IT to test.

@jerryshao jerryshao self-assigned this Oct 21, 2024
@jerryshao jerryshao marked this pull request as ready for review October 21, 2024 07:32
@jerryshao jerryshao changed the title [#4599] feat(core): Add tag support for column [#4599][#4809] feat(core): Add tag support for column Oct 22, 2024
@jerryshao jerryshao added need backport Issues that need to backport to another branch branch-0.7 Automatically cherry-pick commit to branch-0.7 labels Oct 24, 2024
@mchades
Copy link
Contributor

mchades commented Oct 24, 2024

Overall LGTM.

Copy link
Contributor

@mchades mchades left a comment

Choose a reason for hiding this comment

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

@yuqi1129 @jerqi Do you want to take another look?

@jerryshao jerryshao merged commit a89a1ca into apache:main Oct 25, 2024
26 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 25, 2024
### What changes were proposed in this pull request?

This PR proposes to add tag support for columns.

### Why are the changes needed?

With this PR, user could add and remove tags in the column level.

Fix: #4599 
Fix: #4809 

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

No.

### How was this patch tested?

Add new UT and IT to test.

---------

Co-authored-by: Qi Yu <[email protected]>
mplmoknijb pushed a commit to mplmoknijb/gravitino that referenced this pull request Nov 6, 2024
…ache#5186)

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

This PR proposes to add tag support for columns.

### Why are the changes needed?

With this PR, user could add and remove tags in the column level.

Fix: apache#4599 
Fix: apache#4809 

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

No.

### How was this patch tested?

Add new UT and IT to test.

---------

Co-authored-by: Qi Yu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-0.7 Automatically cherry-pick commit to branch-0.7 need backport Issues that need to backport to another branch
Projects
None yet
3 participants