Re: [PR] [#6326] Refactor to add a command context to CLI [gravitino]

2025-02-07 Thread via GitHub
shaofengshi merged PR #6343: URL: https://github.com/apache/gravitino/pull/6343 -- 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...@grav

Re: [PR] [#6326] Refactor to add a command context to CLI [gravitino]

2025-02-07 Thread via GitHub
shaofengshi commented on PR #6343: URL: https://github.com/apache/gravitino/pull/6343#issuecomment-2642443391 Once the CI passed, it can be merged I think. -- 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

Re: [PR] [#6326] Refactor to add a command context to CLI [gravitino]

2025-02-07 Thread via GitHub
shaofengshi commented on code in PR #6343: URL: https://github.com/apache/gravitino/pull/6343#discussion_r1946249403 ## clients/cli/src/main/java/org/apache/gravitino/cli/MetalakeCommandHandler.java: ## @@ -40,15 +39,18 @@ public class MetalakeCommandHandler extends CommandHandl

Re: [PR] [#6326] Refactor to add a command context to CLI [gravitino]

2025-02-06 Thread via GitHub
justinmclean commented on code in PR #6343: URL: https://github.com/apache/gravitino/pull/6343#discussion_r1945581200 ## clients/cli/src/main/java/org/apache/gravitino/cli/MetalakeCommandHandler.java: ## @@ -40,15 +39,18 @@ public class MetalakeCommandHandler extends CommandHand

Re: [PR] [#6326] Refactor to add a command context to CLI [gravitino]

2025-02-06 Thread via GitHub
shaofengshi commented on code in PR #6343: URL: https://github.com/apache/gravitino/pull/6343#discussion_r1944389104 ## clients/cli/src/main/java/org/apache/gravitino/cli/MetalakeCommandHandler.java: ## @@ -40,15 +39,18 @@ public class MetalakeCommandHandler extends CommandHandl