Re: [PR] [#6656 ] fix(trino-connector): support read MySQL time/datetime/timestamp columns with precision [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on PR #6657: URL: https://github.com/apache/gravitino/pull/6657#issuecomment-2791797828 > > How do you handle the type of Timestamp without precision? Should a parameter be added to indicate using the default precision > > Hi @diqiu50, I understand your point of view.

Re: [PR] [#6852] improvement(core): Optimize SchemaOperationDispatcher to get entity id when loading schema [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6853: URL: https://github.com/apache/gravitino/pull/6853#discussion_r2036746720 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -427,7 +442,7 @@ private EntityCombinedSchema internalLoadSchema(NameIdentifie

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2037269982 ## lineage/src/main/java/org/apache/gravitino/lineage/source/HTTPLineageSource.java: ## @@ -31,6 +32,6 @@ public void initialize(Map configs, LineageDispatcher dispat

Re: [PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on code in PR #6866: URL: https://github.com/apache/gravitino/pull/6866#discussion_r2037273299 ## core/src/main/java/org/apache/gravitino/listener/api/event/CreateRoleFailureEvent.java: ## @@ -33,62 +29,29 @@ */ @DeveloperApi public class CreateRoleFailureEv

Re: [I] [FEATURE] Support for column type e.g. array, array, array, etc. [gravitino]

2025-04-10 Thread via GitHub
mchades commented on issue #6875: URL: https://github.com/apache/gravitino/issues/6875#issuecomment-2793762674 It's outdated, would you like to fix the 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 a

[PR] [MINOR] docs: fix several typo in docs [gravitino]

2025-04-10 Thread via GitHub
zhoukangcn opened a new pull request, #6885: URL: https://github.com/apache/gravitino/pull/6885 ### What changes were proposed in this pull request? - fix several typo in docs - add .vscode to gitignore ### Why are the changes needed? typo ### Does this PR intro

[PR] [#6722] fix(jdbc-doris): use `show backends` to get the count of Doris Backend (BE) nodes [gravitino]

2025-04-10 Thread via GitHub
zhoukangcn opened a new pull request, #6884: URL: https://github.com/apache/gravitino/pull/6884 ### What changes were proposed in this pull request? use `show backends` to get the count of Doris Backend (BE) nodes ### Why are the changes needed? (Please clarify why the ch

Re: [PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord commented on PR #6866: URL: https://github.com/apache/gravitino/pull/6866#issuecomment-2794492002 Hi @FANNG1 , I've completed the code updates and would appreciate your review of the PR when you have a moment. Here's a summary of the commits: 1. rename the argument. -- This

Re: [PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on code in PR #6866: URL: https://github.com/apache/gravitino/pull/6866#discussion_r2036897349 ## core/src/main/java/org/apache/gravitino/listener/api/event/AccessControlEventDispatcher.java: ## @@ -346,7 +340,8 @@ public Role createRole( return roleObj

[PR] [#6632] improve(core): Add timestamp information to metrics [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord opened a new pull request, #6882: URL: https://github.com/apache/gravitino/pull/6882 ### What changes were proposed in this pull request? Add timestamp information to metrics ### Why are the changes needed? Fix: #6632 ### Does this PR introduce _any_ us

Re: [PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on PR #6866: URL: https://github.com/apache/gravitino/pull/6866#issuecomment-2792109164 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] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord commented on PR #6866: URL: https://github.com/apache/gravitino/pull/6866#issuecomment-2792315533 Hi @FANNG1 , I've completed the code updates and would appreciate your review of the PR when you have a moment. Here's a summary of the commits: 1. Use `RoleInfo` in `CreateRoleFai

Re: [PR] [#6632] improve(core): Add timestamp information to metrics [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord commented on PR #6882: URL: https://github.com/apache/gravitino/pull/6882#issuecomment-2792316326 Hi @FANNG1 , could you please review this PR when you have time? I’d really appreciate your feedback. -- This is an automated message from the Apache Git Service. To respond to the

[PR] [#6880] improve(core): Refactor EventBus#dispatchEvent [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord opened a new pull request, #6881: URL: https://github.com/apache/gravitino/pull/6881 ### What changes were proposed in this pull request? Refactor EventBus#dispatchEvent. By adding the `accept(EventVisitor visitor)` method to BaseEvent and its subclasses, the handling l

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036411558 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[I] [FEATURE] In version 0.9, make the underlying Casbin permissions an optional plugin. [gravitino]

2025-04-10 Thread via GitHub
an-shi-chi-fan opened a new issue, #6883: URL: https://github.com/apache/gravitino/issues/6883 ### Describe the feature In our existing product, we have implemented Ranger for authorization, and there are already a large number of authorization policies in Ranger. We want to integrat

Re: [I] [FEATURE] In version 0.9, make the underlying Casbin permissions an optional plugin. [gravitino]

2025-04-10 Thread via GitHub
an-shi-chi-fan commented on issue #6883: URL: https://github.com/apache/gravitino/issues/6883#issuecomment-2792538894 @xunliu please cc -- 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

(gravitino) 01/01: Merge branch 'main' of github.com:apache/gravitino into issue_6852

2025-04-10 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a commit to branch issue_6852 in repository https://gitbox.apache.org/repos/asf/gravitino.git commit 7620404c3a6a45ab720c622c62c6dcf45f70371a Merge: 634a7ca565 ecf6534c54 Author: yuqi AuthorDate: Thu Apr 10 15:55

Re: [PR] [#6486] feat(iceberg): supports change Iceberg request in pre event listener [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on PR #6485: URL: https://github.com/apache/gravitino/pull/6485#issuecomment-2791861990 @jerryshao Do we need include this PR in 0.9 version? -- 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] [#6852] improvement(core): Optimize SchemaOperationDispatcher to get entity id when loading schema [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 commented on code in PR #6853: URL: https://github.com/apache/gravitino/pull/6853#discussion_r2036748863 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -427,7 +442,7 @@ private EntityCombinedSchema internalLoadSchema(NameIdenti

(gravitino) branch issue_6852 updated: Remove method isEntityExists

2025-04-10 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a commit to branch issue_6852 in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/issue_6852 by this push: new 985b7c2942 Remove method isEntityEx

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036786770 ## lineage/src/main/java/org/apache/gravitino/lineage/source/HTTPLineageSource.java: ## @@ -31,6 +32,6 @@ public void initialize(Map configs, LineageDispatcher disp

Re: [PR] [#6852] improvement(core): Optimize SchemaOperationDispatcher to get entity id when loading schema [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6853: URL: https://github.com/apache/gravitino/pull/6853#discussion_r2036698958 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -254,7 +254,11 @@ public Schema alterSchema(NameIdentifier ident, SchemaChange

Re: [PR] [#6880] improve(core): Refactor EventBus#dispatchEvent [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord commented on PR #6881: URL: https://github.com/apache/gravitino/pull/6881#issuecomment-2792033889 Hi @FANNG1 , could you please review this PR when you have time? I’d really appreciate your feedback. -- This is an automated message from the Apache Git Service. To respond to the

Re: [I] [EPIC] Python Task Summarization [gravitino]

2025-04-10 Thread via GitHub
xunliu commented on issue #6879: URL: https://github.com/apache/gravitino/issues/6879#issuecomment-2791782545 hi @Abyss-lord, Thanks for helping to organize the issue! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation [gravitino]

2025-04-10 Thread via GitHub
zhengkezhou1 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2791472287 Sure. -- 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 unsu

[I] [Bug report] Possible wrong git reference used in release script [gravitino]

2025-04-10 Thread via GitHub
justinmclean opened a new issue, #6869: URL: https://github.com/apache/gravitino/issues/6869 ### Version main branch ### Describe what's wrong The release script use this to get the git reference: ``` GIT_REF=${GIT_REF:-main} ``` And later checks out the r

(gravitino-site) 01/01: change tagline

2025-04-10 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a commit to branch tag-line in repository https://gitbox.apache.org/repos/asf/gravitino-site.git commit e35563ad30e8857b4f35ce6a6f834ddd2a48bbed Author: Justin Mclean AuthorDate: Tue Apr 8 09:25:54 2025 +0800

(gravitino) branch dependabot/gradle/iceberg-1.8.1 deleted (was 3993123569)

2025-04-10 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/gradle/iceberg-1.8.1 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 3993123569 build(deps): bump iceberg from 1.6.1 to 1.8.1 The revisions that were on t

(gravitino) branch issue_6852 updated: fix

2025-04-10 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a commit to branch issue_6852 in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/issue_6852 by this push: new d33dc7a506 fix d33dc7a506 is descri

Re: [PR] [#6697] fix: filter out all PG Gravitino table properties [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on code in PR #6824: URL: https://github.com/apache/gravitino/pull/6824#discussion_r202168 ## spark-connector/spark-common/src/test/java/org/apache/gravitino/spark/connector/integration/test/jdbc/SparkJdbcPostgreSqlCatalogIT.java: ## @@ -0,0 +1,108 @@ +/* +

(gravitino) branch main updated: [MINOR] docs: fix several typo in docs (#6885)

2025-04-10 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 a685e0e276 [MINOR] docs: fix several typo in docs

[PR] [MINOR] docs: fix several typo in docs [gravitino]

2025-04-10 Thread via GitHub
github-actions[bot] opened a new pull request, #6886: URL: https://github.com/apache/gravitino/pull/6886 ### What changes were proposed in this pull request? - fix several typo in docs - add .vscode to gitignore ### Why are the changes needed? typo ### Does this

Re: [PR] [MINOR] docs: fix several typo in docs [gravitino]

2025-04-10 Thread via GitHub
jerryshao merged PR #6885: URL: https://github.com/apache/gravitino/pull/6885 -- 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.8 updated: [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue (#6876)

2025-04-10 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a commit to branch branch-0.8 in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/branch-0.8 by this push: new cd44ee49be [#6777] fix(server,authn):

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-10 Thread via GitHub
jerryshao merged PR #6876: URL: https://github.com/apache/gravitino/pull/6876 -- 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.8-a685e0e276b807a75a50e7a8a8c96e81afe00878 created (now 95ef21f0aa)

2025-04-10 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.8-a685e0e276b807a75a50e7a8a8c96e81afe00878 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 95ef21f0aa [MINOR] docs: fix several typo in docs (#6

Re: [PR] [MINOR] docs: fix several typo in docs [gravitino]

2025-04-10 Thread via GitHub
jerryshao closed pull request #6886: [MINOR] docs: fix several typo in docs URL: https://github.com/apache/gravitino/pull/6886 -- 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. T

(gravitino) branch cherry-pick-branch-0.8-ecf6534c543be5704d5824c9cffb2b3e8dc09965 deleted (was d785398d55)

2025-04-10 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch cherry-pick-branch-0.8-ecf6534c543be5704d5824c9cffb2b3e8dc09965 in repository https://gitbox.apache.org/repos/asf/gravitino.git was d785398d55 [#6777] fix(server,authn): Fix use catalog and

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
jerryshao commented on PR #6865: URL: https://github.com/apache/gravitino/pull/6865#issuecomment-2794684301 LGTM, @yuqi1129 do you have any more 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 t

Re: [I] [FEATURE] Support for column type e.g. array, array, array, etc. [gravitino]

2025-04-10 Thread via GitHub
rickyma commented on issue #6875: URL: https://github.com/apache/gravitino/issues/6875#issuecomment-2791721487 I see. ``` A list type. Note, this type is not supported in the current version of Gravitino. ``` But why does it say it is not supported: https://github.co

Re: [PR] [#6704] feat(authz): Support model hook dispatcher [gravitino]

2025-04-10 Thread via GitHub
jerryshao commented on code in PR #6844: URL: https://github.com/apache/gravitino/pull/6844#discussion_r2032356694 ## core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/postgresql/OwnerMetaPostgreSQLProvider.java: ## @@ -116,6 +122,11 @@ public String sot

[I] [FEATURE] Gracefully handle views for proxying catalogs that don't support them Trino [gravitino]

2025-04-10 Thread via GitHub
jkolash opened a new issue, #6887: URL: https://github.com/apache/gravitino/issues/6887 ### Describe the feature When proxying a catalog that doesn't support views we can return empty/absent results instead of failing on view requests ``` io.trino.spi.TrinoException: Fa

Re: [PR] [#6841] feat: Introduce an authorization framework. [gravitino]

2025-04-10 Thread via GitHub
hdygxsj commented on PR #6842: URL: https://github.com/apache/gravitino/pull/6842#issuecomment-2786310387 > > I think maybe we need move these java file to `server-common/src/main/java/org/apache/gravitino/server/authentication` path? > > Is this the authentication or authorization? I

Re: [I] [Bug report] Iceberg tables are not visible by using spark connector [gravitino]

2025-04-10 Thread via GitHub
FANNG1 closed issue #6877: [Bug report] Iceberg tables are not visible by using spark connector URL: https://github.com/apache/gravitino/issues/6877 -- 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 issue_6852 updated: fix

2025-04-10 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a commit to branch issue_6852 in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/issue_6852 by this push: new d1f03d5026 fix d1f03d5026 is descri

(gravitino) branch ISSUE-6353 deleted (was 1de53e55e2)

2025-04-10 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch ISSUE-6353 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 1de53e55e2 fix ut The revisions that were on this branch are still contained in other references; therefore, th

(gravitino) branch dependabot/gradle/com.github.jk1.dependency-license-report-2.9 created (now 7d735a4080)

2025-04-10 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/gradle/com.github.jk1.dependency-license-report-2.9 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 7d735a4080 build(deps): bump com.github.jk1.dependency

Re: [I] [Improvement] Currently path based ranger plugin doesn't support schema and catalog [gravitino]

2025-04-10 Thread via GitHub
xunliu closed issue #6692: [Improvement] Currently path based ranger plugin doesn't support schema and catalog URL: https://github.com/apache/gravitino/issues/6692 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [I] [Improvement] Currently path based ranger plugin doesn't support schema and catalog [gravitino]

2025-04-10 Thread via GitHub
xunliu closed issue #6692: [Improvement] Currently path based ranger plugin doesn't support schema and catalog URL: https://github.com/apache/gravitino/issues/6692 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] [#6692] improvement(authz): Support schema level for Ranger path based authz plugin [gravitino]

2025-04-10 Thread via GitHub
xunliu merged PR #6864: URL: https://github.com/apache/gravitino/pull/6864 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@gravitino

(gravitino) branch main updated (a685e0e276 -> e38efeb7ea)

2025-04-10 Thread liuxun
This is an automated email from the ASF dual-hosted git repository. liuxun pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git from a685e0e276 [MINOR] docs: fix several typo in docs (#6885) add e38efeb7ea [#6692] improvement(authz): Support sch

Re: [I] [EPIC] Gravitino event listener bus system [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on issue #2233: URL: https://github.com/apache/gravitino/issues/2233#issuecomment-2795579082 We could leave it open as we may support other events for Gravitino -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [I] [Improvement] Change `SchemaOperationDispatcher` to get schema id when load schemas. [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 closed issue #6852: [Improvement] Change `SchemaOperationDispatcher` to get schema id when load schemas. URL: https://github.com/apache/gravitino/issues/6852 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [I] [Improvement] Change `SchemaOperationDispatcher` to get schema id when load schemas. [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 closed issue #6852: [Improvement] Change `SchemaOperationDispatcher` to get schema id when load schemas. URL: https://github.com/apache/gravitino/issues/6852 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [I] [FEATURE] Gracefully handle views for proxying iceberg catalogs that don't support them [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on issue #6887: URL: https://github.com/apache/gravitino/issues/6887#issuecomment-2795515295 #6396 attempt to fix it, please feel free to leave your comments. For the trino side, the configuration `iceberg.rest-catalog.view-endpoints-enabled` is used to keep compatibility

Re: [I] [Subtask] [event-listener] supports user&group&role event for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 closed issue #2969: [Subtask] [event-listener] supports user&group&role event for Gravitino server URL: https://github.com/apache/gravitino/issues/2969 -- 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] [#6852] improvement(core): Optimize SchemaOperationDispatcher to get entity id when loading schema [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 merged PR #6853: URL: https://github.com/apache/gravitino/pull/6853 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@graviti

(gravitino) branch main updated (b6a61bafb5 -> ff32ceff22)

2025-04-10 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git from b6a61bafb5 [#6760] test(core): Support role failure event to Gravitino server (#6866) add ff32ceff22 [#6852]

Re: [PR] docs: add instruction to set up IDE for python-client [gravitino]

2025-04-10 Thread via GitHub
unknowntpo commented on PR #6888: URL: https://github.com/apache/gravitino/pull/6888#issuecomment-2795599440 @Abyss-lord @yuqi1129 @xunliu Would you like 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

Re: [PR] docs: add instruction to set up IDE for python-client [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord commented on PR #6888: URL: https://github.com/apache/gravitino/pull/6888#issuecomment-2795668223 @unknowntpo Should we add a new scene if we are using Vscode or Pycharm to develop Python Client? -- This is an automated message from the Apache Git Service. To respond to the mes

Re: [PR] [#6722] fix(jdbc-doris): use `show backends` to get the count of Doris BE nodes [gravitino]

2025-04-10 Thread via GitHub
mchades commented on code in PR #6884: URL: https://github.com/apache/gravitino/pull/6884#discussion_r2038700024 ## catalogs/catalog-jdbc-doris/src/main/java/org/apache/gravitino/catalog/doris/operation/DorisTableOperations.java: ## @@ -159,24 +159,28 @@ private Map appendNeces

Re: [I] [EPIC] Gravitino event listener bus system [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord commented on issue #2233: URL: https://github.com/apache/gravitino/issues/2233#issuecomment-2795568775 @FANNG1 This Issue seems to be done as well。 -- 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] docs: add instruction to set up JetBrains Intellij IDEA for python-client [gravitino]

2025-04-10 Thread via GitHub
unknowntpo commented on PR #6888: URL: https://github.com/apache/gravitino/pull/6888#issuecomment-2795694517 I changed this PR back to draft PR because when I run `TestCatalog` with Intellij, I got ``` /Users/unknowntpo/repo/unknowntpo/gravitino/improve-py-client-doc/.gradle/pytho

Re: [PR] docs: add instruction to set up JetBrains Intellij IDEA for python-client [gravitino]

2025-04-10 Thread via GitHub
unknowntpo commented on PR #6888: URL: https://github.com/apache/gravitino/pull/6888#issuecomment-2795692343 > @unknowntpo Should we add a new scene if we are using Vscode or Pycharm to develop Python Client? Yeah, but I think we should do this in different PR. -- This is an automa

Re: [PR] [#6594] feat(deploy): Provide gravitino chart for deploying different storage backend on K8S [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6599: URL: https://github.com/apache/gravitino/pull/6599#discussion_r2038741824 ## dev/charts/gravitino/values.yaml: ## @@ -0,0 +1,422 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. S

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 commented on PR #6865: URL: https://github.com/apache/gravitino/pull/6865#issuecomment-2795746606 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 unsubscri

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 merged PR #6865: URL: https://github.com/apache/gravitino/pull/6865 -- 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] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2038740087 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,77 @@ +/* + * Licensed to the Apache Software Foundation (AS

Re: [I] [Subtask] Support Lineage endpoint for Gravitino [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 closed issue #6780: [Subtask] Support Lineage endpoint for Gravitino URL: https://github.com/apache/gravitino/issues/6780 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment

(gravitino) branch main updated: [#6780] feat(lineage): support lineage endpoint for Gravitino server (#6865)

2025-04-10 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 842cfc3a46 [#6780] feat(lineage): support line

Re: [PR] [#6594] feat(deploy): Provide gravitino chart for deploying different storage backend on K8S [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6599: URL: https://github.com/apache/gravitino/pull/6599#discussion_r2038738236 ## dev/charts/gravitino/README.md: ## @@ -0,0 +1,151 @@ + + +# Apache Gravitino Review Comment: Could you put the documents in our docs directory instead of managin

Re: [PR] [#6759] test(core): Support role event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
Abyss-lord commented on PR #6851: URL: https://github.com/apache/gravitino/pull/6851#issuecomment-2782286750 Hi @FANNG1 , could you please review this PR when you have time? I’d really appreciate your feedback. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [#6779] feat(core): Support lineage framework in Gravitino [gravitino]

2025-04-10 Thread via GitHub
jerryshao commented on code in PR #6782: URL: https://github.com/apache/gravitino/pull/6782#discussion_r2031687553 ## lineage/src/main/java/org/apache/gravitino/lineage/sink/LineageLogSink.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [#6852] improvement(core): Optimize SchemaOperationDispatcher to get entity id when loading schema [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 commented on PR #6853: URL: https://github.com/apache/gravitino/pull/6853#issuecomment-2791815478 @jerqi Could you please help me 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 t

Re: [PR] feat(dev/docker/ranger): build ranger packages in ranger-dependency.sh [gravitino]

2025-04-10 Thread via GitHub
xunliu commented on code in PR #6848: URL: https://github.com/apache/gravitino/pull/6848#discussion_r2032447990 ## dev/docker/hive/hive-dependency.sh: ## @@ -89,7 +90,9 @@ if [ ! -f "${hive_dir}/packages/${ZOOKEEPER_PACKAGE_NAME}" ]; then fi if [ ! -f "${hive_dir}/packages/

(gravitino) branch cherry-pick-branch-0.8-ecf6534c543be5704d5824c9cffb2b3e8dc09965 created (now d785398d55)

2025-04-10 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.8-ecf6534c543be5704d5824c9cffb2b3e8dc09965 in repository https://gitbox.apache.org/repos/asf/gravitino.git at d785398d55 [#6777] fix(server,authn): Fix use catalog

Re: [PR] [#6695] fix(authz): Fix the error privileges [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6821: URL: https://github.com/apache/gravitino/pull/6821#discussion_r2033159837 ## authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java: ## @@ -92,15 +92,26 @@ public String toStri

Re: [PR] [#6852] improvement(core): Optimize SchemaOperationDispatcher to get entity id when loading schema [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6853: URL: https://github.com/apache/gravitino/pull/6853#discussion_r2033126256 ## core/src/main/java/org/apache/gravitino/catalog/OperationDispatcher.java: ## @@ -243,12 +243,16 @@ protected boolean isEntityExist(NameIdentifier ident, Entity.Enti

Re: [PR] [#6697] fix: filter out all PG Gravitino table properties [gravitino]

2025-04-10 Thread via GitHub
FANNG1 commented on code in PR #6824: URL: https://github.com/apache/gravitino/pull/6824#discussion_r2033325647 ## spark-connector/spark-common/src/main/java/org/apache/gravitino/spark/connector/jdbc/postgersql/PostgreSqlPropertiesConverter.java: ## @@ -0,0 +1,75 @@ +/* + * Lice

(gravitino) branch main updated (818362b0ea -> 8d0ac2c2b2)

2025-04-10 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git from 818362b0ea [6346] feat(authz): Add privilege support for model (#6820) add 8d0ac2c2b2 build(deps): bump com.githu

(gravitino) branch issue_6852 updated (d1f03d5026 -> 7620404c3a)

2025-04-10 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a change to branch issue_6852 in repository https://gitbox.apache.org/repos/asf/gravitino.git from d1f03d5026 fix add 634a7ca565 Resolve comments. add 818362b0ea [6346] feat(authz): Add privilege su

Re: [PR] [Docs] Rework docs [gravitino]

2025-04-10 Thread via GitHub
justinmclean commented on code in PR #6849: URL: https://github.com/apache/gravitino/pull/6849#discussion_r2034344463 ## docs/admin/iceberg-server.md: ## @@ -0,0 +1,1351 @@ +--- +title: Iceberg REST catalog service +slug: /iceberg-rest-service +keywords: + - Iceberg REST catalo

(gravitino) branch main updated: [#6760] test(core): Support role failure event to Gravitino server (#6866)

2025-04-10 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 b6a61bafb5 [#6760] test(core): Support role failur

Re: [I] [Subtask] Support role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 closed issue #6760: [Subtask] Support role failure event to Gravitino server URL: https://github.com/apache/gravitino/issues/6760 -- 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] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 merged PR #6866: URL: https://github.com/apache/gravitino/pull/6866 -- 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 role failure event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 closed issue #6760: [Subtask] Support role failure event to Gravitino server URL: https://github.com/apache/gravitino/issues/6760 -- 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: [I] [Subtask] Support role-related event to Gravitino server [gravitino]

2025-04-10 Thread via GitHub
FANNG1 closed issue #6757: [Subtask] Support role-related event to Gravitino server URL: https://github.com/apache/gravitino/issues/6757 -- 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

svn commit: r76197 - in /dev/incubator/gravitino/v0.8.1-incubating-rc3: gravitino-0.8.1-incubating-src.tar.gz gravitino-0.8.1-incubating-src.tar.gz.asc gravitino-0.8.1-incubating-src.tar.gz.sha512

2025-04-10 Thread jmclean
Author: jmclean Date: Fri Apr 11 01:08:05 2025 New Revision: 76197 Log: Removed gradle jar from source release Modified: dev/incubator/gravitino/v0.8.1-incubating-rc3/gravitino-0.8.1-incubating-src.tar.gz dev/incubator/gravitino/v0.8.1-incubating-rc3/gravitino-0.8.1-incubating-src.tar.g

[PR] docs: add instruction to set up IDE for python-client [gravitino]

2025-04-10 Thread via GitHub
unknowntpo opened a new pull request, #6888: URL: https://github.com/apache/gravitino/pull/6888 ### What changes were proposed in this pull request? Add instruction to setup Python Client environment in JetBrains Intellij IDEA. ### Why are the changes needed? Mak

Re: [PR] [MINOR] docs: fix several typo in docs [gravitino]

2025-04-10 Thread via GitHub
mchades commented on code in PR #6886: URL: https://github.com/apache/gravitino/pull/6886#discussion_r2038730136 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -320,6 +320,115 @@ Currently, Gravitino supports two **types** of filesets: The `storageLocation` is the ph

Re: [PR] [#6775] feat(server): Introduce authorization annotations [gravitino]

2025-04-10 Thread via GitHub
xunliu commented on code in PR #6828: URL: https://github.com/apache/gravitino/pull/6828#discussion_r2038735497 ## common/src/main/java/org/apache/gravitino/auth/annotations/AuthorizeApi.java: ## @@ -0,0 +1,72 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#6594] feat(deploy): Provide gravitino chart for deploying different storage backend on K8S [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6599: URL: https://github.com/apache/gravitino/pull/6599#discussion_r2038737198 ## .github/workflows/chart-test.yaml: ## @@ -0,0 +1,85 @@ +name: Test Charts + +# Controls when the workflow will run +on: + pull_request: +branches: [ "main", "br

Re: [PR] [#6594] feat(deploy): Provide gravitino chart for deploying different storage backend on K8S [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6599: URL: https://github.com/apache/gravitino/pull/6599#discussion_r2038737198 ## .github/workflows/chart-test.yaml: ## @@ -0,0 +1,85 @@ +name: Test Charts + +# Controls when the workflow will run +on: + pull_request: +branches: [ "main", "br

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2038738368 ## lineage/src/main/java/org/apache/gravitino/lineage/source/HTTPLineageSource.java: ## @@ -31,6 +32,6 @@ public void initialize(Map configs, LineageDispatcher disp

Re: [PR] [#6594] feat(deploy): Provide gravitino chart for deploying different storage backend on K8S [gravitino]

2025-04-10 Thread via GitHub
jerqi commented on code in PR #6599: URL: https://github.com/apache/gravitino/pull/6599#discussion_r2038738497 ## dev/charts/gravitino/README.md: ## @@ -0,0 +1,151 @@ + + +# Apache Gravitino + +Apache Gravitino is a high-performance, geo-distributed, and federated metadata lake

(gravitino) branch main updated: [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation (#6540)

2025-04-10 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 ca7175e067 [#4024]Refactor: Reduce unnecessary

Re: [PR] [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 merged PR #6540: URL: https://github.com/apache/gravitino/pull/6540 -- 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] [#6722] fix(jdbc-doris): use `show backends` to get the count of Doris BE nodes [gravitino]

2025-04-10 Thread via GitHub
zhoukangcn commented on code in PR #6884: URL: https://github.com/apache/gravitino/pull/6884#discussion_r2038780800 ## catalogs/catalog-jdbc-doris/src/main/java/org/apache/gravitino/catalog/doris/operation/DorisTableOperations.java: ## @@ -159,24 +159,28 @@ private Map appendNe

Re: [PR] [#6852] improvement(core): Optimize SchemaOperationDispatcher to get entity id when loading schema [gravitino]

2025-04-10 Thread via GitHub
yuqi1129 commented on PR #6853: URL: https://github.com/apache/gravitino/pull/6853#issuecomment-2785099168 > > > > > > OperationDispatcher > > > > > > > > > > > > > > > OK, let me take a look. > > > > > > > > > > > > It seems that we can't use it directly as the method `

  1   2   >