[I] [Bug report] Main branch compile error, due to PR merge issue [gravitino]

2024-12-16 Thread via GitHub
shaofengshi opened a new issue, #5884: URL: https://github.com/apache/gravitino/issues/5884 ### Version main branch ### Describe what's wrong Main branch code compile error: ` > Task :clients:client-java:classes > Task :clients:client-java:jar > Task :

Re: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1888035438 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/ChainAuthorizationProperties.java: ## @@ -0,0 +1,160 @@ +/* + * Licensed

[PR] Revert "[Minor] Add column audit command to CLI (#5792)" [gravitino]

2024-12-16 Thread via GitHub
shaofengshi opened a new pull request, #5883: URL: https://github.com/apache/gravitino/pull/5883 This reverts commit 9cf0e9a0cd7dae1a2aa2fdd1d8a03f1c18ff354a. Revert a PR to fix the compilation error in main branch now. ### What changes were proposed in this pull request?

Re: [PR] [#5192] [Feature] Support Catalog Operation DDL for paimon-catalog [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5818: URL: https://github.com/apache/gravitino/pull/5818#discussion_r1888028511 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/paimon/PaimonPropertiesConverter.java: ## @@ -0,0 +1,85 @@ +/* + * Licensed to the Apache S

Re: [PR] [#5739] feat(model-catalog): Implement the model catalog logic [gravitino]

2024-12-16 Thread via GitHub
mchades commented on code in PR #5848: URL: https://github.com/apache/gravitino/pull/5848#discussion_r1888020466 ## catalogs/catalog-model/build.gradle.kts: ## @@ -40,11 +40,17 @@ dependencies { exclude(group = "*") } - compileOnly(libs.guava) - + implementation(libs

Re: [PR] [#5192] [Feature] Support Catalog Operation DDL for paimon-catalog [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5818: URL: https://github.com/apache/gravitino/pull/5818#discussion_r1888026899 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/paimon/FlinkPaimonCatalogIT.java: ## @@ -0,0 +1,174 @@ +/* + * Licensed to

Re: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1888025872 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/ChainAuthorizationProperties.java: ## @@ -0,0 +1,163 @@ +/* + * Licensed

Re: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1888024676 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/ChainAuthorizationProperties.java: ## @@ -0,0 +1,160 @@ +/* + * Licensed

Re: [PR] [#5881] test(authorization): Support Hive test container with SQL based auhtorization [gravitino]

2024-12-16 Thread via GitHub
jerqi commented on code in PR #5882: URL: https://github.com/apache/gravitino/pull/5882#discussion_r1888018543 ## integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/HiveContainer.java: ## @@ -49,6 +49,7 @@ public class HiveContainer extends Bas

Re: [PR] [#5881] test(authorization): Support Hive test container with SQL based auhtorization [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on code in PR #5882: URL: https://github.com/apache/gravitino/pull/5882#discussion_r1888014303 ## integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/ContainerSuite.java: ## @@ -509,6 +538,10 @@ public HiveContainer getKerberosH

Re: [PR] [#5568] Update help content for Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi commented on PR #5809: URL: https://github.com/apache/gravitino/pull/5809#issuecomment-2547677434 The content in these "_help.txt" files are duplicated to some extend with the "cli.md" file. Is it possible to combine them so that easier for maintain? For example, fetch some line

Re: [PR] [#5867] feat(flink-connector) Improve flink connector [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on PR #5868: URL: https://github.com/apache/gravitino/pull/5868#issuecomment-2547675547 Overall LGTM, @sunxiaojian could you add more information in the issue description about what and why you are changing? @coolderli do you have time to review this PR? -- This is

Re: [PR] [#5867] feat(flink-connector) Improve flink connector [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5868: URL: https://github.com/apache/gravitino/pull/5868#discussion_r1888008617 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/store/GravitinoCatalogStore.java: ## @@ -96,43 +102,37 @@ public boolean contains(String ca

Re: [PR] [#5162] Add grant and revoke privileges to the Gravitino CLI. [gravitino]

2024-12-16 Thread via GitHub
shaofengshi commented on code in PR #5783: URL: https://github.com/apache/gravitino/pull/5783#discussion_r1888008066 ## docs/cli.md: ## @@ -688,10 +694,23 @@ gcli group grant --group groupA --role admin ``` Remove a role from a group + ```bash gcli group revoke --grou

Re: [PR] [#5867] feat(flink-connector) Improve flink connector [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5868: URL: https://github.com/apache/gravitino/pull/5868#discussion_r1888004921 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/hive/GravitinoHiveCatalogFactory.java: ## @@ -81,4 +86,39 @@ public Set> requiredOptions()

Re: [PR] [#5527] Add audit command to roles, users and groups in the Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi merged PR #5802: URL: https://github.com/apache/gravitino/pull/5802 -- 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...@grav

Re: [I] [Improvement] Display audit information for roles, users and groups in Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi closed issue #5527: [Improvement] Display audit information for roles, users and groups in Gravitino CLI URL: https://github.com/apache/gravitino/issues/5527 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
jerqi commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1888003809 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/ChainAuthorizationProperties.java: ## @@ -0,0 +1,160 @@ +/* + * Licensed

(gravitino) branch main updated: [#5527] Add audit command to roles, users and groups in the Gravitino CLI (#5802)

2024-12-16 Thread shaofengshi
This is an automated email from the ASF dual-hosted git repository. shaofengshi 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 71005e7bf [#5527] Add audit command to roles

Re: [PR] [#5730] feat(client-python): Add sorts expression [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on PR #5879: URL: https://github.com/apache/gravitino/pull/5879#issuecomment-2547653613 hi @SophieTech88 I help you improve this PR, Please review it, Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[PR] [#5881] test(authorization): Support Hive test container with SQL based auhtorization [gravitino]

2024-12-16 Thread via GitHub
jerqi opened a new pull request, #5882: URL: https://github.com/apache/gravitino/pull/5882 ### What changes were proposed in this pull request? Support Hive test container with SQL based auhtorization ### Why are the changes needed? Fix: #5881 ### Does this PR in

(gravitino) branch main updated: [Minor] Add column audit command to CLI (#5792)

2024-12-16 Thread shaofengshi
This is an automated email from the ASF dual-hosted git repository. shaofengshi 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 9cf0e9a0c [Minor] Add column audit command t

Re: [PR] [Minor] Add column audit command to CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi merged PR #5792: URL: https://github.com/apache/gravitino/pull/5792 -- 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...@grav

Re: [PR] [#5624] feat(bundles): support ADLS credential provider [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on PR #5737: URL: https://github.com/apache/gravitino/pull/5737#issuecomment-2547644712 > @FANNG1 Thank you for providing help and reviewing! Yes, I would like to continue working on it. I can open an issue later today, or if you create it first, please tag me and assign it

Re: [PR] [#4309] feat(core): support tag events for event listener [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5847: URL: https://github.com/apache/gravitino/pull/5847#discussion_r1887975373 ## core/src/main/java/org/apache/gravitino/GravitinoEnv.java: ## @@ -485,5 +489,6 @@ private void initGravitinoServerComponents() { // Tag manager this.tag

Re: [PR] [#5624] feat(bundles): support ADLS credential provider [gravitino]

2024-12-16 Thread via GitHub
orenccl commented on PR #5737: URL: https://github.com/apache/gravitino/pull/5737#issuecomment-2547614386 @FANNG1 Thank you for providing help and reviewing! Yes, I would like to continue working on it. I can open an issue later today, or if you create it first, please tag me and assi

[I] [Subtask] Support Hive with SQL based auhtorization test container [gravitino]

2024-12-16 Thread via GitHub
jerqi opened a new issue, #5881: URL: https://github.com/apache/gravitino/issues/5881 ### Describe the subtask Support Hive with SQL based auhtorization test container. We have added the image in the pull request. We should provide the test container. ### Parent issue #5

[I] [Improvement] support OSSSecretKeyCredential for python client [gravitino]

2024-12-16 Thread via GitHub
FANNG1 opened a new issue, #5880: URL: https://github.com/apache/gravitino/issues/5880 ### What would you like to be improved? support OSSSecretKeyCredential for Python client ### How should we improve? 1. define OSSSecretKeyCredential in Python client 2. add create lo

Re: [PR] [#5624] feat(bundles): support ADLS credential provider [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on PR #5737: URL: https://github.com/apache/gravitino/pull/5737#issuecomment-2547601547 @orenccl , merged to main, it's an import feature for Gravitino, thanks for your work! There are some subsequent issues related like support `AzureSecertKeyCredential`, support ADLS cr

Re: [I] [Subtask] support ADSL credential provider [gravitino]

2024-12-16 Thread via GitHub
FANNG1 closed issue #5624: [Subtask] support ADSL credential provider URL: https://github.com/apache/gravitino/issues/5624 -- 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 un

(gravitino) branch main updated: [#5624] feat(bundles): support ADLS credential provider (#5737)

2024-12-16 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng 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 73d5ffcfe [#5624] feat(bundles): support ADLS cred

Re: [PR] [#5624] feat(bundles): support ADLS credential provider [gravitino]

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

Re: [I] [Subtask] support ADSL credential provider [gravitino]

2024-12-16 Thread via GitHub
FANNG1 closed issue #5624: [Subtask] support ADSL credential provider URL: https://github.com/apache/gravitino/issues/5624 -- 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 un

(gravitino) branch main updated: [#5329] improvement(core): Clarify exception when importing entity multiple times (#5844)

2024-12-16 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 dc655c6f6 [#5329] improvement(core): Clarify excep

Re: [I] [Bug report] Open schema in some postgres catalog, encounter issue: org.h2.jdbc.JdbcSQLIntegrityConstraintViolationException [gravitino]

2024-12-16 Thread via GitHub
jerryshao closed issue #5329: [Bug report] Open schema in some postgres catalog, encounter issue: org.h2.jdbc.JdbcSQLIntegrityConstraintViolationException URL: https://github.com/apache/gravitino/issues/5329 -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [I] [Bug report] Open schema in some postgres catalog, encounter issue: org.h2.jdbc.JdbcSQLIntegrityConstraintViolationException [gravitino]

2024-12-16 Thread via GitHub
jerryshao closed issue #5329: [Bug report] Open schema in some postgres catalog, encounter issue: org.h2.jdbc.JdbcSQLIntegrityConstraintViolationException URL: https://github.com/apache/gravitino/issues/5329 -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] [#5329] improvement(core): Clarify exception when importing entity multiple times [gravitino]

2024-12-16 Thread via GitHub
jerryshao merged PR #5844: URL: https://github.com/apache/gravitino/pull/5844 -- 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] [Minor] Improve missing name error message in Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi merged PR #5823: URL: https://github.com/apache/gravitino/pull/5823 -- 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...@grav

(gravitino) branch main updated: [Minor] Improve missing name error message in Gravitino CLI (#5823)

2024-12-16 Thread shaofengshi
This is an automated email from the ASF dual-hosted git repository. shaofengshi 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 821abb66f [Minor] Improve missing name error

Re: [PR] [Minor] Improve missing name error message in Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi commented on code in PR #5823: URL: https://github.com/apache/gravitino/pull/5823#discussion_r1887928412 ## clients/cli/src/main/java/org/apache/gravitino/cli/ErrorMessages.java: ## @@ -29,7 +29,7 @@ public class ErrorMessages { public static final String UNKNOWN_

(gravitino) branch main updated: [#5822] Print Help information when the help option is passed (#5852)

2024-12-16 Thread shaofengshi
This is an automated email from the ASF dual-hosted git repository. shaofengshi 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 4a3e71758 [#5822] Print Help information whe

Re: [I] [Improvement] In the Gravitino CLI have "gcli metalake --help" and "gcli metalake help" produce the help output [gravitino]

2024-12-16 Thread via GitHub
shaofengshi closed issue #5822: [Improvement] In the Gravitino CLI have "gcli metalake --help" and "gcli metalake help" produce the help output URL: https://github.com/apache/gravitino/issues/5822 -- This is an automated message from the Apache Git Service. To respond to the message, please l

Re: [I] [Improvement] In the Gravitino CLI have "gcli metalake --help" and "gcli metalake help" produce the help output [gravitino]

2024-12-16 Thread via GitHub
shaofengshi closed issue #5822: [Improvement] In the Gravitino CLI have "gcli metalake --help" and "gcli metalake help" produce the help output URL: https://github.com/apache/gravitino/issues/5822 -- This is an automated message from the Apache Git Service. To respond to the message, please l

Re: [PR] [#5822] Print Help information when the help option is passed [gravitino]

2024-12-16 Thread via GitHub
shaofengshi merged PR #5852: URL: https://github.com/apache/gravitino/pull/5852 -- 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...@grav

(gravitino) branch main updated: [Minor] Fix list column output in Gravitino CLI (#5871)

2024-12-16 Thread shaofengshi
This is an automated email from the ASF dual-hosted git repository. shaofengshi 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 fd3a530ae [Minor] Fix list column output in

Re: [PR] [Minor] Fix list column output in Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi merged PR #5871: URL: https://github.com/apache/gravitino/pull/5871 -- 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...@grav

Re: [PR] [Minor] Add support for topic and fileset in set owner in Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
shaofengshi merged PR #5870: URL: https://github.com/apache/gravitino/pull/5870 -- 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...@grav

(gravitino) branch main updated: [Minor] Add support for topic and fileset in set owner in Gravitino CLI (#5870)

2024-12-16 Thread shaofengshi
This is an automated email from the ASF dual-hosted git repository. shaofengshi 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 1f004 [Minor] Add support for topic and

Re: [I] [Subtask] Support transforms expression [gravitino]

2024-12-16 Thread via GitHub
SophieTech88 commented on issue #5732: URL: https://github.com/apache/gravitino/issues/5732#issuecomment-2547538901 This work is blocked by the implementation for partitions in python client. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[PR] [#5730] feat(client-python): Add sorts expression [gravitino]

2024-12-16 Thread via GitHub
SophieTech88 opened a new pull request, #5879: URL: https://github.com/apache/gravitino/pull/5879 ### What changes were proposed in this pull request? Implement sorts expression in python client, add unit test. ### Why are the changes needed? We need to support the sorts expressi

Re: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1887870879 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/ChainAuthorizationProperties.java: ## @@ -0,0 +1,162 @@ +/* + * Licensed

Re: [I] [Improvement] When a malformed name is passed to the CLI command to list columns in a table an exception occurs [gravitino]

2024-12-16 Thread via GitHub
justinmclean closed issue #5808: [Improvement] When a malformed name is passed to the CLI command to list columns in a table an exception occurs URL: https://github.com/apache/gravitino/issues/5808 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [I] [Improvement] When a malformed name is passed to the CLI command to list columns in a table an exception occurs [gravitino]

2024-12-16 Thread via GitHub
justinmclean closed issue #5808: [Improvement] When a malformed name is passed to the CLI command to list columns in a table an exception occurs URL: https://github.com/apache/gravitino/issues/5808 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [#5808] fix(CLI): Fix improper exception throwing When a malformed name is passed to the CLI command [gravitino]

2024-12-16 Thread via GitHub
justinmclean merged PR #5836: URL: https://github.com/apache/gravitino/pull/5836 -- 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...@gra

(gravitino) branch main updated: [#5808] fix(CLI): Fix improper exception throwing When a malformed name is passed to the CLI command (#5836)

2024-12-16 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.git The following commit(s) were added to refs/heads/main by this push: new 286286d78 [#5808] fix(CLI): Fix improper excepti

Re: [I] [Subtask] Add debug log for FUSE API handling. [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on issue #5873: URL: https://github.com/apache/gravitino/issues/5873#issuecomment-2547452561 hi @unknowntpo sure, Please go ahead -- 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

(gravitino) annotated tag hive-ci-v0.1.17 updated (eb8fa67eb -> 75d818781)

2024-12-16 Thread roryqi
This is an automated email from the ASF dual-hosted git repository. roryqi pushed a change to annotated tag hive-ci-v0.1.17 in repository https://gitbox.apache.org/repos/asf/gravitino.git *** WARNING: tag hive-ci-v0.1.17 was modified! *** from eb8fa67eb (commit) to 75d818781 (tag) ta

Re: [PR] [#3088] Improve greedy regular expressions [gravitino]

2024-12-16 Thread via GitHub
xunliu commented on PR #3399: URL: https://github.com/apache/gravitino/pull/3399#issuecomment-2547447287 hi @kiratkumar47 Thank you for your contributions. Please fix the conflict file. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

(gravitino) annotated tag hive-ci-v0.1.16 updated (eb8fa67eb -> 91ac69030)

2024-12-16 Thread roryqi
This is an automated email from the ASF dual-hosted git repository. roryqi pushed a change to annotated tag hive-ci-v0.1.16 in repository https://gitbox.apache.org/repos/asf/gravitino.git *** WARNING: tag hive-ci-v0.1.16 was modified! *** from eb8fa67eb (commit) to 91ac69030 (tag) ta

(gravitino) branch fix-version-generate updated: update

2024-12-16 Thread lipeidian
This is an automated email from the ASF dual-hosted git repository. lipeidian pushed a commit to branch fix-version-generate in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/fix-version-generate by this push: new 098703576 upd

(gravitino) branch main updated: [#5846] build(dev): Hive image supports for JDBC SQL standard authorization (#5849)

2024-12-16 Thread liuxun
This is an automated email from the ASF dual-hosted git repository. liuxun 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 eb8fa67eb [#5846] build(dev): Hive image supports

Re: [I] [Subtask] Add a new standard SQL based Hive image [gravitino]

2024-12-16 Thread via GitHub
xunliu closed issue #5846: [Subtask] Add a new standard SQL based Hive image URL: https://github.com/apache/gravitino/issues/5846 -- 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] [Subtask] Add a new standard SQL based Hive image [gravitino]

2024-12-16 Thread via GitHub
xunliu closed issue #5846: [Subtask] Add a new standard SQL based Hive image URL: https://github.com/apache/gravitino/issues/5846 -- 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: [PR] [#5846] build(dev): Hive image supports for JDBC SQL standard authorization [gravitino]

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

Re: [PR] [#5846] build(dev): Hive image supports for JDBC SQL standard authorization [gravitino]

2024-12-16 Thread via GitHub
jerqi commented on PR #5849: URL: https://github.com/apache/gravitino/pull/5849#issuecomment-2547427056 @xunliu I have tested this image by my test. -- 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: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
jerqi commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1887839500 ## docs/security/authorization-pushdown.md: ## @@ -24,7 +24,8 @@ In order to use the Ranger Hadoop SQL Plugin, you need to configure the followin | `authorization.ran

[PR] [#5877] feat (gvfs-fuse): Implement a common filesystem layer [gravitino]

2024-12-16 Thread via GitHub
diqiu50 opened a new pull request, #5878: URL: https://github.com/apache/gravitino/pull/5878 ### What changes were proposed in this pull request? Implement a common filesystem layer to handle manage file ids, file name mappings, and file relationships. and delegate filesystem APIs to

(gravitino) branch fix-version-generate created (now 58efb92ab)

2024-12-16 Thread lipeidian
This is an automated email from the ASF dual-hosted git repository. lipeidian pushed a change to branch fix-version-generate in repository https://gitbox.apache.org/repos/asf/gravitino.git at 58efb92ab fix: catch the generate version fatal This branch includes the following new commits:

Re: [PR] [#5624] feat(bundles): support ADLS credential provider [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on PR #5737: URL: https://github.com/apache/gravitino/pull/5737#issuecomment-2547380756 LGTM, just one comment, @yuqi1129 do you have time to review? Some changes related to fileset azure properties. -- This is an automated message from the Apache Git Service. To respond

Re: [PR] [#5624] feat(bundles): support ADLS credential provider [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5737: URL: https://github.com/apache/gravitino/pull/5737#discussion_r1887815765 ## core/src/main/java/org/apache/gravitino/credential/config/ADLSCredentialConfig.java: ## @@ -0,0 +1,110 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] [#5808] fix(CLI): Fix improper exception throwing When a malformed name is passed to the CLI command [gravitino]

2024-12-16 Thread via GitHub
Abyss-lord commented on PR #5836: URL: https://github.com/apache/gravitino/pull/5836#issuecomment-2547372144 > @Abyss-lord I can still see two: > > 1. The variable missArguments could have a better name. > 2. Please don't use MetadataObject.Type.xxx.name() fix -- This is a

[I] [Improvement] Support login with grant_type=authorization_code, when Gravitino use OAuth as authenticator [gravitino]

2024-12-16 Thread via GitHub
danhuawang opened a new issue, #5875: URL: https://github.com/apache/gravitino/issues/5875 ### What would you like to be improved? The Client Credentials grant is used for client (server-to-server) authorization, not for authenticating end-users. Currently, the end-users can't

Re: [PR] [#4309] feat(core): support tag events for event listener [gravitino]

2024-12-16 Thread via GitHub
TungYuChiang commented on code in PR #5847: URL: https://github.com/apache/gravitino/pull/5847#discussion_r1887801260 ## api/src/main/java/org/apache/gravitino/SupportsTags.java: ## @@ -17,11 +17,12 @@ * under the License. */ -package org.apache.gravitino.tag; +package org

(gravitino) branch main updated: [#5830] fix(client): add error handling for no tag in cli. (#5857)

2024-12-16 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.git The following commit(s) were added to refs/heads/main by this push: new 530cd95ea [#5830] fix(client): add error handlin

Re: [I] [Improvement] Missing tag name in creating Gravitino CLI command gives an exception. [gravitino]

2024-12-16 Thread via GitHub
justinmclean closed issue #5830: [Improvement] Missing tag name in creating Gravitino CLI command gives an exception. URL: https://github.com/apache/gravitino/issues/5830 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[I] [Improvement] Support login with grant_type=password, when Gravitino use OAuth as authenticator [gravitino]

2024-12-16 Thread via GitHub
danhuawang opened a new issue, #5874: URL: https://github.com/apache/gravitino/issues/5874 ### What would you like to be improved? Currently, the end-user can't login with password grant_type when I configure KeyCloak as the Identity provider. https://github.com/user-attachments/as

Re: [I] [Improvement] Missing tag name in creating Gravitino CLI command gives an exception. [gravitino]

2024-12-16 Thread via GitHub
justinmclean closed issue #5830: [Improvement] Missing tag name in creating Gravitino CLI command gives an exception. URL: https://github.com/apache/gravitino/issues/5830 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] [#5830] fix(client): add error handling for no tag in cli. [gravitino]

2024-12-16 Thread via GitHub
justinmclean merged PR #5857: URL: https://github.com/apache/gravitino/pull/5857 -- 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...@gra

Re: [PR] [doc] Revise the CLI documentation [gravitino]

2024-12-16 Thread via GitHub
justinmclean commented on code in PR #5840: URL: https://github.com/apache/gravitino/pull/5840#discussion_r1881973909 ## docs/cli.md: ## @@ -152,710 +163,464 @@ keytabFile=file.keytab ### Potentially unsafe operations -For operations that delete data or rename a metalake th

Re: [PR] [#5822] Print Help information when the help option is passed [gravitino]

2024-12-16 Thread via GitHub
justinmclean commented on code in PR #5852: URL: https://github.com/apache/gravitino/pull/5852#discussion_r1887625887 ## clients/cli/src/main/java/org/apache/gravitino/cli/Main.java: ## @@ -75,7 +75,8 @@ protected static String resolveCommand(CommandLine line) { return

Re: [PR] [#5808] fix(CLI): Fix improper exception throwing When a malformed name is passed to the CLI command [gravitino]

2024-12-16 Thread via GitHub
justinmclean commented on code in PR #5836: URL: https://github.com/apache/gravitino/pull/5836#discussion_r1879661056 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListTables.java: ## @@ -64,6 +69,16 @@ public void handle() { String all = Joiner.on(System

Re: [PR] [#5808] fix(CLI): Fix improper exception throwing When a malformed name is passed to the CLI command [gravitino]

2024-12-16 Thread via GitHub
justinmclean commented on PR #5836: URL: https://github.com/apache/gravitino/pull/5836#issuecomment-2546821524 @Abyss-lord I can still see two: 1. The variable missArguments could have a better name. 2. Please don't use MetadataObject.Type.xxx.name() -- This is an automated message f

Re: [PR] [#5329] improvement(core): Clarify exception when importing entity multiple times [gravitino]

2024-12-16 Thread via GitHub
mchades commented on code in PR #5844: URL: https://github.com/apache/gravitino/pull/5844#discussion_r1886970535 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -350,6 +351,10 @@ private void importSchema(NameIdentifier identifier) {

Re: [PR] [#5822] Print Help information when the help option is passed [gravitino]

2024-12-16 Thread via GitHub
tengqm commented on code in PR #5852: URL: https://github.com/apache/gravitino/pull/5852#discussion_r1886865590 ## clients/cli/src/main/java/org/apache/gravitino/cli/Main.java: ## @@ -75,7 +75,8 @@ protected static String resolveCommand(CommandLine line) { return action

Re: [PR] [#5830] fix(client): add error handling for no tag in cli. [gravitino]

2024-12-16 Thread via GitHub
tengqm commented on code in PR #5857: URL: https://github.com/apache/gravitino/pull/5857#discussion_r1886861802 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/CreateTag.java: ## @@ -52,11 +52,17 @@ public CreateTag( /** Create tags. */ @Override public v

Re: [PR] [#5842] feat(core): supports credential REST endpoint in Gravitino server [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5841: URL: https://github.com/apache/gravitino/pull/5841#discussion_r1886757840 ## core/src/main/java/org/apache/gravitino/catalog/CredentialOperationDispatcher.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

(gravitino-playground) branch main updated: feat(helm-chart): remove helm-chart support (#110)

2024-12-16 Thread liuxun
This is an automated email from the ASF dual-hosted git repository. liuxun 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 b269620 feat(helm-chart): remove helm

Re: [PR] [#4731] feat(paimon-spark-conenctor): Support delete, update and mergeInto paimon table in paimon spark connector [gravitino]

2024-12-16 Thread via GitHub
tengqm commented on code in PR #5864: URL: https://github.com/apache/gravitino/pull/5864#discussion_r1886700225 ## catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/PaimonSchemaPropertiesMetadata.java: ## @@ -34,7 +34,7 @@ */ public

Re: [PR] feat(helm-chart): remove helm-chart support [gravitino-playground]

2024-12-16 Thread via GitHub
xunliu merged PR #110: URL: https://github.com/apache/gravitino-playground/pull/110 -- 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...@

Re: [I] [Improvement]: Remove helm chart and revert any docker compose unrelated changes [gravitino-playground]

2024-12-16 Thread via GitHub
xunliu closed issue #108: [Improvement]: Remove helm chart and revert any docker compose unrelated changes URL: https://github.com/apache/gravitino-playground/issues/108 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [#5867] feat(flink-connector) Improve flink connector [gravitino]

2024-12-16 Thread via GitHub
tengqm commented on PR #5868: URL: https://github.com/apache/gravitino/pull/5868#issuecomment-2545437705 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 unsubscrib

Re: [PR] [Minor] Add support for topic and fileset in set owner in Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
tengqm commented on PR #5870: URL: https://github.com/apache/gravitino/pull/5870#issuecomment-2545431537 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 unsubsc

Re: [PR] [Minor] Fix list column output in Gravitino CLI [gravitino]

2024-12-16 Thread via GitHub
tengqm commented on PR #5871: URL: https://github.com/apache/gravitino/pull/5871#issuecomment-2545429080 LGTM wrt the scope of 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 go to the specifi

Re: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
mchades commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1886683315 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/ChainAuthorizationProperties.java: ## @@ -0,0 +1,162 @@ +/* + * License

Re: [PR] [#5739] feat(model-catalog): Implement the model catalog logic [gravitino]

2024-12-16 Thread via GitHub
jerryshao commented on PR #5848: URL: https://github.com/apache/gravitino/pull/5848#issuecomment-2545404545 @yuqi1129 @mchades @xloya can you please help to review this PR? Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] [#5790] auth(chain): Chain authorization properties [gravitino]

2024-12-16 Thread via GitHub
jerqi commented on code in PR #5791: URL: https://github.com/apache/gravitino/pull/5791#discussion_r1886665778 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/ChainAuthorizationProperties.java: ## @@ -34,28 +34,25 @@ * Configurati

(gravitino) branch main updated: [#5745] feat(CLI): Table format output for ListCatalogs command (#5759)

2024-12-16 Thread liuxun
This is an automated email from the ASF dual-hosted git repository. liuxun 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 80218128b [#5745] feat(CLI): Table format output

Re: [PR] [#5745] feat(CLI): Table format output for ListCatalogs command [gravitino]

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

Re: [PR] [#5842] feat(core): supports credential REST endpoint in Gravitino server [gravitino]

2024-12-16 Thread via GitHub
jerryshao commented on code in PR #5841: URL: https://github.com/apache/gravitino/pull/5841#discussion_r1886636071 ## core/src/main/java/org/apache/gravitino/catalog/CredentialOperationDispatcher.java: ## @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] [#5624] feat(bundles): support ADLS credential provider [gravitino]

2024-12-16 Thread via GitHub
FANNG1 commented on code in PR #5737: URL: https://github.com/apache/gravitino/pull/5737#discussion_r1886631195 ## catalogs/catalog-common/src/main/java/org/apache/gravitino/storage/ABSProperties.java: ## @@ -22,8 +22,8 @@ public class ABSProperties { Review Comment: could

  1   2   >