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

2024-09-22 Thread via GitHub
unknowntpo commented on PR #56: URL: https://github.com/apache/gravitino-playground/pull/56#issuecomment-2367003012 > @unknowntpo I am going to add CI check for this repository. But there's an issue when I lint this helm-chart. Can you help check this issue? Thanks! > > ``` > helm

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

2024-09-22 Thread via GitHub
unknowntpo commented on code in PR #56: URL: https://github.com/apache/gravitino-playground/pull/56#discussion_r1770666509 ## init/gravitino/init.sh: ## @@ -25,6 +25,10 @@ cp /root/gravitino/catalogs/jdbc-mysql/libs/mysql-connector-java-8.0.27.jar /roo cp /root/gravitino/cata

[PR] [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog [gravitino]

2024-09-22 Thread via GitHub
github-actions[bot] opened a new pull request, #4986: URL: https://github.com/apache/gravitino/pull/4986 ### What changes were proposed in this pull request? 1. Spark,Trino, Iceberg catalog and Iceberg REST server use `getCatalogBackendName` to get catalog backend name. 2.

Re: [PR] [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog [gravitino]

2024-09-22 Thread via GitHub
jerryshao merged PR #4900: URL: https://github.com/apache/gravitino/pull/4900 -- 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-a258a46b651fb728bba2c69f2cf6b2ac7a0a743f created (now 64e16f5f0)

2024-09-22 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-a258a46b651fb728bba2c69f2cf6b2ac7a0a743f in repository https://gitbox.apache.org/repos/asf/gravitino.git at 64e16f5f0 [#4718] fix(iceberg): use unified logic to

(gravitino) branch main updated: [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog (#4900)

2024-09-22 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 a258a46b6 [#4718] fix(iceberg): use unified logic

Re: [PR] [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed pull request #4986: [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog URL: https://github.com/apache/gravitino/pull/4986 -- This is an automated message from the Apache Git Service. To respond to the message, please l

Re: [I] [Bug report] Catalog backend name handle the renaming of catalog [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed issue #4718: [Bug report] Catalog backend name handle the renaming of catalog URL: https://github.com/apache/gravitino/issues/4718 -- 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] [#4959] feat(Iceberg): support Iceberg REST extend API [gravitino]

2024-09-22 Thread via GitHub
jerryshao commented on PR #4987: URL: https://github.com/apache/gravitino/pull/4987#issuecomment-2367155061 I think you'd better discuss with the Pinterest team about the solutions before proposing it, to see if it can satisfy their needs. Moreover, it would be better to let them do it. -

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

2024-09-22 Thread via GitHub
jerryshao commented on code in PR #4982: URL: https://github.com/apache/gravitino/pull/4982#discussion_r1770734115 ## dev/docker/hive/hive-site.xml: ## @@ -42,4 +42,25 @@ hdfs://__REPLACE__HOST_NAME:9000/user/hive/warehouse location of default database for the warehous

(gravitino) branch issue_4979 deleted (was c7f571cd6)

2024-09-22 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a change to branch issue_4979 in repository https://gitbox.apache.org/repos/asf/gravitino.git was c7f571cd6 Add S3 related configuration to support Hive S3 schema/table. The revisions that were on this bran

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

2024-09-22 Thread via GitHub
yuqi1129 commented on code in PR #4982: URL: https://github.com/apache/gravitino/pull/4982#discussion_r1770734844 ## dev/docker/hive/hive-site.xml: ## @@ -42,4 +42,25 @@ hdfs://__REPLACE__HOST_NAME:9000/user/hive/warehouse location of default database for the warehouse

Re: [I] [Improvement] binary distribution package is too large. Optimize the package size [gravitino]

2024-09-22 Thread via GitHub
jerryshao commented on issue #4983: URL: https://github.com/apache/gravitino/issues/4983#issuecomment-2367158730 @melin can you please confirm this again in main branch, I think we're working on fix such problems. -- This is an automated message from the Apache Git Service. To respond to

Re: [I] [Bug report] doc: Datatype in some catalog doc should be correct [gravitino]

2024-09-22 Thread via GitHub
lsyulong commented on issue #4956: URL: https://github.com/apache/gravitino/issues/4956#issuecomment-2367164616 Hello, can I come and take a look at this question? I will communicate with you if there are any further questions -- This is an automated message from the Apache Git Service. T

Re: [I] [Bug report] doc: Datatype in some catalog doc should be correct [gravitino]

2024-09-22 Thread via GitHub
jerryshao commented on issue #4956: URL: https://github.com/apache/gravitino/issues/4956#issuecomment-2367166630 Sure, please go ahead, if you have further question, you can leave message here, or ping @mchades . -- This is an automated message from the Apache Git Service. To respond to t

Re: [I] [Bug report] Catalog backend name handle the renaming of catalog [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed issue #4718: [Bug report] Catalog backend name handle the renaming of catalog URL: https://github.com/apache/gravitino/issues/4718 -- 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

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

2024-09-22 Thread via GitHub
FANNG1 opened a new pull request, #4987: URL: https://github.com/apache/gravitino/pull/4987 ### What changes were proposed in this pull request? support Iceberg REST extend API ### Why are the changes needed? Fix: #4959 ### Does this PR introduce _any_ user-facing

Re: [PR] [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog [gravitino]

2024-09-22 Thread via GitHub
jerryshao commented on PR #4986: URL: https://github.com/apache/gravitino/pull/4986#issuecomment-2367150628 @FANNG1 can you please fix the backport issue here? -- 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: [I] [Bug report] doc: Datatype in some catalog doc should be correct [gravitino]

2024-09-22 Thread via GitHub
jerryshao commented on issue #4956: URL: https://github.com/apache/gravitino/issues/4956#issuecomment-2367159624 @mchades would you please help to check 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 abo

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

2024-09-22 Thread via GitHub
yuqi1129 commented on code in PR #4982: URL: https://github.com/apache/gravitino/pull/4982#discussion_r1770734844 ## dev/docker/hive/hive-site.xml: ## @@ -42,4 +42,25 @@ hdfs://__REPLACE__HOST_NAME:9000/user/hive/warehouse location of default database for the warehouse

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

2024-09-22 Thread via GitHub
yuqi1129 commented on code in PR #4982: URL: https://github.com/apache/gravitino/pull/4982#discussion_r1770735437 ## dev/docker/hive/hive-site.xml: ## @@ -42,4 +42,25 @@ hdfs://__REPLACE__HOST_NAME:9000/user/hive/warehouse location of default database for the warehouse

[I] [Bug report] Loading a Doris table with empty distribution. [gravitino]

2024-09-22 Thread via GitHub
yuqi1129 opened a new issue, #4988: URL: https://github.com/apache/gravitino/issues/4988 ### Version main branch ### Describe what's wrong create doris table success https://github.com/user-attachments/assets/a9ed0d4f-10f0-4ecf-83aa-3656eb231ac5";> `curl 'http:

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

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

(gravitino) branch cherry-pick-branch-0.6-a258a46b651fb728bba2c69f2cf6b2ac7a0a743f updated (64e16f5f0 -> 0e74c9aa1)

2024-09-22 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a change to branch cherry-pick-branch-0.6-a258a46b651fb728bba2c69f2cf6b2ac7a0a743f in repository https://gitbox.apache.org/repos/asf/gravitino.git from 64e16f5f0 [#4718] fix(iceberg): use unified logic to tran

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

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

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

2024-09-22 Thread via GitHub
FANNG1 commented on PR #4575: URL: https://github.com/apache/gravitino/pull/4575#issuecomment-2367187644 @jerryshao do you have time to review this 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 above to

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

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

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

2024-09-22 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1770750160 ## 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-22 Thread via GitHub
jerryshao commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1770751793 ## core/src/main/java/org/apache/gravitino/storage/relational/service/SupportsSkippingFieldsHandlers.java: ## @@ -0,0 +1,49 @@ +/* + * Licensed to the Apache Softwa

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

2024-09-22 Thread via GitHub
jerryshao commented on PR #4055: URL: https://github.com/apache/gravitino/pull/4055#issuecomment-2367196641 @xunliu , @yuqi1129 you wanner another look? -- 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

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

2024-09-22 Thread via GitHub
jerryshao commented on PR #4575: URL: https://github.com/apache/gravitino/pull/4575#issuecomment-236723 I will review it today. -- 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] [#4278] feat(filesystem): Refactor the `getFileLocation` logics in hadoop GVFS [gravitino]

2024-09-22 Thread via GitHub
xloya commented on PR #4320: URL: https://github.com/apache/gravitino/pull/4320#issuecomment-2367095368 > Hi @xloya just some small comments. Seems like currently we only have Java implementation for client and gvfs, we should also have Python counterpart, am I right? Yeah, I will fi

Re: [PR] [#4278] feat(filesystem): Refactor the `getFileLocation` logics in hadoop GVFS [gravitino]

2024-09-22 Thread via GitHub
xloya commented on code in PR #4320: URL: https://github.com/apache/gravitino/pull/4320#discussion_r1770705835 ## clients/filesystem-hadoop3/src/main/java/org/apache/gravitino/filesystem/hadoop/GravitinoVirtualFileSystem.java: ## @@ -381,51 +339,66 @@ NameIdentifier extractIdent

Re: [PR] [#4278] feat(filesystem): Refactor the `getFileLocation` logics in hadoop GVFS [gravitino]

2024-09-22 Thread via GitHub
xloya commented on code in PR #4320: URL: https://github.com/apache/gravitino/pull/4320#discussion_r1770707925 ## catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java: ## @@ -672,4 +721,24 @@ static Path formalizePath(Path path,

Re: [I] [Improvement] Add amazon S3 related configuration to support S3. [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed issue #4979: [Improvement] Add amazon S3 related configuration to support S3. URL: https://github.com/apache/gravitino/issues/4979 -- 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: [I] [Improvement] Add amazon S3 related configuration to support S3. [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed issue #4979: [Improvement] Add amazon S3 related configuration to support S3. URL: https://github.com/apache/gravitino/issues/4979 -- 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

(gravitino) branch main updated: [#4979] improvement(docker-hive): Add S3 related configuration to support Hive S3 schema/table. (#4980)

2024-09-22 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 5b73abd0b [#4979] improvement(docker-hive): Add S3

Re: [PR] [#4979] improvement(docker-hive): Add S3 related configuration to support Hive S3 schema/table. [gravitino]

2024-09-22 Thread via GitHub
jerryshao merged PR #4980: URL: https://github.com/apache/gravitino/pull/4980 -- 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] Unify the modification behavior of the comment field. [gravitino]

2024-09-22 Thread via GitHub
mchades commented on issue #4968: URL: https://github.com/apache/gravitino/issues/4968#issuecomment-2367128243 @koonchen No problem! Please go ahead and feel free to ping me if you have any questions -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [I] [Improvement] Add distribution strategy `random` as the drois table support `random` [gravitino]

2024-09-22 Thread via GitHub
yuqi1129 commented on issue #4984: URL: https://github.com/apache/gravitino/issues/4984#issuecomment-2367131688 This issue is a part of the fix for https://github.com/datastrato/unified-catalog-ui/issues/45#event-14337721711 -- This is an automated message from the Apache Git Service. To

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

2024-09-22 Thread via GitHub
yuqi1129 commented on PR #4055: URL: https://github.com/apache/gravitino/pull/4055#issuecomment-2367306258 > @xunliu , @yuqi1129 you wanner another look? Sure, I will review it today. -- This is an automated message from the Apache Git Service. To respond to the message, please log

(gravitino) branch branch-0.6 updated: [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog (#4986)

2024-09-22 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 a63939038 [#4718] fix(iceberg): use u

[PR] [Minor] Update NOTICE [gravitino]

2024-09-22 Thread via GitHub
justinmclean opened a new pull request, #4989: URL: https://github.com/apache/gravitino/pull/4989 ### What changes were proposed in this pull request? Removed duplicated information as there is no need to repeat any information ### Why are the changes needed? To comply wi

Re: [PR] [#4278] feat(filesystem): Refactor the `getFileLocation` logics in hadoop GVFS [gravitino]

2024-09-22 Thread via GitHub
jerryshao merged PR #4320: URL: https://github.com/apache/gravitino/pull/4320 -- 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] [Subtask] Refactor the `getFileLocation` logic in the Hadoop GVFS [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed issue #4278: [Subtask] Refactor the `getFileLocation` logic in the Hadoop GVFS URL: https://github.com/apache/gravitino/issues/4278 -- 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 t

Re: [I] [Subtask] Refactor the `getFileLocation` logic in the Hadoop GVFS [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed issue #4278: [Subtask] Refactor the `getFileLocation` logic in the Hadoop GVFS URL: https://github.com/apache/gravitino/issues/4278 -- 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 t

(gravitino) branch cherry-pick-branch-0.6-a258a46b651fb728bba2c69f2cf6b2ac7a0a743f deleted (was 0e74c9aa1)

2024-09-22 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-a258a46b651fb728bba2c69f2cf6b2ac7a0a743f in repository https://gitbox.apache.org/repos/asf/gravitino.git was 0e74c9aa1 fix The revisions that were on this branch are

(gravitino) branch justinmclean-patch-2 created (now fcf398373)

2024-09-22 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch justinmclean-patch-2 in repository https://gitbox.apache.org/repos/asf/gravitino.git at fcf398373 Update NOTICE This branch includes the following new commits: new fcf398373 Update

(gravitino) 01/01: Update NOTICE

2024-09-22 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a commit to branch justinmclean-patch-2 in repository https://gitbox.apache.org/repos/asf/gravitino.git commit fcf3983733d7d87f98fea1f192c30d9f21d9fb9a Author: Justin Mclean AuthorDate: Mon Sep 23 15:50:25 2024 +1

Re: [PR] [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalog [gravitino]

2024-09-22 Thread via GitHub
jerryshao merged PR #4986: URL: https://github.com/apache/gravitino/pull/4986 -- 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] [#4545] improvement(paimon-catalog): reduce catalog-lakehouse-paimon libs size from 222MB to 98MB [gravitino]

2024-09-22 Thread via GitHub
jerryshao commented on PR #4547: URL: https://github.com/apache/gravitino/pull/4547#issuecomment-2367287558 @FANNG1 would you please help to review 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 above to

(gravitino) branch main updated (a258a46b6 -> be7a5e6a0)

2024-09-22 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git from a258a46b6 [#4718] fix(iceberg): use unified logic to transform catalog backend name to handle the renaming of catalo

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

2024-09-22 Thread via GitHub
jerryshao commented on PR #4055: URL: https://github.com/apache/gravitino/pull/4055#issuecomment-2367291342 Please fix the IT issue here. -- 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 speci

Re: [I] [EPIC] Prepare for 0.6.0 release [gravitino]

2024-09-22 Thread via GitHub
jerryshao closed issue #4480: [EPIC] Prepare for 0.6.0 release URL: https://github.com/apache/gravitino/issues/4480 -- 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 unsubscri

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

2024-09-22 Thread via GitHub
jerqi commented on PR #4055: URL: https://github.com/apache/gravitino/pull/4055#issuecomment-2367353459 > Please fix the IT issue here. I have fixed the issue. Waiting for CI. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [I] [Improvement] Unify the modification behavior of the comment field. [gravitino]

2024-09-22 Thread via GitHub
koonchen commented on issue #4968: URL: https://github.com/apache/gravitino/issues/4968#issuecomment-2366828628 @mchades i want to have a try. -- 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