[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

Re: [I] [Bug report] federation query using 2 hive metastores does not work when using gravitino [gravitino]

2024-09-23 Thread via GitHub
foryou7242 commented on issue #4932: URL: https://github.com/apache/gravitino/issues/4932#issuecomment-2367645442 @FANNG1 the above query results in the following error ```shell spark-sql (test)> create table a(a int) location 'hdfs://test1/warehouse/tablespace/external/hi

Re: [PR] [#4988] fix(doris-catalog): Fix the missing distribution information when loading Doris tables [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 commented on PR #4991: URL: https://github.com/apache/gravitino/pull/4991#issuecomment-2367721037 This PR is not ready for review until #4985 is merged. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771157205 ## core/src/main/java/org/apache/gravitino/storage/relational/service/SupportsSkippingFieldsHandlers.java: ## @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache Software F

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771162002 ## core/src/main/java/org/apache/gravitino/storage/relational/po/CombinedUserPO.java: ## @@ -0,0 +1,60 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
xunliu commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771110357 ## core/src/main/java/org/apache/gravitino/storage/relational/po/CombinedUserPO.java: ## @@ -0,0 +1,60 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771160198 ## core/src/main/java/org/apache/gravitino/storage/relational/service/UserMetaService.java: ## @@ -265,4 +281,60 @@ public int deleteUserMetasByLegacyTimeline(long leg

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771160198 ## core/src/main/java/org/apache/gravitino/storage/relational/service/UserMetaService.java: ## @@ -265,4 +281,60 @@ public int deleteUserMetasByLegacyTimeline(long leg

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
xunliu commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771168800 ## core/src/main/java/org/apache/gravitino/storage/relational/service/SupportsSkippingFieldsHandlers.java: ## @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache Software

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771162002 ## core/src/main/java/org/apache/gravitino/storage/relational/po/CombinedUserPO.java: ## @@ -0,0 +1,60 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [Minor] Update NOTICE [gravitino]

2024-09-23 Thread via GitHub
jerryshao merged PR #4989: URL: https://github.com/apache/gravitino/pull/4989 -- 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] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771003380 ## core/src/main/java/org/apache/gravitino/EntityStore.java: ## @@ -55,15 +56,40 @@ public interface EntityStore extends Closeable { * Note. Depends on the isola

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771006616 ## core/src/main/java/org/apache/gravitino/authorization/UserGroupManager.java: ## @@ -109,6 +115,35 @@ User getUser(String metalake, String user) throws NoSuchUserExc

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

2024-09-23 Thread via GitHub
yuqi1129 commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2367953706 > 2.9M distribution/package/catalogs/lakehouse-paimon/libs/paimon-shade-guava-30-30.1.1-jre-0.8.0.jar Can we remove these? 2.9M distribution/package/catalogs/lakehouse-

Re: [I] [Bug report] federation query using 2 hive metastores does not work when using gravitino [gravitino]

2024-09-23 Thread via GitHub
FANNG1 commented on issue #4932: URL: https://github.com/apache/gravitino/issues/4932#issuecomment-2367958064 You should set `spark.bypass.spark.sql.hive.metastore.jars` explicitly in Hive catalog properties if using Spark SQL client, or use Spark shell instead. -- This is an automated m

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771016853 ## core/src/main/java/org/apache/gravitino/EntityStore.java: ## @@ -55,15 +56,40 @@ public interface EntityStore extends Closeable { * Note. Depends on the isolatio

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771188612 ## core/src/main/java/org/apache/gravitino/storage/relational/service/SupportsSkippingFieldsHandlers.java: ## @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache Software F

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

2024-09-23 Thread via GitHub
LiuQhahah commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2367870669 > @LiuQhahah can you please list all the jars and paste here on Github for Paimon catalog? You can use `./gradlew compileDistribution` to get a distribution, then check the paimon cat

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

2024-09-23 Thread via GitHub
FANNG1 commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2367882298 Hi, @LiuQhahah , I did some extra work to exclude `yarn` `zookeeper` `curator` jars, please help to review, cc @jerryshao @caican00 ``` du -sh distribution/package/catalogs/l

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

2024-09-23 Thread via GitHub
FANNG1 commented on PR #4987: URL: https://github.com/apache/gravitino/pull/4987#issuecomment-2367893605 @SinghAsDev this PR add a configuration to extend the REST API packages, is it meet your requirement? -- This is an automated message from the Apache Git Service. To respond to the mes

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

2024-09-23 Thread via GitHub
xunliu commented on code in PR #4948: URL: https://github.com/apache/gravitino/pull/4948#discussion_r1771702314 ## core/src/main/java/org/apache/gravitino/storage/relational/database/H2Database.java: ## @@ -54,6 +54,12 @@ public String startH2Database(Config config) { Str

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

2024-09-23 Thread via GitHub
SinghAsDev commented on PR #4987: URL: https://github.com/apache/gravitino/pull/4987#issuecomment-2368774338 Hi @FANNG1 thanks for the change, I think it's in the right direction. Can you help add a test case that adds an extension uri, and run some test against it. -- This is an automat

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

2024-09-23 Thread via GitHub
FANNG1 commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2368378621 > 2.9M distribution/package/catalogs/lakehouse-paimon/libs/paimon-shade-guava-30-30.1.1-jre-0.8.0.jar it's used by Paimon internally, we'd better to keep it. > Can we

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

2024-09-23 Thread via GitHub
jerryshao commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2368351936 Also, why do we need hdfs jar? I assume the hdfs client jar should be enough. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

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

2024-09-23 Thread via GitHub
jerryshao commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2368350194 Why the paimon format jar is so big, is it correct? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
jerryshao commented on code in PR #4373: URL: https://github.com/apache/gravitino/pull/4373#discussion_r1771549795 ## clients/client-python/gravitino/audit/fileset_data_operation.py: ## @@ -0,0 +1,81 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more con

Re: [I] [Bug report] federation query using 2 hive metastores does not work when using gravitino [gravitino]

2024-09-23 Thread via GitHub
foryou7242 commented on issue #4932: URL: https://github.com/apache/gravitino/issues/4932#issuecomment-2368434057 @FANNG1 Thank you so much for your help. First of all, the `spark.bypass.spark.sql.hive.metastore.jars` setting is already set to `builtin`. The root cause is that

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

2024-09-23 Thread via GitHub
FANNG1 commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2368436571 > 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 alternative jars? -- This is an auto

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

2024-09-23 Thread via GitHub
FANNG1 commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-236849 > Also, why do we need hdfs jar? I assume the hdfs client jar should be enough. Kerberos relies on the default HDFS configuration, removing hdfs jar may cause Kerberos operation f

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

2024-09-23 Thread via GitHub
FANNG1 commented on code in PR #4948: URL: https://github.com/apache/gravitino/pull/4948#discussion_r1771589973 ## server-common/src/main/java/org/apache/gravitino/server/web/JettyServer.java: ## @@ -260,6 +260,29 @@ private void initializeWebAppServletContextHandler() { se

Re: [PR] [#4981] Improvement(test): Add integration tests for Hive catalog with S3 location. [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 commented on code in PR #4982: URL: https://github.com/apache/gravitino/pull/4982#discussion_r1771289384 ## integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/ContainerSuite.java: ## @@ -71,6 +71,16 @@ public class ContainerSuite imple

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

2024-09-23 Thread via GitHub
yuqi1129 commented on code in PR #4948: URL: https://github.com/apache/gravitino/pull/4948#discussion_r1771275905 ## authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerHiveE2EIT.java: ## @@ -77,63 +78,207 @@ public

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771245801 ## core/src/main/java/org/apache/gravitino/EntityStore.java: ## @@ -55,15 +57,40 @@ public interface EntityStore extends Closeable { * Note. Depends on the isolatio

(gravitino) branch main updated: [Minor] Update NOTICE (#4989)

2024-09-23 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 d962b6bd4 [Minor] Update NOTICE (#4989) d962b6bd4

Re: [I] [Subtask] Added full code to support non-partitioned tables for OceanBase-CE JDBC catalog [gravitino]

2024-09-23 Thread via GitHub
yuanoOo closed issue #4906: [Subtask] Added full code to support non-partitioned tables for OceanBase-CE JDBC catalog URL: https://github.com/apache/gravitino/issues/4906 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [I] [Subtask] Added full code to support non-partitioned tables for OceanBase-CE JDBC catalog [gravitino]

2024-09-23 Thread via GitHub
yuanoOo commented on issue #4906: URL: https://github.com/apache/gravitino/issues/4906#issuecomment-2367555073 close -- 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 uns

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

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

[PR] [#4988] fix(doris-catalog): Fix the missing distribution information when loading Doris tables [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 opened a new pull request, #4991: URL: https://github.com/apache/gravitino/pull/4991 ### What changes were proposed in this pull request? Loading distribution information when obtaining Doris tables. ### Why are the changes needed? It's a bug to be fixed.

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
xloya closed pull request #4373: [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client URL: https://github.com/apache/gravitino/pull/4373 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
xloya opened a new pull request, #4373: URL: https://github.com/apache/gravitino/pull/4373 ### What changes were proposed in this pull request? Added an interface code skeleton in Python Client for obtaining the file location so that the client can report some necessary information fo

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771215174 ## core/src/main/java/org/apache/gravitino/EntityStore.java: ## @@ -55,15 +57,40 @@ public interface EntityStore extends Closeable { * Note. Depends on the isola

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1771235018 ## core/src/main/java/org/apache/gravitino/storage/relational/service/SupportsSkippingFieldsHandlers.java: ## @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache Softwar

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

2024-09-23 Thread via GitHub
yangyuxia commented on PR #4975: URL: https://github.com/apache/gravitino/pull/4975#issuecomment-2369905932 I test with ranger 2.1.0,found error: [EL Warning]: 2024-09-19 10:02:23.45--ClientSession(1124729650)--Exception [EclipseLink-4002] (Eclipse Persistence Services - 2.5.2.v20140319-9

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

2024-09-23 Thread via GitHub
FANNG1 commented on PR #4987: URL: https://github.com/apache/gravitino/pull/4987#issuecomment-2369788835 > Hi @FANNG1 thanks for the change, I think it's in the right direction. Can you help add a test case that adds an extension uri, and run some test against it. ok -- This is an

Re: [I] [Bug report] federation query using 2 hive metastores does not work when using gravitino [gravitino]

2024-09-23 Thread via GitHub
FANNG1 commented on issue #4932: URL: https://github.com/apache/gravitino/issues/4932#issuecomment-2369796185 > First of all, the spark.bypass.spark.sql.hive.metastore.jars setting is already set to builtin. `spark.bypass.spark.sql.hive.metastore.jars` should be set to the values oth

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
xloya commented on code in PR #4373: URL: https://github.com/apache/gravitino/pull/4373#discussion_r1772451813 ## clients/client-python/gravitino/audit/fileset_data_operation.py: ## @@ -0,0 +1,81 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contrib

Re: [I] [FEATURE] Add github logo, stars number and forks number for UI [gravitino]

2024-09-23 Thread via GitHub
CodingWithError commented on issue #4362: URL: https://github.com/apache/gravitino/issues/4362#issuecomment-2369290879 hey @jenish-thapa if you are not working on this issue can you assign this to me. -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] [#50] Add support for helm chart [gravitino-playground]

2024-09-23 Thread via GitHub
unknowntpo commented on PR #56: URL: https://github.com/apache/gravitino-playground/pull/56#issuecomment-2369867121 @danhuawang in llamaindex jupyter notebook example, at this step of creating fileset `cities`, it will throw `InternalError`, this happed at `docker-compose` and `helm-chart`

[PR] [doc]: Straight-forward improvements [gravitino]

2024-09-23 Thread via GitHub
jun-ma opened a new pull request, #4997: URL: https://github.com/apache/gravitino/pull/4997 ### What changes were proposed in this pull request? This PR addresses the following improvements to the Manage Metalake topic from a tech writing perspective. 1. Move term-specific content

Re: [PR] [#4962] feat(trino-connector): Support the Trino cascading connector [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 commented on code in PR #4935: URL: https://github.com/apache/gravitino/pull/4935#discussion_r1772483233 ## docs/trino-connector/trino-cascading-query.md: ## @@ -0,0 +1,155 @@ +--- +title: "Apache Gravitino connector Trino cascading query" +slug: /trino-connector/trino-

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

2024-09-23 Thread via GitHub
jerqi commented on PR #4879: URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2370022193 https://github.com/apache/gravitino/pull/4055 @LiuQhahah This pull request about listing users have been merged. Do you have time to rework of this pull request? -- This is an automate

Re: [PR] [#4984] improvement(core, doris): Add the random distribution strategy [gravitino]

2024-09-23 Thread via GitHub
mchades commented on code in PR #4985: URL: https://github.com/apache/gravitino/pull/4985#discussion_r1772527071 ## api/src/main/java/org/apache/gravitino/rel/expressions/distributions/Strategy.java: ## @@ -50,7 +50,10 @@ public enum Strategy { RANGE, /** Distributes dat

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

2024-09-23 Thread via GitHub
mchades commented on code in PR #4948: URL: https://github.com/apache/gravitino/pull/4948#discussion_r1772506639 ## integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/RangerContainer.java: ## @@ -38,7 +38,7 @@ public class RangerContainer exten

Re: [I] [Subtask] Add a `getFileLocation` interface code skeleton in the Python Client [gravitino]

2024-09-23 Thread via GitHub
jerryshao closed issue #4279: [Subtask] Add a `getFileLocation` interface code skeleton in the Python Client URL: https://github.com/apache/gravitino/issues/4279 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

(gravitino) branch main updated: [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client (#4373)

2024-09-23 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 39f3ca518 [#4279] feat(client-python): Add the `ge

Re: [I] [Subtask] Add a `getFileLocation` interface code skeleton in the Python Client [gravitino]

2024-09-23 Thread via GitHub
jerryshao closed issue #4279: [Subtask] Add a `getFileLocation` interface code skeleton in the Python Client URL: https://github.com/apache/gravitino/issues/4279 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
jerryshao merged PR #4373: URL: https://github.com/apache/gravitino/pull/4373 -- 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] [doc]: Straight-forward improvements [gravitino]

2024-09-23 Thread via GitHub
jerryshao commented on PR #4997: URL: https://github.com/apache/gravitino/pull/4997#issuecomment-2370242652 Thanks @jun-ma for your contribution, I will take a look at your PR today. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [PR] [#4981] Improvement(test): Add integration tests for Hive catalog with S3 location. [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 commented on PR #4982: URL: https://github.com/apache/gravitino/pull/4982#issuecomment-2369949476 > I'm OK with this PR itself. But I think compared to IT, it is more valuable to add some docs to claim that this can be run on S3 and walk users to some references about how to config

Re: [PR] [#50] Add support for helm chart [gravitino-playground]

2024-09-23 Thread via GitHub
danhuawang commented on PR #56: URL: https://github.com/apache/gravitino-playground/pull/56#issuecomment-2369952892 @jerqi @qqqttt123 Can you help Eric to figure out this problem ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
jerryshao commented on code in PR #4373: URL: https://github.com/apache/gravitino/pull/4373#discussion_r1772463011 ## clients/client-python/gravitino/audit/caller_context.py: ## @@ -0,0 +1,71 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] [#4981] Improvement(test): Add integration tests for Hive catalog with S3 location. [gravitino]

2024-09-23 Thread via GitHub
jerryshao commented on PR #4982: URL: https://github.com/apache/gravitino/pull/4982#issuecomment-2369947003 I'm OK with this PR itself. But I think compared to IT, it is more valuable to add some docs to claim that this can be run on S3 and walk users to some references about how to configu

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerryshao commented on PR #4055: URL: https://github.com/apache/gravitino/pull/4055#issuecomment-2369945162 Just make the CI pass, I'm fine with the current PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

(gravitino) branch main updated: [#4981] Improvement(test): Add integration tests for Hive catalog with S3 location. (#4982)

2024-09-23 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 870c569fa [#4981] Improvement(test): Add integrati

Re: [PR] [#4981] Improvement(test): Add integration tests for Hive catalog with S3 location. [gravitino]

2024-09-23 Thread via GitHub
jerryshao merged PR #4982: URL: https://github.com/apache/gravitino/pull/4982 -- 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] Add integration test for Hive with S3 [gravitino]

2024-09-23 Thread via GitHub
jerryshao closed issue #4981: [Improvement] Add integration test for Hive with S3 URL: https://github.com/apache/gravitino/issues/4981 -- 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 co

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi closed pull request #4055: [#3348] feat(core,server): Add the list operation of the user URL: https://github.com/apache/gravitino/pull/4055 -- 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

[PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
jerqi opened a new pull request, #4055: URL: https://github.com/apache/gravitino/pull/4055 ### What changes were proposed in this pull request? Add the list operation of the user ### Why are the changes needed? Fix: #3348 ### Does this PR introduce _any_ user-facing c

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
jerryshao commented on PR #4373: URL: https://github.com/apache/gravitino/pull/4373#issuecomment-2369966693 Generally LGTM, just one comment. -- 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] [#50] Add support for helm chart [gravitino-playground]

2024-09-23 Thread via GitHub
jerqi commented on PR #56: URL: https://github.com/apache/gravitino-playground/pull/56#issuecomment-2369970390 Could he create a file set using location `file:/tmp/gravitino/data/pdfs`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] [#4279] feat(client-python): Add the `getFileLocation` interface code skeleton in Python Client [gravitino]

2024-09-23 Thread via GitHub
xloya commented on code in PR #4373: URL: https://github.com/apache/gravitino/pull/4373#discussion_r1772479793 ## clients/client-python/gravitino/audit/caller_context.py: ## @@ -0,0 +1,71 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor lic

(gravitino) branch main updated: [#3348] feat(core,server): Add the list operation of the user (#4055)

2024-09-23 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 92a0ec8c1 [#3348] feat(core,server): Add the li

Re: [PR] [#3348] feat(core,server): Add the list operation of the user [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 merged PR #4055: URL: https://github.com/apache/gravitino/pull/4055 -- 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: [I] [FEATURE] Need REST API to list all users' info in a metalake [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 closed issue #3348: [FEATURE] Need REST API to list all users' info in a metalake URL: https://github.com/apache/gravitino/issues/3348 -- 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: [I] [FEATURE] Need REST API to list all users' info in a metalake [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 closed issue #3348: [FEATURE] Need REST API to list all users' info in a metalake URL: https://github.com/apache/gravitino/issues/3348 -- 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] #4873 support list group [gravitino]

2024-09-23 Thread via GitHub
LiuQhahah commented on PR #4879: URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2370210296 > @LiuQhahah #4055 pull request about listing users have been merged. Do you have time to rework this pull request? Hi @jerqi what action need i to do ? just close th

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

2024-09-23 Thread via GitHub
jerqi commented on PR #4879: URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2370292825 > > @LiuQhahah #4055 pull request about listing users have been merged. Do you have time to rework this pull request? > > Hi @jerqi > > what action need i to do ? just close

Re: [PR] [#4962] feat(trino-connector): Support the Trino cascading connector [gravitino]

2024-09-23 Thread via GitHub
diqiu50 commented on code in PR #4935: URL: https://github.com/apache/gravitino/pull/4935#discussion_r1772516992 ## docs/trino-connector/trino-cascading-query.md: ## @@ -0,0 +1,155 @@ +--- +title: "Apache Gravitino connector Trino cascading query" +slug: /trino-connector/trino-c

[I] [Improvement] Make tag interface more property [gravitino]

2024-09-23 Thread via GitHub
jerqi opened a new issue, #5000: URL: https://github.com/apache/gravitino/issues/5000 ### What would you like to be improved? ![image](https://github.com/user-attachments/assets/5a016c21-67f8-49a1-b3ce-28ea586fea16) If type name is duplicated with tag name, it will make users confu

[PR] [#4832] feat(core): Add basic framework to support multiple JDBC backend [gravitino]

2024-09-23 Thread via GitHub
yuqi1129 opened a new pull request, #4998: URL: https://github.com/apache/gravitino/pull/4998 ### What changes were proposed in this pull request? 1. Add a framework to support multiple JDBC backend 2. Modify all the mapper classes related to metalake, catalog, schema, filesetset,

Re: [I] [FEATURE] Support OceanBase-CE JDBC Catalog [gravitino]

2024-09-23 Thread via GitHub
mchades closed issue #4353: [FEATURE] Support OceanBase-CE JDBC Catalog URL: https://github.com/apache/gravitino/issues/4353 -- 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: [I] [FEATURE] Support OceanBase-CE JDBC Catalog [gravitino]

2024-09-23 Thread via GitHub
mchades commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2370310838 close this since it is duplicated with #4848 -- 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