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

2025-01-16 Thread via GitHub
diqiu50 commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1917878392 ## clients/filesystem-fuse/conf/gvfs_fuse.toml: ## @@ -20,6 +20,7 @@ file_mask = 0o600 dir_mask = 0o700 fs_type = "memory" +fuse_debug=true Review Comment: K

(gravitino) branch main updated: [#6272] fix: Remove the unnecessary null checks (#6282)

2025-01-16 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 39e8a09d0b [#6272] fix: Remove the unnecessary nul

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

2025-01-16 Thread via GitHub
diqiu50 commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1918026613 ## clients/filesystem-fuse/src/fuse_api_handle.rs: ## @@ -30,10 +30,174 @@ use fuse3::{Errno, FileType, Inode, SetAttr, Timestamp}; use futures_util::stream; use fu

Re: [PR] [#6272] fix: Remove the unnecessary null checks [gravitino]

2025-01-16 Thread via GitHub
jerryshao closed pull request #6296: [#6272] fix: Remove the unnecessary null checks URL: https://github.com/apache/gravitino/pull/6296 -- 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] [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode [gravitino]

2025-01-16 Thread via GitHub
jerqi commented on code in PR #6214: URL: https://github.com/apache/gravitino/pull/6214#discussion_r1918002720 ## catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogOperations.java: ## @@ -513,6 +514,13 @@ public Table c

(gravitino) branch cherry-pick-branch-0.8-39e8a09d0b8dfbfa551e6bfc58ec56549f41717d created (now 8d05d31d53)

2025-01-16 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-39e8a09d0b8dfbfa551e6bfc58ec56549f41717d in repository https://gitbox.apache.org/repos/asf/gravitino.git at 8d05d31d53 [#6272] fix: Remove the unnecessary null c

Re: [PR] [#6272] fix: Remove the unnecessary null checks [gravitino]

2025-01-16 Thread via GitHub
jerryshao merged PR #6282: URL: https://github.com/apache/gravitino/pull/6282 -- 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] [#3515] feat(flink-connector): Support flink iceberg catalog [gravitino]

2025-01-16 Thread via GitHub
jerryshao merged PR #6295: URL: https://github.com/apache/gravitino/pull/6295 -- 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: [#3515] feat(flink-connector): Support flink iceberg catalog (#6295)

2025-01-16 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 01ec8c48cd [#3515] feat(flink-connecto

Re: [I] [Subtask] [Docs]Adding Documentation for GravitinoPaimonCatalog in Flink Connectors [gravitino]

2025-01-16 Thread via GitHub
jerryshao closed issue #6218: [Subtask] [Docs]Adding Documentation for GravitinoPaimonCatalog in Flink Connectors URL: https://github.com/apache/gravitino/issues/6218 -- 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] [Docs]Adding Documentation for GravitinoPaimonCatalog in Flink Connectors [gravitino]

2025-01-16 Thread via GitHub
jerryshao closed issue #6218: [Subtask] [Docs]Adding Documentation for GravitinoPaimonCatalog in Flink Connectors URL: https://github.com/apache/gravitino/issues/6218 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

(gravitino) branch main updated (39e8a09d0b -> 1017f3ecf4)

2025-01-16 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 39e8a09d0b [#6272] fix: Remove the unnecessary null checks (#6282) add 1017f3ecf4 [#6218] docs: Adding Documentat

Re: [PR] [#6218] docs: Adding Documentation for GravitinoPaimonCatalog in Flink Connectors [gravitino]

2025-01-16 Thread via GitHub
jerryshao closed pull request #6297: [#6218] docs: Adding Documentation for GravitinoPaimonCatalog in Flink Connectors URL: https://github.com/apache/gravitino/pull/6297 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] [#6218] docs: Adding Documentation for GravitinoPaimonCatalog in Flink Connectors [gravitino]

2025-01-16 Thread via GitHub
github-actions[bot] opened a new pull request, #6297: URL: https://github.com/apache/gravitino/pull/6297 ### What changes were proposed in this pull request? close #6218 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] [#6218] docs: Adding Documentation for GravitinoPaimonCatalog in Flink Connectors [gravitino]

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

svn commit: r74232 - /dev/incubator/gravitino/v0.8.0-incubating-rc2/

2025-01-16 Thread fanng
Author: fanng Date: Thu Jan 16 10:58:29 2025 New Revision: 74232 Log: Apache Gravitino v0.8.0-incubating-rc2 Added: dev/incubator/gravitino/v0.8.0-incubating-rc2/ dev/incubator/gravitino/v0.8.0-incubating-rc2/gravitino-0.8.0-incubating-bin.tar.gz (with props) dev/incubator/gravit

Re: [PR] [#6218] docs: Adding Documentation for GravitinoPaimonCatalog in Flink Connectors [gravitino]

2025-01-16 Thread via GitHub
jerryshao merged PR #6297: URL: https://github.com/apache/gravitino/pull/6297 -- 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: [#6218] docs: Adding Documentation for GravitinoPaimonCatalog in Flink Connectors (#6297)

2025-01-16 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 f025777ddb [#6218] docs: Adding Docume

(gravitino) branch cherry-pick-branch-0.8-1017f3ecf48c7143c9675fe6c80cae805781bc9d deleted (was 2f347fe10c)

2025-01-16 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-1017f3ecf48c7143c9675fe6c80cae805781bc9d in repository https://gitbox.apache.org/repos/asf/gravitino.git was 2f347fe10c [#6218] docs: Adding Documentation for Graviti

Re: [PR] [#6097] improve(CLI): Add --quiet option to the Gravition CLI [gravitino]

2025-01-16 Thread via GitHub
jerryshao commented on PR #6198: URL: https://github.com/apache/gravitino/pull/6198#issuecomment-2595011461 I would suggest better revisiting the current CLI code architecture, and make it more extendable and maintainable before continuing to add more and more options. Options like this is

(gravitino) branch branch-0.8 updated: [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode (#6299)

2025-01-16 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 54d1c2a352 [#6196] feat(iceberg): adju

Re: [PR] [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode [gravitino]

2025-01-16 Thread via GitHub
jerryshao merged PR #6299: URL: https://github.com/apache/gravitino/pull/6299 -- 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-e6225a01ea72d614c465a04262c78d2c8f337f49 deleted (was 8014184113)

2025-01-16 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-e6225a01ea72d614c465a04262c78d2c8f337f49 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 8014184113 [#6196] feat(iceberg): adjust table distributio

Re: [PR] [#6272] fix: Remove the unnecessary null checks [gravitino]

2025-01-16 Thread via GitHub
jerryshao merged PR #6296: URL: https://github.com/apache/gravitino/pull/6296 -- 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: [#6272] fix: Remove the unnecessary null checks (#6296)

2025-01-16 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 1edfab305c [#6272] fix: Remove the unn

(gravitino) branch cherry-pick-branch-0.8-39e8a09d0b8dfbfa551e6bfc58ec56549f41717d deleted (was 8d05d31d53)

2025-01-16 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-39e8a09d0b8dfbfa551e6bfc58ec56549f41717d in repository https://gitbox.apache.org/repos/asf/gravitino.git was 8d05d31d53 [#6272] fix: Remove the unnecessary null checks

(gravitino) branch branch-0.8 updated (54d1c2a352 -> 1cdca76ef3)

2025-01-16 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a change to branch branch-0.8 in repository https://gitbox.apache.org/repos/asf/gravitino.git from 54d1c2a352 [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mo

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

2025-01-16 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a commit to tag v0.8.0-incubating-rc2 in repository https://gitbox.apache.org/repos/asf/gravitino.git commit d3d40db282d5173a8329dab4c7d653da66aee15e Author: Xiaojing Fang AuthorDate: Thu Jan 16 10:51:52 2025 +

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

2025-01-16 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a commit to branch branch-0.8 in repository https://gitbox.apache.org/repos/asf/gravitino.git commit 1cdca76ef386e2c8a62ddffd6c7a85980b069ec0 Author: Xiaojing Fang AuthorDate: Thu Jan 16 10:51:53 2025 + Pre

(gravitino) branch main updated: [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode (#6214)

2025-01-16 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 e6225a01ea [#6196] feat(iceberg): adjust table dis

Re: [PR] [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode [gravitino]

2025-01-16 Thread via GitHub
FANNG1 merged PR #6214: URL: https://github.com/apache/gravitino/pull/6214 -- 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] [Improvement] correct the behaviors when creating Iceberg table with none distribution [gravitino]

2025-01-16 Thread via GitHub
FANNG1 closed issue #6196: [Improvement] correct the behaviors when creating Iceberg table with none distribution URL: https://github.com/apache/gravitino/issues/6196 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

(gravitino) branch cherry-pick-branch-0.8-e6225a01ea72d614c465a04262c78d2c8f337f49 created (now 8014184113)

2025-01-16 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-e6225a01ea72d614c465a04262c78d2c8f337f49 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 8014184113 [#6196] feat(iceberg): adjust table distri

[PR] [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode [gravitino]

2025-01-16 Thread via GitHub
github-actions[bot] opened a new pull request, #6299: URL: https://github.com/apache/gravitino/pull/6299 ### What changes were proposed in this pull request? Adjust the distribution mode for creating Iceberg table with none distribution. the following is the Spark adjust logic, the f

Re: [PR] [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode [gravitino]

2025-01-16 Thread via GitHub
FANNG1 closed pull request #6299: [#6196] feat(iceberg): adjust table distribution if creating table without specifying disribution mode URL: https://github.com/apache/gravitino/pull/6299 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

(gravitino) branch cherry-pick-branch-0.8-1017f3ecf48c7143c9675fe6c80cae805781bc9d created (now 2f347fe10c)

2025-01-16 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-1017f3ecf48c7143c9675fe6c80cae805781bc9d in repository https://gitbox.apache.org/repos/asf/gravitino.git at 2f347fe10c [#6218] docs: Adding Documentation for Gr

Re: [PR] [#5901] feat(core): support tag event to Gravitino server [gravitino]

2025-01-16 Thread via GitHub
TungYuChiang commented on PR #5998: URL: https://github.com/apache/gravitino/pull/5998#issuecomment-2594961333 @FANNG1, I have fixed everything except renaming `ListTagsInfoFailureEvent `to `ListTagInfoFailureEvent` -- This is an automated message from the Apache Git Service. To respond t

[I] [Bug report] shouldn't publish the RC release to latest tag of docker image [gravitino]

2025-01-16 Thread via GitHub
FANNG1 opened a new issue, #6300: URL: https://github.com/apache/gravitino/issues/6300 ### Version main branch ### Describe what's wrong in current docker image build workflow, the latest tag of docker image is updated when we trigger a build, this is not correct. ```

Re: [PR] [#5901] feat(core): support tag event to Gravitino server [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on code in PR #5998: URL: https://github.com/apache/gravitino/pull/5998#discussion_r1918618665 ## core/src/main/java/org/apache/gravitino/listener/TagEventDispatcher.java: ## @@ -57,8 +68,9 @@ public TagEventDispatcher(EventBus eventBus, TagDispatcher dispatche

[I] [FEATURE] Flink Multi-Version Support [gravitino]

2025-01-16 Thread via GitHub
hdygxsj opened a new issue, #6301: URL: https://github.com/apache/gravitino/issues/6301 ### Describe the feature Flink Multi-Version Support ### Motivation _No response_ ### Describe the solution _No response_ ### Additional context _No respons

Re: [PR] [MINOR] fix(dos): Fix minor error about spelling and web link [gravitino]

2025-01-16 Thread via GitHub
FANNG1 merged PR #6294: URL: https://github.com/apache/gravitino/pull/6294 -- 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 cherry-pick-branch-0.8-393609be70f929083c7d997bed24b692d7bd358d deleted (was 1f2eceb028)

2025-01-16 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a change to branch cherry-pick-branch-0.8-393609be70f929083c7d997bed24b692d7bd358d in repository https://gitbox.apache.org/repos/asf/gravitino.git was 1f2eceb028 [MINOR] fix(dos): Fix minor error about spellin

Re: [I] [Improvement] improve the docker publish for gravitino-playground:trino [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on issue #6281: URL: https://github.com/apache/gravitino/issues/6281#issuecomment-2594792466 I think this is unrelated to the tag name because we could handle it in the gravitino docker build script. -- This is an automated message from the Apache Git Service. To respon

(gravitino) branch branch-0.8 updated: [MINOR] fix(dos): Fix minor error about spelling and web link (#6294)

2025-01-16 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng 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 7a3f8ea70c [MINOR] fix(dos): Fix minor

Re: [PR] [#6218] docs: Adding Documentation for GravitinoPaimonCatalog in Flink Connectors [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on PR #6266: URL: https://github.com/apache/gravitino/pull/6266#issuecomment-2594787610 LGTM -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe

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

2025-01-16 Thread via GitHub
diqiu50 commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1917885027 ## clients/filesystem-fuse/src/config.rs: ## @@ -265,6 +265,8 @@ pub struct FuseConfig { #[serde(default)] pub fs_type: String, #[serde(default)] +

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

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

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

2025-01-16 Thread via GitHub
diqiu50 commented on PR #5905: URL: https://github.com/apache/gravitino/pull/5905#issuecomment-2594813907 ``` 2025-01-16T08:22:33.123544Z DEBUG gvfs_fuse::fuse_api_handle_debug: getattr started req.unique=6 req=Request { unique: 6, uid: 1000, gid: 1000, pid: 52525 } filename="" fh=No

Re: [PR] [#5900] feat(tag): support tag pre-event to Gravitino server [gravitino]

2025-01-16 Thread via GitHub
schoolLychee commented on PR #5980: URL: https://github.com/apache/gravitino/pull/5980#issuecomment-2596073940 @FANNG1 I commit the implement of the tag pre-event and I am working on the test right now. Thank you! -- This is an automated message from the Apache Git Service. To respond to

Re: [I] [Improvement] Upgrade the version of paimon dependencies to 1.0. [gravitino]

2025-01-16 Thread via GitHub
hdygxsj commented on issue #6292: URL: https://github.com/apache/gravitino/issues/6292#issuecomment-2596074094 Apologies, I expressed myself incorrectly. My original intention was to add integration test cases for multiple versions of Paimon. -- This is an automated message from the Apac

Re: [I] [Improvement] Upgrade the version of paimon dependencies to 1.0. [gravitino]

2025-01-16 Thread via GitHub
hdygxsj closed issue #6292: [Improvement] Upgrade the version of paimon dependencies to 1.0. URL: https://github.com/apache/gravitino/issues/6292 -- 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

[PR] [#6302] refactor:Optimize Flink connector properties converter [gravitino]

2025-01-16 Thread via GitHub
sunxiaojian opened a new pull request, #6303: URL: https://github.com/apache/gravitino/pull/6303 ### What changes were proposed in this pull request? - Unified params format, add flink.bypass. prefix, especially Iceberg. - Extract the flink.bypass. logic to the PropertiesCon

[I] [Bug report] upgrade actions/upload-artifact to v4 [gravitino]

2025-01-16 Thread via GitHub
sunxiaojian opened a new issue, #6304: URL: https://github.com/apache/gravitino/issues/6304 ### Version main branch ### Describe what's wrong https://github.com/apache/gravitino/actions/runs/12813017309/job/35726085181?pr=6303 ### Error message and/or stac

[PR] [#6304] fix: upgrade actions/upload-artifact to v4 [gravitino]

2025-01-16 Thread via GitHub
sunxiaojian opened a new pull request, #6305: URL: https://github.com/apache/gravitino/pull/6305 ### What changes were proposed in this pull request? upgrade actions/upload-artifact to v4 ### Why are the changes needed? Fix: [#(6304)](https://github.com/apach

Re: [PR] [#6302] refactor:Optimize Flink connector properties converter [gravitino]

2025-01-16 Thread via GitHub
sunxiaojian commented on PR #6303: URL: https://github.com/apache/gravitino/pull/6303#issuecomment-2596188457 fixed : https://github.com/apache/gravitino/pull/6305 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

(gravitino) tag v0.8.0-incubating-rc2 created (now d3d40db282)

2025-01-16 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a change to tag v0.8.0-incubating-rc2 in repository https://gitbox.apache.org/repos/asf/gravitino.git at d3d40db282 (commit) This tag includes the following new commits: new d3d40db282 Preparing Gravitin

Re: [I] [Improvement] Upgrade the version of paimon dependencies to 1.0. [gravitino]

2025-01-16 Thread via GitHub
sunxiaojian commented on issue #6292: URL: https://github.com/apache/gravitino/issues/6292#issuecomment-2596052097 If there are no changes or bugs in the catalog module, do we not need to upgrade? We are using the Paimon catalog API, and I understand that the specific version of the package

[I] [Bug report] Graviitno Flink connector start failed [gravitino]

2025-01-16 Thread via GitHub
FANNG1 opened a new issue, #6307: URL: https://github.com/apache/gravitino/issues/6307 ### Version main branch ### Describe what's wrong Graviitno Flink connector start failed ### Error message and/or stacktrace ``` Exception in thread "main" java.lang.No

[PR] [#6306] fix(authz): Fix the OOM of JdbcAuthorizationPlugin [gravitino]

2025-01-16 Thread via GitHub
jerqi opened a new pull request, #6312: URL: https://github.com/apache/gravitino/pull/6312 ### What changes were proposed in this pull request? Fix the OOM of JdbcAuthorizationPlugin ### Why are the changes needed? Fix: #6306 ### Does this PR introduce _any_ user-

Re: [PR] [#6270] (Improvement) unified cache framework [gravitino]

2025-01-16 Thread via GitHub
sunxiaojian commented on PR #6271: URL: https://github.com/apache/gravitino/pull/6271#issuecomment-2597324122 > > > I am concerned that this will introduce extra dependencies that may conflict with Spark runtime jars. Could you analysis which jars are introduced and could we shade them?

Re: [PR] fix(playground.sh): should use checkPlaygroundRunning in start [gravitino-playground]

2025-01-16 Thread via GitHub
unknowntpo closed pull request #125: fix(playground.sh): should use checkPlaygroundRunning in start URL: https://github.com/apache/gravitino-playground/pull/125 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] fix(playground.sh): should use checkPlaygroundRunning in start [gravitino-playground]

2025-01-16 Thread via GitHub
unknowntpo commented on PR #125: URL: https://github.com/apache/gravitino-playground/pull/125#issuecomment-2597512536 I misunderstood the code, closed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] Delete old docker log files but keep the latest 3 [gravitino-playground]

2025-01-16 Thread via GitHub
jerqi merged PR #124: URL: https://github.com/apache/gravitino-playground/pull/124 -- 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...@g

Re: [I] Optimize the playground logs [gravitino-playground]

2025-01-16 Thread via GitHub
jerqi closed issue #123: Optimize the playground logs URL: https://github.com/apache/gravitino-playground/issues/123 -- 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 unsubscr

(gravitino-playground) branch main updated: Delete old docker log files but keep the latest 3 (#124)

2025-01-16 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-playground.git The following commit(s) were added to refs/heads/main by this push: new 4512122 Delete old docker log files b

Re: [PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on PR #6308: URL: https://github.com/apache/gravitino/pull/6308#issuecomment-2597534779 > Have you identified the root cause of this issue? Does it work with 0.7.0-inbucating? yes, it works with 0.7.0-inbucating -- This is an automated message from the Apache Git S

Re: [PR] Set HADOOP_USER_NAME for Spark user [gravitino-playground]

2025-01-16 Thread via GitHub
jerqi commented on code in PR #115: URL: https://github.com/apache/gravitino-playground/pull/115#discussion_r1919629705 ## README.md: ## @@ -202,6 +202,12 @@ You might also consider generating data with SparkSQL and then querying this dat 1. Login Spark container and execut

Re: [PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
yuqi1129 commented on PR #6308: URL: https://github.com/apache/gravitino/pull/6308#issuecomment-2597504963 Have you identified the root cause of this issue? Does it work with 0.7.0-inbucating? -- This is an automated message from the Apache Git Service. To respond to the message, please

[PR] fix(playground.sh): should use checkPlaygroundRunning in start [gravitino-playground]

2025-01-16 Thread via GitHub
unknowntpo opened a new pull request, #125: URL: https://github.com/apache/gravitino-playground/pull/125 In `start`, we should use `checkPlaygroundRunning`, not `checkPlaygroundNotRunning`. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[I] [Bug report] Flink SQL client start failed for missing Paimon package [gravitino]

2025-01-16 Thread via GitHub
FANNG1 opened a new issue, #6309: URL: https://github.com/apache/gravitino/issues/6309 ### Version main branch ### Describe what's wrong Flink SQL client start failed for missing Paimon package ### Error message and/or stacktrace ``` Exception in thr

[I] [Bug report] Gravitino REST Server Defaults to 'jdbc' Instead of Using Catalog Name from Spark Configuration [gravitino]

2025-01-16 Thread via GitHub
dataageek opened a new issue, #6310: URL: https://github.com/apache/gravitino/issues/6310 ### Version main branch ### Describe what's wrong I have multiple catalogs created using the Iceberg JDBC catalog. When I attempt to consume these catalogs via the Gravitino REST se

[I] [Improvement] Add more `AuthorizationMetadataObject` type support for PathBasedMetadataObject [gravitino]

2025-01-16 Thread via GitHub
yuqi1129 opened a new issue, #6311: URL: https://github.com/apache/gravitino/issues/6311 ### What would you like to be improved? Currently, PathBasedMetadataObject only supports metadata object `fileset`, this can't satisfy the need to support other type like `table`, `schema` as the

Re: [PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
yuqi1129 commented on PR #6308: URL: https://github.com/apache/gravitino/pull/6308#issuecomment-2597539498 > > Have you identified the root cause of this issue? Does it work with 0.7.0-inbucating? > > yes, it works with 0.7.0-inbucating So I suggest we may need to find out whic

Re: [PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on PR #6308: URL: https://github.com/apache/gravitino/pull/6308#issuecomment-2597547106 > > > Have you identified the root cause of this issue? Does it work with 0.7.0-inbucating? > > > > > > yes, it works with 0.7.0-inbucating > > So I suggest we may need

Re: [PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
yuqi1129 commented on PR #6308: URL: https://github.com/apache/gravitino/pull/6308#issuecomment-2597554377 > > > > Have you identified the root cause of this issue? Does it work with 0.7.0-inbucating? > > > > > > > > > yes, it works with 0.7.0-inbucating > > > > > > So

Re: [PR] [#6309] fix(flink): remove paimon package from GravitinoPaimonCatalogFactory [gravitino]

2025-01-16 Thread via GitHub
jerryshao closed pull request #6314: [#6309] fix(flink): remove paimon package from GravitinoPaimonCatalogFactory URL: https://github.com/apache/gravitino/pull/6314 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[I] [Bug report] JdbcAuthorizationPlugin will cause OOM [gravitino]

2025-01-16 Thread via GitHub
jerqi opened a new issue, #6306: URL: https://github.com/apache/gravitino/issues/6306 ### Version main branch ### Describe what's wrong JdbcAuthorizationPlugin will cause OOM ### Error message and/or stacktrace JdbcAuthorizationPlugin will cause OOM #

Re: [PR] [#6270] (Improvement) unified cache framework [gravitino]

2025-01-16 Thread via GitHub
jerqi commented on PR #6271: URL: https://github.com/apache/gravitino/pull/6271#issuecomment-2597385383 > > > > I am concerned that this will introduce extra dependencies that may conflict with Spark runtime jars. Could you analysis which jars are introduced and could we shade them? > >

Re: [PR] [#6302] refactor:Optimize Flink connector properties converter [gravitino]

2025-01-16 Thread via GitHub
sunxiaojian commented on PR #6303: URL: https://github.com/apache/gravitino/pull/6303#issuecomment-2597385437 @FANNG1 PTAL , 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 co

Re: [PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on PR #6308: URL: https://github.com/apache/gravitino/pull/6308#issuecomment-2597622938 > > > > > Have you identified the root cause of this issue? Does it work with 0.7.0-inbucating? > > > > > > > > > > > > yes, it works with 0.7.0-inbucating > > > > > >

Re: [PR] [#6309] fix(flink): remove paimon package from GravitinoPaimonCatalogFactory [gravitino]

2025-01-16 Thread via GitHub
jerryshao merged PR #6313: URL: https://github.com/apache/gravitino/pull/6313 -- 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] Flink SQL client start failed for missing Paimon package [gravitino]

2025-01-16 Thread via GitHub
jerryshao closed issue #6309: [Bug report] Flink SQL client start failed for missing Paimon package URL: https://github.com/apache/gravitino/issues/6309 -- 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] Flink SQL client start failed for missing Paimon package [gravitino]

2025-01-16 Thread via GitHub
jerryshao closed issue #6309: [Bug report] Flink SQL client start failed for missing Paimon package URL: https://github.com/apache/gravitino/issues/6309 -- 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 main updated: [#6309] fix(flink): remove paimon package from GravitinoPaimonCatalogFactory (#6313)

2025-01-16 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 85b919cf29 [#6309] fix(flink): remove paimon pack

(gravitino) branch cherry-pick-branch-0.8-85b919cf29a6274fdc4a778433f38ca0adc2a901 created (now 391bb27624)

2025-01-16 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-85b919cf29a6274fdc4a778433f38ca0adc2a901 in repository https://gitbox.apache.org/repos/asf/gravitino.git at 391bb27624 [#6309] fix(flink): remove paimon package

[PR] [#6309] fix(flink): remove paimon package from GravitinoPaimonCatalogFactory [gravitino]

2025-01-16 Thread via GitHub
github-actions[bot] opened a new pull request, #6314: URL: https://github.com/apache/gravitino/pull/6314 ### What changes were proposed in this pull request? remove paimon package from `GravitinoPaimonCatalogFactory`, because Paimon package may not in the Flink classpath when loading

[PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
FANNG1 opened a new pull request, #6308: URL: https://github.com/apache/gravitino/pull/6308 ### What changes were proposed in this pull request? (Please outline the changes and how this PR fixes the issue.) ### Why are the changes needed? (Please clarify why the c

Re: [PR] [#6307] feat(flink): remove log4j from Gravitino Flink connector [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on PR #6308: URL: https://github.com/apache/gravitino/pull/6308#issuecomment-2597443704 @coolderli @jerryshao PTAL -- 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

[PR] [#6309] fix(flink): remove paimon package from GravitinoPaimonCatalogFactory [gravitino]

2025-01-16 Thread via GitHub
FANNG1 opened a new pull request, #6313: URL: https://github.com/apache/gravitino/pull/6313 ### What changes were proposed in this pull request? remove paimon package from GravitinoPaimonCatalogFactory ### Why are the changes needed? Fix: #6309 ### Does this PR in

Re: [PR] [#6309] fix(flink): remove paimon package from GravitinoPaimonCatalogFactory [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on PR #6313: URL: https://github.com/apache/gravitino/pull/6313#issuecomment-2597597504 @hdygxsj @jerryshao PTAL -- 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] [Improvement] In AreYouSure Scanner should be closed (CLI) [gravitino]

2025-01-16 Thread via GitHub
justinmclean closed issue #6277: [Improvement] In AreYouSure Scanner should be closed (CLI) URL: https://github.com/apache/gravitino/issues/6277 -- 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 s

(gravitino) branch main updated: [#6277] improve(CLI): Fix AreYouSure Scanner should close. (#6285)

2025-01-16 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean 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 0fc57376c0 [#6277] improve(CLI): Fix AreYouSure

Re: [PR] [#6277] improve(CLI): Fix AreYouSure Scanner should close. [gravitino]

2025-01-16 Thread via GitHub
justinmclean merged PR #6285: URL: https://github.com/apache/gravitino/pull/6285 -- 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...@gra

Re: [I] [Improvement] In the Gravitino CLI/Java API you can delete the anoymous user [gravitino]

2025-01-16 Thread via GitHub
justinmclean commented on issue #5932: URL: https://github.com/apache/gravitino/issues/5932#issuecomment-2597088307 Yes, that is one approach we could take. -- 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: [I] [Improvement] improve the docker publish for gravitino-playground:trino [gravitino]

2025-01-16 Thread via GitHub
FANNG1 closed issue #6281: [Improvement] improve the docker publish for gravitino-playground:trino URL: https://github.com/apache/gravitino/issues/6281 -- 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 t

Re: [I] [Improvement] improve the docker publish for gravitino-playground:trino [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on issue #6281: URL: https://github.com/apache/gravitino/issues/6281#issuecomment-2597269865 I'm not planning to change the tag name, if we want to keep the tag name, we could handle it in many ways. I think the docker version (435-gravitino-$gravitino_version) for trino wh

(gravitino) branch main updated: [#6278] improve(CLI): Revoke all privileges from a role via the Gravitino CLI. (#6293)

2025-01-16 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean 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 6fb45d6a94 [#6278] improve(CLI): Revoke all priv

Re: [PR] [#6278] improve(CLI): Revoke all privileges from a role via the Gravitino CLI. [gravitino]

2025-01-16 Thread via GitHub
justinmclean merged PR #6293: URL: https://github.com/apache/gravitino/pull/6293 -- 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...@gra

Re: [I] [Improvement] Revoke all privileges from a role via the Gravitino CLI [gravitino]

2025-01-16 Thread via GitHub
justinmclean closed issue #6278: [Improvement] Revoke all privileges from a role via the Gravitino CLI URL: https://github.com/apache/gravitino/issues/6278 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] [#6270] (Improvement) unified cache framework [gravitino]

2025-01-16 Thread via GitHub
FANNG1 commented on PR #6271: URL: https://github.com/apache/gravitino/pull/6271#issuecomment-2597195824 @jerryshao WDYT? -- 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

  1   2   >