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

2024-09-12 Thread via GitHub
FANNG1 commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1756255295 ## catalogs/catalog-lakehouse-iceberg/build.gradle.kts: ## @@ -138,16 +138,10 @@ tasks { } tasks.test { - val skipUTs = project.hasProperty("skipTests") Review C

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

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

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

2024-09-12 Thread via GitHub
xloya closed pull request #4920: [#4890] improvement(IT): Add the Integration Tests for `getFileLocation` interface URL: https://github.com/apache/gravitino/pull/4920 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

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

2024-09-12 Thread via GitHub
mchades commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1756271385 ## catalogs/catalog-lakehouse-iceberg/build.gradle.kts: ## @@ -138,16 +138,10 @@ tasks { } tasks.test { - val skipUTs = project.hasProperty("skipTests") Review

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

2024-09-12 Thread via GitHub
mchades commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1756273223 ## catalogs/catalog-lakehouse-iceberg/build.gradle.kts: ## @@ -138,16 +138,10 @@ tasks { } tasks.test { - val skipUTs = project.hasProperty("skipTests") - if (s

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

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

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

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

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

2024-09-12 Thread via GitHub
xloya closed pull request #4920: [#4890] improvement(IT): Add the Integration Tests for `getFileLocation` interface URL: https://github.com/apache/gravitino/pull/4920 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

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

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

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

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

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

2024-09-12 Thread via GitHub
FANNG1 commented on PR #4827: URL: https://github.com/apache/gravitino/pull/4827#issuecomment-2345536290 LGTM -- 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

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

2024-09-12 Thread via GitHub
yuqi1129 commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1756438417 ## build.gradle.kts: ## @@ -204,6 +204,13 @@ allprojects { throw GradleException("Gravitino integration tests only support [-PtestMode=embedded] or [-Ptest

[PR] [Minor] improvement(docs): Fix some format and spell error in docs [gravitino]

2024-09-12 Thread via GitHub
yuqi1129 opened a new pull request, #4923: URL: https://github.com/apache/gravitino/pull/4923 ### What changes were proposed in this pull request? Fix some document problems. ### Why are the changes needed? To improve document readability. ### Does this PR

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

2024-09-12 Thread via GitHub
mchades commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1756471584 ## .github/workflows/access-control-integration-test.yml: ## @@ -83,7 +87,8 @@ jobs: - name: Authorization Integration Test (JDK${{ matrix.java-version }}-${{

Re: [I] [FEATURE] [Lineage] Support data lineage manage [gravitino]

2024-09-12 Thread via GitHub
Jiayi-Liao commented on issue #4711: URL: https://github.com/apache/gravitino/issues/4711#issuecomment-2345721869 Another interesting idea is, you can integrate lineage ability with LLM like GPT-4o, which is extremely helpful when building a complex system involving different developers wit

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

2024-09-12 Thread via GitHub
mchades commented on code in PR #4827: URL: https://github.com/apache/gravitino/pull/4827#discussion_r1756475125 ## build.gradle.kts: ## @@ -204,6 +204,13 @@ allprojects { throw GradleException("Gravitino integration tests only support [-PtestMode=embedded] or [-PtestM

[PR] [#4922] improvement(test): Add the integration test for client withSimpleAuth(String) [gravitino]

2024-09-12 Thread via GitHub
jerqi opened a new pull request, #4924: URL: https://github.com/apache/gravitino/pull/4924 ### What changes were proposed in this pull request? Add the integration test for client withSimpleAuth(String) ### Why are the changes needed? Fix: #4922 ### Does this PR i

Re: [I] [Improvement] Trim database value in the JDBC URI [gravitino]

2024-09-12 Thread via GitHub
diqiu50 commented on issue #4884: URL: https://github.com/apache/gravitino/issues/4884#issuecomment-2345784579 I think it should throw an error instead of trimming the database. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] [#4514] improvement(iceberg): shrink catalog-lakehouse-iceberg libs size from 184MB to 56MB, IcebergRESTServer size from 162M to 70M [gravitino]

2024-09-12 Thread via GitHub
FANNG1 commented on code in PR #4548: URL: https://github.com/apache/gravitino/pull/4548#discussion_r1756529857 ## catalogs/catalog-lakehouse-iceberg/build.gradle.kts: ## @@ -69,22 +74,14 @@ dependencies { exclude("org.rocksdb") } - testImplementation(libs.hadoop2.com

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

2024-09-12 Thread via GitHub
xloya closed pull request #4320: [#4278] feat(filesystem): Refactor the `getFileLocation` logics in hadoop GVFS 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 ab

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

2024-09-12 Thread via GitHub
xloya opened a new pull request, #4320: URL: https://github.com/apache/gravitino/pull/4320 ### What changes were proposed in this pull request? Refactor the `getFileLocation` logic in Hadoop GVFS so that when sending a request to the server to obtain the file location, the current dat

[PR] [#4757]feat(trino-connector): Support more partition and sort order features of Iceberg catalog [gravitino]

2024-09-12 Thread via GitHub
diqiu50 opened a new pull request, #4925: URL: https://github.com/apache/gravitino/pull/4925 ### What changes were proposed in this pull request? Support iceberg partition expressions like: year(x), month(x), day(x), hour(x), bucket(x, n), truncate(x,n) Support iceberg sort order e

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

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

(gravitino) branch main updated: [#4246] improvement(core): Remove role local cache (#4901)

2024-09-12 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 252b911be [#4246] improvement(core): Remove rol

Re: [I] [Improvement] Local cache should be turned off by default [gravitino]

2024-09-12 Thread via GitHub
yuqi1129 closed issue #4246: [Improvement] Local cache should be turned off by default URL: https://github.com/apache/gravitino/issues/4246 -- 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 specif

Re: [I] [Improvement] Local cache should be turned off by default [gravitino]

2024-09-12 Thread via GitHub
yuqi1129 closed issue #4246: [Improvement] Local cache should be turned off by default URL: https://github.com/apache/gravitino/issues/4246 -- 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 specif

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

2024-09-12 Thread via GitHub
github-actions[bot] opened a new pull request, #4926: URL: https://github.com/apache/gravitino/pull/4926 ### What changes were proposed in this pull request? Remove role local cache ### Why are the changes needed? Fix: #4246 ### Does this PR introduce _any_ user-f

(gravitino) branch cherry-pick-branch-0.6-252b911bedb6a055209613ae3841322a0e475629 created (now 2ce05efff)

2024-09-12 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-252b911bedb6a055209613ae3841322a0e475629 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 2ce05efff [#4246] improvement(core): Remove role loca

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

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

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

2024-09-12 Thread via GitHub
yuqi1129 closed pull request #4926: [#4246] improvement(core): Remove role local cache URL: https://github.com/apache/gravitino/pull/4926 -- 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-12 Thread via GitHub
FANNG1 commented on code in PR #4893: URL: https://github.com/apache/gravitino/pull/4893#discussion_r1756830435 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/util/GeneralDataTypeTransformer.java: ## @@ -103,7 +104,11 @@ public Type getTri

[I] [Subtask] extract a common HMS module [gravitino]

2024-09-12 Thread via GitHub
mchades opened a new issue, #4927: URL: https://github.com/apache/gravitino/issues/4927 ### Describe the subtask Since both the Hive catalog and the Hudi catalog's HMS backend require access to HMS, I would like to have a common HMS module for HMS access for code reuse ### Par

[PR] [#4927] improvement(catalog): extract a common HMS module [gravitino]

2024-09-12 Thread via GitHub
mchades opened a new pull request, #4928: URL: https://github.com/apache/gravitino/pull/4928 ### What changes were proposed in this pull request? extract a common HMS module ### Why are the changes needed? Fix: #4927 ### Does this PR introduce _any_ user-facing ch

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

2024-09-12 Thread via GitHub
jerryshao closed pull request #4926: [#4246] improvement(core): Remove role local cache URL: https://github.com/apache/gravitino/pull/4926 -- 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 specifi

Re: [PR] [Minor] improvement(docs): Fix some format and spell error in docs [gravitino]

2024-09-12 Thread via GitHub
jerryshao merged PR #4923: URL: https://github.com/apache/gravitino/pull/4923 -- 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 main updated: [Minor] improvement(docs): Fix some format and spell error in docs (#4923)

2024-09-12 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 2d819c248 [Minor] improvement(docs): Fix some form

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

2024-09-12 Thread via GitHub
jerryshao commented on code in PR #4920: URL: https://github.com/apache/gravitino/pull/4920#discussion_r1757180824 ## server/src/main/java/org/apache/gravitino/server/web/rest/FilesetOperations.java: ## @@ -268,7 +269,7 @@ public Response getFileLocation( catalog,

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

2024-09-12 Thread via GitHub
jerryshao commented on code in PR #4920: URL: https://github.com/apache/gravitino/pull/4920#discussion_r1757178548 ## catalogs/catalog-hadoop/src/test/java/org/apache/gravitino/catalog/hadoop/integration/test/HadoopCatalogIT.java: ## @@ -609,6 +613,82 @@ public void testDropCata

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

2024-09-12 Thread via GitHub
jerryshao commented on PR #4917: URL: https://github.com/apache/gravitino/pull/4917#issuecomment-2346717290 @FANNG1 did you verify it locally, does it work? -- 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

Re: [PR] [#4795] feat(core): Add column entity in Gravitino [gravitino]

2024-09-12 Thread via GitHub
jerryshao commented on code in PR #4866: URL: https://github.com/apache/gravitino/pull/4866#discussion_r1757185817 ## core/src/main/java/org/apache/gravitino/meta/TableEntity.java: ## @@ -36,6 +37,8 @@ public class TableEntity implements Entity, Auditable, HasIdentifier { pu

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

2024-09-12 Thread via GitHub
jerryshao closed pull request #4926: [#4246] improvement(core): Remove role local cache URL: https://github.com/apache/gravitino/pull/4926 -- 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 specifi

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

2024-09-12 Thread via GitHub
jerryshao commented on code in PR #4897: URL: https://github.com/apache/gravitino/pull/4897#discussion_r1757208115 ## catalogs/catalog-common/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergConstants.java: ## @@ -37,20 +37,13 @@ public class IcebergConstants

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

2024-09-12 Thread via GitHub
jerryshao merged PR #4926: URL: https://github.com/apache/gravitino/pull/4926 -- 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 branch-0.6 updated: [#4246] improvement(core): Remove role local cache (#4926)

2024-09-12 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 e6b3180d9 [#4246] improvement(core): R

(gravitino) branch cherry-pick-branch-0.6-252b911bedb6a055209613ae3841322a0e475629 deleted (was 2ce05efff)

2024-09-12 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-252b911bedb6a055209613ae3841322a0e475629 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 2ce05efff [#4246] improvement(core): Remove role local cac

Re: [PR] [#4514] improvement(iceberg): shrink catalog-lakehouse-iceberg libs size from 184MB to 56MB, IcebergRESTServer size from 162M to 70M [gravitino]

2024-09-12 Thread via GitHub
jerryshao commented on code in PR #4548: URL: https://github.com/apache/gravitino/pull/4548#discussion_r1757672420 ## gradle/libs.versions.toml: ## @@ -138,6 +138,7 @@ hive2-common = { group = "org.apache.hive", name = "hive-common", version.ref = hive2-jdbc = { group = "org.a

Re: [PR] [#4514] improvement(iceberg): shrink catalog-lakehouse-iceberg libs size from 184MB to 56MB, IcebergRESTServer size from 162M to 70M [gravitino]

2024-09-12 Thread via GitHub
jerryshao commented on code in PR #4548: URL: https://github.com/apache/gravitino/pull/4548#discussion_r1757673658 ## iceberg/iceberg-common/build.gradle.kts: ## @@ -83,6 +105,7 @@ dependencies { annotationProcessor(libs.lombok) compileOnly(libs.lombok) + testImplementa

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

2024-09-12 Thread via GitHub
FANNG1 commented on PR #4917: URL: https://github.com/apache/gravitino/pull/4917#issuecomment-2347536870 > @FANNG1 did you verify it locally, does it work? yes, I create catalog though UI, create schema and table though API, the UI could show the table info successfully. -- This i

[I] [Bug report] SparkIcebergCatalogHiveBackendIT35 > testIcebergSQLExtensions() FAILED (Cron Integration Test 20240912) [gravitino]

2024-09-12 Thread via GitHub
danhuawang opened a new issue, #4929: URL: https://github.com/apache/gravitino/issues/4929 ### Version main branch ### Describe what's wrong Run spark IT failed ` ./gradlew -PskipTests -PtestMode=embedded -PjdkVersion=8 -PscalaVersion=2.12 -PskipDockerTests=false :sp

Re: [I] [Bug report] SparkIcebergCatalogHiveBackendIT35 > testIcebergSQLExtensions() FAILED (Cron Integration Test 20240912) [gravitino]

2024-09-12 Thread via GitHub
danhuawang commented on issue #4929: URL: https://github.com/apache/gravitino/issues/4929#issuecomment-2347700475 @FANNG1 Can you help check this issue? -- 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] [#4743]fix(trino-connector): Fix the default precision of time and timestamp in the Iceberg catalog. [gravitino]

2024-09-12 Thread via GitHub
FANNG1 commented on PR #4893: URL: https://github.com/apache/gravitino/pull/4893#issuecomment-2347706416 LGTM -- 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

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

2024-09-12 Thread via GitHub
FANNG1 commented on PR #4893: URL: https://github.com/apache/gravitino/pull/4893#issuecomment-2347708673 @yuqi1129 @mchades , do you have time to review again? -- 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] [#4580] feat(catalog): unify cloud storage configurations [gravitino]

2024-09-12 Thread via GitHub
FANNG1 commented on code in PR #4897: URL: https://github.com/apache/gravitino/pull/4897#discussion_r1757868094 ## catalogs/catalog-common/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergConstants.java: ## @@ -37,20 +37,13 @@ public class IcebergConstants {

Re: [PR] [#4514] improvement(iceberg): shrink catalog-lakehouse-iceberg libs size from 184MB to 56MB, IcebergRESTServer size from 162M to 70M [gravitino]

2024-09-12 Thread via GitHub
FANNG1 commented on code in PR #4548: URL: https://github.com/apache/gravitino/pull/4548#discussion_r1757888485 ## iceberg/iceberg-common/build.gradle.kts: ## @@ -83,6 +105,7 @@ dependencies { annotationProcessor(libs.lombok) compileOnly(libs.lombok) + testImplementatio

Re: [PR] [#4514] improvement(iceberg): shrink catalog-lakehouse-iceberg libs size from 184MB to 56MB, IcebergRESTServer size from 162M to 70M [gravitino]

2024-09-12 Thread via GitHub
FANNG1 commented on code in PR #4548: URL: https://github.com/apache/gravitino/pull/4548#discussion_r1757906228 ## gradle/libs.versions.toml: ## @@ -138,6 +138,7 @@ hive2-common = { group = "org.apache.hive", name = "hive-common", version.ref = hive2-jdbc = { group = "org.apac

Re: [PR] [#4514] improvement(iceberg): shrink catalog-lakehouse-iceberg libs size from 184MB to 56MB, IcebergRESTServer size from 162M to 70M [gravitino]

2024-09-12 Thread via GitHub
FANNG1 commented on code in PR #4548: URL: https://github.com/apache/gravitino/pull/4548#discussion_r1757906228 ## gradle/libs.versions.toml: ## @@ -138,6 +138,7 @@ hive2-common = { group = "org.apache.hive", name = "hive-common", version.ref = hive2-jdbc = { group = "org.apac

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

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

Re: [I] [Bug report] SparkIcebergCatalogHiveBackendIT35 > testIcebergSQLExtensions() FAILED (Cron Integration Test 20240912) [gravitino]

2024-09-12 Thread via GitHub
FANNG1 commented on issue #4929: URL: https://github.com/apache/gravitino/issues/4929#issuecomment-2347864584 Seems caused by the environment, let's check the next test result. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

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

2024-09-12 Thread via GitHub
jerryshao merged PR #4897: URL: https://github.com/apache/gravitino/pull/4897 -- 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 storage configurations [gravitino]

2024-09-12 Thread via GitHub
jerryshao closed issue #4580: [Improvement] unify the storage configurations URL: https://github.com/apache/gravitino/issues/4580 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [I] [Improvement] unify the storage configurations [gravitino]

2024-09-12 Thread via GitHub
jerryshao closed issue #4580: [Improvement] unify the storage configurations URL: https://github.com/apache/gravitino/issues/4580 -- 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.

(gravitino) branch main updated: [#4580] feat(catalog): unify cloud storage configurations (#4897)

2024-09-12 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 23f7f4caa [#4580] feat(catalog): unify cloud stora

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

2024-09-12 Thread via GitHub
jerryshao merged PR #4917: URL: 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 and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gravit

Re: [I] [Improvement] The ui does not support creating paimon catalog [gravitino]

2024-09-12 Thread via GitHub
jerryshao closed issue #4742: [Improvement] The ui does not support creating paimon catalog URL: https://github.com/apache/gravitino/issues/4742 -- 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] [Improvement] The ui does not support creating paimon catalog [gravitino]

2024-09-12 Thread via GitHub
jerryshao closed issue #4742: [Improvement] The ui does not support creating paimon catalog URL: https://github.com/apache/gravitino/issues/4742 -- 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: [#4742] Improvement(ui): support creating paimon catalog (#4917)

2024-09-12 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 ff8405682 [#4742] Improvement(ui): support creatin

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

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

(gravitino) branch cherry-pick-branch-0.6-ff8405682290bfbba11642f8da6896cc6e374e48 created (now e3e74a605)

2024-09-12 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-ff8405682290bfbba11642f8da6896cc6e374e48 in repository https://gitbox.apache.org/repos/asf/gravitino.git at e3e74a605 [#4742] Improvement(ui): support creating p

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

2024-09-12 Thread via GitHub
jerryshao closed pull request #4930: [#4742] Improvement(ui): support creating paimon catalog URL: https://github.com/apache/gravitino/pull/4930 -- 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

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

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

2024-09-12 Thread via GitHub
xunliu commented on code in PR #4055: URL: https://github.com/apache/gravitino/pull/4055#discussion_r1758125216 ## core/src/main/java/com/datastrato/gravitino/storage/relational/service/UserMetaService.java: ## @@ -221,6 +224,39 @@ public UserEntity updateUser( return new

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

2024-09-12 Thread via GitHub
xloya commented on code in PR #4920: URL: https://github.com/apache/gravitino/pull/4920#discussion_r1758125700 ## catalogs/catalog-hadoop/src/test/java/org/apache/gravitino/catalog/hadoop/integration/test/HadoopCatalogIT.java: ## @@ -609,6 +613,82 @@ public void testDropCatalogW

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

2024-09-12 Thread via GitHub
xloya commented on code in PR #4920: URL: https://github.com/apache/gravitino/pull/4920#discussion_r1758127172 ## server/src/main/java/org/apache/gravitino/server/web/rest/FilesetOperations.java: ## @@ -268,7 +269,7 @@ public Response getFileLocation( catalog,

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

2024-09-12 Thread via GitHub
xunliu commented on PR #4055: URL: https://github.com/apache/gravitino/pull/4055#issuecomment-2347972209 Query all data at once or cache buffering in the service also has the problem of dirty data. I think paged queries are better. @jerryshao @jerqi What do you think? -- This is an aut

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

2024-09-12 Thread via GitHub
xunliu commented on PR #4744: URL: https://github.com/apache/gravitino/pull/4744#issuecomment-2347975722 @jerryshao I removed the use properties file to overwrite the default authorization configuration. Please help me review it again, thanks! -- This is an automated message from the A

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

2024-09-12 Thread via GitHub
jerryshao closed pull request #4930: [#4742] Improvement(ui): support creating paimon catalog URL: https://github.com/apache/gravitino/pull/4930 -- 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 cherry-pick-branch-0.6-ff8405682290bfbba11642f8da6896cc6e374e48 deleted (was e3e74a605)

2024-09-12 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-ff8405682290bfbba11642f8da6896cc6e374e48 in repository https://gitbox.apache.org/repos/asf/gravitino.git was e3e74a605 [#4742] Improvement(ui): support creating paimon

(gravitino) branch branch-0.6 updated: [#4742] Improvement(ui): support creating paimon catalog (#4931)

2024-09-12 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 b34e3e83a [#4742] Improvement(ui): sup

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

2024-09-12 Thread via GitHub
jerryshao merged PR #4931: URL: https://github.com/apache/gravitino/pull/4931 -- 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 Integration Tests for `getFileLocation` interface [gravitino]

2024-09-12 Thread via GitHub
jerryshao closed issue #4890: [Subtask] Add the Integration Tests for `getFileLocation` interface URL: https://github.com/apache/gravitino/issues/4890 -- 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

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

2024-09-12 Thread via GitHub
jerryshao merged PR #4920: URL: https://github.com/apache/gravitino/pull/4920 -- 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 main updated: [#4890] improvement(IT): Add the Integration Tests for `getFileLocation` interface (#4920)

2024-09-12 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 9f416c789 [#4890] improvement(IT): Add the Integra

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

2024-09-12 Thread via GitHub
jerryshao commented on PR #4827: URL: https://github.com/apache/gravitino/pull/4827#issuecomment-2348015931 Any more comments here @yuqi1129 ? -- 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

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

2024-09-12 Thread via GitHub
foryou7242 opened a new issue, #4932: URL: https://github.com/apache/gravitino/issues/4932 ### Version main branch ### Describe what's wrong I want to use federation query using hive metastore stored in 2 hadoop clusters. So we added two hive catalogues to metalak

Re: [PR] [#4514] improvement(iceberg): shrink catalog-lakehouse-iceberg libs size from 184MB to 56MB, IcebergRESTServer size from 162M to 70M [gravitino]

2024-09-12 Thread via GitHub
jerryshao commented on PR #4548: URL: https://github.com/apache/gravitino/pull/4548#issuecomment-2348056102 @FANNG1 can you make the CI pass? -- 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] [#4811] improvement(CI): optimize CI jobs trigger conditions [gravitino]

2024-09-12 Thread via GitHub
yuqi1129 commented on PR #4827: URL: https://github.com/apache/gravitino/pull/4827#issuecomment-2348107834 > Any more comments here @yuqi1129 ? I have no further comments on the matter. -- This is an automated message from the Apache Git Service. To respond to the message, please l

Re: [I] [Subtask] Minimize CI trigger conditions [gravitino]

2024-09-12 Thread via GitHub
mchades closed issue #4811: [Subtask] Minimize CI trigger conditions URL: https://github.com/apache/gravitino/issues/4811 -- 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] [#4811] improvement(CI): optimize CI jobs trigger conditions [gravitino]

2024-09-12 Thread via GitHub
mchades merged PR #4827: URL: https://github.com/apache/gravitino/pull/4827 -- 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...@gravitin

(gravitino) branch main updated: [#4811] improvement(CI): optimize CI jobs trigger conditions (#4827)

2024-09-12 Thread mchades
This is an automated email from the ASF dual-hosted git repository. mchades 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 adbf61589 [#4811] improvement(CI): optimize CI j

Re: [I] [Subtask] Minimize CI trigger conditions [gravitino]

2024-09-12 Thread via GitHub
mchades closed issue #4811: [Subtask] Minimize CI trigger conditions URL: https://github.com/apache/gravitino/issues/4811 -- 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

[I] [Subtask] Change the Apache License comment style in client-python [gravitino]

2024-09-12 Thread via GitHub
noidname01 opened a new issue, #4933: URL: https://github.com/apache/gravitino/issues/4933 ### Describe the subtask * Current Apache License using docstring comment style, making Apache License appears in every generated document, which is not easy to read. ### Parent issue