jerryshao merged PR #4590:
URL: https://github.com/apache/gravitino/pull/4590
--
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
jerryshao closed issue #3729: [Improvement] create and alter failed since the
operations in catalog are async
URL: https://github.com/apache/gravitino/issues/3729
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
jerryshao closed issue #3729: [Improvement] create and alter failed since the
operations in catalog are async
URL: https://github.com/apache/gravitino/issues/3729
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL ab
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 f7a0c11cc [#4572][#3729][#3496] refactor(core): re
jerryshao closed issue #4572: [Bug report] CatalogKafkaIT > testAlterTopic()
FAILED
URL: https://github.com/apache/gravitino/issues/4572
--
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
jerryshao closed issue #3496: [Improvement] Creating a Kafka topic may fail
when reloading topic before return
URL: https://github.com/apache/gravitino/issues/3496
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724512836
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724512320
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724515308
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724516409
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724517579
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724518162
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724520020
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724520020
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724525054
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724530797
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,1027 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724534814
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724536521
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724546784
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724551608
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerClientExtend.java:
##
@@ -0,0 +1,184 @@
+/*
+ * Licensed to the Apa
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724555745
##
authorizations/authorization-ranger/src/main/resources/META-INF/services/org.apache.gravitino.connector.authorization.AuthorizationProvider:
##
@@ -0,0 +1,19 @@
+#
+
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724574374
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,1006 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724574374
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,1006 @@
+/*
+ * Licensed to
shaofengshi commented on code in PR #65:
URL:
https://github.com/apache/gravitino-playground/pull/65#discussion_r1724598811
##
docker-compose.yaml:
##
@@ -22,9 +22,9 @@ services:
image: datastrato/hive:2.7.3-no-yarn
ports:
- "3307:3306"
- - "9000:9000"
-
jerqi commented on issue #4246:
URL: https://github.com/apache/gravitino/issues/4246#issuecomment-2301396431
@lw-yang Do you like to submit a pull request to fix the issue?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and u
lw-yang commented on issue #4246:
URL: https://github.com/apache/gravitino/issues/4246#issuecomment-2301408936
@jerqi Ok, I would like to fix 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 t
xunliu commented on PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#issuecomment-2301431331
@yuqi1129
> Policies can't be null, please see
>
> // Only return the policies that are managed by Gravitino.
> if (policies.size() > 1) {
> throw new Au
jerryshao merged PR #4518:
URL: https://github.com/apache/gravitino/pull/4518
--
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
github-actions[bot] opened a new pull request, #4609:
URL: https://github.com/apache/gravitino/pull/4609
### What changes were proposed in this pull request?
This PR proposes to add several release script to automate the release
process.
### Why are the changes needed?
F
jerryshao closed issue #4417: [FEATURE] Add the release script for Gravitino
URL: https://github.com/apache/gravitino/issues/4417
--
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.
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 a4d720d38 [#4417] feat(build): Add release script
jerryshao closed pull request #4609: [#4417] feat(build): Add release script to
automate the release process
URL: https://github.com/apache/gravitino/pull/4609
--
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
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.6-a4d720d38df21f2d5f692caabacaba15d29d830d
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 39f61f5c1 [#4417] feat(build): Add release script to
lingzhengyou opened a new issue, #4610:
URL: https://github.com/apache/gravitino/issues/4610
### Describe the feature
Currently, when creating a catalog, no validation is performed, and any
parameters can be filled in arbitrarily to execute successfully. You need to
proactively check
yuqi1129 opened a new pull request, #4611:
URL: https://github.com/apache/gravitino/pull/4611
### 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 the
coolderli commented on issue #4558:
URL: https://github.com/apache/gravitino/issues/4558#issuecomment-2301481233
> Hi Peidian, I don't have much knowledge about FUSE; For the solution2, is
that only available for python? Which means, only in Python application, fsspec
mounts a remote storag
xunliu commented on PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#issuecomment-2301488589
> Does it support multiple Hive catalog uses one Ranger?
Yes, we support.
Because each Hive catalog have a separate AuthorizationPlugin instance.
--
This is an automated messa
yuqi1129 opened a new pull request, #4612:
URL: https://github.com/apache/gravitino/pull/4612
### What changes were proposed in this pull request?
Fix the backend type configuration.
### Why are the changes needed?
The configuration for the backend type is incorr
jerryshao merged PR #4609:
URL: https://github.com/apache/gravitino/pull/4609
--
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
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch branch-0.6
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.6 by this push:
new ab2f0720b [#4417] feat(build): Add rel
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.6-a4d720d38df21f2d5f692caabacaba15d29d830d
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 39f61f5c1 [#4417] feat(build): Add release script to autom
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724747227
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerClientExtend.java:
##
@@ -0,0 +1,184 @@
+/*
+ * Licensed to the Apa
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724752624
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724753165
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724753495
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724753819
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724755880
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,998 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724759506
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724766504
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
jerqi commented on PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#issuecomment-2301631113
> > Does it support multiple Hive catalog uses one Ranger?
>
> Yes, we support. Because each Hive catalog have a separate
AuthorizationPlugin instance.
All the authorization
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724773530
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,1006 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724774008
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,1006 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724774456
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,1006 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724775759
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,1006 @@
+/*
+ * Licensed to
mchades commented on code in PR #4612:
URL: https://github.com/apache/gravitino/pull/4612#discussion_r1724777201
##
.github/workflows/backend-integration-test.yml:
##
@@ -95,10 +95,10 @@ jobs:
run: |
dev/ci/util_free_space.sh
- - name: Backend Integrat
yuqi1129 commented on code in PR #4612:
URL: https://github.com/apache/gravitino/pull/4612#discussion_r1724782471
##
.github/workflows/backend-integration-test.yml:
##
@@ -95,10 +95,10 @@ jobs:
run: |
dev/ci/util_free_space.sh
- - name: Backend Integra
jerqi commented on PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#issuecomment-2301655274
Thanks for your hard work! The code seems good shape.
There are some points to discuss with you.
1. Could you configure the catalog with other authorization plugin instead
of Ranger
xunliu commented on PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#issuecomment-2301657509
> > > Does it support multiple Hive catalog uses one Ranger?
> >
> >
> > Yes, we support. Because each Hive catalog have a separate
AuthorizationPlugin instance.
>
> All
xunliu commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724783782
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache S
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724800112
##
authorizations/authorization-ranger/src/main/resources/META-INF/services/org.apache.gravitino.connector.authorization.AuthorizationProvider:
##
@@ -0,0 +1,19 @@
+#
+
yuqi1129 commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724790047
##
authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerITEnv.java:
##
@@ -311,4 +314,11 @@ protected s
xunliu commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724849823
##
authorizations/authorization-ranger/src/main/resources/META-INF/services/org.apache.gravitino.connector.authorization.AuthorizationProvider:
##
@@ -0,0 +1,19 @@
+#
xloya closed pull request #4281: [#4277] feat(client/server): Add the
`getFileLocation` interface in the Java Client/Server
URL: https://github.com/apache/gravitino/pull/4281
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
xloya opened a new pull request, #4281:
URL: https://github.com/apache/gravitino/pull/4281
### What changes were proposed in this pull request?
Added an interface for obtaining the file location so that the client can
report some necessary information for the server to audit and simpl
xunliu commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724857748
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724863560
##
authorizations/authorization-ranger/src/main/resources/META-INF/services/org.apache.gravitino.connector.authorization.AuthorizationProvider:
##
@@ -0,0 +1,19 @@
+#
+
jerqi commented on PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#issuecomment-2301785058
Only one point:
1. Some privileges like `use schema`, `use catalog`, `create catalog` could
be pushed down underlying system, too. The check logic will throw exception for
them.
FANNG1 commented on code in PR #4598:
URL: https://github.com/apache/gravitino/pull/4598#discussion_r1724857441
##
iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/GravitinoBasedIcebergTableOpsProvider.java:
##
@@ -0,0 +1,91 @@
+/*
+ * Licensed to the
yuqi1129 merged PR #4612:
URL: https://github.com/apache/gravitino/pull/4612
--
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
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 235d54286 [Minor] fix(CI): Fix CI pipeline conf
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724878878
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724879222
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724882507
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
yuqi1129 commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724881433
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724884494
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724887242
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724888182
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
FANNG1 commented on code in PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#discussion_r1724890644
##
core/src/main/java/org/apache/gravitino/audit/AbstractAuditLogWriter.java:
##
@@ -0,0 +1,27 @@
+package org.apache.gravitino.audit;
Review Comment:
please add a
xunliu commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724893759
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
FANNG1 commented on code in PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#discussion_r1724895664
##
core/src/main/java/org/apache/gravitino/audit/AuditLogConfig.java:
##
@@ -0,0 +1,60 @@
+package org.apache.gravitino.audit;
+
+import java.util.Map;
+import org.apa
yuqi1129 opened a new pull request, #4613:
URL: https://github.com/apache/gravitino/pull/4613
### What changes were proposed in this pull request?
Fix the backend type configuration.
### Why are the changes needed?
The configuration for the backend type is incorre
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724898605
##
authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerITEnv.java:
##
@@ -252,7 +257,7 @@ protected stati
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724902782
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
yuqi1129 commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724903813
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724906342
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724906342
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
FANNG1 commented on code in PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#discussion_r1724907611
##
core/src/main/java/org/apache/gravitino/audit/AuditLogManager.java:
##
@@ -0,0 +1,90 @@
+package org.apache.gravitino.audit;
+
+import com.google.common.annotations
FANNG1 commented on code in PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#discussion_r1724910883
##
core/src/main/java/org/apache/gravitino/audit/AuditLogManager.java:
##
@@ -0,0 +1,90 @@
+package org.apache.gravitino.audit;
+
+import com.google.common.annotations
FANNG1 commented on PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#issuecomment-2301857646
@hanwxx , this PR seems mainly about audit frame, could you split separate
sub-tasks from #3973 ?
--
This is an automated message from the Apache Git Service.
To respond to the message
shaofengshi commented on issue #4558:
URL: https://github.com/apache/gravitino/issues/4558#issuecomment-2301864360
> > Hi Peidian, I don't have much knowledge about FUSE; For the solution2,
is that only available for python? Which means, only in Python application,
fsspec mounts a remote st
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724987141
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,539 @@
+/*
+ * Licensed to the Apache So
yuqi1129 commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724993429
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,556 @@
+/*
+ * Licensed
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1725005020
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerAuthorizationPlugin.java:
##
@@ -0,0 +1,583 @@
+/*
+ * Licensed to
jerryshao commented on issue #4586:
URL: https://github.com/apache/gravitino/issues/4586#issuecomment-2302004440
@heziyi399 would you please check again to see if @xloya 's PR really fix
your problem? Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the m
xunliu commented on PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#issuecomment-2302034040
> Only one point:
>
> 1. Some privileges like `use schema`, `use catalog`, `create catalog`
could be pushed down underlying system, too. The check logic will throw
exception for t
xunliu commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1724962022
##
authorizations/authorization-ranger/src/test/java/org/apache/gravitino/authorization/ranger/integration/test/RangerITEnv.java:
##
@@ -252,7 +257,7 @@ protected stat
yuqi1129 merged PR #4613:
URL: https://github.com/apache/gravitino/pull/4613
--
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
This is an automated email from the ASF dual-hosted git repository.
yuqi4733 pushed a commit to branch branch-0.6
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.6 by this push:
new cdf3988bb [Minor] Fix CI pipeline c
jerqi commented on code in PR #4515:
URL: https://github.com/apache/gravitino/pull/4515#discussion_r1725098417
##
authorizations/authorization-ranger/src/main/java/org/apache/gravitino/authorization/ranger/RangerHelper.java:
##
@@ -0,0 +1,533 @@
+/*
+ * Licensed to the Apache So
yuqi1129 commented on code in PR #4611:
URL: https://github.com/apache/gravitino/pull/4611#discussion_r1725105295
##
core/src/main/java/org/apache/gravitino/storage/relational/mapper/CatalogMetaMapper.java:
##
@@ -38,148 +38,168 @@
public interface CatalogMetaMapper {
String
1 - 100 of 132 matches
Mail list logo
