Re: [PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-09 Thread via GitHub
Abyss-lord commented on PR #6866: URL: https://github.com/apache/gravitino/pull/6866#issuecomment-2791386924 Hi @FANNG1 , I've completed the code updates and would appreciate your review of the PR when you have a moment. Here's a summary of the commits: 1. Remove unrelated codes. -- Th

(gravitino) branch support_matrix created (now a9e93e256e)

2025-04-09 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch support_matrix in repository https://gitbox.apache.org/repos/asf/gravitino.git at a9e93e256e [#6704] feat(authz): Support model hook dispatcher (#6844) No new revisions were added by this

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

2025-04-09 Thread via GitHub
yuqi1129 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2791469741 > > @zhengkezhou1 All tests have passed. do you have any additional updates? > > No, there are other types of entities that also need to be modified, so I prefer to use multiple

Re: [I] [Bug report] List catalog in a long run environment, encounter issue:"org.apache.ibatis.exceptions.PersistenceException: " [gravitino]

2025-04-09 Thread via GitHub
yuqi1129 commented on issue #6874: URL: https://github.com/apache/gravitino/issues/6874#issuecomment-2791471165 Change testOnBorrow to `true` can solve it, let me work on it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
FANNG1 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036444755 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[I] [Bug report] List catalog in a long run environment, encounter issue:"org.apache.ibatis.exceptions.PersistenceException: " [gravitino]

2025-04-09 Thread via GitHub
danhuawang opened a new issue, #6874: URL: https://github.com/apache/gravitino/issues/6874 ### Version main branch ### Describe what's wrong https://github.com/user-attachments/assets/d2dfdccc-c89d-419c-9430-e788a7589973"; /> ### Error message and/or stacktrace

Re: [I] [Bug report] use catalog and show databases failed with krb cluster [gravitino]

2025-04-09 Thread via GitHub
yuqi1129 closed issue #6777: [Bug report] use catalog and show databases failed with krb cluster URL: https://github.com/apache/gravitino/issues/6777 -- 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] [Bug report] Iceberg tables are not visible by using spark connector [gravitino]

2025-04-09 Thread via GitHub
FANNG1 commented on issue #6877: URL: https://github.com/apache/gravitino/issues/6877#issuecomment-2791493342 Could you provide more context like error stack? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[PR] [#6874] improvement(core): Set `testOnBorrow` to true to avoid possible connection timeout problem [gravitino]

2025-04-09 Thread via GitHub
yuqi1129 opened a new pull request, #6878: URL: https://github.com/apache/gravitino/pull/6878 ### What changes were proposed in this pull request? Set t`estOnBorrow` and `setTestWhileIdle` of JDBC connection pool from `false` to `true`. ### Why are the changes needed?

[I] [Bug report] Iceberg tables are not visible by using spark connector [gravitino]

2025-04-09 Thread via GitHub
granewang opened a new issue, #6877: URL: https://github.com/apache/gravitino/issues/6877 ### Version main branch ### Describe what's wrong After running kinit with a keytab, executing the spark-sql command with the following configurations: spark-sql -v --conf s

Re: [PR] [#6779] feat(core): Support lineage framework in Gravitino [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6782: URL: https://github.com/apache/gravitino/pull/6782#discussion_r2031692686 ## lineage/src/main/java/org/apache/gravitino/lineage/sink/LineageLogSink.java: ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-09 Thread via GitHub
FANNG1 commented on PR #6866: URL: https://github.com/apache/gravitino/pull/6866#issuecomment-2791262131 The refactor of `EventBus#dispatchEvent` seems not related to this PR, how about propose a new PR to track it? -- This is an automated message from the Apache Git Service. To respond t

(gravitino) annotated tag v0.8.1-incubating-rc3 updated (134c6bef2e -> 78f8642f7c)

2025-04-09 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to annotated tag v0.8.1-incubating-rc3 in repository https://gitbox.apache.org/repos/asf/gravitino.git *** WARNING: tag v0.8.1-incubating-rc3 was modified! *** from 134c6bef2e (commit) to 78f86

(gravitino) branch fix-version-numbers updated (2fd4e39ea1 -> 134c6bef2e)

2025-04-09 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a change to branch fix-version-numbers in repository https://gitbox.apache.org/repos/asf/gravitino.git from 2fd4e39ea1 remove snapshot add 134c6bef2e update docs versions No new revisions were added by t

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
granewang commented on PR #6778: URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2791328735 > > cc @yuqi1129 Do we need to apply similar fix for Iceberg or other catalogs? > > Yes, Iceberg has a similar problem. Could you please also raise a PR to solve it in `HiveBac

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
granewang commented on PR #6778: URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2791329398 > > > @jerqi Test code add ,could U please help to review, thanks > > > > > > PRINCIPAL "gravitino_client/instance@HADOOPKRB" has not created in the docker image `kerberos-h

[I] [Bug report] use catalog and show databases failed with krb cluster for iceberg [gravitino]

2025-04-09 Thread via GitHub
granewang opened a new issue, #6871: URL: https://github.com/apache/gravitino/issues/6871 ### Version main branch ### Describe what's wrong save issue like #6778 ### Error message and/or stacktrace save issue like #6778 ### How to reproduce s

Re: [PR] [#6779] feat(core): Support lineage framework in Gravitino [gravitino]

2025-04-09 Thread via GitHub
FANNG1 commented on code in PR #6782: URL: https://github.com/apache/gravitino/pull/6782#discussion_r2032215870 ## lineage/build.gradle.kts: ## @@ -0,0 +1,39 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See th

(gravitino) branch main updated: [#6217] flink-connector: Add Integration Tests for GravitinoPaimonCatalog with Catalog-backend as JDBC in the Flink (#6856)

2025-04-09 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 0cecdbfc87 [#6217] flink-connector: Add Integratio

[PR] [Minor] Change tagline [gravitino-site]

2025-04-09 Thread via GitHub
justinmclean opened a new pull request, #51: URL: https://github.com/apache/gravitino-site/pull/51 Change the tagline to a new one. -- 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

[I] [Bug report] Release script PyPy asc file ignored [gravitino]

2025-04-09 Thread via GitHub
justinmclean opened a new issue, #6868: URL: https://github.com/apache/gravitino/issues/6868 ### Version main branch ### Describe what's wrong When uploading apache_gravitino-$PYGRAVITINO_VERSION.tar.gz and apache_gravitino-$PYGRAVITINO_VERSION.tar.gz.asc a warning is g

Re: [PR] [#6775] feat(server): Introduce authorization annotations [gravitino]

2025-04-09 Thread via GitHub
xunliu commented on code in PR #6828: URL: https://github.com/apache/gravitino/pull/6828#discussion_r2036324419 ## server/src/main/java/org/apache/gravitino/server/web/auth/annotations/AuthorizeApi.java: ## @@ -0,0 +1,46 @@ +/* + * Licensed to the Apache Software Foundation (ASF

[I] [Bug report] Release script puts tag in wrong branch [gravitino]

2025-04-09 Thread via GitHub
justinmclean opened a new issue, #6870: URL: https://github.com/apache/gravitino/issues/6870 ### Version main branch ### Describe what's wrong The release script will tag the wrong branch if the tag has not been created - note the use of the main branch below rather than

Re: [I] [Improvement] Inconsistencies Between Python and Java APIs [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on issue #6598: URL: https://github.com/apache/gravitino/issues/6598#issuecomment-2790355318 Please do not break the compatibility when you change the method signature, I think this is the most important thing to me. -- This is an automated message from the Apache Git

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
FANNG1 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2034503202 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [I] [Improvement] Inconsistencies Between Python and Java APIs [gravitino]

2025-04-09 Thread via GitHub
AndreVale69 commented on issue #6598: URL: https://github.com/apache/gravitino/issues/6598#issuecomment-2790397011 > Please do not break the compatibility when you change the method signature, I think this is the most important thing to me. Maybe this is a stupid question, but how can

Re: [PR] [#6815] feat(core): Support update comment for model version [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6861: URL: https://github.com/apache/gravitino/pull/6861#discussion_r2036259176 ## catalogs/catalog-model/src/main/java/org/apache/gravitino/catalog/model/ModelCatalogOperations.java: ## @@ -327,6 +350,81 @@ private ModelEntity updateModelEntit

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
FANNG1 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036308005 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] Draft:A demo on how Gravitino implements metadata authorization. [gravitino]

2025-04-09 Thread via GitHub
unknowntpo commented on PR #6676: URL: https://github.com/apache/gravitino/pull/6676#issuecomment-2791299800 @hdygxsj @xunliu Thanks for this great demo !, is there any instructions / integration tests that can help us to play around on this demo ? -- This is an automated message from the

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036339381 ## server-common/src/main/java/org/apache/gravitino/server/web/Utils.java: ## @@ -48,6 +49,14 @@ public static Response ok(T t) { return Response.status(Resp

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036337967 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (A

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036337967 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (A

Re: [I] [Bug report] There may be a bug in GravitinoVersion.__gt__, a case is here. [gravitino]

2025-04-09 Thread via GitHub
xunliu commented on issue #5362: URL: https://github.com/apache/gravitino/issues/5362#issuecomment-2791312964 hi @unknowntpo Thank you, I assigned this issue to you. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] [Bug report] There may be a bug in GravitinoVersion.__gt__, a case is here. [gravitino]

2025-04-09 Thread via GitHub
unknowntpo commented on issue #5362: URL: https://github.com/apache/gravitino/issues/5362#issuecomment-2791310188 @jerryshao May I take this issue ? -- 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: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2035710280 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (A

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

2025-04-09 Thread via GitHub
yuqi1129 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2789766299 > @yuqi1129 can you please help on this PR? Got -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [I] [FEATURE] Support for column type e.g. array, array, array, etc. [gravitino]

2025-04-09 Thread via GitHub
mchades commented on issue #6875: URL: https://github.com/apache/gravitino/issues/6875#issuecomment-2791476899 It's already supported. see https://gravitino.apache.org/docs/0.8.0-incubating/manage-relational-metadata-using-gravitino#apache-gravitino-table-column-type -- This is an automat

Re: [I] [Bug report] use catalog and show databases failed with krb cluster [gravitino]

2025-04-09 Thread via GitHub
yuqi1129 closed issue #6777: [Bug report] use catalog and show databases failed with krb cluster URL: https://github.com/apache/gravitino/issues/6777 -- 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

[PR] [#6871] fix(server,authn): Fix use catalog and show databases failed with krb cluster for iceberg issue [gravitino]

2025-04-09 Thread via GitHub
granewang opened a new pull request, #6873: URL: https://github.com/apache/gravitino/pull/6873 ### What changes were proposed in this pull request? Fix use catalog and show databases failed with krb cluster for iceberg issue ### Why are the changes needed? Fix: #6871

Re: [PR] [#6815] feat(core): Support update comment for model version [gravitino]

2025-04-09 Thread via GitHub
Abyss-lord commented on PR #6861: URL: https://github.com/apache/gravitino/pull/6861#issuecomment-2791464152 Hi @jerryshao , I've completed the code updates and would appreciate your review of the PR when you have a moment. Here's a summary of the commits: 1. Fix typo problem. 2. Updat

Re: [I] [FEATURE] Support for column type e.g. array, array, array, etc. [gravitino]

2025-04-09 Thread via GitHub
jerqi commented on issue #6875: URL: https://github.com/apache/gravitino/issues/6875#issuecomment-2791467107 @mchades Could you answer this question? -- 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] [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation [gravitino]

2025-04-09 Thread via GitHub
zhengkezhou1 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2791443314 > @zhengkezhou1 All tests have passed. do you have any additional updates? No, there are other types of entities that also need to be modified, so I prefer to use multiple sm

Re: [I] [FEATURE] Support for column type e.g. array, array, array, etc. [gravitino]

2025-04-09 Thread via GitHub
rickyma commented on issue #6875: URL: https://github.com/apache/gravitino/issues/6875#issuecomment-2791466861 Do we support this currently? -- 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 sp

svn commit: r76174 - /dev/incubator/gravitino/v0.8.1-incubating-rc3/

2025-04-09 Thread jmclean
Author: jmclean Date: Thu Apr 10 06:03:21 2025 New Revision: 76174 Log: Adding RC3 Added: dev/incubator/gravitino/v0.8.1-incubating-rc3/ dev/incubator/gravitino/v0.8.1-incubating-rc3/gravitino-0.8.1-incubating-bin.tar.gz (with props) dev/incubator/gravitino/v0.8.1-incubating-rc3/

Re: [I] [Bug report] Hyphen (-) should be supported in model names [gravitino]

2025-04-09 Thread via GitHub
jerqi commented on issue #6544: URL: https://github.com/apache/gravitino/issues/6544#issuecomment-2791676115 Long time no actions. @mchades Maybe you can help this issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [I] [Bug report] Iceberg tables are not visible by using spark connector [gravitino]

2025-04-09 Thread via GitHub
granewang commented on issue #6877: URL: https://github.com/apache/gravitino/issues/6877#issuecomment-2791723279 > stack After investigation, I thinkthis is not a bug . When creating the catalog, you need to explicitly set the property "list-all-tables": true to see Iceberg tables. H

Re: [I] [EPIC] Python Task Summarization [gravitino]

2025-04-09 Thread via GitHub
Abyss-lord commented on issue #6879: URL: https://github.com/apache/gravitino/issues/6879#issuecomment-2791747529 @xunliu -- 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] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036336236 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (A

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
jerryshao commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036337967 ## lineage/src/main/java/org/apache/gravitino/lineage/source/rest/LineageOperations.java: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (A

[PR] [Minor] Add support maxtix table [gravitino]

2025-04-09 Thread via GitHub
justinmclean opened a new pull request, #6872: URL: https://github.com/apache/gravitino/pull/6872 ### What changes were proposed in this pull request? Add support maxtix table to README ### Why are the changes needed? So people know what we support. Fix: # N/A

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

2025-04-09 Thread via GitHub
yuqi1129 commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2791405349 @zhengkezhou1 All tests have passed. do you have any additional updates? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] build(deps): bump com.github.jk1.dependency-license-report from 2.5 to 2.9 [gravitino]

2025-04-09 Thread via GitHub
jerryshao merged PR #6859: URL: https://github.com/apache/gravitino/pull/6859 -- 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] build(deps): bump iceberg from 1.6.1 to 1.8.1 [gravitino]

2025-04-09 Thread via GitHub
dependabot[bot] commented on PR #6858: URL: https://github.com/apache/gravitino/pull/6858#issuecomment-2784201267 OK, I won't notify you again about this release, but will get in touch when a new version is available. You can also ignore all major, minor, or patch releases for a dependency

Re: [I] [Improvement] Enhance Documentation for Setting Up Python Client Development Environment [gravitino]

2025-04-09 Thread via GitHub
unknowntpo commented on issue #4220: URL: https://github.com/apache/gravitino/issues/4220#issuecomment-2791365629 @xunliu May I work on this issue ? This will block new contributors to develop python client. -- This is an automated message from the Apache Git Service. To respond to the me

Re: [I] [Bug report] List catalog in a long run environment, encounter issue:"org.apache.ibatis.exceptions.PersistenceException: " [gravitino]

2025-04-09 Thread via GitHub
jerqi commented on issue #6874: URL: https://github.com/apache/gravitino/issues/6874#issuecomment-2791429560 @yuqi1129 @FANNG1 Could you take a 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] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
yuqi1129 closed pull request #6876: [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue URL: https://github.com/apache/gravitino/pull/6876 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
github-actions[bot] opened a new pull request, #6876: URL: https://github.com/apache/gravitino/pull/6876 ### What changes were proposed in this pull request? Fix use catalog and show databases failed with krb cluster issue ### Why are the changes needed? Fix: #6777

(gravitino) branch main updated (a9e93e256e -> ecf6534c54)

2025-04-09 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 a9e93e256e [#6704] feat(authz): Support model hook dispatcher (#6844) add ecf6534c54 [#6777] fix(server,authn)

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

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

svn commit: r76175 - in /dev/incubator/gravitino/v0.8.1-incubating-rc3: gravitino-0.8.1-incubating-src.tar.gz gravitino-0.8.1-incubating-src.tar.gz.asc gravitino-0.8.1-incubating-src.tar.gz.sha512

2025-04-09 Thread jmclean
Author: jmclean Date: Thu Apr 10 06:07:23 2025 New Revision: 76175 Log: Fixed version number in source release Modified: dev/incubator/gravitino/v0.8.1-incubating-rc3/gravitino-0.8.1-incubating-src.tar.gz dev/incubator/gravitino/v0.8.1-incubating-rc3/gravitino-0.8.1-incubating-src.tar.g

Re: [I] [Improvement] Close `FileSystem` instances explicitly to avoid JVM memory increase fastly [gravitino]

2025-04-09 Thread via GitHub
jerqi closed issue #6487: [Improvement] Close `FileSystem` instances explicitly to avoid JVM memory increase fastly URL: https://github.com/apache/gravitino/issues/6487 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [#6780] feat(lineage): support lineage endpoint for Gravitino server [gravitino]

2025-04-09 Thread via GitHub
FANNG1 commented on code in PR #6865: URL: https://github.com/apache/gravitino/pull/6865#discussion_r2036412467 ## server-common/src/main/java/org/apache/gravitino/server/web/Utils.java: ## @@ -48,6 +49,14 @@ public static Response ok(T t) { return Response.status(Respons

Re: [I] [Improvement] Can Gravitino replace HMS? [gravitino]

2025-04-09 Thread via GitHub
melin commented on issue #6867: URL: https://github.com/apache/gravitino/issues/6867#issuecomment-2789401745 cc @mchades -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
jerqi commented on PR #6778: URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2789638321 > > @jerqi Test code add ,could U please help to review, thanks > > PRINCIPAL "gravitino_client/instance@HADOOPKRB" has not created in the docker image `kerberos-hive`, so the tests

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
yuqi1129 commented on PR #6778: URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2789571228 > cc @yuqi1129 Do we need to apply similar fix for Iceberg or other catalogs? Yes, Iceberg has a similar problem. Could you please also raise a PR to solve it in `HiveBackendProx

Re: [I] [Improvement] Can Gravitino replace HMS? [gravitino]

2025-04-09 Thread via GitHub
hdygxsj commented on issue #6867: URL: https://github.com/apache/gravitino/issues/6867#issuecomment-2789824224 Perhaps we can first add a generic catalog in the relational catalog to replace HMS. This generic catalog will persist metadata to Gravitino's backend storage, such as MySQL or Pos

Re: [I] [Subtask] Support update comment operations for model alteration [gravitino]

2025-04-09 Thread via GitHub
TEOTEO520 commented on issue #6525: URL: https://github.com/apache/gravitino/issues/6525#issuecomment-2785155650 > We’re aiming to release version 0 Thank you, I'll complete it by the 17th. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

Re: [PR] [Docs] Rework docs [gravitino]

2025-04-09 Thread via GitHub
tengqm commented on PR #6849: URL: https://github.com/apache/gravitino/pull/6849#issuecomment-2788589993 > Some problems mentioned above were in the original file, but more have been introduced. This is a ridiculous accusation. You are saying this as if you read the original docs car

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
jerqi commented on PR #6778: URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2789093154 @yuqi1129 Could you give some guidance about the test? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-09 Thread via GitHub
Abyss-lord opened a new pull request, #6866: URL: https://github.com/apache/gravitino/pull/6866 ### What changes were proposed in this pull request? Support role failure event to Gravitino server 1. Support role failure event to Gravitino server 2. Refactor the `EventBus#dispatch

Re: [PR] [#6760] test(core): Support role failure event to Gravitino server [gravitino]

2025-04-09 Thread via GitHub
Abyss-lord commented on PR #6866: URL: https://github.com/apache/gravitino/pull/6866#issuecomment-2789206584 Hi @FANNG1 , 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 respond to the

Re: [PR] [#6777] fix(server,authn): Fix use catalog and show databases failed with krb cluster issue [gravitino]

2025-04-09 Thread via GitHub
granewang commented on PR #6778: URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2789264213 @jerqi Test code add ,could U please help to review, thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th