[I] [Bug report] The ci test case testCheckLinkDocs failed [gravitino]

2024-09-11 Thread via GitHub
LauraXia123 opened a new issue, #4914: URL: https://github.com/apache/gravitino/issues/4914 ### Version main branch ### Describe what's wrong The ci test case testCheckLinkDocs failed ### Error message and/or stacktrace https://github.com/user-attachments/as

[PR] [#4914] ci(web): fix testCheckLinkDocs test case issue [gravitino]

2024-09-11 Thread via GitHub
LauraXia123 opened a new pull request, #4915: URL: https://github.com/apache/gravitino/pull/4915 ### What changes were proposed in this pull request? fix testCheckLinkDocs test case issue ### Why are the changes needed? The test case testCheckLinkDocs failed Fix: #

[PR] [#4742] Improvement(ui): support creating paimon catalog [gravitino]

2024-09-11 Thread via GitHub
LauraXia123 opened a new pull request, #4917: URL: https://github.com/apache/gravitino/pull/4917 ### What changes were proposed in this pull request? support creating paimon catalog https://github.com/user-attachments/assets/e60ee53b-018d-4f0d-a49b-4e5145c74f8e";> https://git

Re: [PR] [#4742] Improvement(ui): support creating paimon catalog [gravitino]

2024-09-11 Thread via GitHub
LauraXia123 commented on PR #4852: URL: https://github.com/apache/gravitino/pull/4852#issuecomment-2342939800 I will close the pr by https://github.com/apache/gravitino/pull/4917 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] [#4742] Improvement(ui): support creating paimon catalog [gravitino]

2024-09-11 Thread via GitHub
LauraXia123 closed pull request #4852: [#4742] Improvement(ui): support creating paimon catalog URL: https://github.com/apache/gravitino/pull/4852 -- 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] Feature(jdbc-catalog):Add code skeleton for oceanbase-ce jdbc catalog [gravitino]

2024-09-11 Thread via GitHub
yuanoOo opened a new pull request, #4918: URL: https://github.com/apache/gravitino/pull/4918 ### What changes were proposed in this pull request? - add catalog-jdbc-oceanbase module - add code skeleton for oceanbase-ce jdbc catalog ### Why are the changes needed? F

Re: [PR] Feature(jdbc-catalog):Add code skeleton for oceanbase-ce jdbc catalog [gravitino]

2024-09-11 Thread via GitHub
yuanoOo commented on PR #4918: URL: https://github.com/apache/gravitino/pull/4918#issuecomment-2343027869 @mchades PTAL. -- 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] [#4889] improvement(catalog-hadoop): Add the implementations for `getFileLocation` interface in Catalog Hadoop [gravitino]

2024-09-11 Thread via GitHub
xloya closed pull request #4912: [#4889] improvement(catalog-hadoop): Add the implementations for `getFileLocation` interface in Catalog Hadoop URL: https://github.com/apache/gravitino/pull/4912 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] [#4889] improvement(catalog-hadoop): Add the implementations for `getFileLocation` interface in Catalog Hadoop [gravitino]

2024-09-11 Thread via GitHub
xloya commented on PR #4912: URL: https://github.com/apache/gravitino/pull/4912#issuecomment-2343191313 @jerryshao This PR is ready for review, the frontend tests will be fixed by #4915. Please take a look when you have time, thanks. -- This is an automated message from the Apache Git Ser

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r1754411670 ## docs/lakehouse-iceberg-catalog.md: ## @@ -103,6 +103,21 @@ For other Iceberg OSS properties not managed by Gravitino like `client.security- Please set the `wareho

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r1754412283 ## docs/iceberg-rest-service.md: ## @@ -321,7 +332,7 @@ For example, we can configure Spark catalog options to use Gravitino Iceberg RES --conf spark.sql.catalog.res

Re: [PR] Replace staging repo with central repo [gravitino-playground]

2024-09-11 Thread via GitHub
jerryshao merged PR #74: URL: https://github.com/apache/gravitino-playground/pull/74 -- 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-playground) branch main updated: Replace staging repo with central repo (#74)

2024-09-11 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-playground.git The following commit(s) were added to refs/heads/main by this push: new b112316 Replace staging repo with cent

Re: [I] [Bug report] The ci test case testCheckLinkDocs failed [gravitino]

2024-09-11 Thread via GitHub
jerryshao closed issue #4914: [Bug report] The ci test case testCheckLinkDocs failed URL: https://github.com/apache/gravitino/issues/4914 -- 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

(gravitino) branch main updated: [#4914] fix(web): fix testCheckLinkDocs test case issue (#4915)

2024-09-11 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 491d173b7 [#4914] fix(web): fix testCheckLinkDocs

Re: [PR] [#4914] fix(web): fix testCheckLinkDocs test case issue [gravitino]

2024-09-11 Thread via GitHub
jerryshao merged PR #4915: URL: https://github.com/apache/gravitino/pull/4915 -- 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-491d173b7118436712e3e5d3cb6ded899d6b9df4 created (now 55ef11fa4)

2024-09-11 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-491d173b7118436712e3e5d3cb6ded899d6b9df4 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 55ef11fa4 [#4914] fix(web): fix testCheckLinkDocs tes

[PR] [#4914] fix(web): fix testCheckLinkDocs test case issue [gravitino]

2024-09-11 Thread via GitHub
github-actions[bot] opened a new pull request, #4919: URL: https://github.com/apache/gravitino/pull/4919 ### What changes were proposed in this pull request? fix testCheckLinkDocs test case issue ### Why are the changes needed? The test case testCheckLinkDocs failed

Re: [PR] [#4914] fix(web): fix testCheckLinkDocs test case issue [gravitino]

2024-09-11 Thread via GitHub
jerryshao closed pull request #4919: [#4914] fix(web): fix testCheckLinkDocs test case issue URL: https://github.com/apache/gravitino/pull/4919 -- 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 sp

Re: [PR] [#4881] improvement(docs): Add documents about PostgreSQL storage backend [gravitino]

2024-09-11 Thread via GitHub
jerryshao merged PR #4885: URL: https://github.com/apache/gravitino/pull/4885 -- 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 document about PostgreSQL storage backend [gravitino]

2024-09-11 Thread via GitHub
jerryshao closed issue #4881: [Improvement] Add document about PostgreSQL storage backend URL: https://github.com/apache/gravitino/issues/4881 -- 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 spe

(gravitino) branch main updated: [#4881] improvement(docs): Add documents about PostgreSQL storage backend (#4885)

2024-09-11 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 3121f11e5 [#4881] improvement(docs): Add documents

Re: [PR] [#4914] fix(web): fix testCheckLinkDocs test case issue [gravitino]

2024-09-11 Thread via GitHub
jerryshao closed pull request #4919: [#4914] fix(web): fix testCheckLinkDocs test case issue URL: https://github.com/apache/gravitino/pull/4919 -- 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 sp

Re: [PR] [#4889] improvement(catalog-hadoop): Add the implementations for `getFileLocation` interface in Catalog Hadoop [gravitino]

2024-09-11 Thread via GitHub
jerryshao merged PR #4912: URL: https://github.com/apache/gravitino/pull/4912 -- 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] Add the implementations for `getFileLocation` interface in Catalog Hadoop [gravitino]

2024-09-11 Thread via GitHub
jerryshao closed issue #4889: [Subtask] Add the implementations for `getFileLocation` interface in Catalog Hadoop URL: https://github.com/apache/gravitino/issues/4889 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

(gravitino) branch main updated: [#4889] improvement(catalog-hadoop): Add the implementations for `getFileLocation` interface in Catalog Hadoop (#4912)

2024-09-11 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 36b701388 [#4889] improvement(catalog-hadoop): Add

Re: [PR] [#4914] fix(web): fix testCheckLinkDocs test case issue [gravitino]

2024-09-11 Thread via GitHub
jerryshao merged PR #4919: URL: https://github.com/apache/gravitino/pull/4919 -- 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-491d173b7118436712e3e5d3cb6ded899d6b9df4 deleted (was 55ef11fa4)

2024-09-11 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-491d173b7118436712e3e5d3cb6ded899d6b9df4 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 55ef11fa4 [#4914] fix(web): fix testCheckLinkDocs test cas

(gravitino) branch branch-0.6 updated: [#4914] fix(web): fix testCheckLinkDocs test case issue (#4919)

2024-09-11 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 3b8b22385 [#4914] fix(web): fix testCh

Re: [PR] [#4811] improvement(CI): optimize CI jobs trigger conditions [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1755239324 ## .github/workflows/backend-integration-test-action.yml: ## @@ -0,0 +1,77 @@ +name: Backend Integration Test Action + +# run backend integration test +on: + workf

Re: [PR] [#4811] improvement(CI): optimize CI jobs trigger conditions [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1755243308 ## .github/workflows/backend-integration-test-action.yml: ## @@ -0,0 +1,77 @@ +name: Backend Integration Test Action + +# run backend integration test +on: + workf

Re: [PR] [#4811] improvement(CI): optimize CI jobs trigger conditions [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on PR #4827: URL: https://github.com/apache/gravitino/pull/4827#issuecomment-2344346885 @diqiu50 @FANNG1 @yuqi1129 @xunliu can you please help to review your parts again? Generally, LGTM, waiting for your check again. -- This is an automated message from the Apa

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r1755268004 ## docs/iceberg-rest-service.md: ## @@ -18,10 +18,7 @@ The Apache Gravitino Iceberg REST Server follows the [Apache Iceberg REST API sp - multi table transactio

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r1755268939 ## docs/iceberg-rest-service.md: ## @@ -18,10 +18,7 @@ The Apache Gravitino Iceberg REST Server follows the [Apache Iceberg REST API sp - multi table transactio

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r1755273008 ## docs/iceberg-rest-service.md: ## @@ -337,7 +348,7 @@ For example, we can configure Spark catalog options to use Gravitino Iceberg RES --conf spark.sql.catalog.

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r179093 ## docs/iceberg-rest-service.md: ## @@ -162,6 +159,20 @@ You should place HDFS configuration file to the classpath of the Iceberg REST se Builds with Hadoop 2.10.

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r178499 ## docs/lakehouse-iceberg-catalog.md: ## @@ -119,6 +119,20 @@ Please make sure the credential file is accessible by Gravitino, like using `exp Please set `warehou

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r1755912686 ## docs/iceberg-rest-service.md: ## @@ -18,10 +18,7 @@ The Apache Gravitino Iceberg REST Server follows the [Apache Iceberg REST API sp - multi table transaction

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4896: URL: https://github.com/apache/gravitino/pull/4896#discussion_r1755912878 ## docs/lakehouse-iceberg-catalog.md: ## @@ -119,6 +119,20 @@ Please make sure the credential file is accessible by Gravitino, like using `exp Please set `warehouse`

Re: [PR] [#4246] improvement(core): Remove role local cache [gravitino]

2024-09-11 Thread via GitHub
jerqi commented on PR #4901: URL: https://github.com/apache/gravitino/pull/4901#issuecomment-2345108195 @yuqi1129 Could you help review this pull request? -- 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 t

Re: [PR] [#4811] improvement(CI): optimize CI jobs trigger conditions [gravitino]

2024-09-11 Thread via GitHub
mchades commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1755991259 ## .github/workflows/backend-integration-test-action.yml: ## @@ -0,0 +1,77 @@ +name: Backend Integration Test Action + +# run backend integration test +on: + workflo

[PR] [#4890] improvement(IT): Add the Integration Tests for `getFileLocation` interface [gravitino]

2024-09-11 Thread via GitHub
xloya opened a new pull request, #4920: URL: https://github.com/apache/gravitino/pull/4920 ### What changes were proposed in this pull request? Add the integration Tests for `getFileLocation` interface. ### Why are the changes needed? Fix: #4890 ### How was this p

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

2024-09-11 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1756020405 ## core/src/main/java/org/apache/gravitino/storage/relational/utils/SessionUtils.java: ## @@ -145,4 +145,20 @@ public static void doMultipleWithCommit(Runnable... oper

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

2024-09-11 Thread via GitHub
jerqi commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1756020793 ## core/src/main/java/org/apache/gravitino/storage/relational/service/UserMetaService.java: ## @@ -221,6 +222,22 @@ public UserEntity updateUser( return newEntit

(gravitino-site) branch justinmclean-patch-1 created (now ca66b90)

2024-09-11 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch justinmclean-patch-1 in repository https://gitbox.apache.org/repos/asf/gravitino-site.git at ca66b90 Add brief instructions on verifying releases. This branch includes the following new

(gravitino-site) 01/01: Add brief instructions on verifying releases.

2024-09-11 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a commit to branch justinmclean-patch-1 in repository https://gitbox.apache.org/repos/asf/gravitino-site.git commit ca66b9013714fbabd17c99bcf110f821bec4ae0c Author: Justin Mclean AuthorDate: Thu Sep 12 12:58:46 20

[PR] Add brief instructions on verifying releases [gravitino-site]

2024-09-11 Thread via GitHub
justinmclean opened a new pull request, #26: URL: https://github.com/apache/gravitino-site/pull/26 (no 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 specific comment. To unsubscri

(gravitino-site) branch justinmclean-patch-1 updated (ca66b90 -> 7ccd24f)

2024-09-11 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch justinmclean-patch-1 in repository https://gitbox.apache.org/repos/asf/gravitino-site.git from ca66b90 Add brief instructions on verifying releases. add 7ccd24f add "before you use it

Re: [PR] [#4698] feat(auth-ranger): Extended Ranger authorization by configuration [gravitino]

2024-09-11 Thread via GitHub
jerqi commented on PR #4744: URL: https://github.com/apache/gravitino/pull/4744#issuecomment-2345171363 This is just a discussion instead of review suggestion. In my opinion, What's the difference between users and developers. Users can only change the configurations. Developers

(gravitino-site) 01/01: Merge pull request #26 from apache/justinmclean-patch-1

2024-09-11 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/gravitino-site.git commit 6f5a51f55acc673553f13fcea67fd724f2711b04 Merge: 1e40e2b 7ccd24f Author: Justin Mclean AuthorDate: Thu Sep 12 13:0

(gravitino-site) branch justinmclean-patch-1 deleted (was 7ccd24f)

2024-09-11 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch justinmclean-patch-1 in repository https://gitbox.apache.org/repos/asf/gravitino-site.git was 7ccd24f add "before you use it" The revisions that were on this branch are still contained in

Re: [PR] Add brief instructions on verifying releases [gravitino-site]

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

(gravitino-site) branch main updated (1e40e2b -> 6f5a51f)

2024-09-11 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino-site.git from 1e40e2b Add release notes for 0.6.0 (#20) add ca66b90 Add brief instructions on verifying releases.

Re: [PR] [#4580] feat(catalog): unify cloud storage configurations [gravitino]

2024-09-11 Thread via GitHub
diqiu50 commented on code in PR #4897: URL: https://github.com/apache/gravitino/pull/4897#discussion_r1756063329 ## catalogs/catalog-common/src/main/java/org/apache/gravitino/storage/OSSProperties.java: ## @@ -0,0 +1,32 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-11 Thread via GitHub
diqiu50 commented on code in PR #4893: URL: https://github.com/apache/gravitino/pull/4893#discussion_r1756066592 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java: ## @@ -20,6 +20,9 @@ package o

Re: [PR] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4893: URL: https://github.com/apache/gravitino/pull/4893#discussion_r1756140557 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/util/GeneralDataTypeTransformer.java: ## @@ -103,7 +104,11 @@ public Type getTri

Re: [PR] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4893: URL: https://github.com/apache/gravitino/pull/4893#discussion_r1756141855 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java: ## @@ -20,6 +20,9 @@ package or

Re: [PR] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-11 Thread via GitHub
diqiu50 commented on code in PR #4893: URL: https://github.com/apache/gravitino/pull/4893#discussion_r1756193526 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/util/GeneralDataTypeTransformer.java: ## @@ -103,7 +104,11 @@ public Type getTr

Re: [PR] [#4890] improvement(IT): Add the Integration Tests for `getFileLocation` interface [gravitino]

2024-09-11 Thread via GitHub
xloya commented on PR #4920: URL: https://github.com/apache/gravitino/pull/4920#issuecomment-2345359430 @jerryshao Please take a look of this PR when you have time, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

Re: [PR] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-11 Thread via GitHub
diqiu50 commented on code in PR #4893: URL: https://github.com/apache/gravitino/pull/4893#discussion_r1756201205 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java: ## @@ -20,6 +20,9 @@ package o

Re: [PR] [#4890] improvement(IT): Add the Integration Tests for `getFileLocation` interface [gravitino]

2024-09-11 Thread via GitHub
jerryshao commented on code in PR #4920: URL: https://github.com/apache/gravitino/pull/4920#discussion_r1756223822 ## clients/client-java/src/test/java/org/apache/gravitino/client/integration/test/FilesetIT.java: ## @@ -0,0 +1,203 @@ +/* + * Licensed to the Apache Software Found

Re: [PR] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4893: URL: https://github.com/apache/gravitino/pull/4893#discussion_r1756231253 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergDataTypeTransformer.java: ## @@ -20,6 +20,9 @@ package or

Re: [I] [Improvement] HiveTableOperations.java has no unit tests/coverage [gravitino]

2024-09-11 Thread via GitHub
justinmclean closed issue #2183: [Improvement] HiveTableOperations.java has no unit tests/coverage URL: https://github.com/apache/gravitino/issues/2183 -- 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: [I] [Improvement] HiveTableOperations.java has no unit tests/coverage [gravitino]

2024-09-11 Thread via GitHub
justinmclean commented on issue #2183: URL: https://github.com/apache/gravitino/issues/2183#issuecomment-2345409651 Thanks for checking that feel free to pick another issue or work on or to improve our test coverage in other places. -- This is an automated message from the Apache Git Serv

Re: [PR] [#4895] docs(iceberg): add document for support not managed storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
jerryshao merged PR #4896: URL: https://github.com/apache/gravitino/pull/4896 -- 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] add document for support other storages for Iceberg [gravitino]

2024-09-11 Thread via GitHub
jerryshao closed issue #4895: [Subtask] add document for support other storages for Iceberg URL: https://github.com/apache/gravitino/issues/4895 -- 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

(gravitino) branch main updated: [#4895] docs(iceberg): add document for support not managed storages for Iceberg (#4896)

2024-09-11 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 55ad5fdee [#4895] docs(iceberg): add document for

Re: [PR] [#4811] improvement(CI): optimize CI jobs trigger conditions [gravitino]

2024-09-11 Thread via GitHub
FANNG1 commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1756248592 ## .github/workflows/flink-integration-test.yml: ## @@ -28,77 +27,45 @@ jobs: - catalogs/** - clients/client-java/** - cli