[PR] [#4170] improve(IT): clarify skip Docker test messages [gravitino]

2024-09-14 Thread via GitHub
xunliu opened a new pull request, #4944: URL: https://github.com/apache/gravitino/pull/4944 ### What changes were proposed in this pull request? 1. Add a extension to the AbstractIT class 2. Before runing ITs check Docker environment, and use log4j to print message in the IDEA cons

[PR] Revert "[#4608] feat(core): Supports to load fields lazily" [gravitino]

2024-09-14 Thread via GitHub
jerqi opened a new pull request, #4945: URL: https://github.com/apache/gravitino/pull/4945 Reverts apache/gravitino#4690 -- 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] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-14 Thread via GitHub
diqiu50 commented on PR #4936: URL: https://github.com/apache/gravitino/pull/4936#issuecomment-2350888356 > I remembered that the package organization is different for trino-connector in 0.6 and main branch, maybe there's an issue directly cherry-picked to branch-0.6, @diqiu50 @FANNG1 can y

Re: [PR] Revert "[#4608] feat(core): Supports to load fields lazily" [gravitino]

2024-09-14 Thread via GitHub
jerqi closed pull request #4945: Revert "[#4608] feat(core): Supports to load fields lazily" URL: https://github.com/apache/gravitino/pull/4945 -- 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] [#4938]feat(lakehouse-paimon): Support S3 filesystem for Paimon catalog. [gravitino]

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4939: URL: https://github.com/apache/gravitino/pull/4939#discussion_r1759699567 ## catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/PaimonCatalogPropertiesMetadata.java: ## @@ -61,6 +62,12 @@ public cla

Re: [PR] [#4938]feat(lakehouse-paimon): Support S3 filesystem for Paimon catalog. [gravitino]

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4939: URL: https://github.com/apache/gravitino/pull/4939#discussion_r1759699866 ## catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/filesystem/FileSystemType.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed t

Re: [PR] [#4938]feat(lakehouse-paimon): Support S3 filesystem for Paimon catalog. [gravitino]

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4939: URL: https://github.com/apache/gravitino/pull/4939#discussion_r1759700166 ## catalogs/catalog-lakehouse-paimon/src/test/java/org/apache/gravitino/catalog/lakehouse/paimon/integration/test/CatalogPaimonS3IT.java: ## @@ -0,0 +1,140 @@ +/* + *

Re: [PR] [#4938]feat(lakehouse-paimon): Support S3 filesystem for Paimon catalog. [gravitino]

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4939: URL: https://github.com/apache/gravitino/pull/4939#discussion_r1759700310 ## catalogs/catalog-lakehouse-paimon/src/test/java/org/apache/gravitino/catalog/lakehouse/paimon/integration/test/CatalogPaimonS3IT.java: ## @@ -0,0 +1,140 @@ +/* + *

Re: [PR] [#4938]feat(lakehouse-paimon): Support S3 filesystem for Paimon catalog. [gravitino]

2024-09-14 Thread via GitHub
yuqi1129 commented on code in PR #4939: URL: https://github.com/apache/gravitino/pull/4939#discussion_r1759700472 ## catalogs/catalog-lakehouse-paimon/src/test/java/org/apache/gravitino/catalog/lakehouse/paimon/integration/test/CatalogPaimonS3IT.java: ## @@ -0,0 +1,140 @@ +/* +

Re: [PR] [#4938]feat(lakehouse-paimon): Support S3 filesystem for Paimon catalog. [gravitino]

2024-09-14 Thread via GitHub
yuqi1129 commented on code in PR #4939: URL: https://github.com/apache/gravitino/pull/4939#discussion_r1759700590 ## catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/filesystem/FileSystemType.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed

Re: [PR] [#4938]feat(lakehouse-paimon): Support S3 filesystem for Paimon catalog. [gravitino]

2024-09-14 Thread via GitHub
yuqi1129 commented on code in PR #4939: URL: https://github.com/apache/gravitino/pull/4939#discussion_r1759700535 ## catalogs/catalog-lakehouse-paimon/src/test/java/org/apache/gravitino/catalog/lakehouse/paimon/integration/test/CatalogPaimonS3IT.java: ## @@ -0,0 +1,140 @@ +/* +

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

2024-09-14 Thread via GitHub
diqiu50 commented on code in PR #4900: URL: https://github.com/apache/gravitino/pull/4900#discussion_r1759700640 ## docs/lakehouse-iceberg-catalog.md: ## @@ -36,12 +36,12 @@ Builds with Apache Iceberg `1.5.2`. The Apache Iceberg table format version is ` ### Catalog properti

[I] [Bug report] The catalog of Hive 3.1.2 with Kerberos authentication enabled cannot list the tables under the schema [gravitino]

2024-09-14 Thread via GitHub
yangyuxia opened a new issue, #4946: URL: https://github.com/apache/gravitino/issues/4946 ### Version 0.6.0 ### Describe what's wrong After gravitino creates a Hive 3.1.2 catalog with Kerberos authentication enabled, all schemas can be viewed. However, when viewing the t

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

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4900: URL: https://github.com/apache/gravitino/pull/4900#discussion_r1759725787 ## docs/lakehouse-iceberg-catalog.md: ## @@ -36,12 +36,12 @@ Builds with Apache Iceberg `1.5.2`. The Apache Iceberg table format version is ` ### Catalog propertie

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

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4900: URL: https://github.com/apache/gravitino/pull/4900#discussion_r1759725922 ## catalogs/catalog-common/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergPropertiesUtils.java: ## @@ -74,4 +76,22 @@ public static Map toIcebergC

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

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4900: URL: https://github.com/apache/gravitino/pull/4900#discussion_r1759726502 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergConnectorAdapter.java: ## @@ -46,7 +46,6 @@ public Iceberg

Re: [I] [Improvement] remove `server-common` from Iceberg catalog [gravitino]

2024-09-14 Thread via GitHub
FANNG1 closed issue #4778: [Improvement] remove `server-common` from Iceberg catalog URL: https://github.com/apache/gravitino/issues/4778 -- 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

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

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

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

2024-09-14 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng 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 8b55591ca [#4743]fix(trino-connector):

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

2024-09-14 Thread via GitHub
FANNG1 commented on code in PR #4900: URL: https://github.com/apache/gravitino/pull/4900#discussion_r1759745041 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergConnectorAdapter.java: ## @@ -46,7 +46,6 @@ public Iceberg

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

2024-09-14 Thread via GitHub
jerryshao commented on code in PR #4744: URL: https://github.com/apache/gravitino/pull/4744#discussion_r1759763261 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationConfig.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed