Re: [PR] [#2869] feat(core): Add the core logic of authorization [gravitino]

2024-08-23 Thread via GitHub
jerqi closed pull request #3050: [#2869] feat(core): Add the core logic of authorization URL: https://github.com/apache/gravitino/pull/3050 -- 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 specif

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728562544 ## docs/iceberg-rest-service.md: ## @@ -215,6 +217,24 @@ You can access different catalogs by setting the `prefix` to the specific catalo ... ``` +# gravitino

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728562544 ## docs/iceberg-rest-service.md: ## @@ -215,6 +217,24 @@ You can access different catalogs by setting the `prefix` to the specific catalo ... ``` +# gravitino

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
theoryxu commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728566750 ## docs/iceberg-rest-service.md: ## @@ -178,6 +178,8 @@ The Gravitino Iceberg REST server supports multiple catalogs and offers a config |

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728568092 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergTableOpsManager.java: ## @@ -97,8 +126,20 @@ private String shelling(String ra

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728571604 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/provider/GravitinoBasedIcebergTableOpsProvider.java: ## @@ -0,0 +1,96 @@ +/* + * Licensed to

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
theoryxu commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728579287 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/provider/GravitinoBasedIcebergTableOpsProvider.java: ## @@ -0,0 +1,96 @@ +/* + * Licensed t

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
theoryxu commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728607986 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergTableOpsManager.java: ## @@ -97,8 +126,20 @@ private String shelling(String

[PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 opened a new pull request, #4652: URL: https://github.com/apache/gravitino/pull/4652 ### What changes were proposed in this pull request? Add the corresponding license for Javax Activation to `LICENSE.BIN` ### Why are the changes needed? To make it more c

(gravitino-playground) branch main updated: Change exposed ports to avoid possible conflicts, based on last pr (#67)

2024-08-23 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/gravitino-playground.git The following commit(s) were added to refs/heads/main by this push: new cfd29f3 Change exposed ports to avoid

Re: [PR] Change exposed ports to avoid possible conflicts, based on last pr [gravitino-playground]

2024-08-23 Thread via GitHub
FANNG1 merged PR #67: URL: https://github.com/apache/gravitino-playground/pull/67 -- 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...@gr

[I] [Bug report] Docker base image lacks `ps` command [gravitino]

2024-08-23 Thread via GitHub
jerqi opened a new issue, #4654: URL: https://github.com/apache/gravitino/issues/4654 ### Version main branch ### Describe what's wrong Docker base image lacks `ps` command ### Error message and/or stacktrace playground-gravitino | /root/gravitino/bin/grav

Re: [PR] [Bug] Avoid OOM (Out of Memory) for Playground-gravitino [gravitino-playground]

2024-08-23 Thread via GitHub
shaofengshi commented on PR #66: URL: https://github.com/apache/gravitino-playground/pull/66#issuecomment-2306664491 Hello Junbo, several questions: 1) In which case you will get the OOM error? 2) I see you add a "gravitino.env.sh" file, but didn't see where it will be called. Will i

Re: [I] [Bug report] Docker base image lacks `ps` command [gravitino]

2024-08-23 Thread via GitHub
jerqi commented on issue #4654: URL: https://github.com/apache/gravitino/issues/4654#issuecomment-2306669895 https://github.com/apache/gravitino/pull/4415 This pull request should backport to branch 0.6. -- This is an automated message from the Apache Git Service. To respond to the messag

[PR] [MINOR] Use Debian as gravitino basic image ,otherwise lack of basic tools like 'wget','apt-get' [gravitino]

2024-08-23 Thread via GitHub
jerqi opened a new pull request, #4655: URL: https://github.com/apache/gravitino/pull/4655 ### What changes were proposed in this pull request? Use the linux distribution include basic tools like wget ,apt-get as basic image The Debian linux distribution include basic tools like wg

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
jerqi commented on code in PR #4652: URL: https://github.com/apache/gravitino/pull/4652#discussion_r1728680834 ## LICENSE.bin: ## @@ -460,3 +460,8 @@ AOP Alliance XZ For Java + + This product bundles various third-party components also under the Review Comment: I

svn commit: r71064 - /dev/incubator/gravitino/KEYS

2024-08-23 Thread jshao
Author: jshao Date: Fri Aug 23 09:42:01 2024 New Revision: 71064 Log: Update KEYS Added: dev/incubator/gravitino/KEYS Added: dev/incubator/gravitino/KEYS == --- dev/incubator/gravitino/KEYS (added) +++ dev/incubator/

Re: [PR] [Improvement] Add port checks for MySQL and Postgres [gravitino-playground]

2024-08-23 Thread via GitHub
shaofengshi commented on PR #22: URL: https://github.com/apache/gravitino-playground/pull/22#issuecomment-2306712939 This is something similar as https://github.com/apache/gravitino-playground/issues/60, I propose to solve that by chaning to an less possible conflict port in local, which i

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 commented on code in PR #4652: URL: https://github.com/apache/gravitino/pull/4652#discussion_r1728694032 ## LICENSE.bin: ## @@ -460,3 +460,8 @@ AOP Alliance XZ For Java + + This product bundles various third-party components also under the Review Comment:

Re: [PR] [TEST]Test CI stability [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 closed pull request #4260: [TEST]Test CI stability URL: https://github.com/apache/gravitino/pull/4260 -- 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

svn commit: r71065 - /dev/incubator/gravitino/KEYS

2024-08-23 Thread jshao
Author: jshao Date: Fri Aug 23 09:46:37 2024 New Revision: 71065 Log: Update KEYS Modified: dev/incubator/gravitino/KEYS Modified: dev/incubator/gravitino/KEYS == --- dev/incubator/gravitino/KEYS (original) +++ dev/i

(gravitino) branch main updated: [MINOR] fix(build): Add gradle.kts to iceberg parent module (#4653)

2024-08-23 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/main by this push: new e997ad2ea [MINOR] fix(build): Add gradle.kts to ic

(gravitino) branch cherry-pick-branch-0.6-e997ad2eabbcf88dc4518cdd10d233e0bc56cad3 created (now f9ed1e9cb)

2024-08-23 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch cherry-pick-branch-0.6-e997ad2eabbcf88dc4518cdd10d233e0bc56cad3 in repository https://gitbox.apache.org/repos/asf/gravitino.git at f9ed1e9cb [MINOR] fix(build): Add gradle.kts to icebe

Re: [PR] [MINOR] fix(build): Add gradle.kts to iceberg parent module [gravitino]

2024-08-23 Thread via GitHub
FANNG1 closed pull request #4656: [MINOR] fix(build): Add gradle.kts to iceberg parent module URL: https://github.com/apache/gravitino/pull/4656 -- 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 s

Re: [PR] [MINOR] fix(build): Add gradle.kts to iceberg parent module [gravitino]

2024-08-23 Thread via GitHub
jerryshao closed pull request #4656: [MINOR] fix(build): Add gradle.kts to iceberg parent module URL: https://github.com/apache/gravitino/pull/4656 -- 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 th

Re: [PR] [MINOR] fix(build): Add gradle.kts to iceberg parent module [gravitino]

2024-08-23 Thread via GitHub
FANNG1 merged PR #4653: URL: https://github.com/apache/gravitino/pull/4653 -- 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...@gravitino

[I] [Bug report] Start the Gravitino Server failed in gravitino container [gravitino]

2024-08-23 Thread via GitHub
danhuawang opened a new issue, #4659: URL: https://github.com/apache/gravitino/issues/4659 ### Version main branch ### Describe what's wrong ``` Start the Gravitino Server Log dir doesn't exist, create /root/gravitino/logs Gravitino Server is not running

Re: [PR] [MINOR] Use Debian as gravitino basic image ,otherwise lack of basic tools like 'wget','apt-get' [gravitino]

2024-08-23 Thread via GitHub
jerryshao merged PR #4655: URL: https://github.com/apache/gravitino/pull/4655 -- 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

(gravitino) branch branch-0.6 updated: [MINOR] Use Debian as gravitino basic image ,otherwise lack of basic tools like 'wget','apt-get' (#4655)

2024-08-23 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a commit to branch branch-0.6 in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/branch-0.6 by this push: new dd7e55087 [MINOR] Use Debian as gravit

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
jerqi commented on code in PR #4652: URL: https://github.com/apache/gravitino/pull/4652#discussion_r1728769084 ## LICENSE.bin: ## @@ -460,3 +460,8 @@ AOP Alliance XZ For Java + + This product bundles various third-party components also under the Review Comment: I

Re: [PR] [MINOR]: fix authorization ranger jar conflict [gravitino]

2024-08-23 Thread via GitHub
jerqi commented on code in PR #4651: URL: https://github.com/apache/gravitino/pull/4651#discussion_r1728771087 ## LICENSE.bin: ## @@ -310,6 +310,8 @@ Apache Yetus - Audience Annotations Apache Kerby Apache Kyuubi + Apache Ranger + Apache Ranger intg Review Comme

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
jerryshao commented on code in PR #4652: URL: https://github.com/apache/gravitino/pull/4652#discussion_r1728773510 ## LICENSE.bin: ## @@ -460,3 +460,8 @@ AOP Alliance XZ For Java + + This product bundles various third-party components also under the Review Comment:

Re: [I] [Bug report] Docker base image lacks `ps` command [gravitino]

2024-08-23 Thread via GitHub
jerqi closed issue #4654: [Bug report] Docker base image lacks `ps` command URL: https://github.com/apache/gravitino/issues/4654 -- 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.

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 commented on code in PR #4652: URL: https://github.com/apache/gravitino/pull/4652#discussion_r1728813158 ## LICENSE.bin: ## @@ -460,3 +460,8 @@ AOP Alliance XZ For Java + + This product bundles various third-party components also under the Review Comment:

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 closed pull request #4652: [Minor] fix(license): Fix the license problem for Javax Activation URL: https://github.com/apache/gravitino/pull/4652 -- 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

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 commented on code in PR #4652: URL: https://github.com/apache/gravitino/pull/4652#discussion_r1728816986 ## LICENSE.bin: ## @@ -460,3 +460,8 @@ AOP Alliance XZ For Java + + This product bundles various third-party components also under the Review Comment:

Re: [PR] [Minor] fix(license): Fix the license problem for Javax Activation [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 commented on PR #4652: URL: https://github.com/apache/gravitino/pull/4652#issuecomment-2306886138 Not a problem -- 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

[I] [Improvement] The release jar should contains LICENSE and NOTICE [gravitino]

2024-08-23 Thread via GitHub
jerqi opened a new issue, #4661: URL: https://github.com/apache/gravitino/issues/4661 ### What would you like to be improved? The release jar should contains LICENSE and NOTICE Our jar is as below: ![12981724412057_ pic](https://github.com/user-attachments/assets/28718ef5-5bf3-

Re: [PR] [MINOR] fix(build): Add gradle.kts to iceberg parent module [gravitino]

2024-08-23 Thread via GitHub
jerryshao merged PR #4656: URL: https://github.com/apache/gravitino/pull/4656 -- 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

(gravitino) branch cherry-pick-branch-0.6-e997ad2eabbcf88dc4518cdd10d233e0bc56cad3 deleted (was f9ed1e9cb)

2024-08-23 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch cherry-pick-branch-0.6-e997ad2eabbcf88dc4518cdd10d233e0bc56cad3 in repository https://gitbox.apache.org/repos/asf/gravitino.git was f9ed1e9cb [MINOR] fix(build): Add gradle.kts to iceberg pa

(gravitino) branch branch-0.6 updated: [MINOR] fix(build): Add gradle.kts to iceberg parent module (#4656)

2024-08-23 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a commit to branch branch-0.6 in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/branch-0.6 by this push: new e841ee54e [MINOR] fix(build): Add grad

Re: [PR] [#4520] Imporvement(jdbc-catalogs): Shrink JDBC catalog binary package size to about 3MB. [gravitino]

2024-08-23 Thread via GitHub
jerryshao commented on code in PR #4521: URL: https://github.com/apache/gravitino/pull/4521#discussion_r1728859688 ## catalogs/catalog-jdbc-doris/build.gradle.kts: ## @@ -61,7 +65,11 @@ tasks { } val copyCatalogLibs by registering(Copy::class) { dependsOn("jar", "runt

Re: [PR] [#4520] Imporvement(jdbc-catalogs): Shrink JDBC catalog binary package size to about 3MB. [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 commented on code in PR #4521: URL: https://github.com/apache/gravitino/pull/4521#discussion_r1728862878 ## catalogs/catalog-jdbc-doris/build.gradle.kts: ## @@ -61,7 +65,11 @@ tasks { } val copyCatalogLibs by registering(Copy::class) { dependsOn("jar", "runti

Re: [PR] [#4520] Imporvement(jdbc-catalogs): Shrink JDBC catalog binary package size to about 3MB. [gravitino]

2024-08-23 Thread via GitHub
yuqi1129 commented on code in PR #4521: URL: https://github.com/apache/gravitino/pull/4521#discussion_r1728866167 ## catalogs/catalog-jdbc-doris/build.gradle.kts: ## @@ -61,7 +65,11 @@ tasks { } val copyCatalogLibs by registering(Copy::class) { dependsOn("jar", "runti

Re: [PR] [#4277] feat(client/server): Add the `getFileLocation` interface in the Java Client/Server [gravitino]

2024-08-23 Thread via GitHub
jerryshao commented on PR #4281: URL: https://github.com/apache/gravitino/pull/4281#issuecomment-2307010161 Hi @xloya This PR is a little big. I would suggest to split into several small PRs. For this PR, I think we can focus on the definition and skeleton code, what do you think? -- Thi

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728913692 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/provider/GravitinoBasedIcebergTableOpsProvider.java: ## @@ -0,0 +1,98 @@ +/* + * Licensed to

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728914097 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergTableOpsManager.java: ## @@ -97,8 +126,19 @@ private String shelling(String ra

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728914709 ## docs/iceberg-rest-service.md: ## @@ -215,6 +217,24 @@ You can access different catalogs by setting the `prefix` to the specific catalo ... ``` +# Gravitino

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728913692 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/provider/GravitinoBasedIcebergTableOpsProvider.java: ## @@ -0,0 +1,98 @@ +/* + * Licensed to

Re: [PR] [#4662] improve(IT): Add ranger authorization Hive E2E test [gravitino]

2024-08-23 Thread via GitHub
xunliu commented on code in PR #4651: URL: https://github.com/apache/gravitino/pull/4651#discussion_r1729077938 ## LICENSE.bin: ## @@ -310,6 +310,8 @@ Apache Yetus - Audience Annotations Apache Kerby Apache Kyuubi + Apache Ranger + Apache Ranger intg Review Comm

[PR] [#4208] improvement(client-python): Add Integration Test of OAuth2TokenProvider of client-python [gravitino]

2024-08-23 Thread via GitHub
noidname01 opened a new pull request, #4663: URL: https://github.com/apache/gravitino/pull/4663 ### What changes were proposed in this pull request? * Add integration test of `OAuth2TokenProvider`, following the steps mentioned in OAuth section of [`/docs/security/how-to-auth

[I] [Bug report] RoleOperation throw null exception [gravitino]

2024-08-23 Thread via GitHub
jerqi opened a new issue, #4664: URL: https://github.com/apache/gravitino/issues/4664 ### Version main branch ### Describe what's wrong ![Uploading 4401724425613_.pic_hd.jpg…]() ### Error message and/or stacktrace Call role interfaces, it will throw Null

[PR] [#4664] fix(server): Add the missing annotations fo access control interfaces [gravitino]

2024-08-23 Thread via GitHub
jerqi opened a new pull request, #4665: URL: https://github.com/apache/gravitino/pull/4665 ### What changes were proposed in this pull request? Add the missing annotations fo access control interfaces ### Why are the changes needed? Fix: #4664 ### Does this PR introduc

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-23 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1729666353 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergTableOpsManager.java: ## @@ -97,8 +126,19 @@ private String shelling(String ra