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

[FEATURE] Support using credential when using fileset with cloud storage in Java GVFS #5973

Closed
Tracked by #5677
yuqi1129 opened this issue Dec 24, 2024 · 0 comments · Fixed by #5974
Closed
Tracked by #5677

[FEATURE] Support using credential when using fileset with cloud storage in Java GVFS #5973

yuqi1129 opened this issue Dec 24, 2024 · 0 comments · Fixed by #5974
Assignees
@yuqi1129
Labels
0.8.0 Release v0.8.0 feature New feature or request

Comments

@yuqi1129
Copy link
Contributor

Describe the feature

Currently, Gravitino only supports static credentials like AKSK to fetch cloud storage, it's not very secure and elegant to use it like this, we need to provide a dynamic credential mechanism to obtain cloud storage.

Motivation

No response

Describe the solution

No response

Additional context

No response
@yuqi1129 yuqi1129 added the feature New feature or request label Dec 24, 2024
This was referenced Dec 24, 2024
Merged
Closed
@jerryshao jerryshao added the 0.8.0 Release v0.8.0 label Dec 30, 2024
@yuqi1129 yuqi1129 mentioned this issue Dec 30, 2024
Closed
2 tasks
@FANNG1 FANNG1 closed this as completed in 78447ce Jan 10, 2025
Abyss-lord pushed a commit to Abyss-lord/gravitino that referenced this issue Jan 10, 2025
@yuqi1129 @Abyss-lord
[apache#5973] feat(hadoop-catalog): Support credential when using fil…
2f81bdc 
…eset catalog with cloud storage (apache#5974)

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

Support dynamic credential in obtaining cloud storage fileset.

### Why are the changes needed?

Static key are not very safe, we need to optimize it. 

Fix: apache#5973 

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

N/A

### How was this patch tested?

ITs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuqi1129 yuqi1129

Labels
0.8.0 Release v0.8.0 feature New feature or request
Projects
None yet
Milestone
No milestone
2 participants
@jerryshao @yuqi1129