Re: [PR] [#139] Improvement: externalize Gravitino metadata and Jupyter notebooks outside of containers [gravitino-playground]

2025-04-01 Thread via GitHub
Copilot commented on code in PR #140: URL: https://github.com/apache/gravitino-playground/pull/140#discussion_r2023011203 ## docker-compose.yaml: ## @@ -165,6 +166,7 @@ services: - "1:" volumes: - ./init/jupyter:/tmp/gravitino + - ./data/jupyter/

[I] [Bug report] Trino hard crash within Docker container on Apple M4 / macOS 15.2 Processors (aarch64) [gravitino-playground]

2025-04-01 Thread via GitHub
ChongWei905 opened a new issue, #141: URL: https://github.com/apache/gravitino-playground/issues/141 ### Version main branch ### Describe what's wrong Cannot launch apache/gravitino-playground:trino-435-gravitino-0.8.0-incubating "java" process within a Docker container

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

2025-04-01 Thread via GitHub
mchades closed pull request #6751: [#6750] improvement(core): property framework supports prefix 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 th

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

2025-04-01 Thread via GitHub
yuqi1129 commented on code in PR #6782: URL: https://github.com/apache/gravitino/pull/6782#discussion_r2021072530 ## core/src/main/java/org/apache/gravitino/lineage/LineageConfig.java: ## @@ -0,0 +1,130 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] [#6693] improvement(authz): Support to rename Hive table in the Ranger authz plugin [gravitino]

2025-04-01 Thread via GitHub
jerqi commented on code in PR #6770: URL: https://github.com/apache/gravitino/pull/6770#discussion_r2022069567 ## authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java: ## @@ -1044,4 +1044,34 @@ public boolean

Re: [PR] [MINOR]fix: Add dependency on gravitino server to Spark in docker compose [gravitino-playground]

2025-04-01 Thread via GitHub
FANNG1 commented on PR #138: URL: https://github.com/apache/gravitino-playground/pull/138#issuecomment-2769329035 @Xander-run thanks for your contribution! -- 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

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

2025-04-01 Thread via GitHub
FANNG1 commented on code in PR #6801: URL: https://github.com/apache/gravitino/pull/6801#discussion_r2022688583 ## docs/gravitino-server-config.md: ## @@ -118,6 +118,7 @@ Gravitino triggers a pre-event before the operation, a post-event after the comp # Post-event + R

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

2025-04-01 Thread via GitHub
granewang commented on code in PR #6778: URL: https://github.com/apache/gravitino/pull/6778#discussion_r2021965328 ## catalogs/catalog-hive/src/main/java/org/apache/gravitino/catalog/hive/HiveProxyPlugin.java: ## @@ -60,13 +60,16 @@ public Object doAs( // The http authe

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

2025-04-01 Thread via GitHub
FANNG1 commented on PR #6782: URL: https://github.com/apache/gravitino/pull/6782#issuecomment-2767778768 @jerryshao , add an new `lineage` module to place lineage related code, and move `LineageService` from `GravitinoEnv` which is included in core module, to `GravitinoServer` -- This is

(gravitino) branch main updated: [#6741] feat(core): Support group event to Gravitino server (#6801)

2025-04-01 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 6cbf907e9a [#6741] feat(core): Support group event

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

2025-04-01 Thread via GitHub
FANNG1 commented on PR #6773: URL: https://github.com/apache/gravitino/pull/6773#issuecomment-2770989193 LGTM, @hdygxsj could you refactor the issue description according to the template? -- This is an automated message from the Apache Git Service. To respond to the message, please log on

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

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

2025-04-01 Thread via GitHub
FANNG1 merged PR #6801: URL: https://github.com/apache/gravitino/pull/6801 -- 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] [#6741] feat(core): Support group event to Gravitino server [gravitino]

2025-04-01 Thread via GitHub
FANNG1 commented on PR #6801: URL: https://github.com/apache/gravitino/pull/6801#issuecomment-2770982786 merged to main, thanks for your contribution. -- 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] [#6771] fix(flink-connector): Optimize the error message when unable to load the catalog while using Flink. [gravitino]

2025-04-01 Thread via GitHub
FANNG1 commented on PR #6773: URL: https://github.com/apache/gravitino/pull/6773#issuecomment-2770995902 @tengqm @sunxiaojian any 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 t

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

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

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

2025-04-01 Thread via GitHub
jerqi commented on PR #6778: URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2771201814 @granewang Could you add a test case for this PR like `ProxyCatalogHiveIT`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

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

2025-04-01 Thread via GitHub
jerqi commented on PR #136: URL: https://github.com/apache/gravitino-playground/pull/136#issuecomment-2771173897 > > 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? Playground d

Re: [PR] [Improvement] Add support for curl and wget [gravitino-playground]

2025-04-01 Thread via GitHub
justinmclean commented on PR #21: URL: https://github.com/apache/gravitino-playground/pull/21#issuecomment-2770877483 Given the age of this, and the many changes made since then, I think it can be closed. -- This is an automated message from the Apache Git Service. To respond to the mess

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

2025-04-01 Thread via GitHub
Abyss-lord opened a new pull request, #6817: URL: https://github.com/apache/gravitino/pull/6817 ### What changes were proposed in this pull request? Support user failure event to Gravitino server ### Why are the changes needed? Fix: #6713 ### Does this PR introduce

(gravitino) branch main updated: build(deps): bump commons-logging:commons-logging from 1.2 to 1.3.5 (#6807)

2025-04-01 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 51aa9946b1 build(deps): bump commons-logging:commo

Re: [PR] build(deps): bump commons-logging:commons-logging from 1.2 to 1.3.5 [gravitino]

2025-04-01 Thread via GitHub
jerryshao merged PR #6807: URL: https://github.com/apache/gravitino/pull/6807 -- 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] [#6771] flink-connector: Optimize the error message when unable to load the catalog while using Flink. [gravitino]

2025-04-01 Thread via GitHub
hdygxsj commented on code in PR #6773: URL: https://github.com/apache/gravitino/pull/6773#discussion_r2022693079 ## flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/hive/FlinkHiveCatalogIT.java: ## @@ -156,9 +157,10 @@ public void testCre

[PR] [#139] Improvement: externalize Gravitino metadata and Jupyter notebooks outside of containers [gravitino-playground]

2025-04-01 Thread via GitHub
shaofengshi opened a new pull request, #140: URL: https://github.com/apache/gravitino-playground/pull/140 ### What changes were proposed in this pull request? Mount a local folder to the Gravitino and Jupyter container, which persistents the metadata of Gravitino (h2 db file by defaul

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

2025-04-01 Thread via GitHub
yuqi1129 merged PR #6818: URL: https://github.com/apache/gravitino/pull/6818 -- 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] [Improvement] Add support for curl and wget [gravitino-playground]

2025-04-01 Thread via GitHub
justinmclean closed pull request #21: [Improvement] Add support for curl and wget URL: https://github.com/apache/gravitino-playground/pull/21 -- 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 spec

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

2025-04-01 Thread via GitHub
FANNG1 commented on code in PR #6817: URL: https://github.com/apache/gravitino/pull/6817#discussion_r2024182276 ## core/src/main/java/org/apache/gravitino/listener/api/event/UserFailureEvent.java: ## @@ -0,0 +1,44 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [I] [Improvement] Readme is missing port 6080 [gravitino-playground]

2025-04-01 Thread via GitHub
shaofengshi closed issue #131: [Improvement] Readme is missing port 6080 URL: https://github.com/apache/gravitino-playground/issues/131 -- 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

Re: [I] [Improvement] Readme is missing port 6080 [gravitino-playground]

2025-04-01 Thread via GitHub
shaofengshi closed issue #131: [Improvement] Readme is missing port 6080 URL: https://github.com/apache/gravitino-playground/issues/131 -- 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

Re: [PR] [Improvement] Add support for curl and wget [gravitino-playground]

2025-04-01 Thread via GitHub
shaofengshi commented on code in PR #21: URL: https://github.com/apache/gravitino-playground/pull/21#discussion_r2022721561 ## init/gravitino/init.sh: ## @@ -2,9 +2,16 @@ # Copyright 2023 Datastrato Pvt Ltd. # This software is licensed under the Apache License version 2. # -

Re: [PR] [Improvement] Add support for curl and wget [gravitino-playground]

2025-04-01 Thread via GitHub
shaofengshi commented on PR #21: URL: https://github.com/apache/gravitino-playground/pull/21#issuecomment-2769138386 > I am a little confused by what going on here but without wget installed the launch playground script fails. I can't see where it uses wget but it fails, users have also re

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

2025-04-01 Thread via GitHub
mchades commented on code in PR #6751: URL: https://github.com/apache/gravitino/pull/6751#discussion_r2022732412 ## core/src/main/java/org/apache/gravitino/connector/PropertiesMetadata.java: ## @@ -102,10 +146,73 @@ default Object getOrDefault(Map properties, String propertyNam

(gravitino) branch dependabot/gradle/commons-logging-commons-logging-1.3.5 deleted (was 2b9f7682fc)

2025-04-01 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/gradle/commons-logging-commons-logging-1.3.5 in repository https://gitbox.apache.org/repos/asf/gravitino.git was 2b9f7682fc build(deps): bump commons-logging:commons-logging