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

2025-03-15 Thread via GitHub
diqiu50 commented on code in PR #5905: URL: https://github.com/apache/gravitino/pull/5905#discussion_r1986545413 ## 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] [#6233] feat(flink): flink jdbc catalog [gravitino]

2025-03-15 Thread via GitHub
hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1995585561 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/jdbc/AbstractJdbcPropertiesConverter.java: ## @@ -0,0 +1,87 @@ +/* + * Licensed to the Apa

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

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

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

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

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

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

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

2025-03-15 Thread via GitHub
justinmclean commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986589892 ## docs/getting-started/hive.md: ## @@ -0,0 +1,33 @@ +--- +title: "Installing Apache Hive" +slug: /getting-started/hive +license: "This software is licensed und

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

2025-03-15 Thread via GitHub
xunliu commented on code in PR #6652: URL: https://github.com/apache/gravitino/pull/6652#discussion_r1990440232 ## dev/docker/hive/download-release.sh: ## Review Comment: @yuqi1129 Please clean invalid comments. -- This is an automated message from the Apache Git Servic

Re: [PR] fix: Remove incorrect @VisibleForTesting annotation & apply Spotless formatting [gravitino]

2025-03-15 Thread via GitHub
Brijeshthummar02 commented on PR #6571: URL: https://github.com/apache/gravitino/pull/6571#issuecomment-2710321844 the only change i made was removing @VisibleForTesting from .java file rest all changes where done by ./gradlew :spark-connector:spark-common:spotlessApply which is mentioned

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

2025-03-15 Thread via GitHub
mchades commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1986546143 ## core/src/main/java/org/apache/gravitino/metalake/MetalakeManager.java: ## @@ -64,6 +71,26 @@ public class MetalakeManager implements MetalakeDispatcher { priv

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

2025-03-15 Thread via GitHub
Brijeshthummar02 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2703744953 @tengqm do we need any changes?? -- 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] [#5873] feat(gvfs-fuse): add debug log for FuseApiHandle [gravitino]

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

(gravitino) branch branch-0.8 updated: [#6595] fix (trino-connector): Disable the Trino cascading query integration test (#6633)

2025-03-15 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 53b65f72a1 [#6595] fix (trino-connecto

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

2025-03-15 Thread via GitHub
jerryshao commented on PR #6615: URL: https://github.com/apache/gravitino/pull/6615#issuecomment-2703768038 @Brijeshthummar02 it is unnecessary to update the branch if there's no conflict, the updating will rerun all the CI tasks, which is time-consuming. GitHub will rebase your PR when mer

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

2025-03-15 Thread via GitHub
liangyouze opened a new issue, #6667: URL: https://github.com/apache/gravitino/issues/6667 ### Version main branch ### Describe what's wrong When creating a JDBC catalog (e.g., MySQL) on Gravitino and then creating some databases or tables, if the MySQL connection become

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

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

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

2025-03-15 Thread via GitHub
jerqi commented on code in PR #6575: URL: https://github.com/apache/gravitino/pull/6575#discussion_r1983260489 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -959,6 +1004,21 @@ protected void

[I] [EPIC] Fileset supports multiple locations [gravitino]

2025-03-15 Thread via GitHub
mchades opened a new issue, #6681: URL: https://github.com/apache/gravitino/issues/6681 ### Describe the proposal support multiple locations for one fileset, for example: https://github.com/user-attachments/assets/9ded6165-3d9c-4e08-baa5-be2b75130c42"; /> ### Task list

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

2025-03-15 Thread via GitHub
jerryshao commented on PR #49: URL: https://github.com/apache/gravitino-site/pull/49#issuecomment-2712300633 > > @tengqm we don't need to sync the changes to this repo every time. Our doc is versioned and released with Gravitino. After the Gravitino is released, we will sync the doc from Gr

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

2025-03-15 Thread via GitHub
justinmclean commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1987109461 ## 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] [#6233] feat(flink): flink jdbc catalog [gravitino]

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

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

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

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

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

[PR] [#131] docs(README): add missing container ports [gravitino-playground]

2025-03-15 Thread via GitHub
AndreVale69 opened a new pull request, #134: URL: https://github.com/apache/gravitino-playground/pull/134 ### What changes were proposed in this pull request? 1. Add missing ranger container port 2. Add missing spark container port 3. Fix postgresql port typo ### Why are

Re: [PR] [#135] improvement: Update Hive image [gravitino-playground]

2025-03-15 Thread via GitHub
jerqi commented on PR #136: URL: https://github.com/apache/gravitino-playground/pull/136#issuecomment-2709607467 Could you verify all the test cases with new image including authorization demo? -- This is an automated message from the Apache Git Service. To respond to the message, please

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

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

(gravitino) branch issue_6651 deleted (was b05b8cd657)

2025-03-15 Thread yuqi4733
This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a change to branch issue_6651 in repository https://gitbox.apache.org/repos/asf/gravitino.git was b05b8cd657 fix The revisions that were on this branch are still contained in other references; therefore, th

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

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

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

2025-03-15 Thread via GitHub
justinmclean commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1987109461 ## 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] [#6656 ] fix(trino-connector): support read MySQL time/datetime/timestamp columns with precision [gravitino]

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