[PR] [MINOR] fix(CI): update dorny/paths-filter@v2 to dorny/paths-filter@v3.0.2 [gravitino]

2025-04-02 Thread via GitHub
mchades opened a new pull request, #6819: URL: https://github.com/apache/gravitino/pull/6819 ### What changes were proposed in this pull request? update dorny/paths-filter@v2 to dorny/paths-filter@v3.0.2 ### Why are the changes needed? update dorny/paths-filter@v2 to do

Re: [I] [Improvement] Optimize the error message when unable to load the catalog while using Flink. [gravitino]

2025-04-02 Thread via GitHub
FANNG1 closed issue #6771: [Improvement] Optimize the error message when unable to load the catalog while using Flink. URL: https://github.com/apache/gravitino/issues/6771 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] [#6771] fix(flink-connector): Optimize the error message when unable to load the catalog while using Flink. [gravitino]

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

(gravitino) branch main updated: [#6771] fix(flink-connector): Optimize the error message when unable to load the catalog while using Flink. (#6773)

2025-04-02 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 bb4662f483 [#6771] fix(flink-connector): Optimize

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

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

[I] [Subtask] Dynamic loading Jcasbin authentication policies [gravitino]

2025-04-02 Thread via GitHub
xunliu opened a new issue, #6826: URL: https://github.com/apache/gravitino/issues/6826 ### Describe the subtask https://github.com/user-attachments/assets/1382207c-757e-4265-9578-31405efa27ae"; /> Base this solution, The single Gravitino service can use full load jcasbi

[I] [Bug report] Release script use wrong branch to create tag [gravitino]

2025-04-02 Thread via GitHub
justinmclean opened a new issue, #6836: URL: https://github.com/apache/gravitino/issues/6836 ### Version main branch ### Describe what's wrong release-build.sh creates the tag on the main branch - this is incorrect if you are making a point release. ``` git clo

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

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

Re: [PR] [6346] feat(authz): Add privilege support for model [gravitino]

2025-04-02 Thread via GitHub
jerqi commented on PR #6820: URL: https://github.com/apache/gravitino/pull/6820#issuecomment-2774590646 > Can you add some tests to see if the model privileges can be set, stored, and fetched using current access control module? OK. -- This is an automated message from the Apache G

svn commit: r76052 - in /dev/incubator/gravitino/v0.8.1-incubating-rc1: gravitino-0.8.1-incubating-bin.tar.gz gravitino-0.8.1-incubating-bin.tar.gz.asc gravitino-0.8.1-incubating-bin.tar.gz.sha512

2025-04-02 Thread jmclean
Author: jmclean Date: Thu Apr 3 06:12:16 2025 New Revision: 76052 Log: removed wrong version jars from binary release Modified: dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-bin.tar.gz dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-b

(gravitino) branch fix-version-numbers created (now f4831fbb8b)

2025-04-02 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 at f4831fbb8b [#6747] improvement(web): can be dropped while set not in-use false (#6776) No new revis

(gravitino) branch main updated (8924ba5f2f -> 06497e424b)

2025-04-02 Thread fanng
This is an automated email from the ASF dual-hosted git repository. fanng pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/gravitino.git from 8924ba5f2f [#5202] feat(client-python): Support Column and its default value part1 - Column (#6795) add 06497e42

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

2025-04-02 Thread via GitHub
FANNG1 closed issue #6742: [Subtask] Support group failure event to Gravitino server URL: https://github.com/apache/gravitino/issues/6742 -- 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

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

2025-04-02 Thread via GitHub
FANNG1 closed issue #6739: [Subtask] Support group-related event to Gravitino server URL: https://github.com/apache/gravitino/issues/6739 -- 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

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

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

[I] [Bug report] Release scripts fail to work if you have multiple keys [gravitino]

2025-04-02 Thread via GitHub
justinmclean opened a new issue, #6831: URL: https://github.com/apache/gravitino/issues/6831 ### Version main branch ### Describe what's wrong The release script can grab the wrong key to sign the artefacts ### Error message and/or stacktrace No error occurs

[I] [Bug report] Release scripts look at snapshot [gravitino]

2025-04-02 Thread via GitHub
justinmclean opened a new issue, #6832: URL: https://github.com/apache/gravitino/issues/6832 ### Version main branch ### Describe what's wrong The release script look at the snapshot which may not exist. There should be no reason to look at this as the environment variab

[I] [Bug report] Wrong path for web license and notice bin files [gravitino]

2025-04-02 Thread via GitHub
justinmclean opened a new issue, #6834: URL: https://github.com/apache/gravitino/issues/6834 ### Version main branch ### Describe what's wrong In release-build.sh the wrong path is use for the license and notice bin files: ``` rm -f gravitino-$GRAVITINO_VERSION-s

[I] [Bug report] Release script get wrong version number [gravitino]

2025-04-02 Thread via GitHub
justinmclean opened a new issue, #6833: URL: https://github.com/apache/gravitino/issues/6833 ### Version main branch ### Describe what's wrong The release script checks the lastest branch for the version number, this can give an incorrect version number when making a poi

(gravitino) branch main updated: [#5202] feat(client-python): Support Column and its default value part1 - Column (#6795)

2025-04-02 Thread liuxun
This is an automated email from the ASF dual-hosted git repository. liuxun 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 8924ba5f2f [#5202] feat(client-python): Support C

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

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

[I] [Bug report] Iceberg REST, HMS specific configs [gravitino]

2025-04-02 Thread via GitHub
metalshanked opened a new issue, #6822: URL: https://github.com/apache/gravitino/issues/6822 ### Version main branch ### Describe what's wrong using Gravitino 0.8.0, Iceberg REST catalog with an HMS backend. I seem to get getting connection errors to the HMS ```

Re: [I] [Subtask] Add privilege support for model [gravitino]

2025-04-02 Thread via GitHub
xunliu commented on issue #6346: URL: https://github.com/apache/gravitino/issues/6346#issuecomment-2772024783 @jerqi Except CREATE_MODEL, CREATE_MODEL_VERSION, USE_MODEL, Is `Modify_Module` through Owner to implement? -- This is an automated message from the Apache Git Service. To respond

(gravitino) branch dependabot/gradle/mysql-mysql-connector-java-8.0.33 deleted (was c55e209bbe)

2025-04-02 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/gradle/mysql-mysql-connector-java-8.0.33 in repository https://gitbox.apache.org/repos/asf/gravitino.git was c55e209bbe build(deps): bump mysql:mysql-connector-java from 8.0.

Re: [PR] [MINOR] fix(CI): update dorny/paths-filter@v2 to dorny/paths-filter@v3.0.2 [gravitino]

2025-04-02 Thread via GitHub
jerryshao merged PR #6819: URL: https://github.com/apache/gravitino/pull/6819 -- 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 mysql:mysql-connector-java from 8.0.23 to 8.0.33 [gravitino]

2025-04-02 Thread via GitHub
jerryshao merged PR #6806: URL: https://github.com/apache/gravitino/pull/6806 -- 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] [#4024]Refactor: Reduce unnecessary queries in catalog JDBC implementation [gravitino]

2025-04-02 Thread via GitHub
jerryshao commented on PR #6540: URL: https://github.com/apache/gravitino/pull/6540#issuecomment-2773256447 @yuqi1129 can you please help 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

[PR] fix(spark-connector): filter out all pg gravitino table properties [gravitino]

2025-04-02 Thread via GitHub
Xander-run opened a new pull request, #6824: URL: https://github.com/apache/gravitino/pull/6824 ### 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 t

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

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

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

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

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

2025-04-02 Thread via GitHub
FANNG1 commented on code in PR #6782: URL: https://github.com/apache/gravitino/pull/6782#discussion_r2025849247 ## lineage/src/main/java/org/apache/gravitino/lineage/source/LineageSource.java: ## @@ -0,0 +1,55 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

(gravitino) branch fix-version-numbers updated: for 0.8.1 release

2025-04-02 Thread jmclean
This is an automated email from the ASF dual-hosted git repository. jmclean pushed a commit to branch fix-version-numbers in repository https://gitbox.apache.org/repos/asf/gravitino.git The following commit(s) were added to refs/heads/fix-version-numbers by this push: new f4cfdc72cf for 0.

Re: [PR] [#6216] flink-connector: Add integration Tests for GravitinoPaimonCatalog with Catalog-backend as HiveMetaStore [gravitino]

2025-04-02 Thread via GitHub
FANNG1 commented on code in PR #6805: URL: https://github.com/apache/gravitino/pull/6805#discussion_r2025889624 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/paimon/FlinkPaimonLocalFileSystemBackendIT.java: ## @@ -0,0 +1,72 @@ +/* +

Re: [PR] [#6742] feat(core): Support group failure event to Gravitino server [gravitino]

2025-04-02 Thread via GitHub
Abyss-lord commented on PR #6823: URL: https://github.com/apache/gravitino/pull/6823#issuecomment-2774093727 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] [#6742] feat(core): Support group failure event to Gravitino server [gravitino]

2025-04-02 Thread via GitHub
FANNG1 merged PR #6823: URL: https://github.com/apache/gravitino/pull/6823 -- 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] [6346] feat(authz): Add privilege support for model [gravitino]

2025-04-02 Thread via GitHub
jerqi commented on PR #6820: URL: https://github.com/apache/gravitino/pull/6820#issuecomment-2774142781 > Do we need a privilege to support modifying model metadata? In my view, we should > You can do this in another PR. #6704 OK. -- This is an automated message from t

Re: [PR] [#6216] flink-connector: Add integration Tests for GravitinoPaimonCatalog with Catalog-backend as HiveMetaStore [gravitino]

2025-04-02 Thread via GitHub
FANNG1 commented on code in PR #6805: URL: https://github.com/apache/gravitino/pull/6805#discussion_r2025890311 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/paimon/FlinkPaimonLocalFileSystemBackendIT.java: ## @@ -0,0 +1,72 @@ +/* +

Re: [PR] [6346] feat(authz): Add privilege support for model [gravitino]

2025-04-02 Thread via GitHub
jerryshao commented on PR #6820: URL: https://github.com/apache/gravitino/pull/6820#issuecomment-2774482910 Can you add some tests to see if the model privileges can be set, stored, and fetched using current access control module? -- This is an automated message from the Apache Git Servic

(gravitino) branch main updated: [#6750] improvement(core): property framework supports prefix (#6751)

2025-04-02 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 5d00b538d5 [#6750] improvement(core): property fra

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

2025-04-02 Thread via GitHub
jerryshao merged PR #6751: URL: https://github.com/apache/gravitino/pull/6751 -- 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] [#6779] feat(core): Support lineage framework in Gravitino [gravitino]

2025-04-02 Thread via GitHub
jerryshao commented on code in PR #6782: URL: https://github.com/apache/gravitino/pull/6782#discussion_r2026271095 ## server/src/main/java/org/apache/gravitino/server/GravitinoServer.java: ## @@ -75,10 +77,13 @@ public class GravitinoServer extends ResourceConfig { private

(gravitino) branch fix-version-numbers updated (f4cfdc72cf -> c384b2d36f)

2025-04-02 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 f4cfdc72cf for 0.8.1 release add c384b2d36f update version to include incubating and snapshot N

Re: [PR] [MINOR] fix version numbers for 0.8.1 release [gravitino]

2025-04-02 Thread via GitHub
justinmclean commented on PR #6825: URL: https://github.com/apache/gravitino/pull/6825#issuecomment-2774635850 Setting the version to '0.8.1-incubating-SNAPSHOT' gives artifacts names like gravitino-0.8.1-incubating-SNAPSHOT-bin.tar.gz, are you sure that is correct? -- This is an automa

(gravitino) branch fix-version-numbers updated (c384b2d36f -> 1f11d751c8)

2025-04-02 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 c384b2d36f update version to include incubating and snapshot add 1f11d751c8 python version can n

svn commit: r75987 - in /dev/incubator/gravitino/v0.8.1-incubating-rc1: 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-02 Thread jmclean
Author: jmclean Date: Wed Apr 2 07:58:51 2025 New Revision: 75987 Log: removing archives from source bundle Modified: dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-src.tar.gz dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-src.tar.gz.

Re: [I] Support `TableFormat` and `PlainFormat` for Tag, role, Fileset and Topic [gravitino]

2025-04-02 Thread via GitHub
unknowntpo commented on issue #6799: URL: https://github.com/apache/gravitino/issues/6799#issuecomment-2771782275 @Abyss-lord I would like to 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 abo

Re: [PR] [6346] feat(authz): Add privilege support for model [gravitino]

2025-04-02 Thread via GitHub
jerryshao commented on PR #6820: URL: https://github.com/apache/gravitino/pull/6820#issuecomment-2773291052 You can do this in another PR. https://github.com/apache/gravitino/issues/6704 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] [6346] feat(authz): Add privilege support for model [gravitino]

2025-04-02 Thread via GitHub
jerryshao commented on PR #6820: URL: https://github.com/apache/gravitino/pull/6820#issuecomment-2773297559 Do we need a privilege to support modifying model metadata? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

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

2025-04-02 Thread via GitHub
FANNG1 merged PR #6817: URL: https://github.com/apache/gravitino/pull/6817 -- 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] [#135] improvement: Update Hive image [gravitino-playground]

2025-04-02 Thread via GitHub
jerqi commented on PR #136: URL: https://github.com/apache/gravitino-playground/pull/136#issuecomment-2772227833 > > > > Could you verify all the test cases with new image including authorization demo? > > > > > > > > > Hi Royi, is this test case covered in the notebook in Jupyte

svn commit: r75988 - /dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-src.tar.gz.asc

2025-04-02 Thread jmclean
Author: jmclean Date: Wed Apr 2 08:25:22 2025 New Revision: 75988 Log: fixing again Modified: dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-src.tar.gz.asc Modified: dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-src.tar.gz.asc =

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

2025-04-02 Thread via GitHub
shaofengshi commented on PR #136: URL: https://github.com/apache/gravitino-playground/pull/136#issuecomment-2771982634 > > > Could you verify all the test cases with new image including authorization demo? > > > > > > Hi Royi, is this test case covered in the notebook in Jupyter?

Re: [I] [Subtask] Support user-related event to Gravitino server [gravitino]

2025-04-02 Thread via GitHub
FANNG1 closed issue #6710: [Subtask] Support user-related event to Gravitino server URL: https://github.com/apache/gravitino/issues/6710 -- 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

svn commit: r75981 - /release/incubator/gravitino/KEYS

2025-04-02 Thread jmclean
Author: jmclean Date: Wed Apr 2 07:23:39 2025 New Revision: 75981 Log: Add my signature Modified: release/incubator/gravitino/KEYS Modified: release/incubator/gravitino/KEYS == --- release/incubator/gravitino/KEYS (

Re: [I] [Improvement] Currently path based ranger authorization plugin doesn't support relational data source to rename [gravitino]

2025-04-02 Thread via GitHub
jerqi closed issue #6693: [Improvement] Currently path based ranger authorization plugin doesn't support relational data source to rename URL: https://github.com/apache/gravitino/issues/6693 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

svn commit: r75986 - /dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-src.tar.gz.asc

2025-04-02 Thread jmclean
Author: jmclean Date: Wed Apr 2 07:55:11 2025 New Revision: 75986 Log: removing archives from source bundle Modified: dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-src.tar.gz.asc Modified: dev/incubator/gravitino/v0.8.1-incubating-rc1/gravitino-0.8.1-incubating-

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

2025-04-02 Thread via GitHub
Abyss-lord commented on PR #6817: URL: https://github.com/apache/gravitino/pull/6817#issuecomment-2771756840 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. make class abstract. -- This

[PR] [6346] feat(authz): Add privilege support for model [gravitino]

2025-04-02 Thread via GitHub
jerqi opened a new pull request, #6820: URL: https://github.com/apache/gravitino/pull/6820 ### What changes were proposed in this pull request? Add privilege support for model ### Why are the changes needed? Fix: #6346 ### Does this PR introduce _any_ user-facing