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] Lazily load the users/groups/roles #4608

Closed
jerqi opened this issue Aug 21, 2024 · 0 comments · Fixed by #4690
Closed

[Improvement] Lazily load the users/groups/roles #4608

jerqi opened this issue Aug 21, 2024 · 0 comments · Fixed by #4690
Assignees
@jerqi
Labels
0.7.0 Release v0.7.0 improvement Improvements on everything

Comments

@jerqi
Copy link
Contributor

jerqi commented Aug 21, 2024

What would you like to be improved?

For user/group/role entities, they contain some relational data. Some times these data won't be used. We should make these data lazily loaded.

How should we improve?

For user/group/role entities, they contain some relational data. Some times these data won't be used. We should make these data lazily loaded.
@jerqi jerqi added the improvement Improvements on everything label Aug 21, 2024
jerqi added a commit to qqqttt123/gravitino that referenced this issue Aug 26, 2024
@jerqi
[apache#4608] feat(core): Supports to load fields lazily
218beeb
@jerqi jerqi mentioned this issue Aug 26, 2024
Merged
jerqi added a commit to qqqttt123/gravitino that referenced this issue Aug 26, 2024
@jerqi
[apache#4608] feat(core): Supports to load fields lazily
e62e8a0
jerqi added a commit to qqqttt123/gravitino that referenced this issue Aug 26, 2024
@jerqi
[apache#4608] feat(core): Supports to load fields lazily
fb7cc8a
@jerryshao jerryshao added 0.6.1 0.7.0 Release v0.7.0 labels Sep 6, 2024
github-actions bot pushed a commit that referenced this issue Sep 6, 2024
@jerqi @web-flow
[#4608] feat(core): Supports to load fields lazily (#4690)
289d5bd 
### What changes were proposed in this pull request?

Supports to load fields lazily.

### Why are the changes needed?

Fix: #4608

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

### How was this patch tested?
Add new uts and pass the existing uts
@github-actions github-actions bot mentioned this issue Sep 6, 2024
Merged
jerryshao added a commit that referenced this issue Sep 8, 2024
@github-actions @jerqi @jerryshao
[#4608] feat(core): Supports to load fields lazily (#4875)
ac8c69a 
### What changes were proposed in this pull request?

Supports to load fields lazily.

### Why are the changes needed?

Fix: #4608

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

### How was this patch tested?
Add new uts and pass the existing uts

Co-authored-by: roryqi <[email protected]>
Co-authored-by: Jerry Shao <[email protected]>
jerqi added a commit to qqqttt123/gravitino that referenced this issue Sep 14, 2024
@jerqi
Revert "[apache#4608] feat(core): Supports to load fields lazily (apa…
7ef9daf 
…che#4690)"

This reverts commit 3ba729f.
jerryshao pushed a commit that referenced this issue Sep 18, 2024
@jerqi
Revert "[#4608] feat(core): Supports to load fields lazily" (#4945)
1debca6 
Reverts #4690
github-actions bot pushed a commit that referenced this issue Sep 18, 2024
@jerqi @web-flow
Revert "[#4608] feat(core): Supports to load fields lazily" (#4945)
0a215ca 
Reverts #4690
jerryshao pushed a commit that referenced this issue Sep 18, 2024
@github-actions @jerqi
Revert "[#4608] feat(core): Supports to load fields lazily" (#4950)
bcd6f80 
Reverts #4690

Co-authored-by: roryqi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jerqi jerqi

Labels
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.

[#4608] feat(core): Supports to load fields lazily qqqttt123/gravitino
2 participants
@jerryshao @jerqi