coolderli commented on PR #4819:
URL: https://github.com/apache/gravitino/pull/4819#issuecomment-2413052497
@FANNG1 Hi, here are the dependency jars. Please take a look.
```
total 16M
-rw-rw-r-- 1 mi mi 41K 10月 15 14:57
gravitino-flink-1.18_2.12-0.7.0-incubating-SNAPSHOT.jar
-rw
FANNG1 commented on code in PR #5092:
URL: https://github.com/apache/gravitino/pull/5092#discussion_r1800561650
##
docs/lakehouse-paimon-catalog.md:
##
@@ -22,38 +22,40 @@ Builds with Apache Paimon `0.8.0`.
### Catalog capabilities
-- Works as a catalog proxy, supporting `F
caican00 commented on code in PR #5092:
URL: https://github.com/apache/gravitino/pull/5092#discussion_r1800558751
##
catalogs/catalog-lakehouse-paimon/build.gradle.kts:
##
@@ -46,7 +46,6 @@ dependencies {
exclude("com.sun.jersey")
exclude("javax.servlet")
exclude(
caican00 commented on PR #5092:
URL: https://github.com/apache/gravitino/pull/5092#issuecomment-2413044328
> Could you add related document in this PR?
updated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
FANNG1 commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800543905
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/**
-
FANNG1 commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800543570
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/**
-
FANNG1 commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800542923
##
core/src/main/java/org/apache/gravitino/listener/EventListenerPluginWrapper.java:
##
@@ -68,13 +70,40 @@ public void onPostEvent(Event event) {
userEventList
coolderli commented on PR #4819:
URL: https://github.com/apache/gravitino/pull/4819#issuecomment-241247
@FANNG1
I can't find a way to print the dependent jar in the connector jar. Here are
the dependencies.
```
Pro
jerqi commented on issue #5132:
URL: https://github.com/apache/gravitino/issues/5132#issuecomment-2412987912
Playground uses 0.6.0. We supports `list roles` in 0.6.1 and 0.7.0. It's
expected behaviour.
--
This is an automated message from the Apache Git Service.
To respond to the message,
Author: mchades
Date: Tue Oct 15 06:14:18 2024
New Revision: 72329
Log:
Update KEYS
Modified:
release/incubator/gravitino/KEYS
Modified: release/incubator/gravitino/KEYS
==
--- release/incubator/gravitino/KEYS (origi
jerqi commented on issue #5132:
URL: https://github.com/apache/gravitino/issues/5132#issuecomment-2412977078
What's your Gravitino client 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
FANNG1 commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800418912
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/**
-
hanwxx commented on PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#issuecomment-2412850572
@jerryshao Hi, adjusted based on review comments, please see again.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
FANNG1 commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800417697
##
core/src/main/java/org/apache/gravitino/listener/EventBus.java:
##
@@ -64,7 +73,15 @@ public void dispatchEvent(Event event) {
* EventBus.
*/
@Visibl
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800416046
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(Fi
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800414396
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/*
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800413669
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/*
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800413212
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(Fi
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800412895
##
core/src/main/java/org/apache/gravitino/listener/EventListenerPluginWrapper.java:
##
@@ -68,13 +70,40 @@ public void onPostEvent(Event event) {
userEventL
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800411737
##
core/src/main/java/org/apache/gravitino/listener/EventBus.java:
##
@@ -64,7 +73,15 @@ public void dispatchEvent(Event event) {
* EventBus.
*/
@Vis
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800408299
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/*
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800408048
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/*
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800405397
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(Fi
FANNG1 opened a new pull request, #5134:
URL: https://github.com/apache/gravitino/pull/5134
### 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 c
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800402748
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/fs/LocalFileSystemProvider.java:
##
@@ -0,0 +1,45 @@
+/*
+ * Licensed to the Apache Sof
diqiu50 commented on code in PR #4996:
URL: https://github.com/apache/gravitino/pull/4996#discussion_r1800401817
##
trino-connector/integration-test/src/test/java/org/apache/gravitino/trino/connector/integration/test/TrinoQueryITBase.java:
##
@@ -68,14 +68,17 @@ public class Tri
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800402350
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/fs/HDFSFileSystemProvider.java:
##
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Soft
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800399496
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(F
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800399496
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(F
diqiu50 commented on code in PR #4996:
URL: https://github.com/apache/gravitino/pull/4996#discussion_r1800395623
##
trino-connector/integration-test/src/test/java/org/apache/gravitino/trino/connector/integration/test/TrinoQueryITBase.java:
##
@@ -68,14 +68,17 @@ public class Tri
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800393954
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(F
FANNG1 commented on PR #4819:
URL: https://github.com/apache/gravitino/pull/4819#issuecomment-2412808204
Could you update the PR title to add the shrink info like `shrink from xxM
to xxM`? and paste the related dependence jars in the comment like
https://github.com/apache/gravitino/pull/45
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800389410
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(F
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800388253
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(Fi
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800384620
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(F
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800383635
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +742,44 @@ private boolean checkSingleFile(F
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800383266
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -72,10 +71,11 @@
public class HadoopCatalogOperat
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800379555
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -119,26 +123,22 @@ public void initialize(
Ma
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800377602
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -72,10 +71,11 @@
public class HadoopCatalogOperat
FANNG1 commented on code in PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#discussion_r1800341207
##
core/src/main/java/org/apache/gravitino/Configs.java:
##
@@ -342,4 +342,31 @@ private Configs() {}
.stringConf()
.toSequence()
.cre
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800373780
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -119,26 +123,22 @@ public void initialize(
M
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1800368627
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -72,10 +71,11 @@
public class HadoopCatalogOpera
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 53bd22713 [#4992] support credential vending frame
jerryshao closed issue #4992: [Subtask] support credential vending framework
URL: https://github.com/apache/gravitino/issues/4992
--
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.
jerryshao merged PR #4995:
URL: https://github.com/apache/gravitino/pull/4995
--
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
hanwxx commented on PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#issuecomment-2412737315
> > > I kind of agree with @hanwxx , AuditLog and event are two things, we
don't have to use interface to put them together. Using a converter to bridge
them together without hard combin
jerryshao commented on code in PR #5078:
URL: https://github.com/apache/gravitino/pull/5078#discussion_r1800351958
##
core/src/main/java/org/apache/gravitino/storage/relational/service/TableColumnMetaService.java:
##
@@ -0,0 +1,130 @@
+/*
+ * Licensed to the Apache Software Foun
jerqi closed issue #5129: [Subtask] Create a role, delete a role and recreate a
role, it will throw an exception
URL: https://github.com/apache/gravitino/issues/5129
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
jerqi commented on issue #5129:
URL: https://github.com/apache/gravitino/issues/5129#issuecomment-2412701098
Because I modify the policy parameter.
--
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 t
coolderli commented on PR #4819:
URL: https://github.com/apache/gravitino/pull/4819#issuecomment-2412696615
@FANNG1 @mchades Please help review this, Thanks.
--
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
FANNG1 commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800332098
##
core/src/main/java/org/apache/gravitino/listener/EventBus.java:
##
@@ -35,25 +37,45 @@ public class EventBus {
// which are meant for synchronous event listening
FANNG1 commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1800331960
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/**
-
coolderli closed pull request #4819: [MINOR] Shrink the flink-connector-runtime
jar
URL: https://github.com/apache/gravitino/pull/4819
--
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
shaofengshi commented on code in PR #5058:
URL: https://github.com/apache/gravitino/pull/5058#discussion_r1800311770
##
gradle/libs.versions.toml:
##
@@ -90,6 +90,7 @@ shadow-plugin = "8.1.1"
kerby = "2.0.3"
node-plugin = "7.0.1"
commons-cli = "1.2"
+commons-cli-new = "1.9.0"
FANNG1 commented on PR #4575:
URL: https://github.com/apache/gravitino/pull/4575#issuecomment-2412653833
> > I kind of agree with @hanwxx , AuditLog and event are two things, we
don't have to use interface to put them together. Using a converter to bridge
them together without hard combinat
yuqi1129 commented on code in PR #5078:
URL: https://github.com/apache/gravitino/pull/5078#discussion_r1800296210
##
core/src/main/java/org/apache/gravitino/storage/relational/service/TableColumnMetaService.java:
##
@@ -0,0 +1,130 @@
+/*
+ * Licensed to the Apache Software Found
FANNG1 commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1800306465
##
common/src/main/java/org/apache/gravitino/credential/Credential.java:
##
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * o
FANNG1 commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1800302418
##
api/src/main/java/org/apache/gravitino/credential/CredentialConstants.java:
##
@@ -0,0 +1,30 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
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 e46888191 [#4605] Handle NPE in Doris table ope
yuqi1129 merged PR #5125:
URL: https://github.com/apache/gravitino/pull/5125
--
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
justinmclean commented on PR #5133:
URL: https://github.com/apache/gravitino/pull/5133#issuecomment-2412496423
I am waiting on #5057 and #5059 to be merged, so those PRs changes are
included.
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
justinmclean opened a new pull request, #5133:
URL: https://github.com/apache/gravitino/pull/5133
### What changes were proposed in this pull request?
Added basic user and group commands.
### Why are the changes needed?
For the CLI to support users and groups.
justinmclean commented on PR #5117:
URL: https://github.com/apache/gravitino/pull/5117#issuecomment-2411967789
@shaofengshi @diqiu50 @yuqi1129 or @mchades if you could review this that
would be appreciated. Note that this also includes changes from #5057, as that
has not been merged yet.
justinmclean commented on issue #5057:
URL: https://github.com/apache/gravitino/issues/5057#issuecomment-2411965298
@shaofengshi @diqiu50 @yuqi1129 or @mchades if you could review this that
would be appreciated
--
This is an automated message from the Apache Git Service.
To respond to the
justinmclean opened a new issue, #5132:
URL: https://github.com/apache/gravitino/issues/5132
### Version
main branch
### Describe what's wrong
Try to display roles and this error occurs:
```Unable to process: Error code: 405, error message: No content to map due
to en
justinmclean commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799749768
##
common/src/main/java/org/apache/gravitino/credential/Credential.java:
##
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
justinmclean commented on code in PR #5123:
URL: https://github.com/apache/gravitino/pull/5123#discussion_r1799742974
##
docs/security/access-control.md:
##
@@ -90,6 +90,18 @@ The owner of each entity has implicit administrative class
privilege, for exampl
Only the Owner of a
justinmclean commented on code in PR #5058:
URL: https://github.com/apache/gravitino/pull/5058#discussion_r1799727928
##
clients/cli/build.gradle.kts:
##
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreement
justinmclean commented on code in PR #5058:
URL: https://github.com/apache/gravitino/pull/5058#discussion_r1799728863
##
clients/cli/src/main/java/org/apache/gravitino/cli/FullName.java:
##
@@ -0,0 +1,123 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
justinmclean commented on code in PR #5058:
URL: https://github.com/apache/gravitino/pull/5058#discussion_r1799727928
##
clients/cli/build.gradle.kts:
##
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreement
justinmclean commented on code in PR #5058:
URL: https://github.com/apache/gravitino/pull/5058#discussion_r1799726223
##
gradle/libs.versions.toml:
##
@@ -90,6 +90,7 @@ shadow-plugin = "8.1.1"
kerby = "2.0.3"
node-plugin = "7.0.1"
commons-cli = "1.2"
+commons-cli-new = "1.9.0
jerryshao commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799605313
##
api/src/main/java/org/apache/gravitino/credential/CredentialConstants.java:
##
@@ -0,0 +1,30 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
yuqi1129 commented on code in PR #4996:
URL: https://github.com/apache/gravitino/pull/4996#discussion_r1799603877
##
trino-connector/integration-test/src/test/java/org/apache/gravitino/trino/connector/integration/test/TrinoQueryITBase.java:
##
@@ -68,14 +68,17 @@ public class Tr
jerryshao commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799593989
##
api/src/main/java/org/apache/gravitino/credential/CredentialConstants.java:
##
@@ -0,0 +1,30 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under o
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1799588401
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +747,64 @@ private boolean checkSingleFile(Fi
FANNG1 commented on PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#issuecomment-2411331828
@jerryshao please help to review again
--
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
jerqi commented on issue #5115:
URL: https://github.com/apache/gravitino/issues/5115#issuecomment-2411234275
[ ] #5129
[ ] #5130
--
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
jerqi opened a new issue, #5130:
URL: https://github.com/apache/gravitino/issues/5130
### Describe the subtask
Fail to delete the role for the authorization plugin
```
024-10-14 12:57:37.589 WARN [Gravitino-webserver-41]
[org.apache.gravitino.authorization.ranger.RangerAutho
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1799445333
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -742,4 +747,64 @@ private boolean checkSingleFile(Fi
jerryshao closed issue #5120: [Improvement] Add the document of metadata object
ownership
URL: https://github.com/apache/gravitino/issues/5120
--
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
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 327e87632 [#5120] improvement(docs): Add the docum
jerryshao closed issue #5120: [Improvement] Add the document of metadata object
ownership
URL: https://github.com/apache/gravitino/issues/5120
--
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
jerryshao merged PR #5123:
URL: https://github.com/apache/gravitino/pull/5123
--
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
jerqi opened a new issue, #5129:
URL: https://github.com/apache/gravitino/issues/5129
### Describe the subtask
```
{"code":1002,"type":"RuntimeException","message":"Failed to operate object
[role1] operation [CREATE] under [metalake_demo], reason [Ranger API
org.apache.ranger.Rang
FANNG1 commented on PR #5092:
URL: https://github.com/apache/gravitino/pull/5092#issuecomment-2411036224
Could you add related document in 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
FANNG1 commented on code in PR #5092:
URL: https://github.com/apache/gravitino/pull/5092#discussion_r1799371275
##
catalogs/catalog-lakehouse-paimon/build.gradle.kts:
##
@@ -46,7 +46,6 @@ dependencies {
exclude("com.sun.jersey")
exclude("javax.servlet")
exclude("o
FANNG1 commented on code in PR #5092:
URL: https://github.com/apache/gravitino/pull/5092#discussion_r1799366005
##
catalogs/catalog-lakehouse-paimon/src/test/java/org/apache/gravitino/catalog/lakehouse/paimon/integration/test/CatalogPaimonBaseIT.java:
##
@@ -712,11 +727,11 @@ pu
jerqi commented on code in PR #5123:
URL: https://github.com/apache/gravitino/pull/5123#discussion_r1799279549
##
docs/security/access-control.md:
##
@@ -90,6 +90,18 @@ The owner of each entity has implicit administrative class
privilege, for exampl
Only the Owner of a securab
xunliu commented on issue #5126:
URL: https://github.com/apache/gravitino/issues/5126#issuecomment-2410871661
This issue is related to CI environment, not the code bug, we can fix it in
the future.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
jerryshao commented on issue #5126:
URL: https://github.com/apache/gravitino/issues/5126#issuecomment-2410861012
@xunliu @jerqi please help to check this.
--
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
FANNG1 commented on PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#issuecomment-2410737834
@jerryshao comments are addressed, please help to review again
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
FANNG1 commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799165801
##
core/src/main/java/org/apache/gravitino/credential/LocationContext.java:
##
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
FANNG1 commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799165166
##
common/src/main/java/org/apache/gravitino/credential/Credential.java:
##
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * o
JunpingDu commented on issue #5108:
URL: https://github.com/apache/gravitino/issues/5108#issuecomment-2410729109
It may not be enough - the users who click docs on website may not aware
which version they are current using/deploying, so go directly to latest
version may confuse them.
Als
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1799136039
##
core/src/main/java/org/apache/gravitino/listener/EventBus.java:
##
@@ -35,25 +37,45 @@ public class EventBus {
// which are meant for synchronous event listen
jerryshao commented on code in PR #5110:
URL: https://github.com/apache/gravitino/pull/5110#discussion_r1799133736
##
core/src/main/java/org/apache/gravitino/listener/api/EventListenerPlugin.java:
##
@@ -95,17 +97,29 @@ enum Mode {
void stop() throws RuntimeException;
/*
shaofengshi commented on code in PR #5058:
URL: https://github.com/apache/gravitino/pull/5058#discussion_r1799121901
##
gradle/libs.versions.toml:
##
@@ -90,6 +90,7 @@ shadow-plugin = "8.1.1"
kerby = "2.0.3"
node-plugin = "7.0.1"
commons-cli = "1.2"
+commons-cli-new = "1.9.0"
FANNG1 commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799120113
##
common/src/main/java/org/apache/gravitino/credential/Credential.java:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * o
jerryshao commented on code in PR #5123:
URL: https://github.com/apache/gravitino/pull/5123#discussion_r1799110412
##
docs/security/access-control.md:
##
@@ -90,6 +90,18 @@ The owner of each entity has implicit administrative class
privilege, for exampl
Only the Owner of a sec
jerryshao commented on code in PR #4995:
URL: https://github.com/apache/gravitino/pull/4995#discussion_r1799104878
##
common/src/main/java/org/apache/gravitino/credential/Credential.java:
##
@@ -0,0 +1,62 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
1 - 100 of 119 matches
Mail list logo
