Re: [PR] [#6641] fix(core): Possible error in the equals method for collection [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 commented on PR #6644: URL: https://github.com/apache/gravitino/pull/6644#issuecomment-2712923004 @sunxiaojian Could you please resolve the conflict? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 commented on PR #6601: URL: https://github.com/apache/gravitino/pull/6601#issuecomment-2712920656 @unknowntpo Thanks for your contributions. -- 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

(gravitino) branch main updated: [#6474] improvement(storage): batch listing securable objects in RoleMetaService (#6601)

2025-03-11 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 0eec636655 [#6474] improvement(storage): batch

Re: [I] [Improvement] resolve todo to get other securable object fullNames using batch retrieving in function listSecurableObjects. [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 closed issue #6474: [Improvement] resolve todo to get other securable object fullNames using batch retrieving in function listSecurableObjects. URL: https://github.com/apache/gravitino/issues/6474 -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 merged PR #6601: URL: https://github.com/apache/gravitino/pull/6601 -- 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] [Minor] Improve log messages [gravitino]

2025-03-11 Thread via GitHub
justinmclean commented on code in PR #6531: URL: https://github.com/apache/gravitino/pull/6531#discussion_r1988552553 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationHDFSPlugin.java: ## @@ -361,9 +361,9 @@ private

Re: [PR] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-11 Thread via GitHub
unknowntpo commented on PR #6601: URL: https://github.com/apache/gravitino/pull/6601#issuecomment-2712931276 > > Although we batch object ids in same SQL queries, For the child metadata object, We still have to fetch id and names of parent object, this is similar to N+1 problems in GraphQL.

(gravitino) branch disclaimer deleted (was 62dec4d3c3)

2025-03-11 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch disclaimer in repository https://gitbox.apache.org/repos/asf/gravitino.git was 62dec4d3c3 move to using standard DISCLAIMER The revisions that were on this branch are still contained in ot

Re: [PR] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 commented on PR #6601: URL: https://github.com/apache/gravitino/pull/6601#issuecomment-2712898915 Generally 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

Re: [I] [FEATURE] The table supports InsertTable privilege [gravitino]

2025-03-11 Thread via GitHub
jerqi commented on issue #6668: URL: https://github.com/apache/gravitino/issues/6668#issuecomment-2713262501 You can use `ModifyTable`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

Re: [I] [FEATURE] The table supports InsertTable privilege [gravitino]

2025-03-11 Thread via GitHub
bw-chuxin closed issue #6668: [FEATURE] The table supports InsertTable privilege URL: https://github.com/apache/gravitino/issues/6668 -- 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 comm

Re: [I] [FEATURE] The table supports InsertTable privilege [gravitino]

2025-03-11 Thread via GitHub
bw-chuxin commented on issue #6668: URL: https://github.com/apache/gravitino/issues/6668#issuecomment-2713269080 ok,think you! -- 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

[PR] [#6079] fix(core): Support model failure event to Gravitino server [gravitino]

2025-03-11 Thread via GitHub
Abyss-lord opened a new pull request, #6670: URL: https://github.com/apache/gravitino/pull/6670 ### What changes were proposed in this pull request? Support model failure event to Gravitino server. ### Why are the changes needed? Fix: #6079 ### Does this PR introd

Re: [I] [Improvement] kerberos-hive file missing ASF headers [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed issue #6330: [Improvement] kerberos-hive file missing ASF headers URL: https://github.com/apache/gravitino/issues/6330 -- 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 c

Re: [PR] [#6641] fix(core): Possible error in the equals method for collection [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 merged PR #6644: URL: https://github.com/apache/gravitino/pull/6644 -- 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] [#6629] Improvement(jdbc-catalog): Optimize load database method in JdbcDatabaseOperations [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 merged PR #6642: URL: https://github.com/apache/gravitino/pull/6642 -- 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] [#6368] improvement(flink-connector): Add tests for REST Catalog to the Flink connector for Iceberg [gravitino]

2025-03-11 Thread via GitHub
sunxiaojian commented on code in PR #6622: URL: https://github.com/apache/gravitino/pull/6622#discussion_r1989641945 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/iceberg/FlinkIcebergRestCatalogIT.java: ## @@ -0,0 +1,135 @@ +/* + *

Re: [PR] [#6368] improvement(flink-connector): Add tests for REST Catalog to the Flink connector for Iceberg [gravitino]

2025-03-11 Thread via GitHub
sunxiaojian commented on code in PR #6622: URL: https://github.com/apache/gravitino/pull/6622#discussion_r1989627448 ## flink-connector/flink/build.gradle.kts: ## @@ -98,6 +98,8 @@ dependencies { testImplementation(libs.testcontainers.mysql) testImplementation(libs.metrics

(gravitino) branch branch-0.8 updated: [MINOR] Add note about known security vulnerabilities. (#6672)

2025-03-11 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 56053796fe [MINOR] Add note about know

Re: [PR] [#6368] improvement(flink-connector): Add tests for REST Catalog to the Flink connector for Iceberg [gravitino]

2025-03-11 Thread via GitHub
sunxiaojian commented on code in PR #6622: URL: https://github.com/apache/gravitino/pull/6622#discussion_r1989509846 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/FlinkEnvIT.java: ## @@ -212,4 +254,14 @@ protected static void clearT

Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-11 Thread via GitHub
hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1989610321 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/jdbc/GravitinoJdbcCatalogFactory.java: ## @@ -0,0 +1,70 @@ +/* + * Licensed to the Apache

Re: [PR] Refactor getting started docs [gravitino-site]

2025-03-11 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1987365598 ## docs/getting-started/index.md: ## @@ -0,0 +1,250 @@ +--- +title: "Getting started with Apache Gravitino" +slug: /getting-started +license: "This software is licens

[PR] A demo on how Gravitino implements metadata authorization. [gravitino]

2025-03-11 Thread via GitHub
hdygxsj opened a new pull request, #6676: URL: https://github.com/apache/gravitino/pull/6676 A demo on how Gravitino implements metadata authorization. -- 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 g

(gravitino) branch branch-0.8 updated: [#6649] feat(iceberg): make warehouse configable for Iceberg memory catalog backend (#6674)

2025-03-11 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 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 e51ac9f571 [#6649] feat(iceberg): m

Re: [PR] [#6649] feat(iceberg): make warehouse configable for Iceberg memory catalog backend [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 merged PR #6674: URL: https://github.com/apache/gravitino/pull/6674 -- 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: [#6629] Improvement(jdbc-catalog): Optimize load database method in JdbcDatabaseOperations (#6642)

2025-03-11 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 5d0f1a76b5 [#6629] Improvement(jdbc-catalog): O

Re: [PR] build(deps-dev): bump @next/bundle-analyzer from 14.2.24 to 15.2.1 in /web/web [gravitino]

2025-03-11 Thread via GitHub
LauraXia123 commented on PR #6660: URL: https://github.com/apache/gravitino/pull/6660#issuecomment-2714473807 @dependabot ignore this major 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 above to go to

Re: [PR] build(deps): bump @mui/x-data-grid from 6.20.4 to 7.27.3 in /web/web [gravitino]

2025-03-11 Thread via GitHub
dependabot[bot] commented on PR #6661: URL: https://github.com/apache/gravitino/pull/6661#issuecomment-2714475726 OK, I won't notify you about version 7.x.x again, unless you re-open this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-11 Thread via GitHub
FANNG1 commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1989377074 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/jdbc/GravitinoJdbcCatalogFactory.java: ## @@ -0,0 +1,70 @@ +/* + * Licensed to the Apache S

[PR] [#6330] Fix headers / license file [gravitino]

2025-03-11 Thread via GitHub
justinmclean opened a new pull request, #: URL: https://github.com/apache/gravitino/pull/ ### What changes were proposed in this pull request? Add 3rd party file ### Why are the changes needed? For ASF license policy compliance. Fix: #6330 ### Does t

Re: [PR] [#6654] fix(bundle): remove log4j from Gravitino cloud bundle jars [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 commented on PR #6664: URL: https://github.com/apache/gravitino/pull/6664#issuecomment-2712854510 @FANNG1 Can you select one jar and manually test its functionality with engines like Spark? -- This is an automated message from the Apache Git Service. To respond to the messag

Re: [I] [Improvement] kerberos-hive file missing ASF headers [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed issue #6330: [Improvement] kerberos-hive file missing ASF headers URL: https://github.com/apache/gravitino/issues/6330 -- 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 c

Re: [PR] [#6629] Improvement(jdbc-catalog): Optimize load database method in JdbcDatabaseOperations [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 commented on code in PR #6642: URL: https://github.com/apache/gravitino/pull/6642#discussion_r1989049873 ## catalogs/catalog-jdbc-common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcDatabaseOperations.java: ## @@ -107,6 +107,28 @@ public List listDataba

Re: [I] [Improvement] A typo on the official website page [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed issue #6635: [Improvement] A typo on the official website page URL: https://github.com/apache/gravitino/issues/6635 -- 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 comme

Re: [PR] [#6629] Improvement(jdbc-catalog): Optimize load database method in JdbcDatabaseOperations [gravitino]

2025-03-11 Thread via GitHub
sunxiaojian commented on code in PR #6642: URL: https://github.com/apache/gravitino/pull/6642#discussion_r1989066292 ## catalogs/catalog-jdbc-common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcDatabaseOperations.java: ## @@ -107,6 +107,28 @@ public List listDat

Re: [PR] [MINOR] Add note about known security vulnerabilities. [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #6672: URL: https://github.com/apache/gravitino/pull/6672 -- 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] [#6649] feat(iceberg): make warehouse configable for Iceberg memory catalog backend [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed pull request #6674: [#6649] feat(iceberg): make warehouse configable for Iceberg memory catalog backend URL: https://github.com/apache/gravitino/pull/6674 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

(gravitino) branch cherry-pick-branch-0.8-aa014fc776d93714096f9241ae52af7b7290d18f deleted (was f3783956c0)

2025-03-11 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-aa014fc776d93714096f9241ae52af7b7290d18f in repository https://gitbox.apache.org/repos/asf/gravitino.git was f3783956c0 [MINOR] Add note about known security vulnerabi

[PR] [#6667]fix(core):can force delete invalid catalog [gravitino]

2025-03-11 Thread via GitHub
liangyouze opened a new pull request, #6669: URL: https://github.com/apache/gravitino/pull/6669 ### What changes were proposed in this pull request? (Please outline the changes and how this PR fixes the issue.) #6667 ### Why are the changes needed? Fix: #6667

Re: [PR] [MINOR] Add note about known security vulnerabilities. [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed pull request #6672: [MINOR] Add note about known security vulnerabilities. URL: https://github.com/apache/gravitino/pull/6672 -- 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: [PR] [#6600] docs: Refactor the how-to-build docs [gravitino]

2025-03-11 Thread via GitHub
tengqm commented on PR #6659: URL: https://github.com/apache/gravitino/pull/6659#issuecomment-2714358974 /assign @jerryshao -- 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] Refactor getting started docs [gravitino-site]

2025-03-11 Thread via GitHub
tengqm commented on PR #49: URL: https://github.com/apache/gravitino-site/pull/49#issuecomment-2714316841 @LauraXia123 I tried `npm start` or `npm run build` followed by `npm run serve`, I cannot find the next version in the preview site. Am I missing something? -- This is an automat

Re: [PR] build(deps): bump @mui/x-data-grid from 6.20.4 to 7.27.3 in /web/web [gravitino]

2025-03-11 Thread via GitHub
LauraXia123 commented on PR #6661: URL: https://github.com/apache/gravitino/pull/6661#issuecomment-2714475217 @dependabot ignore this major 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 above to go to

(gravitino) branch dependabot/npm_and_yarn/web/web/eslint-config-next-15.2.1 deleted (was 5cd3e08f82)

2025-03-11 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/web/web/eslint-config-next-15.2.1 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 5cd3e08f82 build(deps-dev): bump eslint-config-next in /web

Re: [PR] build(deps): bump @mui/x-data-grid from 6.20.4 to 7.27.3 in /web/web [gravitino]

2025-03-11 Thread via GitHub
LauraXia123 closed pull request #6661: build(deps): bump @mui/x-data-grid from 6.20.4 to 7.27.3 in /web/web URL: https://github.com/apache/gravitino/pull/6661 -- 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] build(deps-dev): bump eslint-config-next from 14.0.3 to 15.2.1 in /web/web [gravitino]

2025-03-11 Thread via GitHub
LauraXia123 commented on PR #6662: URL: https://github.com/apache/gravitino/pull/6662#issuecomment-2714477210 @dependabot ignore this major 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 above to go to

Re: [PR] build(deps-dev): bump eslint-config-prettier from 9.1.0 to 10.1.1 in /web/web [gravitino]

2025-03-11 Thread via GitHub
LauraXia123 commented on PR #6663: URL: https://github.com/apache/gravitino/pull/6663#issuecomment-2714478286 @dependabot ignore this major 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 above to go to

Re: [PR] build(deps-dev): bump eslint-config-next from 14.0.3 to 15.2.1 in /web/web [gravitino]

2025-03-11 Thread via GitHub
dependabot[bot] commented on PR #6662: URL: https://github.com/apache/gravitino/pull/6662#issuecomment-2714477551 OK, I won't notify you about version 15.x.x again, unless you re-open this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please l

(gravitino) branch dependabot/npm_and_yarn/web/web/mui/x-data-grid-7.27.3 deleted (was 426c0fedd3)

2025-03-11 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/web/web/mui/x-data-grid-7.27.3 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 426c0fedd3 build(deps): bump @mui/x-data-grid from 6.20.4 to 7

(gravitino) branch dependabot/npm_and_yarn/web/web/next/bundle-analyzer-15.2.1 deleted (was 1fa407c9d3)

2025-03-11 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/web/web/next/bundle-analyzer-15.2.1 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 1fa407c9d3 build(deps-dev): bump @next/bundle-analyzer in

Re: [PR] [#6368] improvement(flink-connector): Add tests for REST Catalog to the Flink connector for Iceberg [gravitino]

2025-03-11 Thread via GitHub
FANNG1 commented on code in PR #6622: URL: https://github.com/apache/gravitino/pull/6622#discussion_r1989329150 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/FlinkEnvIT.java: ## @@ -212,4 +254,14 @@ protected static void clearTableI

Re: [PR] [#6654] fix(bundle): remove log4j from Gravitino cloud bundle jars [gravitino]

2025-03-11 Thread via GitHub
justinmclean commented on PR #6664: URL: https://github.com/apache/gravitino/pull/6664#issuecomment-2713024559 Thanks @FANNG1 for the license/notice comment - that seems fine to me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] [#6330] chore: add BSD license to missing files [gravitino]

2025-03-11 Thread via GitHub
justinmclean commented on PR #6355: URL: https://github.com/apache/gravitino/pull/6355#issuecomment-2713004503 @zhengkezhou1 thanks for your help with this. I just finished off the last little bit of work in #. -- This is an automated message from the Apache Git Service. To respond to

Re: [I] [Bug report] Possible error in the `equals` method for collection [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 closed issue #6641: [Bug report] Possible error in the `equals` method for collection URL: https://github.com/apache/gravitino/issues/6641 -- 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 th

Re: [PR] [#6330] Fix headers / license file [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #: URL: https://github.com/apache/gravitino/pull/ -- 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] [#3854] feat(core): Add a cache service to cache the mapping of name identifer to id [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 commented on PR #3986: URL: https://github.com/apache/gravitino/pull/3986#issuecomment-2713208943 > This isn't good enough. Maybe we need a pluggable cache framework to let production users to implement their cache. Maybe Xiaomi wants to implement Redis cache. cc @lw-yang Th

Re: [I] [Bug report] Possible error in the `equals` method for collection [gravitino]

2025-03-11 Thread via GitHub
yuqi1129 closed issue #6641: [Bug report] Possible error in the `equals` method for collection URL: https://github.com/apache/gravitino/issues/6641 -- 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 th

(gravitino) branch main updated: [#6641] fix(core): Possible error in the equals method for collection (#6644)

2025-03-11 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 fdcb7ae187 [#6641] fix(core): Possible error in

Re: [PR] Refactor getting started docs [gravitino-site]

2025-03-11 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1988410845 ## docs/getting-started/index.md: ## @@ -0,0 +1,250 @@ +--- +title: "Getting started with Apache Gravitino" +slug: /getting-started +license: "This software is licens

Re: [PR] [#6330] Fix headers / license file [gravitino]

2025-03-11 Thread via GitHub
jerryshao commented on PR #: URL: https://github.com/apache/gravitino/pull/#issuecomment-2713371010 @justinmclean this should also be backport to branch-0.8 if we need a non-WIP release for graduation. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [#3854] feat(core): Add a cache service to cache the mapping of name identifer to id [gravitino]

2025-03-11 Thread via GitHub
lw-yang commented on PR #3986: URL: https://github.com/apache/gravitino/pull/3986#issuecomment-2713296660 > This isn't good enough. Maybe we need a pluggable cache framework to let production users to implement their cache. Maybe Xiaomi wants to implement Redis cache. cc @lw-yang Yes

Re: [PR] [#3854] feat(core): Add a cache service to cache the mapping of name identifer to id [gravitino]

2025-03-11 Thread via GitHub
jerqi commented on PR #3986: URL: https://github.com/apache/gravitino/pull/3986#issuecomment-2713201360 This isn't good enough. Maybe we need a pluggable cache framework to let production users to implement their cache. Maybe Xiaomi wants to implement Redis cache. cc @lw-yang -- This is

(gravitino) 01/01: fix

2025-03-11 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a commit to branch test_pat in repository https://gitbox.apache.org/repos/asf/gravitino.git commit 17f68d01bd0396323f829325551521aba7dbb8af Author: yuqi AuthorDate: Tue Mar 11 18:02:18 2025 +0800 fix --- .g

(gravitino) branch cherry-pick-branch-0.8-20718cff26a4467119aea2b2caa552544887d328 created (now 05df74b52c)

2025-03-11 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-20718cff26a4467119aea2b2caa552544887d328 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 05df74b52c [#6667]fix(core):can force delete invalid

(gravitino) branch branch-0.8 updated: [#6623] fix(spark): Load JDBC driver explictly to fix `No suitable driver found for mysql:xx` error (#6673)

2025-03-11 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 92600c6b88 [#6623] fix(spark): Load JD

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

2025-03-11 Thread via GitHub
mchades commented on code in PR #6657: URL: https://github.com/apache/gravitino/pull/6657#discussion_r1988720750 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/jdbc/mysql/MySQLDataTypeTransformer.java: ## @@ -45,18 +45,36 @@ public

(gravitino) branch main updated: [MINOR] Add note about known security vulnerabilities. (#6665)

2025-03-11 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 aa014fc776 [MINOR] Add note about known security v

Re: [PR] [MINOR] Add note about known security vulnerabilities. [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #6665: URL: https://github.com/apache/gravitino/pull/6665 -- 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

[PR] [MINOR] Add note about known security vulnerabilities. [gravitino]

2025-03-11 Thread via GitHub
github-actions[bot] opened a new pull request, #6672: URL: https://github.com/apache/gravitino/pull/6672 ### What changes were proposed in this pull request? Add note about known security vulnerabilities ### Why are the changes needed? Some users are using older versions

(gravitino) branch cherry-pick-branch-0.8-78c0114ab7eb2eb97d85ae5f54b5774a01185a26 created (now cba0c606ac)

2025-03-11 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-78c0114ab7eb2eb97d85ae5f54b5774a01185a26 in repository https://gitbox.apache.org/repos/asf/gravitino.git at cba0c606ac [#6623] fix(spark): Load JDBC driver expli

(gravitino) branch main updated: [#6623] fix(spark): Load JDBC driver explictly to fix `No suitable driver found for mysql:xx` error (#6621)

2025-03-11 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 78c0114ab7 [#6623] fix(spark): Load JDBC driver ex

Re: [PR] [#6623] fix(spark): Load JDBC driver explictly to fix `No suitable driver found for mysql:xx` error [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #6621: URL: https://github.com/apache/gravitino/pull/6621 -- 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] use spark iceberg catalog error in spark sql client [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed issue #6623: [Bug report] use spark iceberg catalog error in spark sql client URL: https://github.com/apache/gravitino/issues/6623 -- 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 th

[PR] [#6623] fix(spark): Load JDBC driver explictly to fix `No suitable driver found for mysql:xx` error [gravitino]

2025-03-11 Thread via GitHub
github-actions[bot] opened a new pull request, #6673: URL: https://github.com/apache/gravitino/pull/6673 ### What changes were proposed in this pull request? If user set `spark.sql.hive.metastore.jars` to `path` in spark configuration and hive metastore uri is not set explictly, Spark wil

Re: [PR] [#3854] feat(core): Add a cache service to cache the mapping of name identifer to id [gravitino]

2025-03-11 Thread via GitHub
jerqi commented on PR #3986: URL: https://github.com/apache/gravitino/pull/3986#issuecomment-2713230567 > > This isn't good enough. Maybe we need a pluggable cache framework to let production users to implement their cache. Maybe Xiaomi wants to implement Redis cache. cc @lw-yang > >

Re: [PR] [#6467] improve(core): Allow JDBC backend configs be configurable [gravitino]

2025-03-11 Thread via GitHub
jerqi commented on code in PR #6469: URL: https://github.com/apache/gravitino/pull/6469#discussion_r1988778792 ## docs/gravitino-server-config.md: ## @@ -51,18 +51,20 @@ If you are going to use H2 in the production environment, Gravitino will not gua The following table list

(gravitino) branch test_pat created (now 17f68d01bd)

2025-03-11 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a change to branch test_pat in repository https://gitbox.apache.org/repos/asf/gravitino.git at 17f68d01bd fix This branch includes the following new commits: new 17f68d01bd fix The 1 revisions liste

(gravitino) branch test_pat updated: fix

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

(gravitino) branch cherry-pick-branch-0.8-aa014fc776d93714096f9241ae52af7b7290d18f created (now f3783956c0)

2025-03-11 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-aa014fc776d93714096f9241ae52af7b7290d18f in repository https://gitbox.apache.org/repos/asf/gravitino.git at f3783956c0 [MINOR] Add note about known security vuln

Re: [PR] [#6649] feat(iceberg): make warehouse configable for Iceberg memory catalog backend [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #6658: URL: https://github.com/apache/gravitino/pull/6658 -- 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: [#6649] feat(iceberg): make warehouse configable for Iceberg memory catalog backend (#6658)

2025-03-11 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 c934958f02 [#6649] feat(iceberg): make warehouse c

[I] [FEATURE] The table supports InsertTable privilege [gravitino]

2025-03-11 Thread via GitHub
bw-chuxin opened a new issue, #6668: URL: https://github.com/apache/gravitino/issues/6668 ### Describe the feature Can a InsertTable be added to the privilege? We are developing a data access gateway based on Gravitino, which needs to distinguish read and write permissions when check

[PR] [#5289][Improvement] CatalogMysqlDriverIT miss driver url [gravitino]

2025-03-11 Thread via GitHub
lsyulong opened a new pull request, #6671: URL: https://github.com/apache/gravitino/pull/6671 What changes were proposed in this pull request? #5289 [Improvement] CatalogMysqlDriverIT miss driver url Why are the changes needed? #5289 [Improvement] CatalogMysqlDriverIT miss driver

Re: [I] [Bug report] warehouse is hard code in Iceberg memory catalog backend [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed issue #6649: [Bug report] warehouse is hard code in Iceberg memory catalog backend URL: https://github.com/apache/gravitino/issues/6649 -- 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

Re: [I] [Bug report] warehouse is hard code in Iceberg memory catalog backend [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed issue #6649: [Bug report] warehouse is hard code in Iceberg memory catalog backend URL: https://github.com/apache/gravitino/issues/6649 -- 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

(gravitino) branch cherry-pick-branch-0.8-c934958f021f5341058d8b32f467cd8a10109817 created (now 317df3c5bb)

2025-03-11 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-c934958f021f5341058d8b32f467cd8a10109817 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 317df3c5bb [#6649] feat(iceberg): make warehouse conf

Re: [PR] [#6623] fix(spark): Load JDBC driver explictly to fix `No suitable driver found for mysql:xx` error [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed pull request #6673: [#6623] fix(spark): Load JDBC driver explictly to fix `No suitable driver found for mysql:xx` error URL: https://github.com/apache/gravitino/pull/6673 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] [#6649] feat(iceberg): make warehouse configable for Iceberg memory catalog backend [gravitino]

2025-03-11 Thread via GitHub
github-actions[bot] opened a new pull request, #6674: URL: https://github.com/apache/gravitino/pull/6674 ### What changes were proposed in this pull request? The warehouse location for Iceberg memory catalog backend is hard coded to '/tmp', make it configurable. ### Why

(gravitino) branch cherry-pick-branch-0.8-78c0114ab7eb2eb97d85ae5f54b5774a01185a26 deleted (was cba0c606ac)

2025-03-11 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-78c0114ab7eb2eb97d85ae5f54b5774a01185a26 in repository https://gitbox.apache.org/repos/asf/gravitino.git was cba0c606ac [#6623] fix(spark): Load JDBC driver explictly

Re: [PR] [#6667]fix(core):can force delete invalid catalog [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #6669: URL: https://github.com/apache/gravitino/pull/6669 -- 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: [#6667]fix(core):can force delete invalid catalog (#6669)

2025-03-11 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 20718cff26 [#6667]fix(core):can force delete inval

[PR] [#6667]fix(core):can force delete invalid catalog [gravitino]

2025-03-11 Thread via GitHub
github-actions[bot] opened a new pull request, #6675: URL: https://github.com/apache/gravitino/pull/6675 ### What changes were proposed in this pull request? (Please outline the changes and how this PR fixes the issue.) #6667 ### Why are the changes needed? Fi

Re: [I] [Bug report] The invalid JDBC catalog can never be deleted. [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed issue #6667: [Bug report] The invalid JDBC catalog can never be deleted. URL: https://github.com/apache/gravitino/issues/6667 -- 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: [PR] [#6667]fix(core):can force delete invalid catalog [gravitino]

2025-03-11 Thread via GitHub
jerryshao closed pull request #6675: [#6667]fix(core):can force delete invalid catalog URL: https://github.com/apache/gravitino/pull/6675 -- 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] [#6623] fix(spark): Load JDBC driver explictly to fix `No suitable driver found for mysql:xx` error [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #6673: URL: https://github.com/apache/gravitino/pull/6673 -- 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] Refactor getting started docs [gravitino-site]

2025-03-11 Thread via GitHub
justinmclean commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986972278 ## docs/getting-started/aws-remote-access.md: ## @@ -0,0 +1,36 @@ +--- +title: "Remote Access for Apache Gravitino on AWS" +slug: /getting-started/aws-remote-ac

Re: [PR] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-11 Thread via GitHub
unknowntpo commented on PR #6601: URL: https://github.com/apache/gravitino/pull/6601#issuecomment-2712367759 Although we batch object ids in same SQL queries, For the child metadata object, We still have to fetch id and names of parent object, this is similar to N+1 problems in GraphQL.

Re: [PR] [MINOR] fix(doc): Revise syntax errors in getting-started tutorial [gravitino]

2025-03-11 Thread via GitHub
jerryshao merged PR #6655: URL: https://github.com/apache/gravitino/pull/6655 -- 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 (826863c5b0 -> 153da2a22a)

2025-03-11 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 826863c5b0 [MINOR] fix(doc): Revise syntax errors in getting-started tutorial (#6655) add 153da2a22a [#6566] imp

(gravitino) branch dependabot/npm_and_yarn/web/web/eslint-config-next-15.2.1 created (now 5cd3e08f82)

2025-03-11 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/web/web/eslint-config-next-15.2.1 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 5cd3e08f82 build(deps-dev): bump eslint-config-next in /web

  1   2   3   >