Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

xunliu commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2689509176 @tsungchih Thank you for your contutions. :-) hi @tengqm > The reason I asked this question is that I am (still) wondering whether we can write the code in a more Python way.

Re: [PR] [#6459][Followup] fix(test): add default sleep for DataItemInList [gravitino]

yuqi1129 closed pull request #6567: [#6459][Followup] fix(test): add default sleep for DataItemInList URL: https://github.com/apache/gravitino/pull/6567 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

(gravitino) branch main updated (dbb3b961d5 -> 359198b93a)

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 dbb3b961d5 build(deps): bump actions/checkout from 2 to 4 (#6549) add 359198b93a [Minor] Update unknowntpo's o

Re: [PR] [Minor] Update unknowntpo's organization due to career change in MAINTAINERS.md [gravitino]

unknowntpo commented on PR #6572: URL: https://github.com/apache/gravitino/pull/6572#issuecomment-2689590778 Thanks @yuqi1129 -- 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] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974879819 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -201,68 +213,73 @@ public Schema loadSchema(NameIdentifier ident) throws No

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

jerryshao commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1974896684 ## core/src/main/java/org/apache/gravitino/catalog/CatalogManager.java: ## @@ -260,7 +260,7 @@ private ModelCatalog asModels() { private final Config config;

Re: [PR] [#4620] improvement(authz): Throw the necessary exception when handling Ranger plugin exception [gravitino]

jerqi commented on code in PR #6515: URL: https://github.com/apache/gravitino/pull/6515#discussion_r1974898365 ## authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerHiveIT.java: ## @@ -138,6 +138,9 @@ public void te

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974899904 ## core/src/main/java/org/apache/gravitino/authorization/AccessControlManager.java: ## @@ -57,78 +61,136 @@ public AccessControlManager(EntityStore store, IdGenerato

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

jerryshao commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974886527 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -201,68 +213,73 @@ public Schema loadSchema(NameIdentifier ident) throws N

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

jerryshao commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1974902287 ## core/src/main/java/org/apache/gravitino/catalog/CatalogManager.java: ## @@ -791,12 +791,17 @@ private static boolean catalogInUse(EntityStore store, NameIdentif

Re: [PR] build(deps): bump qs and @types/qs in /web/web [gravitino]

jerryshao merged PR #6556: URL: https://github.com/apache/gravitino/pull/6556 -- 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] [#6566] improvement(core): Add the cache mechanism for matalake and use cache to load `in-use` information. [gravitino]

yuqi1129 commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1974902705 ## core/src/main/java/org/apache/gravitino/catalog/CatalogManager.java: ## @@ -260,7 +260,7 @@ private ModelCatalog asModels() { private final Config config;

Re: [PR] build(deps-dev): bump postcss from 8.4.39 to 8.5.3 in /web/web [gravitino]

jerryshao merged PR #6555: URL: https://github.com/apache/gravitino/pull/6555 -- 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] [#6566] improvement(core): Add the cache mechanism for matalake and use cache to load `in-use` information. [gravitino]

jerryshao commented on PR #6569: URL: https://github.com/apache/gravitino/pull/6569#issuecomment-2689885514 What is the performance gaining after cache? -- 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

(gravitino) branch main updated (589ef41231 -> 2980674103)

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 589ef41231 build(deps): bump qs and @types/qs in /web/web (#6556) add 2980674103 build(deps-dev): bump postcss fr

(gravitino) branch dependabot/npm_and_yarn/web/web/postcss-8.5.3 deleted (was 3d77749f80)

This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch dependabot/npm_and_yarn/web/web/postcss-8.5.3 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 3d77749f80 build(deps-dev): bump postcss from 8.4.39 to 8.5.3 in /web/web T

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

yuqi1129 commented on PR #6569: URL: https://github.com/apache/gravitino/pull/6569#issuecomment-2689891888 > What is the performance gaining after cache? The picture attached to this PR is of the performance with the cache, about the picture without cache, please see the corresponding

(gravitino) branch main updated (359198b93a -> 2af41cade9)

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 359198b93a [Minor] Update unknowntpo's organization due to career change in MAINTAINERS.md (#6572) add 2af41cade

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

yuqi1129 commented on code in PR #6540: URL: https://github.com/apache/gravitino/pull/6540#discussion_r1974628247 ## core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/base/CatalogMetaBaseSQLProvider.java: ## @@ -22,10 +22,57 @@ import static org.apache

Re: [PR] [#6468] feature(core): add metrics for mysql backend connect pool [gravitino]

jerryshao merged PR #6470: URL: https://github.com/apache/gravitino/pull/6470 -- 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] Add Connection Pool Metrics for JDBC Backend Storage [gravitino]

jerryshao closed issue #6468: [Improvement] Add Connection Pool Metrics for JDBC Backend Storage URL: https://github.com/apache/gravitino/issues/6468 -- 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] [#6523] fix(core): Filter null prop when init properties [gravitino]

jerryshao commented on code in PR #6524: URL: https://github.com/apache/gravitino/pull/6524#discussion_r1974635303 ## core/src/main/java/org/apache/gravitino/catalog/EntityCombinedFileset.java: ## @@ -77,6 +77,7 @@ public String storageLocation() { public Map properties() {

(gravitino) branch dependabot/npm_and_yarn/web/web/mui/material-5.16.14 updated (0c446a1269 -> 6aadae9efa)

This is an automated email from the ASF dual-hosted git repository. lauraxia pushed a change to branch dependabot/npm_and_yarn/web/web/mui/material-5.16.14 in repository https://gitbox.apache.org/repos/asf/gravitino.git from 0c446a1269 build(deps): bump @mui/material from 5.15.21 to 5.16.14

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

Brijeshthummar02 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2689820574 @justinmclean it was taking too long for me as there where multiple files but finally made my way, now it's up to you, you may go through the changes and feel free to mention a

Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

tsungchih commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2689620443 > @tsungchih Thank you for your contributions. :-) > > hi @tengqm > > > The reason I asked this question is that I am (still) wondering whether we can write the code in a

Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

tengqm commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2689784321 > > @tsungchih Thank you for your contributions. :-) > > hi @tengqm > > > The reason I asked this question is that I am (still) wondering whether we can write the code in a more Pyt

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974919791 ## core/src/main/java/org/apache/gravitino/authorization/AccessControlManager.java: ## @@ -57,78 +61,136 @@ public AccessControlManager(EntityStore store, IdGenerato

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974919255 ## core/src/main/java/org/apache/gravitino/authorization/AccessControlManager.java: ## @@ -57,78 +61,136 @@ public AccessControlManager(EntityStore store, IdGenerato

Re: [PR] [#6459][Followup] fix(test): add default sleep for DataItemInList [gravitino]

jerryshao commented on PR #6567: URL: https://github.com/apache/gravitino/pull/6567#issuecomment-2689528012 Let's run for like 10 times to see if this issue still happens. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] [Minor] Update unknowntpo's organization due to career change in MAINTAINERS.md [gravitino]

yuqi1129 merged PR #6572: URL: https://github.com/apache/gravitino/pull/6572 -- 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] [#6459][Followup] fix(test): add default sleep for DataItemInList [gravitino]

yuqi1129 closed pull request #6567: [#6459][Followup] fix(test): add default sleep for DataItemInList URL: https://github.com/apache/gravitino/pull/6567 -- 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]

yuqi1129 commented on code in PR #6540: URL: https://github.com/apache/gravitino/pull/6540#discussion_r1974628247 ## core/src/main/java/org/apache/gravitino/storage/relational/mapper/provider/base/CatalogMetaBaseSQLProvider.java: ## @@ -22,10 +22,57 @@ import static org.apache

Re: [PR] [#6459][Followup] fix(test): add default sleep for DataItemInList [gravitino]

yuqi1129 closed pull request #6567: [#6459][Followup] fix(test): add default sleep for DataItemInList URL: https://github.com/apache/gravitino/pull/6567 -- 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] refactor(cli): Improve CLI commands with better error handling and output formatting [gravitino]

yuqi1129 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2689834772 > @yuqi1129 need your review 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 abo

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

Brijeshthummar02 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2689836204 @yuqi1129 That was too quick, Thanks!! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

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

Brijeshthummar02 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2689832252 @yuqi1129 need your review on this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974879819 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -201,68 +213,73 @@ public Schema loadSchema(NameIdentifier ident) throws No

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974879819 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -201,68 +213,73 @@ public Schema loadSchema(NameIdentifier ident) throws No

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974879819 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -201,68 +213,73 @@ public Schema loadSchema(NameIdentifier ident) throws No

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974879819 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -201,68 +213,73 @@ public Schema loadSchema(NameIdentifier ident) throws No

Re: [PR] [#6459][Followup] fix(test): add default sleep for DataItemInList [gravitino]

LauraXia123 closed pull request #6567: [#6459][Followup] fix(test): add default sleep for DataItemInList URL: https://github.com/apache/gravitino/pull/6567 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] [#4620] improvement(authz): Throw the necessary exception when handling Ranger plugin exception [gravitino]

xunliu commented on code in PR #6515: URL: https://github.com/apache/gravitino/pull/6515#discussion_r1974875914 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -576,8 +576,9 @@ public Boolean on

Re: [PR] [#4620] improvement(authz): Throw the necessary exception when handling Ranger plugin exception [gravitino]

xunliu commented on code in PR #6515: URL: https://github.com/apache/gravitino/pull/6515#discussion_r1974888545 ## authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerHiveIT.java: ## @@ -266,6 +269,9 @@ public void t

(gravitino) branch dependabot/npm_and_yarn/web/web/multi-69aca920c7 deleted (was b51c29e675)

This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/web/web/multi-69aca920c7 in repository https://gitbox.apache.org/repos/asf/gravitino.git was b51c29e675 build(deps): bump qs and @types/qs in /web/web The revis

(gravitino) branch dependabot/npm_and_yarn/web/web/types/node-22.13.5 updated (8d9fff8b17 -> 79467d38c0)

This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/web/web/types/node-22.13.5 in repository https://gitbox.apache.org/repos/asf/gravitino.git discard 8d9fff8b17 build(deps-dev): bump @types/node from 20.14.9 to 22.13

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

mchades commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1974876857 ## core/src/main/java/org/apache/gravitino/metalake/MetalakeManager.java: ## @@ -60,6 +67,21 @@ public class MetalakeManager implements MetalakeDispatcher { priv

(gravitino) branch main updated (2af41cade9 -> 589ef41231)

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 2af41cade9 [#6468] feature(core): add metrics for mysql backend connect pool (#6470) add 589ef41231 build(deps):

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

Brijeshthummar02 commented on PR #6573: URL: https://github.com/apache/gravitino/pull/6573#issuecomment-2689871816 @yuqi1129 need your review on this PR again -- 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

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974949521 ## core/src/main/java/org/apache/gravitino/catalog/TableOperationDispatcher.java: ## @@ -194,70 +198,75 @@ public Table createTable( public Table alterTable(NameI

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

jerqi commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974930461 ## core/src/main/java/org/apache/gravitino/authorization/AccessControlManager.java: ## @@ -57,78 +61,136 @@ public AccessControlManager(EntityStore store, IdGenerator

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on code in PR #2873: URL: https://github.com/apache/gravitino/pull/2873#discussion_r1974949325 ## core/src/main/java/org/apache/gravitino/catalog/SchemaOperationDispatcher.java: ## @@ -201,68 +213,73 @@ public Schema loadSchema(NameIdentifier ident) throws No

Re: [PR] [#2780] Improvement(server,core): Move tree lock from rest api to the corresponding implementation to minimize tree lock range. [gravitino]

yuqi1129 commented on PR #2873: URL: https://github.com/apache/gravitino/pull/2873#issuecomment-2689966315 > I think we can split into two locks, it should be fine. Changed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

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

yuqi1129 commented on code in PR #6569: URL: https://github.com/apache/gravitino/pull/6569#discussion_r1974963141 ## core/src/main/java/org/apache/gravitino/metalake/MetalakeManager.java: ## @@ -60,6 +67,21 @@ public class MetalakeManager implements MetalakeDispatcher { pri

(gravitino) branch dependabot/npm_and_yarn/web/web/antd-5.24.2 updated (3f7175a8e0 -> a75142184e)

This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch dependabot/npm_and_yarn/web/web/antd-5.24.2 in repository https://gitbox.apache.org/repos/asf/gravitino.git from 3f7175a8e0 build(deps): bump antd from 5.19.0 to 5.24.2 in /web/web add 6d

Re: [I] [Improvement] resolve todo to get other securable object fullNames using batch retrieving in function listSecurableObjects. [gravitino]

xunliu commented on issue #6474: URL: https://github.com/apache/gravitino/issues/6474#issuecomment-2689386542 hi @unknowntpo. I assign to you. Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

tengqm commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2689392452 Yes. There are two extremes here. Either you are auto-generating the Python code from Java, you only care about the correctness of the generator, you don't care if the code is complicate

Re: [I] [Improvement] resolve todo to get other securable object fullNames using batch retrieving in function listSecurableObjects. [gravitino]

unknowntpo commented on issue #6474: URL: https://github.com/apache/gravitino/issues/6474#issuecomment-2689316775 @jerqi 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 to th

[PR] [Minor] Update unknowntpo's organization due to career change in MAINTAINERS.md [gravitino]

unknowntpo opened a new pull request, #6572: URL: https://github.com/apache/gravitino/pull/6572 Update unknowntpo's organization due to career change in MAINTAINERS.md. ### What changes were proposed in this pull request? Update unknowntpo's organization due to career ch

Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

tsungchih commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2689471930 > Yes. There are two extremes here. Either you are auto-generating the Python code from Java, you only care about the correctness of the generator, you don't care if the code is compl

Re: [PR] [#6459][Followup] fix(test): add default sleep for DataItemInList [gravitino]

yuqi1129 closed pull request #6567: [#6459][Followup] fix(test): add default sleep for DataItemInList URL: https://github.com/apache/gravitino/pull/6567 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

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

maythorn300 commented on issue #6544: URL: https://github.com/apache/gravitino/issues/6544#issuecomment-2689464404 收到，我来试试 -- 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

(gravitino) branch main updated: Revert "build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 (#6550)" (#6564)

This is an automated email from the ASF dual-hosted git repository. yuqi4733 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 ce26de8d3c Revert "build(deps): bump carloscas

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

Abyss-lord commented on issue #6566: URL: https://github.com/apache/gravitino/issues/6566#issuecomment-2687381077 I would like to work on it. -- 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] Revert "build(deps): bump carloscastrojumo/github-cherry-pick-action from 1.0.9 to 1.0.10 (#6550)" [gravitino]

yuqi1129 merged PR #6564: URL: https://github.com/apache/gravitino/pull/6564 -- 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] [MINOR] Move to using standard DISCLAIMER [gravitino]

jerryshao commented on PR #6565: URL: https://github.com/apache/gravitino/pull/6565#issuecomment-2687373401 @justinmclean can you manually cherry pick this commit to branch-0.8? The auto cherry-pick action is failed. -- This is an automated message from the Apache Git Service. To respond

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

hdygxsj commented on code in PR #6543: URL: https://github.com/apache/gravitino/pull/6543#discussion_r1973735725 ## 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]

hdygxsj commented on PR #6543: URL: https://github.com/apache/gravitino/pull/6543#issuecomment-2688198977 > Thanks for the PR, could you add a related document? And besides Mysql, does Flink JDBC connector support other RDMS? The FlinkJdbcCatalog supports only MySQL and PostgreSQL dat

Re: [PR] [#6459][Followup] fix(test): add default sleep for DataItemInList [gravitino]

yuqi1129 closed pull request #6567: [#6459][Followup] fix(test): add default sleep for DataItemInList URL: https://github.com/apache/gravitino/pull/6567 -- 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

[I] [Subtask] Improvment: Optimize method `getParentEntityIdByNamespace` to reduce the number of accesses to the database [gravitino]

yuqi1129 opened a new issue, #6570: URL: https://github.com/apache/gravitino/issues/6570 ### Describe the subtask Method `getParentEntityIdByNamespace` wil access database 3 times at most if the name space is a schema, in fact, we can use JOIN to reduce the access times, or it's rath

Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

tengqm commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2688124903 > @tengqm The Python code is manually written according to the corresponding Java code. Only the unit tests were auto-generated and revised by myself. Are there concerns with the auto-ge

Re: [PR] [#4620] improvement(authz): Throw the necessary exception when handling Ranger plugin exception [gravitino]

tengqm commented on code in PR #6515: URL: https://github.com/apache/gravitino/pull/6515#discussion_r1973698302 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -273,8 +273,13 @@ public Boolean o

Re: [I] [FEATURE] Storage system needs a id-name mapping cache [gravitino]

yuqi1129 commented on issue #3854: URL: https://github.com/apache/gravitino/issues/3854#issuecomment-2688127999 This should be the biggest improvement for #6560, however, we need to consider the following things - Distributed cache - Maintaining consistency between the cache and datab

Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

tsungchih commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2688376438 Here I got another question when I was implementing the xxxDTO classes. Most of them have a factory method `of` or builder class inside. I was wondering if I should implement those bu

Re: [PR] [#5202] feat(client-python): Support Column and its default value part1 [gravitino]

tsungchih commented on PR #6542: URL: https://github.com/apache/gravitino/pull/6542#issuecomment-2688345631 > > @tengqm The Python code is manually written according to the corresponding Java code. Only the unit tests were auto-generated and revised by myself. Are there concerns with the au

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

hdygxsj closed pull request #6543: [#6233] feat(flink): flink jdbc catalog URL: https://github.com/apache/gravitino/pull/6543 -- 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

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

Brijeshthummar02 opened a new pull request, #6571: URL: https://github.com/apache/gravitino/pull/6571 **Title:** ``` [#6529 ] fix: Remove incorrect @VisibleForTesting annotation & apply Spotless formatting ``` * * * * * ### **What changes were proposed in this

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

Brijeshthummar02 commented on PR #6571: URL: https://github.com/apache/gravitino/pull/6571#issuecomment-2688558176 @justinmclean much clean now?? https://github.com/apache/gravitino/pull/6532#issuecomment-2683928569 -- This is an automated message from the Apache Git Service. To re

[PR] hide update info for doc page [gravitino-site]

LauraXia123 opened a new pull request, #46: URL: https://github.com/apache/gravitino-site/pull/46 https://github.com/user-attachments/assets/dfc414ac-ea0d-4d2c-b460-deafa1053b28"; /> -- This is an automated message from the Apache Git Service. To respond to the message, please log on

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

yuqi1129 opened a new issue, #6566: URL: https://github.com/apache/gravitino/issues/6566 ### What would you like to be improved? `catalogInUse` and `metalakeInUse` is called frequently and very time-consuming as it directly load data from backend storage, which can be optimized.

Re: [PR] hide update info for doc page [gravitino-site]

jerryshao merged PR #46: URL: https://github.com/apache/gravitino-site/pull/46 -- 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

(gravitino-site) branch main updated: hide update info for doc page (#46)

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 632df67e hide update info for doc page (#46)

(gravitino) branch main updated: build(deps): bump dorny/paths-filter from 2 to 3 (#6552)

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 acf57adbfb build(deps): bump dorny/paths-filter fr

(gravitino) branch dependabot/github_actions/dorny/paths-filter-3 deleted (was 4e0d621547)

This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/dorny/paths-filter-3 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 4e0d621547 build(deps): bump dorny/paths-filter from 2 to 3 The revis

Re: [PR] build(deps): bump dorny/paths-filter from 2 to 3 [gravitino]

jerryshao merged PR #6552: URL: https://github.com/apache/gravitino/pull/6552 -- 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 dependabot/github_actions/actions/checkout-4 updated (166c1913ad -> c128e86d9a)

This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/actions/checkout-4 in repository https://gitbox.apache.org/repos/asf/gravitino.git omit 166c1913ad build(deps): bump actions/checkout from 2 to 4 add acf57

(gravitino) 01/01: build(deps): bump actions/checkout from 2 to 4

This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch dependabot/github_actions/actions/checkout-4 in repository https://gitbox.apache.org/repos/asf/gravitino.git commit c128e86d9ad7016624d60ede96fee7dcef0ac512 Author: dependabot[bot] <49699333+

(gravitino) branch main updated: [MINOR] Move to using standard DISCLAIMER (#6565)

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 f5f1222328 [MINOR] Move to using standard DISCLAIM

Re: [PR] [MINOR] Move to using standard DISCLAIMER [gravitino]

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

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

maythorn300 commented on issue #6544: URL: https://github.com/apache/gravitino/issues/6544#issuecomment-2687568852 你好，我看到在 web/web/src/lib/utils/regex.js 中这里对name的输入做了正则限制，这里做修改应该可以解决，请问这个可以分配给我作为我的第一个任务吗？ -- This is an automated message from the Apache Git Service. To respond to the mess

[PR] fix(test): add default sleep for DataItemInList [gravitino]

LauraXia123 opened a new pull request, #6567: URL: https://github.com/apache/gravitino/pull/6567 ### What changes were proposed in this pull request? Fix ci issue by default sleep ### Why are the changes needed? Need a sleep time for render latest data Only verifyShowD

Re: [I] [Bug report] Integration Test fails when run Locally [gravitino]

jerryshao closed issue #6508: [Bug report] Integration Test fails when run Locally URL: https://github.com/apache/gravitino/issues/6508 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific c

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

yuqi1129 opened a new pull request, #6569: URL: https://github.com/apache/gravitino/pull/6569 ### What changes were proposed in this pull request? 1. Add cache mechanism for MatalakeManage. 2. Use cache to load `in-use` information for metalake and catalog. ### Why are the ch

Re: [I] [Bug report] Integration Test fails when run Locally [gravitino]

yuqi1129 commented on issue #6508: URL: https://github.com/apache/gravitino/issues/6508#issuecomment-2687774831 > I think you need to run `./gradlew compileDistribution -x test` first for python client test. So, the Python-related tests require a distribution package in advance? --

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

yuqi1129 commented on issue #6566: URL: https://github.com/apache/gravitino/issues/6566#issuecomment-2687780102 > I would like to work on it. Sorry, I'm already working on it. If you are interested in this point, you can take part in https://github.com/apache/gravitino/issues/6560

Re: [PR] build(deps): bump docker/setup-qemu-action from 2 to 3 [gravitino]

jerryshao merged PR #6546: URL: https://github.com/apache/gravitino/pull/6546 -- 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: build(deps): bump docker/setup-qemu-action from 2 to 3 (#6546)

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 8b0d405a33 build(deps): bump docker/setup-qemu-act

Re: [I] [EPIC] Support another KV storage except RocksDB [gravitino]

zzzk1 commented on issue #2290: URL: https://github.com/apache/gravitino/issues/2290#issuecomment-2687710835 @yuqi1129 In https://github.com/apache/gravitino/issues/5633 we currently do not require K-V storage. I would like to know if this ticket is still open now? -- This is an automated

Re: [I] [EPIC] Support another KV storage except RocksDB [gravitino]

yuqi1129 closed issue #2290: [EPIC] Support another KV storage except RocksDB URL: https://github.com/apache/gravitino/issues/2290 -- 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: [I] [Bug report] SparkJdbcMysqlCatalogIT33 failed [gravitino]

yuqi1129 closed issue #6392: [Bug report] SparkJdbcMysqlCatalogIT33 failed URL: https://github.com/apache/gravitino/issues/6392 -- 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.

(gravitino) branch main updated: [#6392] fix(test): SparkJdbcMysqlCatalogIT33 failed in some enviroment (#6432)

This is an automated email from the ASF dual-hosted git repository. yuqi4733 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 b8a2349ba6 [#6392] fix(test): SparkJdbcMysqlCat

  1   2   >