[PR] build(deps): bump docker/setup-buildx-action from 2 to 3 [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6585: URL: https://github.com/apache/gravitino/pull/6585 Bumps [docker/setup-buildx-action](https://github.com/docker/setup-buildx-action) from 2 to 3. Release notes Sourced from https://github.com/docker/setup-buildx-action/releases";>d

(gravitino) branch dependabot/github_actions/carloscastrojumo/github-cherry-pick-action-1.0.10 created (now dcd5b95ffe)

2025-03-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/carloscastrojumo/github-cherry-pick-action-1.0.10 in repository https://gitbox.apache.org/repos/asf/gravitino.git at dcd5b95ffe build(deps): bump carloscastro

[PR] build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6587: URL: https://github.com/apache/gravitino/pull/6587 Bumps [carloscastrojumo/github-cherry-pick-action](https://github.com/carloscastrojumo/github-cherry-pick-action) from 1.0.9 to 1.0.10. Release notes Sourced from https://github.co

(gravitino) branch dependabot/github_actions/docker/login-action-3 created (now 4a11eb3dde)

2025-03-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/docker/login-action-3 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 4a11eb3dde build(deps): bump docker/login-action from 2 to 3 No new

[PR] build(deps-dev): bump tailwindcss from 3.4.17 to 4.0.9 in /web/web [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6588: URL: https://github.com/apache/gravitino/pull/6588 Bumps [tailwindcss](https://github.com/tailwindlabs/tailwindcss/tree/HEAD/packages/tailwindcss) from 3.4.17 to 4.0.9. Release notes Sourced from https://github.com/tailwindlabs/tai

[PR] [MINOR] docs: fix typo in docs/model-catalog.md [gravitino]

2025-03-03 Thread via GitHub
zhoukangcn opened a new pull request, #6584: URL: https://github.com/apache/gravitino/pull/6584 ### What changes were proposed in this pull request? fix typo in docs/model-catalog.md ### Why are the changes needed? typo ### Does this PR introduce _any_ user-facing

[PR] build(deps): bump @mui/x-tree-view from 6.17.0 to 7.26.0 in /web/web [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6590: URL: https://github.com/apache/gravitino/pull/6590 Bumps [@mui/x-tree-view](https://github.com/mui/mui-x/tree/HEAD/packages/x-tree-view) from 6.17.0 to 7.26.0. Release notes Sourced from https://github.com/mui/mui-x/releases";>@​mu

[PR] build(deps-dev): bump @iconify/react from 4.1.1 to 5.2.0 in /web/web [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6591: URL: https://github.com/apache/gravitino/pull/6591 Bumps [@iconify/react](https://github.com/iconify/iconify/tree/HEAD/components/react) from 4.1.1 to 5.2.0. Commits See full diff in https://github.com/iconify/iconify/commits/H

(gravitino) branch dependabot/npm_and_yarn/web/web/tailwindcss-4.0.9 created (now d38be39532)

2025-03-03 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/tailwindcss-4.0.9 in repository https://gitbox.apache.org/repos/asf/gravitino.git at d38be39532 build(deps-dev): bump tailwindcss from 3.4.17 to 4.0.9 i

(gravitino) branch dependabot/npm_and_yarn/web/web/mui/x-tree-view-7.26.0 created (now 53ec8f62be)

2025-03-03 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-tree-view-7.26.0 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 53ec8f62be build(deps): bump @mui/x-tree-view from 6.17.0 to 7

[PR] build(deps): bump docker/login-action from 2 to 3 [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6586: URL: https://github.com/apache/gravitino/pull/6586 Bumps [docker/login-action](https://github.com/docker/login-action) from 2 to 3. Release notes Sourced from https://github.com/docker/login-action/releases";>docker/login-action's

(gravitino) branch dependabot/npm_and_yarn/web/web/iconify/react-5.2.0 created (now 34f1370f52)

2025-03-03 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/iconify/react-5.2.0 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 34f1370f52 build(deps-dev): bump @iconify/react from 4.1.1 to 5.2

(gravitino) branch dependabot/github_actions/docker/setup-buildx-action-3 created (now e2f59a3172)

2025-03-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/docker/setup-buildx-action-3 in repository https://gitbox.apache.org/repos/asf/gravitino.git at e2f59a3172 build(deps): bump docker/setup-buildx-action from 2

[PR] build(deps): bump @mui/lab from 5.0.0-alpha.170 to 5.0.0-alpha.175 in /web/web [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6589: URL: https://github.com/apache/gravitino/pull/6589 Bumps [@mui/lab](https://github.com/mui/material-ui/tree/HEAD/packages/mui-lab) from 5.0.0-alpha.170 to 5.0.0-alpha.175. Commits See full diff in https://github.com/mui/materia

Re: [PR] [#5873] feat(gvfs-fuse): add debug log for FuseApiHandle [gravitino]

2025-03-03 Thread via GitHub
diqiu50 commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1977053414 ## clients/filesystem-fuse/src/fuse_api_handle_debug.rs: ## @@ -0,0 +1,871 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contribut

Re: [I] [FEATURE] Support Fluss catalog [gravitino]

2025-03-03 Thread via GitHub
SteNicholas commented on issue #6578: URL: https://github.com/apache/gravitino/issues/6578#issuecomment-2693715022 @jerryshao, I would like to add solution design document into description in this week. -- This is an automated message from the Apache Git Service. To respond to the message

Re: [PR] build(deps): bump @mui/material from 5.15.21 to 5.16.14 in /web/web [gravitino]

2025-03-03 Thread via GitHub
jerryshao merged PR #6558: URL: https://github.com/apache/gravitino/pull/6558 -- 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] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

2025-03-03 Thread via GitHub
yuqi1129 closed pull request #6569: [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. URL: https://github.com/apache/gravitino/pull/6569 -- This is an automated message from the Apache Git Service. To respond to the message, please log

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

2025-03-03 Thread via GitHub
jerryshao commented on PR #6469: URL: https://github.com/apache/gravitino/pull/6469#issuecomment-2693774687 I think you should also update the related documents about newly added configurations. -- This is an automated message from the Apache Git Service. To respond to the message, please

(gravitino) branch dependabot/npm_and_yarn/web/web/types/node-22.13.8 created (now c92161b3e1)

2025-03-03 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/types/node-22.13.8 in repository https://gitbox.apache.org/repos/asf/gravitino.git at c92161b3e1 build(deps-dev): bump @types/node from 20.17.19 to 22.1

[I] [FEATURE] Add HA mechanism to CLI [gravitino]

2025-03-03 Thread via GitHub
Abyss-lord opened a new issue, #6580: URL: https://github.com/apache/gravitino/issues/6580 ### Describe the feature Currently, the CLI can specify the address of the service to connect to via the 'uri' parameter, otherwise http://localhost:8090 is used by default. We expect Gravitino

Re: [I] [FEATURE] Add HA mechanism to CLI [gravitino]

2025-03-03 Thread via GitHub
Abyss-lord commented on issue #6580: URL: https://github.com/apache/gravitino/issues/6580#issuecomment-2693863223 @justinmclean Plz help see if this idea works. -- 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] [#6076] improve(CLI): Support model pre event to Gravitino server [gravitino]

2025-03-03 Thread via GitHub
Abyss-lord commented on PR #6250: URL: https://github.com/apache/gravitino/pull/6250#issuecomment-2693818928 @FANNG1 I’ve finished updating the code. Please take a look at the PR again when you have time. -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

2025-03-03 Thread via GitHub
jerryshao commented on PR #6569: URL: https://github.com/apache/gravitino/pull/6569#issuecomment-2693988151 I think it is because that you change the cache to static, which make it a global cache (not per object), so some UTs may be affected and failed. -- This is an automated message fro

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2694059939 > @tengqm Just to confirm, should I go ahead and remove the return statement from all cases, including if statements? Yup. If the decision was that 'return' is unnecessary, please

Re: [PR] [#5873] feat(gvfs-fuse): add debug log for FuseApiHandle [gravitino]

2025-03-03 Thread via GitHub
diqiu50 commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1977053414 ## clients/filesystem-fuse/src/fuse_api_handle_debug.rs: ## @@ -0,0 +1,871 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contribut

(gravitino) branch dependabot/npm_and_yarn/web/web/mui/material-5.16.14 deleted (was 46cd1e5d7e)

2025-03-03 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch dependabot/npm_and_yarn/web/web/mui/material-5.16.14 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 46cd1e5d7e fix conflict The revisions that were on this branch are s

Re: [PR] build(deps-dev): bump @types/node from 20.14.9 to 22.13.5 in /web/web [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] closed pull request #6557: build(deps-dev): bump @types/node from 20.14.9 to 22.13.5 in /web/web URL: https://github.com/apache/gravitino/pull/6557 -- 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] build(deps-dev): bump @types/node from 20.17.19 to 22.13.8 in /web/web [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] opened a new pull request, #6579: URL: https://github.com/apache/gravitino/pull/6579 Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 20.17.19 to 22.13.8. Commits See full diff in https://github.com/DefinitelyTyp

Re: [PR] build(deps-dev): bump @types/node from 20.14.9 to 22.13.5 in /web/web [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] commented on PR #6557: URL: https://github.com/apache/gravitino/pull/6557#issuecomment-2693748077 Superseded by #6579. -- 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

(gravitino) branch dependabot/npm_and_yarn/web/web/types/node-22.13.5 deleted (was 79467d38c0)

2025-03-03 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/types/node-22.13.5 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 79467d38c0 build(deps-dev): bump @types/node from 20.14.9 to 22.13

Re: [PR] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

2025-03-03 Thread via GitHub
jerryshao commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1977300386 ## core/src/main/java/org/apache/gravitino/catalog/CatalogManager.java: ## @@ -848,12 +848,17 @@ private static boolean catalogInUse(EntityStore store, NameIdentif

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
Brijeshthummar02 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2693994249 @tengqm Apologies for any confusion. As Justin mentioned, the decision was to remove the return statement, and I’ve tried to remove it from almost all files. I didn’t mean to t

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
Brijeshthummar02 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2694001251 @tengqm Just to confirm, should I go ahead and remove the return statement from all cases, including if statements? -- This is an automated message from the Apache Git Servic

Re: [I] [FEATURE] Support Fluss catalog [gravitino]

2025-03-03 Thread via GitHub
jerryshao commented on issue #6578: URL: https://github.com/apache/gravitino/issues/6578#issuecomment-2693677588 Looks great, can we propose a solution about how to integrate with Fluss? -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on code in PR #6573: URL: https://github.com/apache/gravitino/pull/6573#discussion_r1977265823 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListCatalogProperties.java: ## @@ -49,10 +49,14 @@ public ListCatalogProperties(CommandContext context,

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

2025-03-03 Thread via GitHub
zzzk1 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2694077660 @yuqi1129, is our test infrastructure broken? I also encountered failures while testing locally. I think they are caused by TestContainers. -- This is an automated message from the Apac

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
Brijeshthummar02 commented on code in PR #6573: URL: https://github.com/apache/gravitino/pull/6573#discussion_r1977377043 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListCatalogProperties.java: ## @@ -49,10 +49,14 @@ public ListCatalogProperties(CommandContext

Re: [I] [FEATURE] Add HA mechanism to CLI [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on issue #6580: URL: https://github.com/apache/gravitino/issues/6580#issuecomment-2693939193 I don't get the idea of "HA". If we mean that there are more than one servers we could use, there will be a load balancer sitting in front of the servers for L7/L4 traffics. Another

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977424172 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -91,9 +94,42 @@ protected RangerAut

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on code in PR #6573: URL: https://github.com/apache/gravitino/pull/6573#discussion_r1977427676 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListCatalogProperties.java: ## @@ -49,10 +49,14 @@ public ListCatalogProperties(CommandContext context,

[PR] [#6577] docs: fix param typo from 'type' to 'catalog_type' [gravitino]

2025-03-03 Thread via GitHub
AndreVale69 opened a new pull request, #6581: URL: https://github.com/apache/gravitino/pull/6581 ### What changes were proposed in this pull request? Fix parameter typo for `create_catalog` function in `hadoop-catalog` fileset documentation and `manage-fileset` and `manage-model`. Cha

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977420196 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationHDFSPlugin.java: ## @@ -679,4 +680,57 @@ public Boolea

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on code in PR #6573: URL: https://github.com/apache/gravitino/pull/6573#discussion_r1977439250 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListColumns.java: ## @@ -51,18 +50,40 @@ public ListColumns( /** Displays the details of a table's co

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977459351 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationHDFSPlugin.java: ## @@ -679,4 +680,57 @@ public Boolea

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2693929565 > @tengqm made relevant changes. Two points here: - If you don't agree to a feedback, you can leave it there, but don't mark it as "resolved". Marking a feedback as "resolve

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977417628 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -935,8 +971,19 @@ protected void r

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977416894 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -91,9 +94,42 @@ protected RangerAut

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977418641 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationHDFSPlugin.java: ## @@ -679,4 +680,57 @@ public Boolea

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977414190 ## authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/RangerAuthorizationProperties.java: ## @@ -76,15 +76,19 @@ public void va

Re: [I] [Bug report] Document bug in fileset python sample code [gravitino]

2025-03-03 Thread via GitHub
AndreVale69 commented on issue #6577: URL: https://github.com/apache/gravitino/issues/6577#issuecomment-2694239564 Hi, I would like to work on this issue. I noticed that other documents also have the same problem. For example: https://github.com/apache/gravitino/blob/0e31573f5dc15f43

Re: [PR] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on code in PR #6573: URL: https://github.com/apache/gravitino/pull/6573#discussion_r1977266938 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListFilesetProperties.java: ## @@ -59,10 +59,15 @@ public ListFilesetProperties( @Override public

[I] Broken Link in Issue Template Documentation [gravitino]

2025-03-03 Thread via GitHub
Brijeshthummar02 opened a new issue, #6582: URL: https://github.com/apache/gravitino/issues/6582 **Description:**\ The link in the issue template documentation currently leads to a 404 page:\ 🔗 **** **Steps to Re

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1977589211 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -745,6 +763,108 @@ public Boolean

[PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

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

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

2025-03-03 Thread via GitHub
tengqm commented on PR #6584: URL: https://github.com/apache/gravitino/pull/6584#issuecomment-2695900191 lgtm thanks. -- 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: [PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

2025-03-03 Thread via GitHub
tengqm commented on code in PR #6583: URL: https://github.com/apache/gravitino/pull/6583#discussion_r1978437013 ## .github/ISSUE_TEMPLATE/improvement.yml: ## @@ -1,23 +1,23 @@ -# See https://gh-community.github.io/issue-template-feedback/structured/ +# See https://github.com/or

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

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

[I] [Improvement] UI: supports copying the identity information of table via a button. [gravitino]

2025-03-03 Thread via GitHub
zhoukangcn opened a new issue, #6592: URL: https://github.com/apache/gravitino/issues/6592 ### What would you like to be improved? Add a button to the page containing the table, model, fileset. When clicked, this button should copy the identity string, for example: `metalake_demo.cat

Re: [PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

2025-03-03 Thread via GitHub
Brijeshthummar02 commented on code in PR #6583: URL: https://github.com/apache/gravitino/pull/6583#discussion_r1978492845 ## .github/ISSUE_TEMPLATE/improvement.yml: ## @@ -1,23 +1,23 @@ -# See https://gh-community.github.io/issue-template-feedback/structured/ +# See https://git

Re: [PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

2025-03-03 Thread via GitHub
Brijeshthummar02 commented on code in PR #6583: URL: https://github.com/apache/gravitino/pull/6583#discussion_r1978492276 ## .github/ISSUE_TEMPLATE/improvement.yml: ## @@ -1,23 +1,23 @@ -# See https://gh-community.github.io/issue-template-feedback/structured/ +# See https://git

Re: [I] Broken Link in Issue Template Documentation [gravitino]

2025-03-03 Thread via GitHub
Brijeshthummar02 commented on issue #6582: URL: https://github.com/apache/gravitino/issues/6582#issuecomment-2695998202 @justinmclean thanks for pointing it. I have updated the path in issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [I] Broken Link in Issue Template Documentation [gravitino]

2025-03-03 Thread via GitHub
justinmclean commented on issue #6582: URL: https://github.com/apache/gravitino/issues/6582#issuecomment-2695747622 Thanks for reporting this. Can you include where the link is broken? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
xunliu commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978583886 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -745,6 +763,110 @@ public Boolean

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978706648 ## core/src/main/java/org/apache/gravitino/hook/CatalogHookDispatcher.java: ## @@ -131,18 +131,17 @@ public boolean dropCatalog(NameIdentifier ident, boolean force)

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978707342 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -745,6 +763,110 @@ public Boolean o

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r197876 ## core/src/main/java/org/apache/gravitino/hook/CatalogHookDispatcher.java: ## @@ -131,18 +131,17 @@ public boolean dropCatalog(NameIdentifier ident, boolean force)

Re: [PR] [#4620] improvement(authz): Throw the necessary exception when handling Ranger plugin exception [gravitino]

2025-03-03 Thread via GitHub
jerqi merged PR #6515: URL: https://github.com/apache/gravitino/pull/6515 -- 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] Consistency of failure of the underlying system for operational authorization [gravitino]

2025-03-03 Thread via GitHub
jerqi closed issue #4620: [Subtask] Consistency of failure of the underlying system for operational authorization URL: https://github.com/apache/gravitino/issues/4620 -- 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] [Subtask] Consistency of failure of the underlying system for operational authorization [gravitino]

2025-03-03 Thread via GitHub
jerqi closed issue #4620: [Subtask] Consistency of failure of the underlying system for operational authorization URL: https://github.com/apache/gravitino/issues/4620 -- 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] [#6577] docs: fix param typo from 'type' to 'catalog_type' [gravitino]

2025-03-03 Thread via GitHub
AndreVale69 commented on PR #6581: URL: https://github.com/apache/gravitino/pull/6581#issuecomment-2696319209 > Thanks! but unfortunately, I think we don't need this, as `Catalog.Type` is already clear enough. As @yuqi1129 suggested, I could try to modify the Python API to achieve co

Re: [PR] [#5873] feat(gvfs-fuse): add debug log for FuseApiHandle [gravitino]

2025-03-03 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1978575688 ## clients/filesystem-fuse/src/fuse_api_handle_debug.rs: ## @@ -0,0 +1,871 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contri

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978515696 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -91,9 +94,42 @@ protected RangerAut

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978522824 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -745,6 +763,108 @@ public Boolean o

Re: [PR] build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] commented on PR #6587: URL: https://github.com/apache/gravitino/pull/6587#issuecomment-2696052024 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, l

Re: [PR] build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 [gravitino]

2025-03-03 Thread via GitHub
jerryshao commented on PR #6587: URL: https://github.com/apache/gravitino/pull/6587#issuecomment-2696051852 It is because ASF doesn't support using different version of cherry-pick plugin, so we will not upgrade this action. I'm going to close this PR. -- This is an automated message from

Re: [PR] build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 [gravitino]

2025-03-03 Thread via GitHub
jerryshao commented on PR #6587: URL: https://github.com/apache/gravitino/pull/6587#issuecomment-2696053394 @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 t

(gravitino) branch dependabot/github_actions/carloscastrojumo/github-cherry-pick-action-1.0.10 deleted (was dcd5b95ffe)

2025-03-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/carloscastrojumo/github-cherry-pick-action-1.0.10 in repository https://gitbox.apache.org/repos/asf/gravitino.git was dcd5b95ffe build(deps): bump carloscastro

Re: [PR] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978523448 ## core/src/main/java/org/apache/gravitino/connector/authorization/BaseAuthorization.java: ## @@ -45,6 +45,9 @@ public abstract class BaseAuthorization implements

Re: [PR] [#6577] docs: fix param typo from 'type' to 'catalog_type' [gravitino]

2025-03-03 Thread via GitHub
yuqi1129 commented on PR #6581: URL: https://github.com/apache/gravitino/pull/6581#issuecomment-2696057844 > Thanks! but unfortunately, I think we don't need this, as `Catalog.Type` is already clear enough. This sounds reasonable. Java API is as follows ``` public Catalog

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

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

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

2025-03-03 Thread via GitHub
github-actions[bot] opened a new pull request, #6593: URL: https://github.com/apache/gravitino/pull/6593 ### What changes were proposed in this pull request? fix several typo in docs ### Why are the changes needed? typo ### Does this PR introduce _any_ user-facing

(gravitino) branch cherry-pick-branch-0.8-783f990cd971cca9d8162d030a378d7e613181de created (now 86d6cfff38)

2025-03-03 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-783f990cd971cca9d8162d030a378d7e613181de in repository https://gitbox.apache.org/repos/asf/gravitino.git at 86d6cfff38 [MINOR] docs: fix several typo in docs (#6

(gravitino) branch branch-0.8 updated: [MINOR] docs: fix several typo in docs (#6593)

2025-03-03 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 a55bf98511 [MINOR] docs: fix severa

Re: [PR] build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 [gravitino]

2025-03-03 Thread via GitHub
jerryshao closed pull request #6587: build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 URL: https://github.com/apache/gravitino/pull/6587 -- 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] [#6536] improvement: Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978537039 ## core/src/main/java/org/apache/gravitino/hook/CatalogHookDispatcher.java: ## @@ -131,18 +131,18 @@ public boolean dropCatalog(NameIdentifier ident, boolean force)

Re: [PR] [#6577] docs: fix param typo from 'type' to 'catalog_type' [gravitino]

2025-03-03 Thread via GitHub
yuqi1129 commented on PR #6581: URL: https://github.com/apache/gravitino/pull/6581#issuecomment-2696052700 > Thanks! but unfortunately, I think we don't need this, as `Catalog.Type` is already clear enough. This sounds reasonable. If we want to keep it is, we need to change the API s

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

2025-03-03 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 783f990cd9 [MINOR] docs: fix several typo in do

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

2025-03-03 Thread via GitHub
yuqi1129 merged PR #6593: URL: https://github.com/apache/gravitino/pull/6593 -- 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] build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 [gravitino]

2025-03-03 Thread via GitHub
dependabot[bot] commented on PR #6587: URL: https://github.com/apache/gravitino/pull/6587#issuecomment-2696053455 OK, I won't notify you about version 1.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] [#6076] improve(CLI): Support model pre event to Gravitino server [gravitino]

2025-03-03 Thread via GitHub
FANNG1 commented on code in PR #6250: URL: https://github.com/apache/gravitino/pull/6250#discussion_r1978542889 ## core/src/main/java/org/apache/gravitino/listener/ModelEventDispatcher.java: ## @@ -0,0 +1,268 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [#6076] improve(CLI): Support model pre event to Gravitino server [gravitino]

2025-03-03 Thread via GitHub
FANNG1 commented on PR #6250: URL: https://github.com/apache/gravitino/pull/6250#issuecomment-2696093911 Generally LGTM except minor comments, @jerryshao do you have time to review again? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

(gravitino) branch main updated: [#4620] improvement(authz): Throw the necessary exception when handling Ranger plugin exception (#6515)

2025-03-03 Thread roryqi
This is an automated email from the ASF dual-hosted git repository. roryqi 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 873c6afeb3 [#4620] improvement(authz): Throw the

Re: [PR] [#5873] feat(gvfs-fuse): add debug log for FuseApiHandle [gravitino]

2025-03-03 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1978575447 ## clients/filesystem-fuse/src/fuse_api_handle_debug.rs: ## @@ -0,0 +1,871 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contri

Re: [PR] [6582] fix: Broken Link in Issue Template Documentation [gravitino]

2025-03-03 Thread via GitHub
justinmclean commented on PR #6583: URL: https://github.com/apache/gravitino/pull/6583#issuecomment-2696324474 Yes, it would be best to only include the URL change. I'm not 100% sure that the new URL is correct as it has what seems to be extra characters at the end. https://github.com/o

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r197876 ## core/src/main/java/org/apache/gravitino/hook/CatalogHookDispatcher.java: ## @@ -131,18 +131,17 @@ public boolean dropCatalog(NameIdentifier ident, boolean force)

Re: [PR] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

2025-03-03 Thread via GitHub
yuqi1129 commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1978702497 ## core/src/main/java/org/apache/gravitino/catalog/CatalogManager.java: ## @@ -260,7 +260,7 @@ private ModelCatalog asModels() { private final Config config;

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978705115 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -745,6 +763,110 @@ public Boolean o

Re: [PR] [#6536] improvement(authz): Create Ranger service if service is absent [gravitino]

2025-03-03 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1978713739 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -909,59 +1031,4 @@ private void rem

Re: [PR] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

2025-03-03 Thread via GitHub
yuqi1129 commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1978706464 ## core/src/main/java/org/apache/gravitino/catalog/CatalogManager.java: ## @@ -260,7 +260,7 @@ private ModelCatalog asModels() { private final Config config;

  1   2   >