Abyss-lord commented on PR #6428:
URL: https://github.com/apache/gravitino/pull/6428#issuecomment-2647216693
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 t
Abyss-lord opened a new pull request, #6428:
URL: https://github.com/apache/gravitino/pull/6428
### What changes were proposed in this pull request?
Add table command context CLI.
### Why are the changes needed?
Fix: #6419
### Does this PR introduce _any_ user-fac
diqiu50 commented on code in PR #6405:
URL: https://github.com/apache/gravitino/pull/6405#discussion_r1948645629
##
clients/filesystem-fuse/README.md:
##
@@ -0,0 +1,95 @@
+
+
+# Gvfs-fuse
+
+Gvfs-fuse is the Fuse client for Gravitino fileset. It allows users to mount
Gravitino
diqiu50 commented on PR #6405:
URL: https://github.com/apache/gravitino/pull/6405#issuecomment-2647341836
> By the way, where do you plan to place this page?
I think placing it in a submodule is sufficient. It is mainly for helping
developers with building and testing.
--
This is a
mchades opened a new issue, #6429:
URL: https://github.com/apache/gravitino/issues/6429
### Version
main branch
### Describe what's wrong
There are some wrong log format codes, like
https://github.com/apache/gravitino/blob/708b61055c196c2d7042b452d41de3f97092024f/core
Abyss-lord opened a new pull request, #6430:
URL: https://github.com/apache/gravitino/pull/6430
### What changes were proposed in this pull request?
Add model command context CLI
### Why are the changes needed?
Fix: #6417
### Does this PR introduce _any_ user-faci
Abyss-lord commented on PR #6430:
URL: https://github.com/apache/gravitino/pull/6430#issuecomment-2647383089
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
Abyss-lord commented on issue #6429:
URL: https://github.com/apache/gravitino/issues/6429#issuecomment-2647434080
I would like to work on 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 s
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch branch-0.8
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-0.8 by this push:
new 1c4750b5a9 [#6409] fix(client-python):
This is an automated email from the ASF dual-hosted git repository.
jshao pushed a change to branch
cherry-pick-branch-0.8-708b61055c196c2d7042b452d41de3f97092024f
in repository https://gitbox.apache.org/repos/asf/gravitino.git
was 9c3ec132be [#6409] fix(client-python): Change to use `Gene
jerryshao merged PR #6426:
URL: https://github.com/apache/gravitino/pull/6426
--
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
Pranaykarvi commented on PR #6400:
URL: https://github.com/apache/gravitino/pull/6400#issuecomment-2647499292
> plz make the CI pass. see:
https://github.com/apache/gravitino/actions/runs/13173847662/job/36768887540?pr=6400
@mchades CI has been passed
--
This is an automated messa
Pranaykarvi commented on PR #6400:
URL: https://github.com/apache/gravitino/pull/6400#issuecomment-2647500070
> plz make the CI pass. see:
https://github.com/apache/gravitino/actions/runs/13173847662/job/36768887540?pr=6400
@mchades CI has been passed
--
This is an automated m
Abyss-lord commented on PR #6431:
URL: https://github.com/apache/gravitino/pull/6431#issuecomment-2647511322
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
yuqi1129 commented on code in PR #6432:
URL: https://github.com/apache/gravitino/pull/6432#discussion_r1949012530
##
integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/MySQLContainer.java:
##
@@ -119,6 +119,14 @@ public void createDatabase(Test
FANNG1 opened a new pull request, #6432:
URL: https://github.com/apache/gravitino/pull/6432
### 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
FANNG1 commented on PR #6432:
URL: https://github.com/apache/gravitino/pull/6432#issuecomment-2647888622
@jerryshao @shaofengshi PTAL.
--
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
aryanmahawar205 opened a new pull request, #6433:
URL: https://github.com/apache/gravitino/pull/6433
### What changes were proposed in this pull request?
The PR updates incorrect log format codes from `%s` to `{}` in the following
lines:
- Line 420
- Line 483
- Line 504
Abyss-lord commented on PR #6434:
URL: https://github.com/apache/gravitino/pull/6434#issuecomment-2649679258
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
This is an automated email from the ASF dual-hosted git repository.
jmclean 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 0dc49caee0 [#5932] improve(CLI): Fix the CLI de
justinmclean closed issue #5932: [Improvement] In the Gravitino CLI/Java API
you can delete the anoymous user
URL: https://github.com/apache/gravitino/issues/5932
--
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
justinmclean closed issue #5932: [Improvement] In the Gravitino CLI/Java API
you can delete the anoymous user
URL: https://github.com/apache/gravitino/issues/5932
--
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
Abyss-lord commented on PR #6435:
URL: https://github.com/apache/gravitino/pull/6435#issuecomment-2649710709
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
justinmclean closed issue #6415: Add Fileset command context CLI
URL: https://github.com/apache/gravitino/issues/6415
--
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 unsubsc
justinmclean closed issue #6417: Add model command context CLI
URL: https://github.com/apache/gravitino/issues/6417
--
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 unsubscri
justinmclean closed issue #6416: Add schema command context CLI
URL: https://github.com/apache/gravitino/issues/6416
--
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 unsubscr
Abyss-lord commented on PR #6425:
URL: https://github.com/apache/gravitino/pull/6425#issuecomment-2649655796
Hi @justinmclean , I’ve finished updating the code. Please take a look at
the PR again when you have time.
--
This is an automated message from the Apache Git Service.
To respond t
FANNG1 commented on code in PR #6362:
URL: https://github.com/apache/gravitino/pull/6362#discussion_r1950147563
##
flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/catalog/BaseCatalog.java:
##
@@ -554,28 +554,31 @@ protected CatalogBaseTable toFlinkTable(
This is an automated email from the ASF dual-hosted git repository.
jmclean 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 d883068551 [#6419] improve(CLI): Add table comma
justinmclean merged PR #6428:
URL: https://github.com/apache/gravitino/pull/6428
--
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...@gra
justinmclean closed issue #6419: Add table command context CLI
URL: https://github.com/apache/gravitino/issues/6419
--
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 unsubscri
justinmclean closed issue #6419: Add table command context CLI
URL: https://github.com/apache/gravitino/issues/6419
--
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 unsubscri
This is an automated email from the ASF dual-hosted git repository.
jmclean 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 c24f1d60e9 [#6417] improve(CLI): Add model comma
This is an automated email from the ASF dual-hosted git repository.
jmclean pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
from d883068551 [#6419] improve(CLI): Add table command context CLI (#6428)
add c8ef9b8ea8 [#6415] improve(CLI): Add
FANNG1 commented on code in PR #6432:
URL: https://github.com/apache/gravitino/pull/6432#discussion_r1950075622
##
integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/MySQLContainer.java:
##
@@ -119,6 +119,14 @@ public void createDatabase(TestDa
justinmclean merged PR #6431:
URL: https://github.com/apache/gravitino/pull/6431
--
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...@gra
justinmclean closed issue #6415: Add Fileset command context CLI
URL: https://github.com/apache/gravitino/issues/6415
--
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 unsubsc
justinmclean merged PR #6430:
URL: https://github.com/apache/gravitino/pull/6430
--
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...@gra
This is an automated email from the ASF dual-hosted git repository.
jmclean pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
from c24f1d60e9 [#6417] improve(CLI): Add model command context CLI (#6430)
add 086278e8f9 [#6416] improve(CLI): Add
justinmclean merged PR #6427:
URL: https://github.com/apache/gravitino/pull/6427
--
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...@gra
FANNG1 commented on code in PR #6405:
URL: https://github.com/apache/gravitino/pull/6405#discussion_r1950077020
##
clients/filesystem-fuse/README.md:
##
@@ -0,0 +1,95 @@
+
+
+# Gvfs-fuse
+
+Gvfs-fuse is the Fuse client for Gravitino fileset. It allows users to mount
Gravitino f
justinmclean commented on code in PR #6425:
URL: https://github.com/apache/gravitino/pull/6425#discussion_r1949971366
##
clients/cli/src/test/java/org/apache/gravitino/cli/TestCatalogCommands.java:
##
@@ -253,24 +246,22 @@ void testSetCatalogPropertyCommand() {
doReturn(moc
justinmclean commented on code in PR #6425:
URL: https://github.com/apache/gravitino/pull/6425#discussion_r1949971366
##
clients/cli/src/test/java/org/apache/gravitino/cli/TestCatalogCommands.java:
##
@@ -253,24 +246,22 @@ void testSetCatalogPropertyCommand() {
doReturn(moc
justinmclean commented on code in PR #6425:
URL: https://github.com/apache/gravitino/pull/6425#discussion_r1949971366
##
clients/cli/src/test/java/org/apache/gravitino/cli/TestCatalogCommands.java:
##
@@ -253,24 +246,22 @@ void testSetCatalogPropertyCommand() {
doReturn(moc
justinmclean merged PR #6359:
URL: https://github.com/apache/gravitino/pull/6359
--
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...@gra
lsyulong commented on issue #6406:
URL: https://github.com/apache/gravitino/issues/6406#issuecomment-2649727612
[https://spark.apache.org/error-message-guidelines.html](url)
--
This is an automated me
Abyss-lord opened a new pull request, #6437:
URL: https://github.com/apache/gravitino/pull/6437
### What changes were proposed in this pull request?
Add user and group command context CLI
### Why are the changes needed?
Fix: #6421
### Does this PR introduce _any_
Abyss-lord commented on PR #6437:
URL: https://github.com/apache/gravitino/pull/6437#issuecomment-2649958989
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
Abyss-lord commented on PR #6435:
URL: https://github.com/apache/gravitino/pull/6435#issuecomment-2649965967
@justinmclean I’ve finished updating the code. Please take a look at the PR
again when you have time.
--
This is an automated message from the Apache Git Service.
To respond to the
lsyulong commented on issue #6406:
URL: https://github.com/apache/gravitino/issues/6406#issuecomment-2649732462
[https://spark.apache.org/error-message-guidelines.html](url)
Spark reads Iceberg using
justinmclean commented on code in PR #6435:
URL: https://github.com/apache/gravitino/pull/6435#discussion_r1950194591
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/TagEntity.java:
##
@@ -130,7 +129,7 @@ public void handle() {
String all = tagsToAdd.length
justinmclean commented on code in PR #6435:
URL: https://github.com/apache/gravitino/pull/6435#discussion_r1950194762
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/UntagEntity.java:
##
@@ -132,10 +131,9 @@ public void handle() {
}
if (tags.length > 1
justinmclean commented on code in PR #6435:
URL: https://github.com/apache/gravitino/pull/6435#discussion_r1950194343
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/RemoveAllTags.java:
##
@@ -152,10 +150,9 @@ public void handle() {
}
if (tags.length >
Abyss-lord opened a new pull request, #6436:
URL: https://github.com/apache/gravitino/pull/6436
### What changes were proposed in this pull request?
Add column command context CLI
### Why are the changes needed?
Fix: #6420
### Does this PR introduce _any_ user-fac
justinmclean commented on code in PR #6435:
URL: https://github.com/apache/gravitino/pull/6435#discussion_r1950194405
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/RemoveAllTags.java:
##
@@ -152,10 +150,9 @@ public void handle() {
}
if (tags.length >
Abyss-lord commented on PR #6436:
URL: https://github.com/apache/gravitino/pull/6436#issuecomment-2649739660
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
justinmclean commented on code in PR #6437:
URL: https://github.com/apache/gravitino/pull/6437#discussion_r1950327735
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/GroupDetails.java:
##
@@ -62,6 +62,6 @@ public void handle() {
String all = roles.isEmpty()
justinmclean commented on code in PR #6437:
URL: https://github.com/apache/gravitino/pull/6437#discussion_r1950328141
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListUsers.java:
##
@@ -55,6 +55,6 @@ public void handle() {
String all = users.length == 0
justinmclean commented on code in PR #6437:
URL: https://github.com/apache/gravitino/pull/6437#discussion_r1950327886
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListGroups.java:
##
@@ -55,6 +55,6 @@ public void handle() {
String all = groups.length ==
justinmclean commented on code in PR #6435:
URL: https://github.com/apache/gravitino/pull/6435#discussion_r1950331236
##
clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListAllTags.java:
##
@@ -55,6 +55,6 @@ public void handle() {
String all = tags.length == 0
justinmclean merged PR #6425:
URL: https://github.com/apache/gravitino/pull/6425
--
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...@gra
This is an automated email from the ASF dual-hosted git repository.
jmclean 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 918fc49b77 [#6420] improve(CLI): Add column comm
justinmclean merged PR #6436:
URL: https://github.com/apache/gravitino/pull/6436
--
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...@gra
justinmclean closed issue #6420: Add column command context CLI
URL: https://github.com/apache/gravitino/issues/6420
--
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 unsubscr
This is an automated email from the ASF dual-hosted git repository.
jmclean 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 82087ef6ec [#6418] improve(CLI): Add stream comm
This is an automated email from the ASF dual-hosted git repository.
jmclean 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 1e85c3d333 [#6414] improve(CLI): Add catalog com
justinmclean closed issue #6418: Add stream command context CLI
URL: https://github.com/apache/gravitino/issues/6418
--
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 unsubscr
justinmclean merged PR #6434:
URL: https://github.com/apache/gravitino/pull/6434
--
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...@gra
justinmclean closed issue #6414: Add catalog command context CLI
URL: https://github.com/apache/gravitino/issues/6414
--
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 unsubsc
justinmclean closed issue #6414: Add catalog command context CLI
URL: https://github.com/apache/gravitino/issues/6414
--
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 unsubsc
hdygxsj commented on code in PR #6362:
URL: https://github.com/apache/gravitino/pull/6362#discussion_r1950099772
##
flink-connector/flink/src/main/java/org/apache/gravitino/flink/connector/catalog/BaseCatalog.java:
##
@@ -521,12 +554,30 @@ protected CatalogBaseTable toFlinkTable
71 matches
Mail list logo