theoryxu commented on code in PR #4937:
URL: https://github.com/apache/gravitino/pull/4937#discussion_r1776194554
##
iceberg/iceberg-rest-server/src/test/java/org/apache/gravitino/iceberg/service/rest/TestIcebergViewOperations.java:
##
@@ -0,0 +1,311 @@
+/*
+ * Licensed to the A
jerryshao commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1776196134
##
build.gradle.kts:
##
@@ -744,7 +744,8 @@ tasks {
if (!it.name.startsWith("catalog") &&
!it.name.startsWith("authorization") &&
!it.name.
theoryxu commented on PR #4937:
URL: https://github.com/apache/gravitino/pull/4937#issuecomment-2375589195
> @theoryxu , thanks for the PR, after supporting views, could Spark create
the view in the Iceberg REST catalog? If yes, could you add corresponding
integration test?
Sure, I t
jerryshao commented on PR #5022:
URL: https://github.com/apache/gravitino/pull/5022#issuecomment-2375597963
@jerqi please review this backport.
--
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
jerryshao commented on issue #5014:
URL: https://github.com/apache/gravitino/issues/5014#issuecomment-2375599812
Hi @haitaoyao will you work on this, or just a wish for the community?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
jerqi commented on PR #4879:
URL: https://github.com/apache/gravitino/pull/4879#issuecomment-2375600467
@LiuQhahah Do you have time to continue this work? Because we want to
release this feature in 0.6.1. The 0.6.1 will be released at the end of this
month.
--
This is an automated messag
xiaozcy commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1776206338
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/DefaultConfigurationProvider.java:
##
@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache Sof
yuanoOo commented on code in PR #5013:
URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776211821
##
catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java:
##
@@ -45,17 +55,62 @@ private sta
LindaSummer opened a new pull request, #5025:
URL: https://github.com/apache/gravitino/pull/5025
### What changes were proposed in this pull request?
Make catalog property `cloud.name` and `cloud.region-code` mutable.
### Why are the changes needed?
As #4999 mentioned, th
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 d65b41c9a [#5010] Add LICENSE and NOTI
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.6-61a69f92504d7ff78de31db632e4abd257616bb3
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was e9c5d73c1 [#5010] Add LICENSE and NOTICE files for Iceberg
jerryshao merged PR #5024:
URL: https://github.com/apache/gravitino/pull/5024
--
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
justinmclean commented on code in PR #5011:
URL: https://github.com/apache/gravitino/pull/5011#discussion_r1776042560
##
NOTICE.rest:
##
@@ -0,0 +1,728 @@
+Apache Gravitino (incubating)
+Copyright 2024 The Apache Software Foundation
+
+This product includes software developed at
jerryshao closed issue #3346: [FEATURE] Need REST API to list all existing
roles in a metalake
URL: https://github.com/apache/gravitino/issues/3346
--
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
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 912a42469 [#3346] feat(core,server): Supports to l
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.6-912a42469583a88e31a777a9357eaae4afe83e76
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at 0dd4f7478 [#3346] feat(core,server): Supports to list
github-actions[bot] opened a new pull request, #5022:
URL: https://github.com/apache/gravitino/pull/5022
### What changes were proposed in this pull request?
Supports to list roles operations
### Why are the changes needed?
Fix: #3346
### Does this PR introduce _any_ u
jerryshao closed pull request #5022: [#3346] feat(core,server): Supports to
list roles operations
URL: https://github.com/apache/gravitino/pull/5022
--
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
FANNG1 commented on code in PR #5006:
URL: https://github.com/apache/gravitino/pull/5006#discussion_r1776297079
##
docs/hive-catalog-with-s3.md:
##
@@ -0,0 +1,206 @@
+---
+title: "Hive catalog with s3"
+slug: /hive-catalog
+date: 2024-9-24
+keyword: Hive catalog cloud storage S3
FANNG1 commented on code in PR #5006:
URL: https://github.com/apache/gravitino/pull/5006#discussion_r1776297958
##
docs/hive-catalog-with-s3.md:
##
@@ -0,0 +1,206 @@
+---
+title: "Hive catalog with s3"
+slug: /hive-catalog
+date: 2024-9-24
+keyword: Hive catalog cloud storage S3
FANNG1 commented on code in PR #5006:
URL: https://github.com/apache/gravitino/pull/5006#discussion_r1776297765
##
docs/hive-catalog-with-s3.md:
##
@@ -0,0 +1,206 @@
+---
+title: "Hive catalog with s3"
+slug: /hive-catalog
+date: 2024-9-24
+keyword: Hive catalog cloud storage S3
FANNG1 commented on code in PR #5006:
URL: https://github.com/apache/gravitino/pull/5006#discussion_r1776300768
##
docs/hive-catalog-with-s3.md:
##
@@ -0,0 +1,206 @@
+---
+title: "Hive catalog with s3"
+slug: /hive-catalog
+date: 2024-9-24
+keyword: Hive catalog cloud storage S3
FANNG1 commented on code in PR #5006:
URL: https://github.com/apache/gravitino/pull/5006#discussion_r1776301369
##
docs/apache-hive-catalog.md:
##
@@ -197,3 +197,7 @@ As Gravitino has a separate interface for updating the
comment of a table, the H
:::note
Support for altering
FANNG1 commented on code in PR #5006:
URL: https://github.com/apache/gravitino/pull/5006#discussion_r1776298285
##
docs/hive-catalog-with-s3.md:
##
@@ -0,0 +1,206 @@
+---
+title: "Hive catalog with s3"
+slug: /hive-catalog
+date: 2024-9-24
+keyword: Hive catalog cloud storage S3
FANNG1 commented on code in PR #5006:
URL: https://github.com/apache/gravitino/pull/5006#discussion_r1776298935
##
docs/hive-catalog-with-s3.md:
##
@@ -0,0 +1,206 @@
+---
+title: "Hive catalog with s3"
+slug: /hive-catalog
+date: 2024-9-24
+keyword: Hive catalog cloud storage S3
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1776319796
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/DefaultConfigurationProvider.java:
##
@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache So
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1776322135
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/HadoopCatalogOperations.java:
##
@@ -132,6 +132,20 @@ public void initialize(
mchades commented on code in PR #5013:
URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776276454
##
catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java:
##
@@ -45,17 +55,62 @@ private sta
jerryshao commented on code in PR #5023:
URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776460186
##
clients/client-java/src/main/java/org/apache/gravitino/client/GravitinoClient.java:
##
@@ -369,6 +369,20 @@ public String[] listRoleNames() throws
NoSuchMetalak
jerryshao commented on PR #4975:
URL: https://github.com/apache/gravitino/pull/4975#issuecomment-2376028475
@xunliu can you please review 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 to go to the
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 b7ceca005 [#4709] improvement(client-python): Add
jerryshao closed issue #4709: [Subtask] Add the implementations for
`getFileLocation` interface in Python Client
URL: https://github.com/apache/gravitino/issues/4709
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
jerryshao merged PR #5017:
URL: https://github.com/apache/gravitino/pull/5017
--
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 #4709: [Subtask] Add the implementations for
`getFileLocation` interface in Python Client
URL: https://github.com/apache/gravitino/issues/4709
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
yuanoOo commented on code in PR #5013:
URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776452196
##
catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java:
##
@@ -45,17 +55,62 @@ private sta
yuqi1129 commented on code in PR #5023:
URL: https://github.com/apache/gravitino/pull/5023#discussion_r1776447171
##
core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java:
##
@@ -93,49 +78,41 @@ public List listRolesByUserId(Long userId) {
}
xiaozcy commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1776466211
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/DefaultConfigurationProvider.java:
##
@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache Sof
yuqi1129 commented on code in PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#discussion_r1776470931
##
catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/DefaultConfigurationProvider.java:
##
@@ -0,0 +1,31 @@
+/*
+ * Licensed to the Apache So
mchades commented on code in PR #5013:
URL: https://github.com/apache/gravitino/pull/5013#discussion_r1776471055
##
catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java:
##
@@ -45,17 +55,62 @@ private sta
yuqi1129 commented on code in PR #4985:
URL: https://github.com/apache/gravitino/pull/4985#discussion_r1776475326
##
api/src/main/java/org/apache/gravitino/rel/expressions/distributions/Strategy.java:
##
@@ -50,7 +50,10 @@ public enum Strategy {
RANGE,
/** Distributes da
jun-ma commented on PR #4997:
URL: https://github.com/apache/gravitino/pull/4997#issuecomment-2375550737
> @jun-ma Please resolve the conflicts if you have free time. Thanks.
Conflicts resolved. Thanks for your reminder. @yuqi1129
--
This is an automated message from the Apache Git
jerryshao merged PR #4894:
URL: https://github.com/apache/gravitino/pull/4894
--
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 merged PR #5022:
URL: https://github.com/apache/gravitino/pull/5022
--
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 merged PR #4997:
URL: https://github.com/apache/gravitino/pull/4997
--
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 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 07dd3a07d [MINOR] improve(doc): Straight-forward i
jerryshao commented on PR #4997:
URL: https://github.com/apache/gravitino/pull/4997#issuecomment-2375629162
Thanks a lot for your contribution, @jun-ma
--
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
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.6-912a42469583a88e31a777a9357eaae4afe83e76
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 0dd4f7478 [#3346] feat(core,server): Supports to list role
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 dd981346e [#3346] feat(core,server): S
jerqi opened a new pull request, #5023:
URL: https://github.com/apache/gravitino/pull/5023
### What changes were proposed in this pull request?
Supports to list roles by object
### Why are the changes needed?
Fix: #4886
### Does this PR introduce _any_ user-facing chan
jerryshao merged PR #5011:
URL: https://github.com/apache/gravitino/pull/5011
--
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 #5010: Iceberg REST service
URL: https://github.com/apache/gravitino/issues/5010
--
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:
jerryshao closed issue #5010: Iceberg REST service
URL: https://github.com/apache/gravitino/issues/5010
--
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:
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 61a69f925 [#5010] Add LICENSE and NOTICE files for
jerryshao closed pull request #5024: [#5010] Add LICENSE and NOTICE files for
Iceberg REST binary
URL: https://github.com/apache/gravitino/pull/5024
--
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
github-actions[bot] opened a new pull request, #5024:
URL: https://github.com/apache/gravitino/pull/5024
### What changes were proposed in this pull request?
Add LICENSE and NOTICE files for Iceberg REST binary
### Why are the changes needed?
To comply with ASF policy.
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.6-61a69f92504d7ff78de31db632e4abd257616bb3
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at e9c5d73c1 [#5010] Add LICENSE and NOTICE files for Ic
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 eaffd524b [#4962] feat(trino-connector): Suppor
yuqi1129 closed issue #4962: [FEATURE] Support the Trino cascading connector
URL: https://github.com/apache/gravitino/issues/4962
--
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
yuqi1129 merged PR #4935:
URL: https://github.com/apache/gravitino/pull/4935
--
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
yuqi1129 closed issue #4962: [FEATURE] Support the Trino cascading connector
URL: https://github.com/apache/gravitino/issues/4962
--
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
xloya opened a new pull request, #5017:
URL: https://github.com/apache/gravitino/pull/5017
### What changes were proposed in this pull request?
Add the implementations for getFileLocation interface in Python Client.
### Why are the changes needed?
Fix: #4709
### H
github-actions[bot] opened a new pull request, #5018:
URL: https://github.com/apache/gravitino/pull/5018
### What changes were proposed in this pull request?
Fix API description issue
### Why are the changes needed?
Fix: # (issue)
### Does this PR introduce _an
yuqi1129 merged PR #5016:
URL: https://github.com/apache/gravitino/pull/5016
--
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
FANNG1 commented on PR #80:
URL:
https://github.com/apache/gravitino-playground/pull/80#issuecomment-2373244759
This PR should be merged before next release, with updated spark connector
and trino connector
--
This is an automated message from the Apache Git Service.
To respond to the me
tyoushinya opened a new pull request, #5016:
URL: https://github.com/apache/gravitino/pull/5016
### What changes were proposed in this pull request?
Fix API description issue
### Why are the changes needed?
Fix: # (issue)
### Does this PR introduce _any_ user-f
xloya commented on PR #5017:
URL: https://github.com/apache/gravitino/pull/5017#issuecomment-2373416928
@jerryshao This PR is ready for review, please take a look when you have
time, thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
This is an automated email from the ASF dual-hosted git repository.
yuqi4733 pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
from 1f9895633 [#5008] improve(core): reorganize the sql provider package
to make it more clear (#5009)
add ed7fef
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
cherry-pick-branch-0.6-ed7fefbc3abb2805c95d78b882707d2a2c4ce61a
in repository https://gitbox.apache.org/repos/asf/gravitino.git
at cde80b1de Fix API description issue (#5016)
No new r
yuqi1129 closed pull request #5018: Fix API description issue
URL: https://github.com/apache/gravitino/pull/5018
--
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,
mchades commented on code in PR #5013:
URL: https://github.com/apache/gravitino/pull/5013#discussion_r1774731239
##
integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/OceanBaseContainer.java:
##
@@ -0,0 +1,160 @@
+/*
+ * Licensed to the Apache
yuqi1129 opened a new issue, #5019:
URL: https://github.com/apache/gravitino/issues/5019
### Describe the feature
Currently, the Hadoop catalog only supports local files and HDFS file
systems. We plan to support more fileset storage like S3, GCS, and so on.
This proposal propo
FANNG1 commented on code in PR #4985:
URL: https://github.com/apache/gravitino/pull/4985#discussion_r1774633847
##
api/src/main/java/org/apache/gravitino/rel/expressions/distributions/Strategy.java:
##
@@ -50,7 +50,10 @@ public enum Strategy {
RANGE,
/** Distributes data
jerryshao commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775314670
##
core/src/main/java/org/apache/gravitino/authorization/RoleManager.java:
##
@@ -129,6 +133,26 @@ boolean deleteRole(String metalake, String role) {
}
}
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775337925
##
common/src/main/java/org/apache/gravitino/dto/authorization/RoleDTO.java:
##
@@ -93,6 +95,10 @@ public Map properties() {
*/
@Override
public List securabl
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775359874
##
core/src/main/java/org/apache/gravitino/authorization/RoleManager.java:
##
@@ -129,6 +133,26 @@ boolean deleteRole(String metalake, String role) {
}
}
+ S
yuqi1129 commented on code in PR #4985:
URL: https://github.com/apache/gravitino/pull/4985#discussion_r1775048398
##
api/src/main/java/org/apache/gravitino/rel/expressions/distributions/Strategy.java:
##
@@ -50,7 +50,10 @@ public enum Strategy {
RANGE,
/** Distributes da
xunliu commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775093805
##
core/src/main/java/org/apache/gravitino/authorization/UserGroupManager.java:
##
@@ -117,26 +114,21 @@ User getUser(String metalake, String user) throws
NoSuchUserE
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775106796
##
core/src/main/java/org/apache/gravitino/storage/relational/service/RoleMetaService.java:
##
@@ -237,6 +239,26 @@ private List
listSecurableObjectsByRoleId(Long role
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775106281
##
core/src/main/java/org/apache/gravitino/EntityStore.java:
##
@@ -66,7 +64,7 @@ public interface EntityStore extends Closeable {
*/
default List list(
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775108552
##
core/src/main/java/org/apache/gravitino/authorization/UserGroupManager.java:
##
@@ -117,26 +114,21 @@ User getUser(String metalake, String user) throws
NoSuchUserEx
FANNG1 commented on code in PR #4985:
URL: https://github.com/apache/gravitino/pull/4985#discussion_r1775012071
##
api/src/main/java/org/apache/gravitino/rel/expressions/distributions/Strategy.java:
##
@@ -50,7 +50,10 @@ public enum Strategy {
RANGE,
/** Distributes data
jerryshao commented on code in PR #5011:
URL: https://github.com/apache/gravitino/pull/5011#discussion_r1775049033
##
NOTICE.rest:
##
@@ -0,0 +1,728 @@
+Apache Gravitino (incubating)
+Copyright 2024 The Apache Software Foundation
+
+This product includes software developed at
+T
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 2c2fd0ab2 Fix API description issue (#
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.6-ed7fefbc3abb2805c95d78b882707d2a2c4ce61a
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was cde80b1de Fix API description issue (#5016)
The revisions
jerryshao merged PR #5018:
URL: https://github.com/apache/gravitino/pull/5018
--
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
yuqi1129 opened a new pull request, #5020:
URL: https://github.com/apache/gravitino/pull/5020
### What changes were proposed in this pull request?
Add a framework to support multiple storage system within Hadoop catalog
### Why are the changes needed?
Some users want Gra
yuqi1129 commented on PR #5020:
URL: https://github.com/apache/gravitino/pull/5020#issuecomment-2373840209
@xiaozcy
Please help to take a look if this PR meets your requirements, thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
yuqi1129 commented on PR #4997:
URL: https://github.com/apache/gravitino/pull/4997#issuecomment-2373843212
@jun-ma
Please resolve the conflicts if you have free time. Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
jerqi opened a new pull request, #5021:
URL: https://github.com/apache/gravitino/pull/5021
### What changes were proposed in this pull request?
Supports to grant or revoke privileges for a role
### Why are the changes needed?
Fix: #4903
### Does this PR introduce
jerryshao commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775206328
##
core/src/main/java/org/apache/gravitino/authorization/RoleManager.java:
##
@@ -129,6 +133,26 @@ boolean deleteRole(String metalake, String role) {
}
}
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775211378
##
common/src/main/java/org/apache/gravitino/dto/authorization/RoleDTO.java:
##
@@ -186,9 +192,6 @@ public S withAudit(AuditDTO audit) {
public RoleDTO build() {
jerryshao commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775211545
##
core/src/main/java/org/apache/gravitino/authorization/RoleManager.java:
##
@@ -129,6 +133,26 @@ boolean deleteRole(String metalake, String role) {
}
}
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775213046
##
core/src/main/java/org/apache/gravitino/authorization/RoleManager.java:
##
@@ -129,6 +133,26 @@ boolean deleteRole(String metalake, String role) {
}
}
+ S
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775220575
##
core/src/main/java/org/apache/gravitino/authorization/RoleManager.java:
##
@@ -129,6 +133,26 @@ boolean deleteRole(String metalake, String role) {
}
}
+ S
jerryshao commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775198116
##
common/src/main/java/org/apache/gravitino/dto/authorization/RoleDTO.java:
##
@@ -186,9 +192,6 @@ public S withAudit(AuditDTO audit) {
public RoleDTO build()
jerryshao commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775196442
##
common/src/main/java/org/apache/gravitino/dto/authorization/RoleDTO.java:
##
@@ -93,6 +95,10 @@ public Map properties() {
*/
@Override
public List secu
jerryshao commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775208152
##
core/src/main/java/org/apache/gravitino/authorization/RoleManager.java:
##
@@ -129,6 +133,26 @@ boolean deleteRole(String metalake, String role) {
}
}
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775211378
##
common/src/main/java/org/apache/gravitino/dto/authorization/RoleDTO.java:
##
@@ -186,9 +192,6 @@ public S withAudit(AuditDTO audit) {
public RoleDTO build() {
jerqi commented on code in PR #4894:
URL: https://github.com/apache/gravitino/pull/4894#discussion_r1775228297
##
common/src/main/java/org/apache/gravitino/dto/responses/RoleResponse.java:
##
@@ -64,8 +64,5 @@ public void validate() throws IllegalArgumentException {
Precond
jerqi commented on PR #5021:
URL: https://github.com/apache/gravitino/pull/5021#issuecomment-2373939785
@lw-yang Could you help me review this pull request?
--
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
100 matches
Mail list logo
