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

[Subtask] Make NameIdentifier more generic for better client use #3627

Closed
shaofengshi opened this issue May 29, 2024 · 0 comments · Fixed by #3696
Closed

[Subtask] Make NameIdentifier more generic for better client use #3627

shaofengshi opened this issue May 29, 2024 · 0 comments · Fixed by #3696
Assignees
@shaofengshi
Labels
subtask Subtasks of umbrella issue

Comments

@shaofengshi
Copy link
Contributor

Describe the subtask

See https://docs.google.com/document/d/1AXEXTDMyiXLpfMGPCH2q6mIeTDZq31p7O9GZQSueHkY/edit

Parent issue

#3626
@shaofengshi shaofengshi added the subtask Subtasks of umbrella issue label May 29, 2024
@shaofengshi shaofengshi self-assigned this May 29, 2024
This was referenced May 29, 2024
Closed
Merged
shaofengshi added a commit that referenced this issue Jun 4, 2024
@shaofengshi
[#3627] refactor(API): Make NameIdentifier more generic for better cl…
f973dbb 
…ient use (#3696)

### What changes were proposed in this pull request?
See
https://docs.google.com/document/d/1AXEXTDMyiXLpfMGPCH2q6mIeTDZq31p7O9GZQSueHkY/edit

### Why are the changes needed?

To make the API and java client more clear and easy to use.

Fix: #3627

### Does this PR introduce _any_ user-facing change?
See
https://docs.google.com/document/d/1AXEXTDMyiXLpfMGPCH2q6mIeTDZq31p7O9GZQSueHkY/edit

### How was this patch tested?

Purely refactoring, no adding new logics; Current test cases will cover
the change.
diqiu50 pushed a commit to diqiu50/gravitino that referenced this issue Jun 13, 2024
@shaofengshi @diqiu50
[apache#3627] refactor(API): Make NameIdentifier more generic for bet…
8ce338f 
…ter client use (apache#3696)

### What changes were proposed in this pull request?
See
https://docs.google.com/document/d/1AXEXTDMyiXLpfMGPCH2q6mIeTDZq31p7O9GZQSueHkY/edit

### Why are the changes needed?

To make the API and java client more clear and easy to use.

Fix: apache#3627

### Does this PR introduce _any_ user-facing change?
See
https://docs.google.com/document/d/1AXEXTDMyiXLpfMGPCH2q6mIeTDZq31p7O9GZQSueHkY/edit

### How was this patch tested?

Purely refactoring, no adding new logics; Current test cases will cover
the change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shaofengshi shaofengshi

Labels
subtask Subtasks of umbrella issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#3627] refactor(API): Make NameIdentifier more generic for better client use shaofengshi/gravitino
1 participant
@shaofengshi