[PR] [MINOR]: fix authorization ranger compile lib path [gravitino]

2024-08-22 Thread via GitHub
xunliu opened a new pull request, #4651: URL: https://github.com/apache/gravitino/pull/4651 ### What changes were proposed in this pull request? Compile distribution authorization ranger using the error path. ### Why are the changes needed? Doesn't copy authorization rang

Re: [PR] [#4646] docs(iceberg): fix Iceberg REST service doc [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4650: URL: https://github.com/apache/gravitino/pull/4650 -- 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-68011457adfde427b5a8da098cf41bf25291c222 deleted (was cfce29dff)

2024-08-22 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-68011457adfde427b5a8da098cf41bf25291c222 in repository https://gitbox.apache.org/repos/asf/gravitino.git was cfce29dff [#4646] docs(iceberg): fix Iceberg REST service

(gravitino) branch branch-0.6 updated: [#4646] docs(iceberg): fix Iceberg REST service doc (#4650)

2024-08-22 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 5acfb2c9f [#4646] docs(iceberg): fix I

Re: [PR] [#4646] docs(iceberg): fix Iceberg REST service doc [gravitino]

2024-08-22 Thread via GitHub
FANNG1 closed pull request #4650: [#4646] docs(iceberg): fix Iceberg REST service doc URL: https://github.com/apache/gravitino/pull/4650 -- 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] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
theoryxu commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728436981 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergTableOpsManager.java: ## @@ -35,16 +43,41 @@ public class IcebergTableOpsMan

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
theoryxu commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728436981 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergTableOpsManager.java: ## @@ -35,16 +43,41 @@ public class IcebergTableOpsMan

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
theoryxu commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728431895 ## iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java: ## @@ -166,6 +166,13 @@ public class IcebergConfig extends Config imp

svn commit: r71063 - /dev/incubator/gravitino/v0.6.0-incubating-rc2/

2024-08-22 Thread jshao
Author: jshao Date: Fri Aug 23 06:24:58 2024 New Revision: 71063 Log: Apache Gravitino v0.6.0-incubating-rc2 Added: dev/incubator/gravitino/v0.6.0-incubating-rc2/ dev/incubator/gravitino/v0.6.0-incubating-rc2/gravitino-0.6.0-incubating-bin.tar.gz (with props) dev/incubator/gravit

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
theoryxu commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728431647 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/provider/GravitinoBasedIcebergTableOpsProvider.java: ## @@ -0,0 +1,101 @@ +/* + * Licensed

(gravitino) branch cherry-pick-branch-0.6-68011457adfde427b5a8da098cf41bf25291c222 created (now cfce29dff)

2024-08-22 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-68011457adfde427b5a8da098cf41bf25291c222 in repository https://gitbox.apache.org/repos/asf/gravitino.git at cfce29dff [#4646] docs(iceberg): fix Iceberg REST ser

[PR] [#4646] docs(iceberg): fix Iceberg REST service doc [gravitino]

2024-08-22 Thread via GitHub
github-actions[bot] opened a new pull request, #4650: URL: https://github.com/apache/gravitino/pull/4650 ### What changes were proposed in this pull request? fix Iceberg REST service doc ### Why are the changes needed? Fix: #4646 ### Does this PR introduce _any_ user-

(gravitino) branch main updated: [#4646] docs(iceberg): fix Iceberg REST service doc (#4649)

2024-08-22 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 68011457a [#4646] docs(iceberg): fix Iceberg RE

Re: [I] [Bug report] Doc: The link of Spark Iceberg start guide in iceberg-rest-service.md is not found [gravitino]

2024-08-22 Thread via GitHub
yuqi1129 closed issue #4646: [Bug report] Doc: The link of Spark Iceberg start guide in iceberg-rest-service.md is not found URL: https://github.com/apache/gravitino/issues/4646 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [I] [Bug report] Doc: The link of Spark Iceberg start guide in iceberg-rest-service.md is not found [gravitino]

2024-08-22 Thread via GitHub
yuqi1129 closed issue #4646: [Bug report] Doc: The link of Spark Iceberg start guide in iceberg-rest-service.md is not found URL: https://github.com/apache/gravitino/issues/4646 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [#4646] docs(iceberg): fix Iceberg REST service doc [gravitino]

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

Re: [PR] solved npe isssue #4605 [gravitino]

2024-08-22 Thread via GitHub
justinmclean commented on PR #4618: URL: https://github.com/apache/gravitino/pull/4618#issuecomment-2306364452 Looks like you still need to run `./gradlew :catalogs:catalog-jdbc-doris:spotlessApply` to reformat and commit the changes. -- This is an automated message from the Apache Git Se

(gravitino) branch branch-0.6 updated (7932c66ec -> df77dc4fc)

2024-08-22 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch branch-0.6 in repository https://gitbox.apache.org/repos/asf/gravitino.git from 7932c66ec [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash (#4648)

(gravitino) 01/01: Preparing development version 0.6.1-incubating-SNAPSHOT

2024-08-22 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 commit df77dc4fced83b236b2819be06393ec912147988 Author: Jerry Shao AuthorDate: Fri Aug 23 14:09:38 2024 +0800 Prepar

(gravitino) tag v0.6.0-incubating-rc2 created (now 8e9d600ea)

2024-08-22 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to tag v0.6.0-incubating-rc2 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 8e9d600ea (commit) This tag includes the following new commits: new 8e9d600ea Preparing Gravitino

(gravitino) 01/01: Preparing Gravitino release v0.6.0-incubating-rc2

2024-08-22 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a commit to tag v0.6.0-incubating-rc2 in repository https://gitbox.apache.org/repos/asf/gravitino.git commit 8e9d600ead615ec77f11d00fa35e395724a7437f Author: Jerry Shao AuthorDate: Fri Aug 23 14:09:38 2024 +0800

Re: [PR] feat(jdbc-catalog): Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
yuqi1129 commented on PR #4626: URL: https://github.com/apache/gravitino/pull/4626#issuecomment-2306359267 > @xunliu I just fixed failing test cases in integration tests, Please help trigger the github workflow again. done -- This is an automated message from the Apache Git Service

(gravitino) branch cherry-pick-branch-0.6-823dc6a1e8c0e26c5287855da45611203c298b2d deleted (was 0c25b8d77)

2024-08-22 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-823dc6a1e8c0e26c5287855da45611203c298b2d in repository https://gitbox.apache.org/repos/asf/gravitino.git was 0c25b8d77 [Minor] improve(doc): Update docs about partitio

(gravitino) branch cherry-pick-branch-0.6-ea6ae5510437eff17061d0eefbd381930d49ccea deleted (was 4ff7c74fd)

2024-08-22 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-ea6ae5510437eff17061d0eefbd381930d49ccea in repository https://gitbox.apache.org/repos/asf/gravitino.git was 4ff7c74fd [#4586] fix(filesystem): Fix convert the actual

(gravitino) branch cherry-pick-branch-0.6-1fce8ef07731ee151b47cf1ecd49b5728b574f0a deleted (was 8f399d219)

2024-08-22 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-1fce8ef07731ee151b47cf1ecd49b5728b574f0a in repository https://gitbox.apache.org/repos/asf/gravitino.git was 8f399d219 [#4637] improvement: Rename the table `securabl

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4648: URL: https://github.com/apache/gravitino/pull/4648 -- 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: [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash (#4648)

2024-08-22 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 7932c66ec [#4586] fix(filesystem): Fix

Re: [PR] feat(jdbc-catalog): Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
yuanoOo commented on PR #4626: URL: https://github.com/apache/gravitino/pull/4626#issuecomment-2306332934 @xunliu I just fixed failing test cases in integration tests, Please help trigger the github workflow again. -- This is an automated message from the Apache Git Service. To respond to

(gravitino) branch branch-0.6 updated: [#4637] improvement(core): Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` (#4647)

2024-08-22 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 01c43c479 [#4637] improvement(core): R

Re: [PR] [#4637] improvement(core): Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4647: URL: https://github.com/apache/gravitino/pull/4647 -- 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] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed pull request #4648: [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash URL: https://github.com/apache/gravitino/pull/4648 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [I] [Bug report] bug about hdfs root catalog [gravitino]

2024-08-22 Thread via GitHub
jerryshao commented on issue #4586: URL: https://github.com/apache/gravitino/issues/4586#issuecomment-2306287266 Thanks @xloya for your fix, thanks @heziyi399 for your report, greatly appreciated. This PR will be merged in to main and branch-0.6. -- This is an automated message from the A

[PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
github-actions[bot] opened a new pull request, #4648: URL: https://github.com/apache/gravitino/pull/4648 ### What changes were proposed in this pull request? When the storage location of fileset has a trailing slash, the problem of not finding files may occur when traversing the direc

(gravitino) branch cherry-pick-branch-0.6-ea6ae5510437eff17061d0eefbd381930d49ccea created (now 4ff7c74fd)

2024-08-22 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-ea6ae5510437eff17061d0eefbd381930d49ccea in repository https://gitbox.apache.org/repos/asf/gravitino.git at 4ff7c74fd [#4586] fix(filesystem): Fix convert the ac

Re: [I] [Bug report] bug about hdfs root catalog [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed issue #4586: [Bug report] bug about hdfs root catalog URL: https://github.com/apache/gravitino/issues/4586 -- 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 u

Re: [I] [Bug report] bug about hdfs root catalog [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed issue #4586: [Bug report] bug about hdfs root catalog URL: https://github.com/apache/gravitino/issues/4586 -- 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 u

(gravitino) branch main updated: [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash (#4592)

2024-08-22 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 ea6ae5510 [#4586] fix(filesystem): Fix convert the

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4592: URL: https://github.com/apache/gravitino/pull/4592 -- 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] [#4633] refactor(integration-test): remove integration-test module [gravitino]

2024-08-22 Thread via GitHub
diqiu50 commented on code in PR #4628: URL: https://github.com/apache/gravitino/pull/4628#discussion_r1728305349 ## integration-test-common/docker-script/init/hive/init.sh: ## Review Comment: Why we don't move this directory to the Trino integration test directory -- Th

Re: [PR] [#4633] refactor(integration-test): remove integration-test module [gravitino]

2024-08-22 Thread via GitHub
diqiu50 commented on code in PR #4628: URL: https://github.com/apache/gravitino/pull/4628#discussion_r1728303276 ## integration-test-common/docker-script/docker-compose.yaml: ## Review Comment: Why we don't move it to the Trino integration test directory -- This is an a

Re: [PR] [#4633] refactor(integration-test): remove integration-test module [gravitino]

2024-08-22 Thread via GitHub
diqiu50 commented on code in PR #4628: URL: https://github.com/apache/gravitino/pull/4628#discussion_r1728272394 ## .github/workflows/trino-integration-test.yml: ## @@ -0,0 +1,104 @@ +name: Trino Integration Test + +# Controls when the workflow will run +on: + # Triggers the wo

Re: [PR] [#4633] refactor(integration-test): remove integration-test module [gravitino]

2024-08-22 Thread via GitHub
diqiu50 commented on code in PR #4628: URL: https://github.com/apache/gravitino/pull/4628#discussion_r1728261218 ## .github/workflows/trino-integration-test.yml: ## @@ -0,0 +1,104 @@ +name: Trino Integration Test + +# Controls when the workflow will run +on: + # Triggers the wo

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
xloya commented on code in PR #4592: URL: https://github.com/apache/gravitino/pull/4592#discussion_r1728225124 ## clients/filesystem-hadoop3/src/test/java/org/apache/gravitino/filesystem/hadoop/TestGvfsBase.java: ## @@ -607,4 +651,52 @@ public void testGetDefaultBlockSize() thro

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
jerryshao commented on code in PR #4592: URL: https://github.com/apache/gravitino/pull/4592#discussion_r1728217630 ## clients/filesystem-hadoop3/src/test/java/org/apache/gravitino/filesystem/hadoop/TestGvfsBase.java: ## @@ -607,4 +651,52 @@ public void testGetDefaultBlockSize()

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
xloya commented on code in PR #4592: URL: https://github.com/apache/gravitino/pull/4592#discussion_r1728214117 ## clients/client-python/gravitino/filesystem/gvfs.py: ## @@ -500,6 +501,12 @@ def _convert_actual_path(self, path, context: FilesetContext): f"Path {

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
xloya commented on code in PR #4592: URL: https://github.com/apache/gravitino/pull/4592#discussion_r1728214117 ## clients/client-python/gravitino/filesystem/gvfs.py: ## @@ -500,6 +501,12 @@ def _convert_actual_path(self, path, context: FilesetContext): f"Path {

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
xloya commented on code in PR #4592: URL: https://github.com/apache/gravitino/pull/4592#discussion_r1728212383 ## clients/filesystem-hadoop3/src/test/java/org/apache/gravitino/filesystem/hadoop/TestGvfsBase.java: ## @@ -607,4 +651,52 @@ public void testGetDefaultBlockSize() thro

Re: [PR] [#4630] feat(iceberg): remove S3 SDK jar from Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4645: URL: https://github.com/apache/gravitino/pull/4645 -- 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-a14cc56002bd8692453003d928c1211ba1dddbec deleted (was 152a7534f)

2024-08-22 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-a14cc56002bd8692453003d928c1211ba1dddbec in repository https://gitbox.apache.org/repos/asf/gravitino.git was 152a7534f [#4630] feat(iceberg): remove S3 SDK jar from Gr

(gravitino) branch branch-0.6 updated: [#4630] feat(iceberg): remove S3 SDK jar from Gravitino Iceberg REST server (#4645)

2024-08-22 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 f3c0220db [#4630] feat(iceberg): remov

Re: [PR] [#4637] improvement(core): Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` [gravitino]

2024-08-22 Thread via GitHub
jerqi closed pull request #4647: [#4637] improvement(core): Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` URL: https://github.com/apache/gravitino/pull/4647 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
jerryshao commented on code in PR #4592: URL: https://github.com/apache/gravitino/pull/4592#discussion_r1728202699 ## clients/client-python/gravitino/filesystem/gvfs.py: ## @@ -500,6 +501,12 @@ def _convert_actual_path(self, path, context: FilesetContext): f"Pa

Re: [PR] [#4586] fix(filesystem): Fix convert the actual path which storage location end with slash [gravitino]

2024-08-22 Thread via GitHub
jerryshao commented on code in PR #4592: URL: https://github.com/apache/gravitino/pull/4592#discussion_r1728200634 ## clients/filesystem-hadoop3/src/test/java/org/apache/gravitino/filesystem/hadoop/TestGvfsBase.java: ## @@ -607,4 +651,52 @@ public void testGetDefaultBlockSize()

[PR] [#4637] improvement: Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` [gravitino]

2024-08-22 Thread via GitHub
jerqi opened a new pull request, #4647: URL: https://github.com/apache/gravitino/pull/4647 ### What changes were proposed in this pull request? Rename the table securable_object_meta field entity_id to metadata_object_id ### Why are the changes needed? Fix: #4637 #

Re: [I] [Bug report] Doc: The link of Spark Iceberg start guide in iceberg-rest-service.md is not found [gravitino]

2024-08-22 Thread via GitHub
danhuawang commented on issue #4646: URL: https://github.com/apache/gravitino/issues/4646#issuecomment-2306106617 + playground link is not workable https://github.com/user-attachments/assets/f74a5479-97fd-4172-8725-431f1fb31e37";> -- This is an automated message from the Apache Git

Re: [PR] [#3963] feat(core): Call the method ownerset of authorization plugin [gravitino]

2024-08-22 Thread via GitHub
yuqi1129 merged PR #4643: URL: https://github.com/apache/gravitino/pull/4643 -- 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 branch-0.6 updated: [#3963] feat(core): Call the method ownerset of authorization plugin (#4643)

2024-08-22 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 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 dcb325c3f [#3963] feat(core): Call

Re: [I] [Bug report] Doc: The link of Spark Iceberg start guide in iceberg-rest-service.md is not found [gravitino]

2024-08-22 Thread via GitHub
danhuawang commented on issue #4646: URL: https://github.com/apache/gravitino/issues/4646#issuecomment-2306101223 @FANNG1 Would you like to help update the link in doc? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[I] [Bug report] Doc: The link of Spark Iceberg start guide in iceberg-rest-service.md is not found [gravitino]

2024-08-22 Thread via GitHub
danhuawang opened a new issue, #4646: URL: https://github.com/apache/gravitino/issues/4646 ### Version main branch ### Describe what's wrong https://github.com/user-attachments/assets/352616cd-d8a5-4bdf-befd-5d1c7596fbb6";> https://github.com/user-attachmen

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
justinmclean commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306098299 I've taken a look at the distribution artifact and it looks good to me. Due to the Category X license, this would not normally be OK, but it is in this case it is OK, due to h

Re: [PR] [#4606] Improvement(trino-connector): Possible null pointer expection/code handling [gravitino]

2024-08-22 Thread via GitHub
jerryshao commented on PR #4619: URL: https://github.com/apache/gravitino/pull/4619#issuecomment-2306098365 @diqiu50 can you please help to review this? -- 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] [#3766] improvement(client-python): Fix broad-exception-caught Pylint rule for client-python [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4621: URL: https://github.com/apache/gravitino/pull/4621 -- 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: [#3766] improvement(client-python): Fix broad-exception-caught Pylint rule for client-python (#4621)

2024-08-22 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 1913e7510 [#3766] improvement(client-python): Fix

Re: [I] [Subtask] Fix broad-exception-caught Pylint rule for client-python [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed issue #3766: [Subtask] Fix broad-exception-caught Pylint rule for client-python URL: https://github.com/apache/gravitino/issues/3766 -- 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: [I] [Subtask] Fix broad-exception-caught Pylint rule for client-python [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed issue #3766: [Subtask] Fix broad-exception-caught Pylint rule for client-python URL: https://github.com/apache/gravitino/issues/3766 -- 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] [#4630] feat(iceberg): remove S3 SDK jar from Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
FANNG1 closed pull request #4645: [#4630] feat(iceberg): remove S3 SDK jar from Gravitino Iceberg REST server URL: https://github.com/apache/gravitino/pull/4645 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
FANNG1 commented on PR #4598: URL: https://github.com/apache/gravitino/pull/4598#issuecomment-2306086765 LGTM except minor comments -- 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 co

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728178727 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/provider/GravitinoBasedIcebergTableOpsProvider.java: ## @@ -0,0 +1,101 @@ +/* + * Licensed to

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728178113 ## iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java: ## @@ -166,6 +166,13 @@ public class IcebergConfig extends Config imple

(gravitino) branch cherry-pick-branch-0.6-a14cc56002bd8692453003d928c1211ba1dddbec created (now 152a7534f)

2024-08-22 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-a14cc56002bd8692453003d928c1211ba1dddbec in repository https://gitbox.apache.org/repos/asf/gravitino.git at 152a7534f [#4630] feat(iceberg): remove S3 SDK jar fr

Re: [PR] [#4581]feat(iceberg) support Gravitino-based multi catalog support for Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
FANNG1 commented on code in PR #4598: URL: https://github.com/apache/gravitino/pull/4598#discussion_r1728177500 ## iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergTableOpsManager.java: ## @@ -35,16 +43,41 @@ public class IcebergTableOpsManag

Re: [I] [Improvement] remove S3 SDK jar from Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed issue #4630: [Improvement] remove S3 SDK jar from Gravitino Iceberg REST server URL: https://github.com/apache/gravitino/issues/4630 -- 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] [#4630] feat(iceberg): remove S3 SDK jar from Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4631: URL: https://github.com/apache/gravitino/pull/4631 -- 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] remove S3 SDK jar from Gravitino Iceberg REST server [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed issue #4630: [Improvement] remove S3 SDK jar from Gravitino Iceberg REST server URL: https://github.com/apache/gravitino/issues/4630 -- 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) branch main updated: [#4630] feat(iceberg): remove S3 SDK jar from Gravitino Iceberg REST server (#4631)

2024-08-22 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 a14cc5600 [#4630] feat(iceberg): remove S3 SDK jar

Re: [I] [Bug report] UI: scroll bar on the catalog detail page is no longer needed [gravitino]

2024-08-22 Thread via GitHub
LauraXia123 commented on issue #4641: URL: https://github.com/apache/gravitino/issues/4641#issuecomment-2306082083 The substring of value is 22 for now, we can set the length of substring smaller. But the scroll bar still work when open with sm screen. IMO, we can label the issue 'first

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
jerryshao commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306080309 @yuanoOo thanks a lot for your contribution. May I ask what is your email address or wechat id, I think we can have a talk first 😄 . -- This is an automated message from the Ap

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
justinmclean commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306077234 It may or may not be, just because they have the same or similar licenses is not enough to go on. By default, anything with a Category X license (which includes GPL-style lice

(gravitino) branch cherry-pick-branch-0.6-da9c440115f12a7a0b2d738b6b3d18692cf7cda5 deleted (was 94e7b8d7c)

2024-08-22 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-da9c440115f12a7a0b2d738b6b3d18692cf7cda5 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 94e7b8d7c [Minor] Remove URL parameter and add a coueple o

(gravitino) branch branch-0.6 updated: [Minor] Remove URL parameter and add a coueple of more pages (#4644)

2024-08-22 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 54ff13c55 [Minor] Remove URL parameter

Re: [PR] [Minor] Remove URL parameter and add a coueple of more pages [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4644: URL: https://github.com/apache/gravitino/pull/4644 -- 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] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
shaofengshi commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306072746 > Possibly as GPL is not compatible with ALv2 see https://www.apache.org/licenses/GPL-compatibility.html. Note that merely linking to it (i.e. having it as a dependency) is an

Re: [PR] [#3963][followup][0.6] feat(core): Call the method ownerset of authorization plugin [gravitino]

2024-08-22 Thread via GitHub
jerqi closed pull request #4643: [#3963][followup][0.6] feat(core): Call the method ownerset of authorization plugin URL: https://github.com/apache/gravitino/pull/4643 -- 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] [Minor] Remove URL parameter and add a coueple of more pages [gravitino]

2024-08-22 Thread via GitHub
jerryshao closed pull request #4644: [Minor] Remove URL parameter and add a coueple of more pages URL: https://github.com/apache/gravitino/pull/4644 -- 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

[PR] [Minor] Remove URL parameter and add a coueple of more pages [gravitino]

2024-08-22 Thread via GitHub
github-actions[bot] opened a new pull request, #4644: URL: https://github.com/apache/gravitino/pull/4644 ### What changes were proposed in this pull request? Remove URL parameter and add a coueple of more pages. ### Why are the changes needed? Track a few more useful page

(gravitino) branch cherry-pick-branch-0.6-da9c440115f12a7a0b2d738b6b3d18692cf7cda5 created (now 94e7b8d7c)

2024-08-22 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-da9c440115f12a7a0b2d738b6b3d18692cf7cda5 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 94e7b8d7c [Minor] Remove URL parameter and add a coue

Re: [PR] [Minor] Remove URL parameter and add a coueple of more pages [gravitino]

2024-08-22 Thread via GitHub
jerryshao merged PR #4642: URL: https://github.com/apache/gravitino/pull/4642 -- 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] Remove URL parameter and add a coueple of more pages (#4642)

2024-08-22 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 da9c44011 [Minor] Remove URL parameter and add a c

Re: [PR] [#4637] improvement: Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` [gravitino]

2024-08-22 Thread via GitHub
jerqi closed pull request #4639: [#4637] improvement: Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` URL: https://github.com/apache/gravitino/pull/4639 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] [#3963][followup] feat(core): Call the method ownerset of authorization plugin [gravitino]

2024-08-22 Thread via GitHub
jerqi opened a new pull request, #4643: URL: https://github.com/apache/gravitino/pull/4643 ### What changes were proposed in this pull request? This is the follow-up pull request of #3963. #3963 add authorization plugin to set owner. This pull request is to call them. ### Wh

Re: [PR] [#4637] improvement: Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` [gravitino]

2024-08-22 Thread via GitHub
jerqi closed pull request #4639: [#4637] improvement: Rename the table `securable_object_meta` field `entity_id` to `metadata_object_id` URL: https://github.com/apache/gravitino/pull/4639 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
justinmclean commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306044768 Possibly as GPL is not compatible with ALv2 see https://www.apache.org/licenses/GPL-compatibility.html. I'd need to take a look at the final artifacts to be sure. -- This i

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
yuanoOo commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306032128 > The MySQL connector and the Oceanbase client for Jave are licensed under the GPL/LGPL license. This is not compatible with the ALv2. As far as I know, the mysql connector/j

Re: [PR] feat(jdbc-catalog): Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
justinmclean commented on code in PR #4626: URL: https://github.com/apache/gravitino/pull/4626#discussion_r1728108316 ## catalogs/catalog-jdbc-oceanbase/src/main/resources/jdbc-oceanbase.conf: ## @@ -0,0 +1,22 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
justinmclean commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306011861 The MySQL connector and the Oceanbase client for Jave are licensed under the GPL/LGPL license. This is not compatible with the ALv2. -- This is an automated message from the

Re: [I] [Improvement] Support KerberosAuthProvider for Python client [gravitino]

2024-08-22 Thread via GitHub
xloya commented on issue #3756: URL: https://github.com/apache/gravitino/issues/3756#issuecomment-2306010061 > May i work on this issue ~ Sure, you can refer to the implementation in Java Client. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
yuanoOo commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306007892 > To clarify, we would need answers to: > > 1. What software does a user need to connect Gravitino to OceanBase and how is it licensed? > 2. Is that software truly optiona

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
yuanoOo commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2306002382 > I see a PR has been made. Are you sure this is compatible with the ALv2 and ASF distribution and licensing policies? @justinmclean I am not too clear about the compatibilit

Re: [I] [FEATURE] Support OceanBase JDBC Catalog [gravitino]

2024-08-22 Thread via GitHub
justinmclean commented on issue #4353: URL: https://github.com/apache/gravitino/issues/4353#issuecomment-2305997869 To clarify, we would need answers to: 1. What does a user need to connect Gravitino to OceanBase and how is it licensed? 2. Is that software truly optional? 3. Is that

[PR] [Minor] Remove URL parameter and add a coueple of more pages [gravitino]

2024-08-22 Thread via GitHub
justinmclean opened a new pull request, #4642: URL: https://github.com/apache/gravitino/pull/4642 ### What changes were proposed in this pull request? Remove URL parameter and add a coueple of more pages. ### Why are the changes needed? Track a few more useful pages. URL

  1   2   3   >