Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-12-01 Thread via GitHub
jerryshao merged PR #5714: URL: https://github.com/apache/gravitino/pull/5714 -- 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

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-30 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1864725205 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +79,19 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

[PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-30 Thread via GitHub
waukin opened a new pull request, #5714: URL: https://github.com/apache/gravitino/pull/5714 ### What changes were proposed in this pull request? Enhance the Gravitino CLI with formatted output. https://github.com/user-attachments/assets/9655c081-1575-44c3-bcce-49ca2b49a2

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2507253470 Reopen the 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 the specific comment. To un

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2507252963 > Hi @xunliu, I pushed a commit and Github shows "Processing updates" in the past 3 hours. Do you know how to fix this problem? ![Screenshot 2024-11-29 14](https://private-user-image

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2507250940 I tried closing and reopening the PR to see if GitHub could back to normal. Ref: https://github.com/orgs/community/discussions/51962 -- This is an automated message from the Apache G

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2507250326 > Hi @xunliu, I pushed a commit and Github shows "Processing updates" in the past 3 hours. Do you know how to fix this problem? ![Screenshot 2024-11-29 14](https://private-user-image

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin closed pull request #5606: [#5506] feat(CLI): Table formatted output URL: https://github.com/apache/gravitino/pull/5606 -- 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. T

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2507176587 Hi @xunliu, I pushed a commit and Github shows "Processing updates" in the past 3 hours. Do you know how to fix this problem? ![Screenshot 2024-11-29 14](https://github.com/user-

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1862711817 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +79,19 @@ protected GravitinoAdminClient buildAdminClient() { ret

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1862258098 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +79,19 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1862258098 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +79,19 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1862245855 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +79,19 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
tengqm commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1862223611 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +79,19 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-28 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1862117539 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/CatalogDetails.java: ## @@ -37,9 +37,11 @@ public class CatalogDetails extends Command { * @param ig

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-27 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1861465557 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/CatalogDetails.java: ## @@ -37,9 +37,11 @@ public class CatalogDetails extends Command { * @param ig

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-27 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1860629621 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +77,17 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-27 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1860631880 ## clients/cli/src/test/java/org/apache/gravitino/cli/integration/test/TableFormatOutputIT.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-26 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1859652820 ## clients/cli/src/test/java/org/apache/gravitino/cli/integration/test/TableFormatOutputIT.java: ## @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-26 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1859275498 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +77,17 @@ protected GravitinoAdminClient buildAdminClient() { ret

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-26 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1859275498 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +77,17 @@ protected GravitinoAdminClient buildAdminClient() { ret

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-26 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1858708524 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +77,17 @@ protected GravitinoAdminClient buildAdminClient() { return Gr

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-25 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1857739848 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java: ## @@ -69,4 +77,17 @@ protected GravitinoAdminClient buildAdminClient() { ret

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-25 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1856499732 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/MetalakeDetails.java: ## @@ -33,27 +34,24 @@ public class MetalakeDetails extends Command { * @param

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-25 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1856497861 ## clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormats.java: ## @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-25 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1856497861 ## clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormats.java: ## @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1855909668 ## clients/cli/src/main/java/org/apache/gravitino/cli/outputs/TableFormats.java: ## @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Software Foundation (ASF) un

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1855905530 ## clients/cli/src/main/java/org/apache/gravitino/cli/outputs/CommaSeparatorFormats.java: ## @@ -0,0 +1,54 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1855892542 ## clients/cli/src/test/java/org/apache/gravitino/cli/TestMetalakeCommands.java: ## @@ -59,7 +59,9 @@ void testListMetalakesCommand() { spy(

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1855895395 ## clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoOptions.java: ## @@ -89,6 +89,7 @@ public Options options() { // Force delete entity and rena

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1855892542 ## clients/cli/src/test/java/org/apache/gravitino/cli/TestMetalakeCommands.java: ## @@ -59,7 +59,9 @@ void testListMetalakesCommand() { spy(

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1855889032 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/MetalakeDetails.java: ## @@ -33,27 +34,24 @@ public class MetalakeDetails extends Command { *

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1855887054 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/MetalakeDetails.java: ## @@ -33,27 +34,24 @@ public class MetalakeDetails extends Command { *

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-24 Thread via GitHub
xunliu commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2496580672 hi @waukin I will help you improve your this PR, Please wait. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-21 Thread via GitHub
tengqm commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1853165264 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-21 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852970659 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-21 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-21 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-21 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-21 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1852220285 ## clients/cli/src/main/java/org/apache/gravitino/cli/CommandLineOutputs.java: ## @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-20 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1847846692 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListMetalakes.java: ## @@ -50,13 +51,12 @@ public void handle() { return; } -List meta

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-20 Thread via GitHub
xunliu commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2485699094 hi @waukin I help you improve this PR, please get the latest code in this PR to continue development, Tanks! -- This is an automated message from the Apache Git Service. To respond

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1849554303 ## clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java: ## @@ -551,4 +551,8 @@ public String getUrl() { // Return the default local

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1849480387 ## clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java: ## @@ -551,4 +551,8 @@ public String getUrl() { // Return the default localhost U

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1849409986 ## clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java: ## @@ -551,4 +551,8 @@ public String getUrl() { // Return the default local

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1849396805 ## clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java: ## @@ -551,4 +551,8 @@ public String getUrl() { // Return the default localhost U

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1849060154 ## clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java: ## @@ -551,4 +551,8 @@ public String getUrl() { // Return the default local

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
justinmclean commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1849058732 ## clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java: ## @@ -93,7 +93,7 @@ public class GravitinoCommandLine { private final Com

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
waukin commented on PR #5606: URL: https://github.com/apache/gravitino/pull/5606#issuecomment-2485811403 Hi @xunliu, thank you for the update! -- 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

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
waukin commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1848424335 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/MetalakeDetails.java: ## @@ -54,6 +58,10 @@ public void handle() { return; } -System.o

Re: [PR] [#5506] feat(CLI): Table formatted output [gravitino]

2024-11-19 Thread via GitHub
xunliu commented on code in PR #5606: URL: https://github.com/apache/gravitino/pull/5606#discussion_r1847846692 ## clients/cli/src/main/java/org/apache/gravitino/cli/commands/ListMetalakes.java: ## @@ -50,13 +51,12 @@ public void handle() { return; } -List meta