Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
mchades commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776649647 ## catalogs/catalog-jdbc-common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcDatabaseOperations.java: ## @@ -129,4 +137,13 @@ protected Connection ge

Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
yuanoOo commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776641546 ## catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java: ## @@ -45,17 +55,62 @@ private sta

Re: [PR] [#4959] feat(Iceberg): support Iceberg REST extend API [gravitino]

2024-09-26 Thread via GitHub
FANNG1 commented on PR #4987: URL: https://github.com/apache/gravitino/pull/4987#issuecomment-2376191497 Add a test to produce "hello" message, please help to review this when you are free. @SinghAsDev @puchengy @jerryshao -- This is an automated message from the Apache Git Service. To

Re: [PR] [#3973] improvement(core):support audit log [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #4575: URL: https://github.com/apache/gravitino/pull/4575#discussion_r1776573289 ## core/src/main/java/org/apache/gravitino/audit/AbstractAuditLogWriter.java: ## @@ -0,0 +1,44 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#3973] improvement(core):support audit log [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #4575: URL: https://github.com/apache/gravitino/pull/4575#discussion_r1776572132 ## core/src/main/java/org/apache/gravitino/audit/AuditLogWriter.java: ## @@ -0,0 +1,44 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776606475 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalak

Re: [PR] [#4089] fix(hive catalog): the problem of slow acquisition of hive table list [gravitino]

2024-09-26 Thread via GitHub
mygrsun commented on PR #4469: URL: https://github.com/apache/gravitino/pull/4469#issuecomment-237629 > @mygrsun Is there any progress? May I take on this? thanks,you can do it -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776605536 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalak

Re: [PR] [#4089] fix(hive catalog): the problem of slow acquisition of hive table list [gravitino]

2024-09-26 Thread via GitHub
mchades commented on PR #4469: URL: https://github.com/apache/gravitino/pull/4469#issuecomment-2376279251 @mygrsun Is there any progress? May I take on this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] [#3475] fix(all): Fix the problem of multiple version jars in the release tar. [gravitino]

2024-09-26 Thread via GitHub
jerryshao closed pull request #3476: [#3475] fix(all): Fix the problem of multiple version jars in the release tar. URL: https://github.com/apache/gravitino/pull/3476 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776681324 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalakeExc

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776681324 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalakeExc

Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
yuanoOo commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776700382 ## catalogs/catalog-jdbc-common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcDatabaseOperations.java: ## @@ -129,4 +137,13 @@ protected Connection ge

Re: [PR] fix: support check for `docker compose` [gravitino-playground]

2024-09-26 Thread via GitHub
xunliu commented on PR #79: URL: https://github.com/apache/gravitino-playground/pull/79#issuecomment-2376413068 hi @kahnwong Thank you for your contribution to this PR. @unknowntpo Please help review this PR If you have time, Thanks. -- This is an automated message from the Apache Git

Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
yuanoOo commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776701225 ## catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java: ## @@ -44,22 +52,65 @@ private sta

Re: [PR] [#4280] improvement(PyGVFS): Refactor the `getFileLocation` logic in the Python GVFS [gravitino]

2024-09-26 Thread via GitHub
xloya commented on PR #5026: URL: https://github.com/apache/gravitino/pull/5026#issuecomment-2376433796 @jerryshao This PR is ready for review, please take a look when you have time, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] [#4952] feat(hudi-catalog): add implementation of HMSBackend for Hudi catalog [gravitino]

2024-09-26 Thread via GitHub
mchades commented on code in PR #4942: URL: https://github.com/apache/gravitino/pull/4942#discussion_r1776753445 ## catalogs/catalog-lakehouse-hudi/src/main/java/org/apache/gravitino/catalog/lakehouse/hudi/backend/hms/HudiHMSTable.java: ## @@ -44,15 +49,24 @@ protected HudiHMSTa

Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
mchades commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776763918 ## catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java: ## @@ -41,25 +47,56 @@ private sta

[PR] [#5000] improvement(server): Move tag object API to object path [gravitino]

2024-09-26 Thread via GitHub
jerqi opened a new pull request, #5027: URL: https://github.com/apache/gravitino/pull/5027 ### What changes were proposed in this pull request? Move tag related about metadata object API to object path ### Why are the changes needed? Fix: #5000 ### Does this PR int

Re: [PR] [#5000] improvement(server): Move tag object API to object path [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on PR #5027: URL: https://github.com/apache/gravitino/pull/5027#issuecomment-2376550338 cc @LauraXia123 -- 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

Re: [PR] [#4370]feat(iceberg): support view interface for Iceberg REST server [gravitino]

2024-09-26 Thread via GitHub
theoryxu commented on PR #4937: URL: https://github.com/apache/gravitino/pull/4937#issuecomment-2376544168 @FANNG1 Integration tests have been added. Please review. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] [#4759] feat(auth-ranger): Use Spark verify Ranger authorization Hive [gravitino]

2024-09-26 Thread via GitHub
xunliu commented on PR #4948: URL: https://github.com/apache/gravitino/pull/4948#issuecomment-2376543059 @FANNG1 @mchades @yuqi1129 I refactored this PR, Please help me review it again, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776493450 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalakeExc

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776494602 ## core/src/main/java/org/apache/gravitino/SupportsRelationOperations.java: ## @@ -48,8 +48,27 @@ enum Type { * @return The list of entities * @throws IOExcepti

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776496238 ## server/src/main/java/org/apache/gravitino/server/web/rest/ObjectRoleOperations.java: ## @@ -0,0 +1,85 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

Re: [I] [Improvement] Make property `cloud.name` and `cloud.region-code` mutable [gravitino]

2024-09-26 Thread via GitHub
yuqi1129 closed issue #4999: [Improvement] Make property `cloud.name` and `cloud.region-code` mutable URL: https://github.com/apache/gravitino/issues/4999 -- 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 g

(gravitino) branch main updated: [#4999] feat(catalog): make `cloud.name` and `cloud.region-code` mutable. (#5025)

2024-09-26 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 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 c64ded670 [#4999] feat(catalog): make `cloud.na

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776498393 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalak

Re: [PR] [#4999] feat(catalog): make `cloud.name` and `cloud.region-code` mutable. [gravitino]

2024-09-26 Thread via GitHub
yuqi1129 merged PR #5025: URL: https://github.com/apache/gravitino/pull/5025 -- 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...@graviti

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776500444 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -93,49 +78,41 @@ public List listRolesByUserId(Long userId) { }

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776501042 ## core/src/main/java/org/apache/gravitino/storage/relational/JDBCBackend.java: ## @@ -382,20 +380,23 @@ public List listEntitiesByRelation( case METADATA_OBJE

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776501042 ## core/src/main/java/org/apache/gravitino/storage/relational/JDBCBackend.java: ## @@ -382,20 +380,23 @@ public List listEntitiesByRelation( case METADATA_OBJE

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776501042 ## core/src/main/java/org/apache/gravitino/storage/relational/JDBCBackend.java: ## @@ -382,20 +380,23 @@ public List listEntitiesByRelation( case METADATA_OBJE

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r177650 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalakeExc

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776521762 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalakeExc

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776521762 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalakeExc

Re: [PR] [#5019] feat: (hadoop-catalog): Add a framework to support multi-storage in a pluginized manner for fileset catalog [gravitino]

2024-09-26 Thread via GitHub
xiaozcy commented on code in PR #5020: URL: https://github.com/apache/gravitino/pull/5020#discussion_r1776535995 ## catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/DefaultConfigurationProvider.java: ## @@ -0,0 +1,31 @@ +/* + * Licensed to the Apache Sof

[PR] [#4280] improvement(PyGVFS): Refactor the `getFileLocation` logic in the Python GVFS [gravitino]

2024-09-26 Thread via GitHub
xloya opened a new pull request, #5026: URL: https://github.com/apache/gravitino/pull/5026 ### What changes were proposed in this pull request? Refactor the logic of getting the file location in Python GVFS. ### Why are the changes needed? Fix: #4280 ### How was t

Re: [PR] [#4959] feat(Iceberg): support Iceberg REST extend API [gravitino]

2024-09-26 Thread via GitHub
FANNG1 commented on code in PR #4987: URL: https://github.com/apache/gravitino/pull/4987#discussion_r1776557605 ## iceberg/iceberg-rest-server/src/test/java/org/apache/gravitino/iceberg/service/extension/HelloOperations.java: ## @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache S

Re: [PR] [#4886] feat(server, core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776558271 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -369,6 +369,20 @@ public String[] listRoleNames() throws NoSuchMetalakeExc

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777037915 ## clients/client-java/src/main/java/org/apache/gravitino/client/MetadataObjectRoleOperations.java: ## @@ -0,0 +1,56 @@ +/* + * Licensed to the Apache Software Foundati

Re: [PR] #4873 support list group [gravitino]

2024-09-26 Thread via GitHub
LiuQhahah commented on PR #4879: URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2376870233 > @LiuQhahah Do you have time to continue this work? Because we want to release this feature in 0.6.1. The 0.6.1 will be released at the end of this month. Hi @jerqi I'll s

Re: [PR] #4873 support list group [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on PR #4879: URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2376883467 > > @LiuQhahah Do you have time to continue this work? Because we want to release this feature in 0.6.1. The 0.6.1 will be released at the end of this month. > > Hi @jerqi I'll sub

Re: [PR] #4873 support list group [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #4879: URL: https://github.com/apache/gravitino/pull/4879#discussion_r1777020822 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -227,6 +227,15 @@ public Group getGroup(String group) throws NoSuchGroupEx

Re: [PR] #4873 support list group [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #4879: URL: https://github.com/apache/gravitino/pull/4879#discussion_r1777020822 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -227,6 +227,15 @@ public Group getGroup(String group) throws NoSuchGroupEx

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777027453 ## clients/client-java/src/main/java/org/apache/gravitino/client/MetadataObjectRoleOperations.java: ## @@ -0,0 +1,56 @@ +/* + * Licensed to the Apache Software Foun

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777083831 ## api/src/main/java/org/apache/gravitino/authorization/SupportsRoles.java: ## @@ -0,0 +1,38 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777082615 ## server/src/main/java/org/apache/gravitino/server/web/rest/MetadataObjectRoleOperations.java: ## @@ -0,0 +1,85 @@ +/* + * Licensed to the Apache Software Foundation (

Re: [PR] #4873 support list group [gravitino]

2024-09-26 Thread via GitHub
LiuQhahah commented on code in PR #4879: URL: https://github.com/apache/gravitino/pull/4879#discussion_r1777059715 ## clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java: ## @@ -227,6 +227,15 @@ public Group getGroup(String group) throws NoSuchGro

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777058398 ## core/src/main/java/org/apache/gravitino/storage/relational/JDBCBackend.java: ## @@ -382,20 +380,23 @@ public List listEntitiesByRelation( case METADATA_OBJE

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777063231 ## server/src/main/java/org/apache/gravitino/server/web/rest/MetadataObjectRoleOperations.java: ## @@ -0,0 +1,85 @@ +/* + * Licensed to the Apache Software Foundati

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
yuqi1129 commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777056482 ## api/src/main/java/org/apache/gravitino/authorization/SupportsRoles.java: ## @@ -0,0 +1,38 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777096825 ## api/src/main/java/org/apache/gravitino/authorization/SupportsRoles.java: ## @@ -0,0 +1,38 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777099255 ## server/src/main/java/org/apache/gravitino/server/web/rest/MetadataObjectRoleOperations.java: ## @@ -0,0 +1,85 @@ +/* + * Licensed to the Apache Software Foundation (

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777093899 ## api/src/main/java/org/apache/gravitino/authorization/SupportsRoles.java: ## @@ -0,0 +1,38 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * o

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1777050988 ## core/src/main/java/org/apache/gravitino/storage/relational/JDBCBackend.java: ## @@ -382,20 +380,23 @@ public List listEntitiesByRelation( case METADATA_

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776896523 ## clients/client-java/src/test/java/org/apache/gravitino/client/integration/test/authorization/AccessControlIT.java: ## @@ -184,6 +184,12 @@ void testManageRoles()

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776950149 ## clients/client-java/src/test/java/org/apache/gravitino/client/integration/test/authorization/AccessControlIT.java: ## @@ -184,6 +184,12 @@ void testManageRoles() {

Re: [PR] #4873 support list group [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on PR #4879: URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2377040829 @LiuQhahah Could you give the permission of your repo? We are eager to finish the feature before 0.6.1. -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] #4873 support list group [gravitino]

2024-09-26 Thread via GitHub
LiuQhahah commented on PR #4879: URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2377059023 > @LiuQhahah Could you give the permission of your repo? I can help you finish this pull request. We are eager to finish the feature before 0.6.1. Okay, thanks. https

Re: [PR] [#4759] feat(auth-ranger): Use Spark verify Ranger authorization Hive [gravitino]

2024-09-26 Thread via GitHub
yuqi1129 commented on code in PR #4948: URL: https://github.com/apache/gravitino/pull/4948#discussion_r1777133895 ## authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerITEnv.java: ## @@ -106,11 +108,30 @@ public sta

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1778075194 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -291,11 +259,49 @@ public int deleteRoleMetasByLegacyTimeline(long le

Re: [I] [Improvement] Support more HCFS-implemented storages in the Gravitino Virtual File System [gravitino]

2024-09-26 Thread via GitHub
xloya closed issue #2310: [Improvement] Support more HCFS-implemented storages in the Gravitino Virtual File System URL: https://github.com/apache/gravitino/issues/2310 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1778074275 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -291,11 +259,49 @@ public int deleteRoleMetasByLegacyTimeline(long le

Re: [PR] fix(authorization-ranger):the descr field of ranger table x_group is not null [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on PR #4975: URL: https://github.com/apache/gravitino/pull/4975#issuecomment-2378493833 @yangyuxia would you please fix the conflicts here, we will merge it to 0.6.1 release. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [#5000] improvement(server): Move tag object API to object path [gravitino]

2024-09-26 Thread via GitHub
jerryshao merged PR #5027: URL: https://github.com/apache/gravitino/pull/5027 -- 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: [I] [Improvement] Make tag interface more proper [gravitino]

2024-09-26 Thread via GitHub
jerryshao closed issue #5000: [Improvement] Make tag interface more proper URL: https://github.com/apache/gravitino/issues/5000 -- 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.

[PR] [#5000] improvement(server): Move tag object API to object path [gravitino]

2024-09-26 Thread via GitHub
github-actions[bot] opened a new pull request, #5028: URL: https://github.com/apache/gravitino/pull/5028 ### What changes were proposed in this pull request? Move tag related about metadata object API to object path ### Why are the changes needed? Fix: #5000 ### Do

(gravitino) branch cherry-pick-branch-0.6-8c62fc5f67ad74020e132fffb7c45c8fb204584e created (now da6cccde8)

2024-09-26 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-8c62fc5f67ad74020e132fffb7c45c8fb204584e in repository https://gitbox.apache.org/repos/asf/gravitino.git at da6cccde8 [#5000] improvement(server): Move tag objec

(gravitino) branch main updated: [#5000] improvement(server): Move tag object API to object path (#5027)

2024-09-26 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao 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 8c62fc5f6 [#5000] improvement(server): Move tag ob

Re: [I] [Improvement] Make tag interface more proper [gravitino]

2024-09-26 Thread via GitHub
jerryshao closed issue #5000: [Improvement] Make tag interface more proper URL: https://github.com/apache/gravitino/issues/5000 -- 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] [#5005] improvement(docs): Add a document about how to create Hive S3 tables through Gravitino. [gravitino]

2024-09-26 Thread via GitHub
yuqi1129 commented on code in PR #5006: URL: https://github.com/apache/gravitino/pull/5006#discussion_r1777902700 ## docs/hive-catalog-with-s3.md: ## @@ -0,0 +1,210 @@ +--- +title: "Hive catalog with s3" +slug: /hive-catalog +date: 2024-9-24 +keyword: Hive catalog cloud storage

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1778053980 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -109,35 +94,18 @@ public List listRolesByMetadataObjectIdentAndTy

Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
yuanoOo commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1778054955 ## catalogs/catalog-jdbc-common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcDatabaseOperations.java: ## @@ -114,8 +115,15 @@ protected void dropData

Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
yuanoOo commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1778055729 ## catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java: ## @@ -41,25 +47,56 @@ private sta

Re: [PR] feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. [gravitino]

2024-09-26 Thread via GitHub
yuanoOo commented on code in PR #5013: URL: https://github.com/apache/gravitino/pull/5013#discussion_r1778055982 ## catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java: ## @@ -41,25 +47,56 @@ private sta

Re: [PR] [#5000] improvement(server): Move tag object API to object path [gravitino]

2024-09-26 Thread via GitHub
jerryshao closed pull request #5028: [#5000] improvement(server): Move tag object API to object path URL: https://github.com/apache/gravitino/pull/5028 -- 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 t

Re: [PR] [#5005] improvement(docs): Add a document about how to create Hive S3 tables through Gravitino. [gravitino]

2024-09-26 Thread via GitHub
yuqi1129 commented on code in PR #5006: URL: https://github.com/apache/gravitino/pull/5006#discussion_r1777927452 ## docs/hive-catalog-with-s3.md: ## @@ -0,0 +1,210 @@ +--- +title: "Hive catalog with s3" +slug: /hive-catalog +date: 2024-9-24 +keyword: Hive catalog cloud storage

Re: [PR] [#5000] improvement(server): Move tag object API to object path [gravitino]

2024-09-26 Thread via GitHub
jerryshao merged PR #5028: URL: https://github.com/apache/gravitino/pull/5028 -- 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] [#4952] feat(hudi-catalog): add implementation of HMSBackend for Hudi catalog [gravitino]

2024-09-26 Thread via GitHub
mchades commented on PR #4942: URL: https://github.com/apache/gravitino/pull/4942#issuecomment-2378416366 All comments resolved, plz help to review again when you have time, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1778061846 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -291,11 +259,49 @@ public int deleteRoleMetasByLegacyTimeline(long

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerryshao commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1778057589 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -291,11 +259,49 @@ public int deleteRoleMetasByLegacyTimeline(long

[I] [Improvement] When we delete the entity, we should delete the relation between entity and tag or role [gravitino]

2024-09-26 Thread via GitHub
jerqi opened a new issue, #5029: URL: https://github.com/apache/gravitino/issues/5029 ### What would you like to be improved? When we delete the entity, we should delete the relation between entity and tag or role ### How should we improve? When we delete the entity, we

[PR] [#3758] Improvement(PyGVFS): Support Oauth2 authentication in Python GVFS [gravitino]

2024-09-26 Thread via GitHub
xloya opened a new pull request, #5030: URL: https://github.com/apache/gravitino/pull/5030 ### What changes were proposed in this pull request? Support Oauth2 authentication in PyGVFS so that it can interact with the Gravitino server. This is supported in Java GVFS and we should also

[I] [EPIC] Unify drop metalake and catalog behavior [gravitino]

2024-09-26 Thread via GitHub
mchades opened a new issue, #5031: URL: https://github.com/apache/gravitino/issues/5031 ### Describe the proposal metalake and catalog are completely managed by Gravitino, we need to clarify the drop semantics and behavior for metalake/catalog. This issues is uesed to track rel

Re: [PR] [#4886] feat(server,core): Supports to list roles by object [gravitino]

2024-09-26 Thread via GitHub
jerqi commented on code in PR #5023: URL: https://github.com/apache/gravitino/pull/5023#discussion_r1778068870 ## core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java: ## @@ -109,35 +94,18 @@ public List listRolesByMetadataObjectIdentAndType(

(gravitino) branch main updated: [#4759] feat(auth-ranger): Use Spark verify Ranger authorization Hive (#4948)

2024-09-26 Thread liuxun
This is an automated email from the ASF dual-hosted git repository. liuxun 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 46a6a20cf [#4759] feat(auth-ranger): Use Spark ve

Re: [PR] [#4759] feat(auth-ranger): Use Spark verify Ranger authorization Hive [gravitino]

2024-09-26 Thread via GitHub
xunliu commented on code in PR #4948: URL: https://github.com/apache/gravitino/pull/4948#discussion_r1777889680 ## authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerITEnv.java: ## @@ -106,11 +108,30 @@ public stati

Re: [PR] [#4759] feat(auth-ranger): Use Spark verify Ranger authorization Hive [gravitino]

2024-09-26 Thread via GitHub
xunliu merged PR #4948: URL: https://github.com/apache/gravitino/pull/4948 -- 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

Re: [I] [Subtask] Use Spark verify Ranger authorization Hive [gravitino]

2024-09-26 Thread via GitHub
xunliu closed issue #4759: [Subtask] Use Spark verify Ranger authorization Hive URL: https://github.com/apache/gravitino/issues/4759 -- 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 comme

Re: [I] [Subtask] Use Spark verify Ranger authorization Hive [gravitino]

2024-09-26 Thread via GitHub
xunliu closed issue #4759: [Subtask] Use Spark verify Ranger authorization Hive URL: https://github.com/apache/gravitino/issues/4759 -- 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 comme

Re: [PR] [#4545] improvement(paimon-catalog): reduce catalog-lakehouse-paimon libs size from 222MB to 75MB [gravitino]

2024-09-26 Thread via GitHub
caican00 commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2378331488 > > > > Why the paimon format jar is so big, is it correct? > > > > > > > > > Paimon format jar packages the dependences in the jar, @caican00 do you know any other alternati

Re: [PR] [#5005] improvement(docs): Add a document about how to create Hive S3 tables through Gravitino. [gravitino]

2024-09-26 Thread via GitHub
FANNG1 commented on code in PR #5006: URL: https://github.com/apache/gravitino/pull/5006#discussion_r1777887353 ## docs/hive-catalog-with-s3.md: ## @@ -0,0 +1,210 @@ +--- +title: "Hive catalog with s3" +slug: /hive-catalog +date: 2024-9-24 +keyword: Hive catalog cloud storage S3

Re: [PR] [#5005] improvement(docs): Add a document about how to create Hive S3 tables through Gravitino. [gravitino]

2024-09-26 Thread via GitHub
FANNG1 commented on code in PR #5006: URL: https://github.com/apache/gravitino/pull/5006#discussion_r1777888636 ## docs/hive-catalog-with-s3.md: ## @@ -0,0 +1,210 @@ +--- +title: "Hive catalog with s3" +slug: /hive-catalog +date: 2024-9-24 +keyword: Hive catalog cloud storage S3