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-10 Thread via GitHub
FANNG1 commented on issue #6646: URL: https://github.com/apache/gravitino/issues/6646#issuecomment-2712181389 Yes, it seems reasonable to remove the checks, and the underlying Iceberg and jdbc library doesn't enforce the check either. -- This is an automated message from the Apache Git Se

(gravitino-site) branch main updated: fix spelling on contrib.mdx (#48)

2025-03-10 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-site.git The following commit(s) were added to refs/heads/main by this push: new b160d7ff fix spelling on contrib.mdx (#48) b1

Re: [I] [Bug report] Cannot create tables by following the docs with JDBC backend and S3 [gravitino]

2025-03-10 Thread via GitHub
jovezhong commented on issue #6648: URL: https://github.com/apache/gravitino/issues/6648#issuecomment-2711335782 Reporting back * I still got the missing field error with 0.8 after changing s3 endpoint to `https://s3.us-west-2.amazonaws.com` * I setup 0.7 rest server and use `aws-bundl

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

2025-03-10 Thread via GitHub
unknowntpo commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1988220923 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -214,4 +233,407 @@ public static String getMetadataObjectFu

Re: [PR] fix spelling on contrib.mdx [gravitino-site]

2025-03-10 Thread via GitHub
jerryshao merged PR #48: URL: https://github.com/apache/gravitino-site/pull/48 -- 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...@gravi

Re: [I] [Improvement] Support login with grant_type=authorization_code, when Gravitino use OAuth as authenticator [gravitino]

2025-03-10 Thread via GitHub
danhuawang commented on issue #5875: URL: https://github.com/apache/gravitino/issues/5875#issuecomment-2712824449 > > Any updates on this? I'd like to be able to use Keycloak with Gravitino. > > This isn't a blocker for using Keycloak with Gravitino. [@danhuawang](https://github.com/d

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

2025-03-10 Thread via GitHub
FANNG1 commented on PR #6664: URL: https://github.com/apache/gravitino/pull/6664#issuecomment-2712849112 > It likely the LICENSE/NOTICE files will also need updating. This PR only removing the log4j from Gravitino cloud bundle jars not the Gravitino package, seems we should keep log4j

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

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

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

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

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

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

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

2025-03-10 Thread via GitHub
yuqi1129 commented on PR #6644: URL: https://github.com/apache/gravitino/pull/6644#issuecomment-2710486668 @sunxiaojian could you help to check if this issue can solve the https://github.com/apache/gravitino/pull/6455/files#r1955682884 by the way? -- This is an automated message from the

[PR] [SIP] remove log4j from Gravitino bundle jars [gravitino]

2025-03-10 Thread via GitHub
FANNG1 opened a new pull request, #6664: URL: https://github.com/apache/gravitino/pull/6664 ### 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] Refactor getting started docs [gravitino-site]

2025-03-10 Thread via GitHub
tengqm commented on PR #49: URL: https://github.com/apache/gravitino-site/pull/49#issuecomment-2712488059 @LauraXia123 Thanks for the response. Let me try explain where I'm confused. I'm tasked to refactor the Gravitino docs where the review happens in the `gravitino` project, not her

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

2025-03-10 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

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

2025-03-10 Thread via GitHub
FANNG1 commented on PR #6250: URL: https://github.com/apache/gravitino/pull/6250#issuecomment-2712643624 merged to main, thanks @Abyss-lord for your contribution and @tengqm for the review. -- This is an automated message from the Apache Git Service. To respond to the message, please log

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

2025-03-10 Thread via GitHub
FANNG1 merged PR #6250: URL: https://github.com/apache/gravitino/pull/6250 -- 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] [#6656 ] fix(trino-connector): support read MySQL time/datetime/timestamp columns with precision [gravitino]

2025-03-10 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

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

2025-03-10 Thread via GitHub
FANNG1 closed issue #6076: [Subtask] Support model pre event to Gravitino server URL: https://github.com/apache/gravitino/issues/6076 -- 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] [Subtask] Support model pre event to Gravitino server [gravitino]

2025-03-10 Thread via GitHub
FANNG1 closed issue #6076: [Subtask] Support model pre event to Gravitino server URL: https://github.com/apache/gravitino/issues/6076 -- 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

(gravitino) branch main updated: [#6076] improve(core): Support model pre event to Gravitino server (#6250)

2025-03-10 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 c9c9a99dff [#6076] improve(core): Support model pr

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

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

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

2025-03-10 Thread via GitHub
unknowntpo commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1988236171 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -214,4 +233,407 @@ public static String getMetadataObjectFu

Re: [I] [Bug report] Cannot create tables by following the docs with JDBC backend and S3 [gravitino]

2025-03-10 Thread via GitHub
FANNG1 commented on issue #6648: URL: https://github.com/apache/gravitino/issues/6648#issuecomment-2712279699 Thanks for your feedback and glad to hear not blocks you, I will try to reproduce it. By the way the error means Iceberg client gets an HTTP response with the status code not 2xx, b

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

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

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

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

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

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

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

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

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

2025-03-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1988392167 ## 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] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-10 Thread via GitHub
unknowntpo commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1988230827 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -214,4 +233,407 @@ public static String getMetadataObjectFu

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

2025-03-10 Thread via GitHub
unknowntpo commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1988231517 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -214,4 +233,407 @@ public static String getMetadataObjectFu

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

2025-03-10 Thread via GitHub
tengqm commented on PR #49: URL: https://github.com/apache/gravitino-site/pull/49#issuecomment-2712352143 > Hi @tengqm I think we don't need to cache the sidebar, the sidebar can be generated when we release a new version. Also the sidebar is only used for Docusaurus, so it is not needed by

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

2025-03-10 Thread via GitHub
LauraXia123 commented on PR #49: URL: https://github.com/apache/gravitino-site/pull/49#issuecomment-2712378062 > > Hi @tengqm I think we don't need to cache the sidebar, the sidebar can be generated when we release a new version. Also the sidebar is only used for Docusaurus, so it is not ne

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

2025-03-10 Thread via GitHub
FANNG1 commented on PR #6664: URL: https://github.com/apache/gravitino/pull/6664#issuecomment-2712372122 @yuqi1129 @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 com

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

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

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

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

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

2025-03-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1988388970 ## 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] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-10 Thread via GitHub
jerqi commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1986699518 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -125,6 +142,8 @@ public static String getMetadataObjectFullName(

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

2025-03-10 Thread via GitHub
FANNG1 opened a new issue, #6654: URL: https://github.com/apache/gravitino/issues/6654 ### Version main branch ### Describe what's wrong The Iceberg REST server log may empty if adding Gravitino aws bundle jar, original from https://github.com/apache/gravitino/discus

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

2025-03-10 Thread via GitHub
mchades commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1986749848 ## core/src/main/java/org/apache/gravitino/metalake/MetalakeManager.java: ## @@ -86,8 +89,8 @@ public class MetalakeManager implements MetalakeDispatcher {

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

2025-03-10 Thread via GitHub
FANNG1 commented on issue #6654: URL: https://github.com/apache/gravitino/issues/6654#issuecomment-2709678802 There may be package conflict with log4j in Gravitino bundle jar and user packages, seems similar to https://github.com/apache/iceberg/issues/10534, How about excluding log4j packa

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

2025-03-10 Thread via GitHub
FANNG1 commented on issue #6654: URL: https://github.com/apache/gravitino/issues/6654#issuecomment-2709768379 yes -- 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 unsubsc

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

2025-03-10 Thread via GitHub
tengqm opened a new pull request, #49: URL: https://github.com/apache/gravitino-site/pull/49 This is a PR sync'ing the changes made to the `gravitino` docs. -- 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] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

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

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

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

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

2025-03-10 Thread via GitHub
yuqi1129 opened a new pull request, #6652: URL: https://github.com/apache/gravitino/pull/6652 ### What changes were proposed in this pull request? - Change the docker Hive CI image from 0.1.17 to 0.1.18 - Add some change log. ### Why are the changes needed? To sup

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

2025-03-10 Thread via GitHub
yuqi1129 commented on issue #6654: URL: https://github.com/apache/gravitino/issues/6654#issuecomment-2709694182 > How about excluding log4j packages from the Gravitino bundle jar? What do you mean by `excluding` log2j packages? are we going to make it `compileOnly`? -- This is a

Re: [I] [Improvement] Support login with grant_type=authorization_code, when Gravitino use OAuth as authenticator [gravitino]

2025-03-10 Thread via GitHub
jerqi commented on issue #5875: URL: https://github.com/apache/gravitino/issues/5875#issuecomment-2709609596 > Any updates on this? I'd like to be able to use Keycloak with Gravitino. This isn't a blocker for using Keycloak with Gravitino. @danhuawang can share some experience about K

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

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

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

2025-03-10 Thread via GitHub
jerqi commented on PR #6644: URL: https://github.com/apache/gravitino/pull/6644#issuecomment-2709861537 > Member Actually I think this is weird that `null` equals to empty collection. -- This is an automated message from the Apache Git Service. To respond to the message, please log

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

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

(gravitino) branch main updated (0e41493414 -> 826863c5b0)

2025-03-10 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 0e41493414 [#6612] fix(hive-image): Support Chinese in database, table and column commens for Hive catalog (#6625)

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

2025-03-10 Thread via GitHub
jerryshao merged PR #6569: 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 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: [#6650] improvement(test): add IT to cover altering catalog location (#6653)

2025-03-10 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 10252e77ee [#6650] improvement(test): add IT to co

Re: [I] [Bug report] Table comment doesn't support UTF-8 [gravitino]

2025-03-10 Thread via GitHub
shaofengshi commented on issue #6612: URL: https://github.com/apache/gravitino/issues/6612#issuecomment-2710295924 > This is the limitation of Hive Metastoreat present. For MySQL, Hive metastore only supports latin1 and does not support UTF8, see https://issues.apache.org/jira/browse/HIVE-1

Re: [I] [Improvement] Catalog location isn't immutable. If we modify location, will the catalog field variable be updated? [gravitino]

2025-03-10 Thread via GitHub
jerryshao closed issue #6650: [Improvement] Catalog location isn't immutable. If we modify location, will the catalog field variable be updated? URL: https://github.com/apache/gravitino/issues/6650 -- This is an automated message from the Apache Git Service. To respond to the message, please

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

2025-03-10 Thread via GitHub
FANNG1 opened a new pull request, #6658: URL: https://github.com/apache/gravitino/pull/6658 ### What changes were proposed in this pull request? The warehouse location for Iceberg memory catalog backend is hard coded to '/tmp', make it configurable. ### Why are the chan

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

2025-03-10 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] [#6474] improvement(storage): batch listing securable objects in RoleMetaService [gravitino]

2025-03-10 Thread via GitHub
yuqi1129 commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1987113581 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -214,4 +233,407 @@ public static String getMetadataObjectFull

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

2025-03-10 Thread via GitHub
jerryshao commented on PR #6571: URL: https://github.com/apache/gravitino/pull/6571#issuecomment-2710311184 The changes you made bring in so many unrelated changes, I'm curious why do you need to update the Gradle script? -- This is an automated message from the Apache Git Service. To res

Re: [I] [Improvement] Catalog location isn't immutable. If we modify location, will the catalog field variable be updated? [gravitino]

2025-03-10 Thread via GitHub
jerryshao closed issue #6650: [Improvement] Catalog location isn't immutable. If we modify location, will the catalog field variable be updated? URL: https://github.com/apache/gravitino/issues/6650 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [#6650] improvement(test): add IT to cover altering catalog location [gravitino]

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

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

2025-03-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986904613 ## 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-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986900769 ## 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-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986896761 ## 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-10 Thread via GitHub
justinmclean commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986895494 ## 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] [#6651] improvement(docker-image): Update docker Hive CI image to 0.1.18 [gravitino]

2025-03-10 Thread via GitHub
tengqm commented on code in PR #6652: URL: https://github.com/apache/gravitino/pull/6652#discussion_r1986919518 ## docs/docker-image-details.md: ## @@ -185,6 +185,10 @@ You can use this kind of image to test the catalog of Apache Hive. Changelog + +- apache/gravitino-ci:hi

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

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

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

2025-03-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986906213 ## 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-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986907369 ## 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-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986897653 ## 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] [#6636] fix(client): remove py 3.12 classifiers in setup.py [gravitino]

2025-03-10 Thread via GitHub
AndreVale69 commented on PR #6637: URL: https://github.com/apache/gravitino/pull/6637#issuecomment-2709613973 > @AndreVale69 I just check the EOL of Python 3.8, looks like Python 3.8 has already been EOL, so I think we can remove the support of Python 3.8, move on to support 3.12, what do y

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

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

[I] [Bug report] Trino Connector Fails to Read MySQL Timestamp Columns with Precision [gravitino]

2025-03-10 Thread via GitHub
predator4ann opened a new issue, #6656: URL: https://github.com/apache/gravitino/issues/6656 ### Version main branch ### Describe what's wrong When using the Trino connector to read MySQL tables containing timestamp columns with precision specification (e.g., TIMESTAMP(6

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

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

Re: [I] [Improvement] Get `catalogInUse` and `metalakeInUse` from cache instead of retrieve them from storage [gravitino]

2025-03-10 Thread via GitHub
jerryshao closed issue #6566: [Improvement] Get `catalogInUse` and `metalakeInUse` from cache instead of retrieve them from storage URL: https://github.com/apache/gravitino/issues/6566 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [I] [Bug report] software/amazon/awssdk/services/s3/model/S3Exception not in 212MB gravitino-aws-0.8.0-incubating.jar [gravitino]

2025-03-10 Thread via GitHub
shaofengshi commented on issue #6647: URL: https://github.com/apache/gravitino/issues/6647#issuecomment-2709311836 Thanks Jove for reporting this. Seems this is a normal behavior, but we can optimize our documentation so that the users can easily know which jars should be put to the classpa

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

2025-03-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986883351 ## 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-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986903586 ## 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-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986892287 ## docs/getting-started/index.md: ## @@ -0,0 +1,250 @@ +--- +title: "Getting started with Apache Gravitino" +slug: /getting-started +license: "This software is licens

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

2025-03-10 Thread via GitHub
predator4ann opened a new pull request, #6657: URL: https://github.com/apache/gravitino/pull/6657 ### Why are the changes needed? Fix: #6566 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? UT already added. -- T

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

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

[PR] [Minor] Revise syntax errors in getting-started tutorial [gravitino]

2025-03-10 Thread via GitHub
tengqm opened a new pull request, #6655: URL: https://github.com/apache/gravitino/pull/6655 Related: #6600 -- 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 unsubscrib

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

2025-03-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1987005947 ## 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: [I] [Bug report] The CLI.sh can't find the Jar [gravitino]

2025-03-10 Thread via GitHub
Abyss-lord commented on issue #6603: URL: https://github.com/apache/gravitino/issues/6603#issuecomment-2709959271 @justinmclean Whether CLI.sh should be copied into package/bin during the build phase? -- This is an automated message from the Apache Git Service. To respond to the message,

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

2025-03-10 Thread via GitHub
unknowntpo commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1987080538 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -214,4 +235,412 @@ public static String getMetadataObjectFu

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

2025-03-10 Thread via GitHub
tengqm commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1987357612 ## 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-10 Thread via GitHub
justinmclean commented on code in PR #49: URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1986591231 ## 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] [#6641] fix(core): Possible error in the equals method for collection [gravitino]

2025-03-10 Thread via GitHub
sunxiaojian commented on PR #6644: URL: https://github.com/apache/gravitino/pull/6644#issuecomment-2710207419 > > Member > > Actually I think this is weird that `null` equals to empty collection. @jerqi Then let's replace it `public boolean safeIsEqualCollection(Collection li

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

2025-03-10 Thread via GitHub
yuqi1129 commented on code in PR #6601: URL: https://github.com/apache/gravitino/pull/6601#discussion_r1987150834 ## core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java: ## @@ -214,4 +233,407 @@ public static String getMetadataObjectFull

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

2025-03-10 Thread via GitHub
tengqm opened a new pull request, #6659: URL: https://github.com/apache/gravitino/pull/6659 ### What changes were proposed in this pull request? This PR refactors the how-to-build guide. ### Why are the changes needed? The current how-to guide has some duplicated contents

Re: [I] [Bug report] Cannot create tables by following the docs with JDBC backend and S3 [gravitino]

2025-03-10 Thread via GitHub
FANNG1 commented on issue #6648: URL: https://github.com/apache/gravitino/issues/6648#issuecomment-2710635540 I failed to reproduce the error by using your the configuration besides `s3-endpoint`. There are some errors if using `https://tp-internal2.s3.us-west-2.amazonaws.com` style, it wo

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

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