Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011527552 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -320,24 +320,41 @@ Currently, Gravitino supports two **types** of filesets: The `storageLocation` is the p

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
mchades commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011950471 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -320,24 +320,41 @@ Currently, Gravitino supports two **types** of filesets: The `storageLocation` is the ph

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
mchades commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011528499 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -320,24 +320,41 @@ Currently, Gravitino supports two **types** of filesets: The `storageLocation` is the ph

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
mchades commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011527252 ## docs/hadoop-catalog.md: ## @@ -129,6 +129,15 @@ Refer to [Schema operation](./manage-fileset-metadata-using-gravitino.md#schema- | `authentication.kerberos.princ

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
mchades commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011510896 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -320,24 +320,41 @@ Currently, Gravitino supports two **types** of filesets: The `storageLocation` is the ph

Re: [PR] [#6133] Improvement(core): Supports get Fileset schema location in the AuthorizationUtils [gravitino]

2025-03-25 Thread via GitHub
jerqi commented on PR #6211: URL: https://github.com/apache/gravitino/pull/6211#issuecomment-2750684654 @Abyss-lord Would you like to continue 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 g

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
mchades commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011493149 ## docs/hadoop-catalog.md: ## @@ -129,6 +129,15 @@ Refer to [Schema operation](./manage-fileset-metadata-using-gravitino.md#schema- | `authentication.kerberos.princ

[PR] [#6712] feat(core): Support user event to Gravitino server [gravitino]

2025-03-25 Thread via GitHub
Abyss-lord opened a new pull request, #6746: URL: https://github.com/apache/gravitino/pull/6746 ### What changes were proposed in this pull request? Support user event to Gravitino server ![image](https://github.com/user-attachments/assets/1ad67c05-ebed-441b-a02f-d4257195c8cb)

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011516123 ## docs/hadoop-catalog.md: ## @@ -129,6 +129,15 @@ Refer to [Schema operation](./manage-fileset-metadata-using-gravitino.md#schema- | `authentication.kerberos.prin

[PR] [#6747] improvement(web): can be dropped while set not in-use false [gravitino]

2025-03-25 Thread via GitHub
LauraXia123 opened a new pull request, #6748: URL: https://github.com/apache/gravitino/pull/6748 ### What changes were proposed in this pull request? can be droped while set not in-use false https://github.com/user-attachments/assets/77c792db-9546-48d0-8048-1938b9ec3f48"; />

(gravitino) branch dependabot/gradle/testcontainers-1.20.6 updated (1c84a29db4 -> f97250912b)

2025-03-25 Thread jshao
This is an automated email from the ASF dual-hosted git repository. jshao pushed a change to branch dependabot/gradle/testcontainers-1.20.6 in repository https://gitbox.apache.org/repos/asf/gravitino.git from 1c84a29db4 Merge branch 'main' into dependabot/gradle/testcontainers-1.20.6 a

Re: [PR] [#5743] refactor: use gradle plugin to build rust sub-project [gravitino]

2025-03-25 Thread via GitHub
diqiu50 commented on code in PR #6726: URL: https://github.com/apache/gravitino/pull/6726#discussion_r2013272201 ## clients/filesystem-fuse/build.gradle.kts: ## @@ -26,58 +37,14 @@ val checkRustEnvironment by tasks.registering(Exec::class) { isIgnoreExitValue = false } Re

(gravitino) branch dependabot/npm_and_yarn/web/web/antd-5.24.5 updated (4d8dcba842 -> bcd44a8968)

2025-03-25 Thread jshao
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.5 in repository https://gitbox.apache.org/repos/asf/gravitino.git from 4d8dcba842 build(deps): bump antd from 5.24.3 to 5.24.5 in /web/web add ed

Re: [PR] [#5743] refactor: use gradle plugin to build rust sub-project [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 commented on PR #6726: URL: https://github.com/apache/gravitino/pull/6726#issuecomment-2752956584 @diqiu50 Please take some time on 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 abov

(gravitino) branch main updated (e4a049e17a -> 41580cc006)

2025-03-25 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 e4a049e17a [#6740] feat(core): Support group pre-event to Gravitino server (#6743) add 41580cc006 build(deps): b

Re: [PR] build(deps): bump testcontainers from 1.19.0 to 1.20.6 [gravitino]

2025-03-25 Thread via GitHub
jerryshao merged PR #6731: URL: https://github.com/apache/gravitino/pull/6731 -- 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: [#6680] feat(fileset): support fileset location placeholder (#6724)

2025-03-25 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 cff80824d2 [#6680] feat(fileset): support fileset

Re: [I] [Bug report] NullPointerException after alter operation [gravitino]

2025-03-25 Thread via GitHub
jerryshao closed issue #6523: [Bug report] NullPointerException after alter operation URL: https://github.com/apache/gravitino/issues/6523 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

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

2025-03-25 Thread via GitHub
jerryshao closed issue #6680: [FEATURE] support location placeholder when creating fileset URL: https://github.com/apache/gravitino/issues/6680 -- 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

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
jerryshao merged PR #6724: URL: https://github.com/apache/gravitino/pull/6724 -- 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: [#6523] fix(core): Filter null prop when init properties (#6524)

2025-03-25 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 44d55effba [#6523] fix(core): Filter null prop whe

Re: [PR] [#6523] fix(core): Filter null prop when init properties [gravitino]

2025-03-25 Thread via GitHub
jerryshao merged PR #6524: URL: https://github.com/apache/gravitino/pull/6524 -- 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] [FEATURE] support location placeholder when creating fileset [gravitino]

2025-03-25 Thread via GitHub
jerryshao closed issue #6680: [FEATURE] support location placeholder when creating fileset URL: https://github.com/apache/gravitino/issues/6680 -- 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

Re: [I] [Bug report] Spark create table failed for PG catalog [gravitino]

2025-03-25 Thread via GitHub
FANNG1 commented on issue #6697: URL: https://github.com/apache/gravitino/issues/6697#issuecomment-2751313776 > All right, I have a better understanding of the issue's context. May I take this on and work on the fix? sure, please go ahead -- This is an automated message from the Ap

[PR] [#6744] improvment(core): Batch get metadata objects for TagMetaService [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 opened a new pull request, #6752: URL: https://github.com/apache/gravitino/pull/6752 ### What changes were proposed in this pull request? Group by metadata object by metadata type and batch get all metadata with the same type. ### Why are the changes needed? To

Re: [PR] [#6694] improvement: Avoid binding type `PATH` to `FILESET` [gravitino]

2025-03-25 Thread via GitHub
jerqi commented on code in PR #6698: URL: https://github.com/apache/gravitino/pull/6698#discussion_r2013276162 ## authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java: ## @@ -27,20 +27,48 @@ public class PathBa

Re: [PR] [#6720] fix(core): fix entity schema NullPointException on null properties [gravitino]

2025-03-25 Thread via GitHub
jerqi closed pull request #6728: [#6720] fix(core): fix entity schema NullPointException on null properties URL: https://github.com/apache/gravitino/pull/6728 -- 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

[PR] Fix the No key to store error when Running the Spark connector with a Kerberos ticket cache [gravitino]

2025-03-25 Thread via GitHub
granewang opened a new pull request, #6753: URL: https://github.com/apache/gravitino/pull/6753 ### What changes were proposed in this pull request? Fix the No key to store error when Running the Spark connector with a Kerberos ticket cache ### Why are the changes needed?

Re: [I] Why does the createGravitinoClient method in GravitinoDriverPlugin.java check for Kerberos configuration when authType is simple? [gravitino]

2025-03-25 Thread via GitHub
241600489 closed issue #6725: Why does the createGravitinoClient method in GravitinoDriverPlugin.java check for Kerberos configuration when authType is simple? URL: https://github.com/apache/gravitino/issues/6725 -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [#6720] fix(core): fix entity schema NullPointException on null properties [gravitino]

2025-03-25 Thread via GitHub
jerqi commented on PR #6728: URL: https://github.com/apache/gravitino/pull/6728#issuecomment-2753142812 closed. Because this is duplicated. -- 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 spe

[PR] cherrypick license and notice changes into 0.8 branch [gravitino]

2025-03-25 Thread via GitHub
justinmclean opened a new pull request, #6754: URL: https://github.com/apache/gravitino/pull/6754 ### What changes were proposed in this pull request? Update license and notice files. ### Why are the changes needed? To comply with ASF policy. Fix: #(issue) #

Re: [PR] cherrypick license and notice changes into 0.8 branch [gravitino]

2025-03-25 Thread via GitHub
justinmclean commented on PR #6754: URL: https://github.com/apache/gravitino/pull/6754#issuecomment-2753391522 Please ignore this PR some went wrong. -- 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] [#6562] improvement(catalogs): Optimize the code path in createFileset and optimize path [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 commented on PR #6689: URL: https://github.com/apache/gravitino/pull/6689#issuecomment-2750625020 > @yuqi1129 @jerryshao Should I continue with this pr, or should we pause to discuss? OK, please revert the code about `thread-pool` and only change the poll interval of `awaiti

Re: [PR] cherrypick license and notice changes into 0.8 branch [gravitino]

2025-03-25 Thread via GitHub
justinmclean closed pull request #6754: cherrypick license and notice changes into 0.8 branch URL: https://github.com/apache/gravitino/pull/6754 -- 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] Fix the No key to store error when Running the Spark connector with a Kerberos ticket cache [gravitino]

2025-03-25 Thread via GitHub
granewang commented on PR #6753: URL: https://github.com/apache/gravitino/pull/6753#issuecomment-2753047418 Hi Sir, could you please review this PR when you have time?Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] [#6712] feat(core): Support user event to Gravitino server [gravitino]

2025-03-25 Thread via GitHub
Abyss-lord commented on PR #6746: URL: https://github.com/apache/gravitino/pull/6746#issuecomment-2753063336 @FANNG1 I have a question whether assigning a role to a user should count as a role-related event or a user-related event? -- This is an automated message from the Apache Git Servi

Re: [PR] [#6694] improvement: Avoid binding type `PATH` to `FILESET` [gravitino]

2025-03-25 Thread via GitHub
xunliu commented on PR #6698: URL: https://github.com/apache/gravitino/pull/6698#issuecomment-2753241848 LGTM. I have no other comments. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the sp

Re: [PR] build(deps): bump testcontainers from 1.19.0 to 1.20.6 [gravitino]

2025-03-25 Thread via GitHub
jerryshao closed pull request #6731: build(deps): bump testcontainers from 1.19.0 to 1.20.6 URL: https://github.com/apache/gravitino/pull/6731 -- 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 spe

Re: [PR] build(deps): bump testcontainers from 1.19.0 to 1.20.6 [gravitino]

2025-03-25 Thread via GitHub
dependabot[bot] commented on PR #6731: URL: https://github.com/apache/gravitino/pull/6731#issuecomment-2752032021 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: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 commented on PR #6724: URL: https://github.com/apache/gravitino/pull/6724#issuecomment-2751385418 I have no further comments 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 go to

Re: [PR] [#6694] improvement: Avoid binding type `PATH` to `FILESET` [gravitino]

2025-03-25 Thread via GitHub
xunliu commented on code in PR #6698: URL: https://github.com/apache/gravitino/pull/6698#discussion_r2012208939 ## authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java: ## @@ -27,20 +27,48 @@ public class PathB

[PR] [#6750] improvement(core): property framework supports prefix [gravitino]

2025-03-25 Thread via GitHub
mchades opened a new pull request, #6751: URL: https://github.com/apache/gravitino/pull/6751 ### What changes were proposed in this pull request? property framework supports prefix ### Why are the changes needed? some properties with fixed prefixes maybe immutable or requ

Re: [PR] [#6720] fix(core): fix entity schema NullPointException on null properties [gravitino]

2025-03-25 Thread via GitHub
TEOTEO520 commented on PR #6728: URL: https://github.com/apache/gravitino/pull/6728#issuecomment-2750307827 > Seems be the same issue as #6524 Yes, it's same question, I've add UT in this PR, and add null prop check for all EntityCombinedObject -- This is an automated message from

(gravitino) branch main updated: [#6740] feat(core): Support group pre-event to Gravitino server (#6743)

2025-03-25 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 e4a049e17a [#6740] feat(core): Support group pre-e

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

2025-03-25 Thread via GitHub
FANNG1 closed issue #6740: [Subtask] Support group pre-event to Gravitino server URL: https://github.com/apache/gravitino/issues/6740 -- 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: [PR] [#6740] feat(core): Support group pre-event to Gravitino server [gravitino]

2025-03-25 Thread via GitHub
FANNG1 merged PR #6743: URL: https://github.com/apache/gravitino/pull/6743 -- 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] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011435578 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -16,7 +16,7 @@ filesets to manage non-tabular data like training datasets and other raw data. Typically,

Re: [PR] [#6740] feat(core): Support group pre-event to Gravitino server [gravitino]

2025-03-25 Thread via GitHub
FANNG1 commented on PR #6743: URL: https://github.com/apache/gravitino/pull/6743#issuecomment-2750288185 merged to main, thanks @Abyss-lord for your work! -- 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 t

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
mchades commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011513805 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -16,7 +16,7 @@ filesets to manage non-tabular data like training datasets and other raw data. Typically, a

Re: [I] [Bug report] Spark create table failed for PG catalog [gravitino]

2025-03-25 Thread via GitHub
Xander-run commented on issue #6697: URL: https://github.com/apache/gravitino/issues/6697#issuecomment-2750469048 All right, I have a better understanding of the issue's context. May I take this on and work on the fix? -- This is an automated message from the Apache Git Service. To respon

Re: [PR] build(deps): bump @reduxjs/toolkit from 1.9.7 to 2.6.1 in /web/web [gravitino]

2025-03-25 Thread via GitHub
LauraXia123 commented on PR #6737: URL: https://github.com/apache/gravitino/pull/6737#issuecomment-2750504579 @dependabot ignore this major version -- 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] build(deps): bump @reduxjs/toolkit from 1.9.7 to 2.6.1 in /web/web [gravitino]

2025-03-25 Thread via GitHub
LauraXia123 closed pull request #6737: build(deps): bump @reduxjs/toolkit from 1.9.7 to 2.6.1 in /web/web URL: https://github.com/apache/gravitino/pull/6737 -- 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 dependabot/npm_and_yarn/web/web/reduxjs/toolkit-2.6.1 deleted (was 762b0895fa)

2025-03-25 Thread github-bot
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/reduxjs/toolkit-2.6.1 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 762b0895fa build(deps): bump @reduxjs/toolkit from 1.9.7 to 2.6

Re: [PR] build(deps): bump @reduxjs/toolkit from 1.9.7 to 2.6.1 in /web/web [gravitino]

2025-03-25 Thread via GitHub
dependabot[bot] commented on PR #6737: URL: https://github.com/apache/gravitino/pull/6737#issuecomment-2750504889 OK, I won't notify you about version 2.x.x again, unless you re-open this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] [#6712] feat(core): Support user event to Gravitino server [gravitino]

2025-03-25 Thread via GitHub
Abyss-lord commented on PR #6746: URL: https://github.com/apache/gravitino/pull/6746#issuecomment-2750392272 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 m

Re: [PR] [#6680] feat(fileset): support fileset location placeholder [gravitino]

2025-03-25 Thread via GitHub
yuqi1129 commented on code in PR #6724: URL: https://github.com/apache/gravitino/pull/6724#discussion_r2011982623 ## docs/manage-fileset-metadata-using-gravitino.md: ## @@ -320,24 +320,41 @@ Currently, Gravitino supports two **types** of filesets: The `storageLocation` is the p

Re: [PR] [#6133] Improvement(core): Supports get Fileset schema location in the AuthorizationUtils [gravitino]

2025-03-25 Thread via GitHub
jerqi commented on PR #6211: URL: https://github.com/apache/gravitino/pull/6211#issuecomment-2751107101 After discussing offline, I will take 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