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

2025-03-12 Thread via GitHub
diqiu50 commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1991017228 ## clients/filesystem-fuse/src/main.rs: ## @@ -193,6 +213,33 @@ fn main() -> Result<(), i32> { path.to_string_lossy().to_string() }; +

Re: [PR] [#6651] improvement(docker-image): Update docker Hive CI image to 0.1.18 [gravitino]

2025-03-12 Thread via GitHub
jerryshao commented on code in PR #6652: URL: https://github.com/apache/gravitino/pull/6652#discussion_r1990726046 ## dev/docker/hive/download-release.sh: ## @@ -0,0 +1,59 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributo

Re: [I] [Bug report] Fail to import schema entity to the store when list tag for pg column and inherited are true [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 commented on issue #6678: URL: https://github.com/apache/gravitino/issues/6678#issuecomment-2716746608 @danhuawang You'd better provide details on how I can reproduce it, such as the data I need to create metalake/tags/ or... -- This is an automated message from the Apache Gi

[I] [FEATURE] support location placeholder when creating fileset [gravitino]

2025-03-12 Thread via GitHub
mchades opened a new issue, #6680: URL: https://github.com/apache/gravitino/issues/6680 ### Describe the feature user can define the storage location with placeholders and replace them when creating a fileset ### Motivation make fileset location definition more flexible

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

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

Re: [PR] [#6651] improvement(docker-image): Update docker Hive CI image to 0.1.18 [gravitino]

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

Re: [I] [Improvement] Upgrade Gravitino Hive CI image to 0.1.18 [gravitino]

2025-03-12 Thread via GitHub
jerryshao closed issue #6651: [Improvement] Upgrade Gravitino Hive CI image to 0.1.18 URL: https://github.com/apache/gravitino/issues/6651 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

(gravitino) branch main updated: [#6651] improvement(docker-image): Update docker Hive CI image to 0.1.18 (#6652)

2025-03-12 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 92bf69fbf7 [#6651] improvement(docker-image): Upda

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

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

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

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

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

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

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

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

Re: [PR] [#6561] improvement(core): Cache Hadoop Filesystem instance on Gravitino server to improve the performance [gravitino]

2025-03-12 Thread via GitHub
sunxiaojian commented on PR #6619: URL: https://github.com/apache/gravitino/pull/6619#issuecomment-2718171101 > If I understand correctly, the current version of FileSystem Provider will also be used for Hadoop GVFS. I think using Hadoop's own cache here will cause security issues, because

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

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

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

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

Re: [PR] [Minor] Improve log messages [gravitino]

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

(gravitino) branch main updated: [Minor] Improve log messages (#6531)

2025-03-12 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 3f019042aa [Minor] Improve log messages (#6531) 3f

Re: [I] [Bug report] revoking permissions does not take effect when a Role has multiple previlege for Metadata. [gravitino]

2025-03-12 Thread via GitHub
FourFriends commented on issue #6682: URL: https://github.com/apache/gravitino/issues/6682#issuecomment-2717656890 @jerqi please assign to me -- 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

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

2025-03-12 Thread via GitHub
FANNG1 commented on PR #6664: URL: https://github.com/apache/gravitino/pull/6664#issuecomment-2717662600 > @FANNG1 Can you select one jar and manually test its functionality with engines like Spark? I verified with Spark and Gravitino AWS bundle jar, the log is normal. -- This is a

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

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

Re: [PR] [#6600] docs: Refactor the how-to-build docs [gravitino]

2025-03-12 Thread via GitHub
justinmclean commented on code in PR #6659: URL: https://github.com/apache/gravitino/pull/6659#discussion_r1991461982 ## docs/how-to-build.md: ## @@ -4,244 +4,232 @@ slug: /how-to-build license: "This software is licensed under the Apache License version 2." --- -- [Prerequi

Re: [PR] [#6651] improvement(docker-image): Update docker Hive CI image to 0.1.18 [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 commented on code in PR #6652: URL: https://github.com/apache/gravitino/pull/6652#discussion_r1990715291 ## dev/docker/hive/download-release.sh: ## @@ -0,0 +1,59 @@ +#!/usr/bin/env bash +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

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

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

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

2025-03-12 Thread via GitHub
yuqi1129 commented on PR #6664: URL: https://github.com/apache/gravitino/pull/6664#issuecomment-2717754063 > > @FANNG1 Can you select one jar and manually test its functionality with engines like Spark? > > I verified with Spark and Gravitino AWS bundle jar, the log is normal.

Re: [PR] [MINOR] refactor: Tweak the build script [gravitino]

2025-03-12 Thread via GitHub
tengqm commented on code in PR #6677: URL: https://github.com/apache/gravitino/pull/6677#discussion_r1992460182 ## build.gradle.kts: ## @@ -82,28 +82,26 @@ project.extra["extraJvmArgs"] = if (extra["jdkVersion"] in listOf("8", "11")) { listOf( "-XX:+IgnoreUnrecognizedVM

(gravitino) branch cherry-pick-branch-0.8-b237637ca42c6a11082d6aa6bf57ef574715ccaf created (now bd18880036)

2025-03-12 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-b237637ca42c6a11082d6aa6bf57ef574715ccaf in repository https://gitbox.apache.org/repos/asf/gravitino.git at bd18880036 [#6654] fix(bundle): remove log4j from Gra

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

2025-03-12 Thread via GitHub
yuqi1129 merged PR #6664: URL: https://github.com/apache/gravitino/pull/6664 -- 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] [#6654] fix(bundle): remove log4j from Gravitino cloud bundle jars [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 closed pull request #6684: [#6654] fix(bundle): remove log4j from Gravitino cloud bundle jars URL: https://github.com/apache/gravitino/pull/6684 -- 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: [PR] [#6530] improvement(authorization): Reduce repeated strings in Ranger classes [gravitino]

2025-03-12 Thread via GitHub
Abyss-lord commented on PR #6685: URL: https://github.com/apache/gravitino/pull/6685#issuecomment-2719610491 @justinmclean @tengqm could you please review this PR when you have time? I’d really appreciate your feedback. -- This is an automated message from the Apache Git Service. To respo

Re: [I] [Bug report] The CLI.sh can't find the Jar [gravitino]

2025-03-12 Thread via GitHub
justinmclean commented on issue #6603: URL: https://github.com/apache/gravitino/issues/6603#issuecomment-2719643723 Good question. Setting up an alias, as suggested in the docs, is just one way of running it. Some other steps may be required to make it work other than those two copy command

(gravitino) branch branch-0.8 updated: [#6654] fix(bundle): remove log4j from Gravitino cloud bundle jars (#6684)

2025-03-12 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 83fcb56fd4 [#6654] fix(bundle): remove

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

2025-03-12 Thread via GitHub
FANNG1 merged PR #6684: URL: https://github.com/apache/gravitino/pull/6684 -- 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: [PR] [MINOR] refactor: Tweak the build script [gravitino]

2025-03-12 Thread via GitHub
jerryshao commented on code in PR #6677: URL: https://github.com/apache/gravitino/pull/6677#discussion_r1992581710 ## build.gradle.kts: ## @@ -823,16 +846,16 @@ tasks { register("copyCatalogLibAndConfigs", Copy::class) { dependsOn( + ":catalogs:catalog-hadoop:copy

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

2025-03-12 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1992580732 ## clients/filesystem-fuse/src/main.rs: ## @@ -193,6 +213,33 @@ fn main() -> Result<(), i32> { path.to_string_lossy().to_string() };

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

2025-03-12 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1992580732 ## clients/filesystem-fuse/src/main.rs: ## @@ -193,6 +213,33 @@ fn main() -> Result<(), i32> { path.to_string_lossy().to_string() };

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

2025-03-12 Thread via GitHub
yuqi1129 commented on PR #6671: URL: https://github.com/apache/gravitino/pull/6671#issuecomment-2719643844 @lsyulong Please see PR https://github.com/apache/gravitino/pull/6683. I have found and fixed the problem in your PR. -- This is an automated message from the Apache Git Service

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

2025-03-12 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1992580553 ## clients/filesystem-fuse/src/main.rs: ## @@ -193,6 +213,33 @@ fn main() -> Result<(), i32> { path.to_string_lossy().to_string() };

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

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

(gravitino) branch main updated: [#6079] fix(core): Support model failure event to Gravitino server (#6670)

2025-03-12 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 8cc094c81b [#6079] fix(core): Support model failur

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

2025-03-12 Thread via GitHub
FANNG1 closed issue #6079: [Subtask] Support model failure event to Gravitino server URL: https://github.com/apache/gravitino/issues/6079 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [I] [Bug report] The CLI.sh can't find the Jar [gravitino]

2025-03-12 Thread via GitHub
justinmclean commented on issue #6603: URL: https://github.com/apache/gravitino/issues/6603#issuecomment-2719720848 Shure give it a go -- 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

[I] [Bug report] revoking permissions does not take effect when a Role has multiple previlege for Metadata. [gravitino]

2025-03-12 Thread via GitHub
FourFriends opened a new issue, #6682: URL: https://github.com/apache/gravitino/issues/6682 ### Version main branch ### Describe what's wrong When granting a Role permissions corresponding to Metadata (e.g. ReadFileset, WriteFileset, DropFileset), if there are multiple p

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

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

Re: [PR] [#6600] docs: Refactor the how-to-build docs [gravitino]

2025-03-12 Thread via GitHub
justinmclean commented on code in PR #6659: URL: https://github.com/apache/gravitino/pull/6659#discussion_r1991461982 ## docs/how-to-build.md: ## @@ -4,244 +4,232 @@ slug: /how-to-build license: "This software is licensed under the Apache License version 2." --- -- [Prerequi

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

2025-03-12 Thread via GitHub
jerryshao merged PR #6679: URL: https://github.com/apache/gravitino/pull/6679 -- 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] [#6600] docs: Refactor the how-to-build docs [gravitino]

2025-03-12 Thread via GitHub
tengqm commented on code in PR #6659: URL: https://github.com/apache/gravitino/pull/6659#discussion_r1991394483 ## docs/how-to-build.md: ## @@ -4,244 +4,232 @@ slug: /how-to-build license: "This software is licensed under the Apache License version 2." --- -- [Prerequisites]

Re: [PR] [#6600] docs: Refactor the how-to-build docs [gravitino]

2025-03-12 Thread via GitHub
tengqm commented on code in PR #6659: URL: https://github.com/apache/gravitino/pull/6659#discussion_r1991479017 ## docs/how-to-build.md: ## @@ -4,244 +4,232 @@ slug: /how-to-build license: "This software is licensed under the Apache License version 2." --- -- [Prerequisites]

Re: [I] [Improvement] Upgrade Gravitino Hive CI image to 0.1.18 [gravitino]

2025-03-12 Thread via GitHub
jerryshao closed issue #6651: [Improvement] Upgrade Gravitino Hive CI image to 0.1.18 URL: https://github.com/apache/gravitino/issues/6651 -- 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: [PR] [#5289][Improvement] CatalogMysqlDriverIT miss driver url [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 commented on PR #6671: URL: https://github.com/apache/gravitino/pull/6671#issuecomment-2716697060 @lsyulong There are two files under folder `hadoop-catalog`, one is for Gravitino server and one is for container log, please check it https://github.com/user-attachments/assets/

Re: [PR] [#6600] docs: Refactor the how-to-build docs [gravitino]

2025-03-12 Thread via GitHub
tengqm commented on code in PR #6659: URL: https://github.com/apache/gravitino/pull/6659#discussion_r1991479017 ## docs/how-to-build.md: ## @@ -4,244 +4,232 @@ slug: /how-to-build license: "This software is licensed under the Apache License version 2." --- -- [Prerequisites]

Re: [PR] [#6600] docs: Refactor the how-to-build docs [gravitino]

2025-03-12 Thread via GitHub
justinmclean commented on code in PR #6659: URL: https://github.com/apache/gravitino/pull/6659#discussion_r1991512472 ## docs/how-to-build.md: ## @@ -4,244 +4,232 @@ slug: /how-to-build license: "This software is licensed under the Apache License version 2." --- -- [Prerequi

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

2025-03-12 Thread via GitHub
jerryshao commented on code in PR #6599: URL: https://github.com/apache/gravitino/pull/6599#discussion_r1991452403 ## dev/charts/gravitino/README.md: ## @@ -0,0 +1,118 @@ + +# gravitino + +Gravitino is a high-performance, geo-distributed, and federated metadata lake. It manages

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

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

[PR] [TEST] Do NOT review [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 opened a new pull request, #6683: URL: https://github.com/apache/gravitino/pull/6683 ### 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

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

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

Re: [PR] [#6651] improvement(docker-image): Update docker Hive CI image to 0.1.18 [gravitino]

2025-03-12 Thread via GitHub
jerryshao commented on PR #6652: URL: https://github.com/apache/gravitino/pull/6652#issuecomment-2716838530 @xunliu do you want to take another look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] [#6487] improvement(hadoop-catalog): Close FileSystem instance manually. [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 closed pull request #6488: [#6487] improvement(hadoop-catalog): Close FileSystem instance manually. URL: https://github.com/apache/gravitino/pull/6488 -- 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] [#6487] improvement(hadoop-catalog): Close FileSystem instance manually. [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 commented on PR #6488: URL: https://github.com/apache/gravitino/pull/6488#issuecomment-2717323529 No need anymore. -- 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] [#6079] fix(core): Support model failure event to Gravitino server [gravitino]

2025-03-12 Thread via GitHub
Abyss-lord commented on PR #6670: URL: https://github.com/apache/gravitino/pull/6670#issuecomment-2719507395 @FANNG1 @jerryshao 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 t

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

2025-03-12 Thread via GitHub
github-actions[bot] opened a new pull request, #6684: URL: https://github.com/apache/gravitino/pull/6684 ### What changes were proposed in this pull request? Remove log4j from Gravitino bundle jars ### Why are the changes needed? Fix: #6654 ### Does this PR introd

(gravitino) branch main updated (5191e26130 -> b237637ca4)

2025-03-12 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git from 5191e26130 [#6368] improvement(flink-connector): Add tests for REST Catalog to the Flink connector for Iceberg (#6

Re: [I] [Bug report] The log may empty if including Gravitino bundle jars [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 closed issue #6654: [Bug report] The log may empty if including Gravitino bundle jars URL: https://github.com/apache/gravitino/issues/6654 -- 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: [#6368] improvement(flink-connector): Add tests for REST Catalog to the Flink connector for Iceberg (#6622)

2025-03-12 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 5191e26130 [#6368] improvement(flink-connector): A

Re: [I] [Bug report] The log may empty if including Gravitino bundle jars [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 closed issue #6654: [Bug report] The log may empty if including Gravitino bundle jars URL: https://github.com/apache/gravitino/issues/6654 -- 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] [#5873] feat(gvfs-fuse): add debug log for FuseApiHandle [gravitino]

2025-03-12 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1992580883 ## clients/filesystem-fuse/src/main.rs: ## @@ -193,6 +213,33 @@ fn main() -> Result<(), i32> { path.to_string_lossy().to_string() };

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

2025-03-12 Thread via GitHub
FANNG1 commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1992608604 ## flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/catalog/BaseCatalogFactory.java: ## @@ -45,7 +46,7 @@ public interface BaseCatalogFactory e

[PR] [#6603] chore(*): fix CLI.sh can't find the Jar [gravitino]

2025-03-12 Thread via GitHub
Abyss-lord opened a new pull request, #6686: URL: https://github.com/apache/gravitino/pull/6686 ### What changes were proposed in this pull request? fix CLI.sh can't find the Jar 1. Changing the packaging flow, Copy the CLI-related jars to `package/libs` and add gcli.sh to `bin`.

Re: [PR] [#6603] chore(*): fix CLI.sh can't find the Jar [gravitino]

2025-03-12 Thread via GitHub
Abyss-lord commented on PR #6686: URL: https://github.com/apache/gravitino/pull/6686#issuecomment-2720119823 @justinmclean @tengqm could you please review this PR when you have time? I’d really appreciate your feedback. -- This is an automated message from the Apache Git Service. To resp

Re: [I] [Improvement] Allow User Configuration of JDBC Backend Connection Pool Parameters [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 closed issue #6467: [Improvement] Allow User Configuration of JDBC Backend Connection Pool Parameters URL: https://github.com/apache/gravitino/issues/6467 -- 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] Refactor getting started docs [gravitino-site]

2025-03-12 Thread via GitHub
justinmclean commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1987114747 ## 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] [#6079] fix(core): Support model failure event to Gravitino server [gravitino]

2025-03-12 Thread via GitHub
FANNG1 commented on PR #6670: URL: https://github.com/apache/gravitino/pull/6670#issuecomment-2719539882 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] [#6561] improvement(core): Cache Hadoop Filesystem instance on Gravitino server to improve the performance [gravitino]

2025-03-12 Thread via GitHub
yuqi1129 commented on PR #6619: URL: https://github.com/apache/gravitino/pull/6619#issuecomment-2719777926 > If I understand correctly, the current version of FileSystem Provider will also be used for Hadoop GVFS. I think using Hadoop's own cache here will cause security issues, because the

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

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

Re: [PR] [#6561] improvement(core): Cache Hadoop Filesystem instance on Gravitino server to improve the performance [gravitino]

2025-03-12 Thread via GitHub
sunxiaojian commented on PR #6619: URL: https://github.com/apache/gravitino/pull/6619#issuecomment-2719881426 The GVFS side uses its own maintained cache, and when GVS initializing Filesystem, it will force not to use FS cache -- This is an automated message from the Apache Git Service. T

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

2025-03-12 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1992580732 ## clients/filesystem-fuse/src/main.rs: ## @@ -193,6 +213,33 @@ fn main() -> Result<(), i32> { path.to_string_lossy().to_string() };

Re: [PR] [#6561] improvement(core): Cache Hadoop Filesystem instance on Gravitino server to improve the performance [gravitino]

2025-03-12 Thread via GitHub
xloya commented on PR #6619: URL: https://github.com/apache/gravitino/pull/6619#issuecomment-2719886580 > > If I understand correctly, the current version of FileSystem Provider will also be used for Hadoop GVFS. I think using Hadoop's own cache here will cause security issues, because the

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

2025-03-12 Thread via GitHub
unknowntpo commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1992580732 ## clients/filesystem-fuse/src/main.rs: ## @@ -193,6 +213,33 @@ fn main() -> Result<(), i32> { path.to_string_lossy().to_string() };

Re: [PR] [#6530] improvement(authorization): Reduce repeated strings in Ranger classes [gravitino]

2025-03-12 Thread via GitHub
tengqm commented on PR #6685: URL: https://github.com/apache/gravitino/pull/6685#issuecomment-2719961596 looks good to me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. T

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

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

2025-03-12 Thread via GitHub
FANNG1 commented on PR #6622: URL: https://github.com/apache/gravitino/pull/6622#issuecomment-2719460197 @sunxiaojian thanks for your contribution -- 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] [#6079] fix(core): Support model failure event to Gravitino server [gravitino]

2025-03-12 Thread via GitHub
FANNG1 commented on code in PR #6670: URL: https://github.com/apache/gravitino/pull/6670#discussion_r1992493246 ## core/src/main/java/org/apache/gravitino/listener/api/event/DeleteModelFailureEvent.java: ## @@ -0,0 +1,51 @@ +/* + * Licensed to the Apache Software Foundation (ASF

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

2025-03-12 Thread via GitHub
FANNG1 commented on PR #6670: URL: https://github.com/apache/gravitino/pull/6670#issuecomment-2719476153 LGTM except minor comments -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific co

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

2025-03-12 Thread via GitHub
FANNG1 commented on code in PR #6670: URL: https://github.com/apache/gravitino/pull/6670#discussion_r1992493740 ## core/src/main/java/org/apache/gravitino/listener/api/event/ModelFailureEvent.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

Re: [I] [Bug report] when jdbc user and password in the `gravitino.iceberg-rest.uri`, should not show jdbc.user is null error [gravitino]

2025-03-12 Thread via GitHub
FANNG1 commented on issue #6646: URL: https://github.com/apache/gravitino/issues/6646#issuecomment-2719443816 @sunxiaojian , thanks for taking! -- 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

Re: [I] [Bug report] when jdbc user and password in the `gravitino.iceberg-rest.uri`, should not show jdbc.user is null error [gravitino]

2025-03-12 Thread via GitHub
sunxiaojian commented on issue #6646: URL: https://github.com/apache/gravitino/issues/6646#issuecomment-2718161468 @FANNG1 can assign it to me. -- 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

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

2025-03-12 Thread via GitHub
diqiu50 commented on code in PR #6657: URL: https://github.com/apache/gravitino/pull/6657#discussion_r1992563780 ## trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/catalog/jdbc/mysql/MySQLDataTypeTransformer.java: ## @@ -55,24 +55,54 @@ public

Re: [I] [Bug report] The CLI.sh can't find the Jar [gravitino]

2025-03-12 Thread via GitHub
Abyss-lord commented on issue #6603: URL: https://github.com/apache/gravitino/issues/6603#issuecomment-2719614259 @justinmclean Hi Justin, Whether `CLI.sh` should be copied into `package/bin` during the build phase? and copy cli-related jar to `package/libs`? -- This is an automated mes