Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao merged PR #4995: URL: https://github.com/apache/gravitino/pull/4995 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gravit

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1800306465 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,63 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1800302418 ## api/src/main/java/org/apache/gravitino/credential/CredentialConstants.java: ## @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
justinmclean commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799749768 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,63 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799605313 ## api/src/main/java/org/apache/gravitino/credential/CredentialConstants.java: ## @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799593989 ## api/src/main/java/org/apache/gravitino/credential/CredentialConstants.java: ## @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on PR #4995: URL: https://github.com/apache/gravitino/pull/4995#issuecomment-2410737834 @jerryshao comments are addressed, please help to review again -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799165801 ## core/src/main/java/org/apache/gravitino/credential/LocationContext.java: ## @@ -0,0 +1,57 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799165166 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,63 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799120113 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799104878 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799096933 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1798941527 ## core/src/main/java/org/apache/gravitino/credential/Context.java: ## @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1798925186 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,63 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1798876861 ## common/build.gradle.kts: ## @@ -28,6 +28,7 @@ plugins { dependencies { implementation(project(":api")) + implementation(project(":catalogs:catalog-common"

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1798910543 ## common/build.gradle.kts: ## @@ -28,6 +28,7 @@ plugins { dependencies { implementation(project(":api")) + implementation(project(":catalogs:catalog-common"))

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-14 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1798876861 ## common/build.gradle.kts: ## @@ -28,6 +28,7 @@ plugins { dependencies { implementation(project(":api")) + implementation(project(":catalogs:catalog-common"

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-09 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1793044758 ## core/src/main/java/org/apache/gravitino/credential/CredentialProvider.java: ## @@ -0,0 +1,58 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-09 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1793025519 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-09 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1793025519 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-09 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1793008835 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-09 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1792998137 ## common/build.gradle.kts: ## @@ -28,6 +28,7 @@ plugins { dependencies { implementation(project(":api")) + implementation(project(":catalogs:catalog-common"))

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-09 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1792973905 ## core/src/main/java/org/apache/gravitino/credential/CredentialProvider.java: ## @@ -0,0 +1,58 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-08 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1792973195 ## core/src/main/java/org/apache/gravitino/credential/CredentialProviderFactory.java: ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-08 Thread via GitHub
FANNG1 commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1792961793 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4992] support credential vending framework [gravitino]

2024-10-08 Thread via GitHub
jerryshao commented on code in PR #4995: URL: https://github.com/apache/gravitino/pull/4995#discussion_r1792745595 ## common/src/main/java/org/apache/gravitino/credential/Credential.java: ## @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

[PR] [#4992] support credential vending framework [gravitino]

2024-09-23 Thread via GitHub
FANNG1 opened a new pull request, #4995: URL: https://github.com/apache/gravitino/pull/4995 ### What changes were proposed in this pull request? support credential vending framework ### Why are the changes needed? Fix: #4992 ### Does this PR introduce _any_ user-faci