yuqi1129 commented on code in PR #6770:
URL: https://github.com/apache/gravitino/pull/6770#discussion_r2022019458
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -734,19 +735,18 @@ public Boolea
FANNG1 commented on code in PR #6782:
URL: https://github.com/apache/gravitino/pull/6782#discussion_r2022005886
##
server/build.gradle.kts:
##
@@ -38,6 +38,9 @@ dependencies {
implementation(libs.jackson.datatype.jsr310)
implementation(libs.jackson.databind)
implementat
FANNG1 commented on code in PR #6782:
URL: https://github.com/apache/gravitino/pull/6782#discussion_r2022005886
##
server/build.gradle.kts:
##
@@ -38,6 +38,9 @@ dependencies {
implementation(libs.jackson.datatype.jsr310)
implementation(libs.jackson.databind)
implementat
Abyss-lord commented on issue #6815:
URL: https://github.com/apache/gravitino/issues/6815#issuecomment-2768022233
@jerryshao I would like to work on it. should I assign myself?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
jerqi closed issue #6697: [Bug report] Spark create table failed for PG catalog
URL: https://github.com/apache/gravitino/issues/6697
--
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 comme
jerqi merged PR #6770:
URL: https://github.com/apache/gravitino/pull/6770
--
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.
jerryshao commented on code in PR #6752:
URL: https://github.com/apache/gravitino/pull/6752#discussion_r2021447510
##
core/src/main/java/org/apache/gravitino/storage/relational/service/TagMetaService.java:
##
@@ -230,20 +245,32 @@ public List
listAssociatedMetadataObjectsForTag
jerqi commented on issue #6346:
URL: https://github.com/apache/gravitino/issues/6346#issuecomment-2767904907
> [@jerqi](https://github.com/jerqi) In your design, it’s difficult to grant
others access to share the entire schema.
>
> I think the scenario is described as follows: The use
yuqi1129 commented on code in PR #6770:
URL: https://github.com/apache/gravitino/pull/6770#discussion_r2022014688
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -1044,4 +1044,34 @@ public boole
Abyss-lord commented on PR #6796:
URL: https://github.com/apache/gravitino/pull/6796#issuecomment-2768080268
> You did not complete the python **client** support, also you don't add
python test.
I will fix that.
--
This is an automated message from the Apache Git Service.
To respon
jerryshao commented on PR #6796:
URL: https://github.com/apache/gravitino/pull/6796#issuecomment-2768044876
You did not complete the python **client** support, also you don't add
python test.
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
jerryshao commented on PR #6782:
URL: https://github.com/apache/gravitino/pull/6782#issuecomment-2767485799
I was think that if we should move the lineage related framework and
implementations to the separated module, the main reason is that lineage is
indirectly related to core metadata mo
Abyss-lord commented on issue #6814:
URL: https://github.com/apache/gravitino/issues/6814#issuecomment-2768022606
@jerryshao I would like to work on it. should I assign myself?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
Abyss-lord commented on PR #6801:
URL: https://github.com/apache/gravitino/pull/6801#issuecomment-2767920607
@FANNG1 I’ve also updated `GroupPreEvent` to be consistent with
`UserPreEvent`. Additionally, i finish the grant\revoke group event in this PR.
--
This is an automated message from
jerqi commented on code in PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#discussion_r2022048078
##
catalogs/catalog-hive/src/main/java/org/apache/gravitino/catalog/hive/HiveProxyPlugin.java:
##
@@ -60,19 +60,22 @@ public Object doAs(
// The http authentic
granewang commented on code in PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#discussion_r2022061436
##
catalogs/catalog-hive/src/main/java/org/apache/gravitino/catalog/hive/HiveProxyPlugin.java:
##
@@ -60,19 +60,22 @@ public Object doAs(
// The http authe
FANNG1 commented on code in PR #6773:
URL: https://github.com/apache/gravitino/pull/6773#discussion_r2022074229
##
flink-connector/flink/src/test/java/org/apache/gravitino/flink/connector/integration/test/hive/FlinkHiveCatalogIT.java:
##
@@ -156,9 +157,10 @@ public void testCrea
FANNG1 commented on PR #6782:
URL: https://github.com/apache/gravitino/pull/6782#issuecomment-2767713310
> I was think that if we should move the lineage related framework and
implementations to the separated module, the main reason is that lineage is
indirectly related to core metadata mod
jerqi commented on PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#issuecomment-2767993247
cc @yuqi1129 Do we need to apply this fix for Iceberg or other catalogs?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
jerqi commented on code in PR #6770:
URL: https://github.com/apache/gravitino/pull/6770#discussion_r2022082405
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -734,19 +735,18 @@ public Boolean o
jerqi commented on code in PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#discussion_r2022084008
##
catalogs/catalog-hive/src/main/java/org/apache/gravitino/catalog/hive/HiveProxyPlugin.java:
##
@@ -60,19 +60,22 @@ public Object doAs(
// The http authentic
Abyss-lord commented on issue #6816:
URL: https://github.com/apache/gravitino/issues/6816#issuecomment-2768022422
@jerryshao I would like to work on it. should I assign myself?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
FANNG1 commented on code in PR #6782:
URL: https://github.com/apache/gravitino/pull/6782#discussion_r2022002004
##
core/src/main/java/org/apache/gravitino/GravitinoEnv.java:
##
@@ -354,6 +359,26 @@ public FutureGrantManager futureGrantManager() {
return futureGrantManager;
Abyss-lord commented on PR #6746:
URL: https://github.com/apache/gravitino/pull/6746#issuecomment-2765321629
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. Add null check for roles in User
jerqi commented on code in PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#discussion_r2020512774
##
server-common/src/main/java/org/apache/gravitino/server/authentication/KerberosAuthenticator.java:
##
@@ -168,7 +168,7 @@ private Principal retrievePrincipalFromToke
granewang commented on code in PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#discussion_r2020801093
##
server-common/src/main/java/org/apache/gravitino/server/authentication/KerberosAuthenticator.java:
##
@@ -170,14 +168,15 @@ private Principal retrievePrincipalFr
Abyss-lord opened a new pull request, #6800:
URL: https://github.com/apache/gravitino/pull/6800
### What changes were proposed in this pull request?
Uniform CLI output format for Model, User and Group commands.
### Why are the changes needed?
Uniform CLI output format
Abyss-lord commented on PR #6800:
URL: https://github.com/apache/gravitino/pull/6800#issuecomment-2766022479
Hi @justinmclean , 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
241600489 commented on issue #6490:
URL: https://github.com/apache/gravitino/issues/6490#issuecomment-2765638042
ok ,i‘ll try 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 specific comme
Abyss-lord opened a new pull request, #6801:
URL: https://github.com/apache/gravitino/pull/6801
### What changes were proposed in this pull request?
Support group event to Gravitino server.
### Why are the changes needed?
Fix: #6741
### Does this PR introduce _any
Abyss-lord commented on PR #6801:
URL: https://github.com/apache/gravitino/pull/6801#issuecomment-2766196578
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
mchades commented on code in PR #6751:
URL: https://github.com/apache/gravitino/pull/6751#discussion_r2021150303
##
core/src/main/java/org/apache/gravitino/connector/PropertiesMetadata.java:
##
@@ -102,10 +149,34 @@ default Object getOrDefault(Map
properties, String propertyNam
hdygxsj opened a new pull request, #6805:
URL: https://github.com/apache/gravitino/pull/6805
### Why are the changes needed?
(Please clarify why the changes are needed. For instance,
1. If you propose a new API, clarify the use case for a new API.
2. If you fix a bug,
jerqi commented on code in PR #6770:
URL: https://github.com/apache/gravitino/pull/6770#discussion_r2020621717
##
api/src/main/java/org/apache/gravitino/authorization/MetadataObjectChange.java:
##
@@ -100,7 +117,8 @@ public boolean equals(Object o) {
if (o == null || getC
zhoukangcn commented on issue #6346:
URL: https://github.com/apache/gravitino/issues/6346#issuecomment-2766500322
@jerqi In your design, it’s difficult to grant others access to share the
entire schema.
I think the scenario is described as follows: The user creates a
`metalake.catal
FANNG1 commented on code in PR #6746:
URL: https://github.com/apache/gravitino/pull/6746#discussion_r2020626251
##
core/src/main/java/org/apache/gravitino/utils/NameIdentifierUtil.java:
##
@@ -103,6 +103,28 @@ public static NameIdentifier ofTable(
public static NameIdentifier
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/next/bundle-analyzer-14.2.26
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at fcbc3821ad build(deps-dev): bump @next/bundle-analyzer i
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/react-hook-form-7.55.0
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 687e8c35a6 build(deps): bump react-hook-form from 7.54.2 to 7.
jerryshao commented on PR #6796:
URL: https://github.com/apache/gravitino/pull/6796#issuecomment-2766861097
Generally LGTM, please also add the Python client implementation.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
jerryshao commented on code in PR #6752:
URL: https://github.com/apache/gravitino/pull/6752#discussion_r2021427326
##
core/src/main/java/org/apache/gravitino/storage/relational/mapper/TableColumnSQLProviderFactory.java:
##
@@ -90,4 +90,8 @@ public static String selectColumnIdByT
jerryshao commented on code in PR #6752:
URL: https://github.com/apache/gravitino/pull/6752#discussion_r2021432891
##
core/src/main/java/org/apache/gravitino/storage/relational/service/MetadataObjectService.java:
##
@@ -57,6 +59,18 @@ public class MetadataObjectService {
pr
dependabot[bot] opened a new pull request, #6812:
URL: https://github.com/apache/gravitino/pull/6812
Bumps
[eslint-config-next](https://github.com/vercel/next.js/tree/HEAD/packages/eslint-config-next)
from 14.0.3 to 14.1.4.
Commits
https://github.com/vercel/next.js/commit/f1fc
Abyss-lord commented on PR #6746:
URL: https://github.com/apache/gravitino/pull/6746#issuecomment-2765729112
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. move `ofUser` method from NameId
jerqi commented on code in PR #6770:
URL: https://github.com/apache/gravitino/pull/6770#discussion_r2020621717
##
api/src/main/java/org/apache/gravitino/authorization/MetadataObjectChange.java:
##
@@ -100,7 +117,8 @@ public boolean equals(Object o) {
if (o == null || getC
jerqi commented on code in PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#discussion_r2020867733
##
catalogs/catalog-hive/src/main/java/org/apache/gravitino/catalog/hive/HiveProxyPlugin.java:
##
@@ -60,13 +60,16 @@ public Object doAs(
// The http authentic
Abyss-lord opened a new issue, #6797:
URL: https://github.com/apache/gravitino/issues/6797
### What would you like to be improved?
The CLI lacks a standardized output format for many commands, and the
processing logic for multiple entities is directly implemented within the
handle me
FANNG1 commented on PR #6746:
URL: https://github.com/apache/gravitino/pull/6746#issuecomment-2766058700
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 g
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 b382cba719 [#6712] feat(core): Support user event
FANNG1 closed issue #6712: [Subtask] Support user event to Gravitino server
URL: https://github.com/apache/gravitino/issues/6712
--
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.
FANNG1 closed issue #6712: [Subtask] Support user event to Gravitino server
URL: https://github.com/apache/gravitino/issues/6712
--
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.
an-shi-chi-fan opened a new issue, #6802:
URL: https://github.com/apache/gravitino/issues/6802
### Describe the feature
Currently, the permission granularity supported by Gravitino is relatively
coarse. We hope that Gravitino can support more granular permissions.
### Motiv
hdygxsj opened a new issue, #6803:
URL: https://github.com/apache/gravitino/issues/6803
### Describe the subtask
Enable the Gravitino Python Cli to carry a token when requesting the
Gravitino Server for metadata authorization.
### Parent issue
#6762
--
This is an au
jerqi commented on code in PR #6778:
URL: https://github.com/apache/gravitino/pull/6778#discussion_r2020659067
##
server-common/src/main/java/org/apache/gravitino/server/authentication/KerberosAuthenticator.java:
##
@@ -170,14 +168,15 @@ private Principal retrievePrincipalFromTo
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/next-14.2.26
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 01664e3e1b build(deps): bump next from 14.2.21 to 14.2.26 in /web/web
N
dependabot[bot] opened a new pull request, #6809:
URL: https://github.com/apache/gravitino/pull/6809
Bumps [@hookform/resolvers](https://github.com/react-hook-form/resolvers)
from 3.10.0 to 4.1.3.
Release notes
Sourced from https://github.com/react-hook-form/resolvers/releases";>@
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/hookform/resolvers-4.1.3
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 7cb183caa1 build(deps): bump @hookform/resolvers from 3.10.0
jerryshao commented on code in PR #6751:
URL: https://github.com/apache/gravitino/pull/6751#discussion_r2021464201
##
core/src/main/java/org/apache/gravitino/connector/PropertiesMetadata.java:
##
@@ -35,8 +37,18 @@ public interface PropertiesMetadata {
* @return true if the
jerryshao commented on code in PR #6751:
URL: https://github.com/apache/gravitino/pull/6751#discussion_r2021482788
##
core/src/main/java/org/apache/gravitino/connector/PropertiesMetadata.java:
##
@@ -102,10 +146,73 @@ default Object getOrDefault(Map
properties, String propertyN
jerryshao commented on code in PR #6751:
URL: https://github.com/apache/gravitino/pull/6751#discussion_r2021473564
##
core/src/main/java/org/apache/gravitino/connector/PropertiesMetadata.java:
##
@@ -102,10 +146,73 @@ default Object getOrDefault(Map
properties, String propertyN
59 matches
Mail list logo
