eric-maynard commented on code in PR #2643:
URL: https://github.com/apache/polaris/pull/2643#discussion_r2369595821


##########
client/python/cli/polaris_cli.py:
##########
@@ -119,101 +109,6 @@ def _try_print_exception(e):
                 f"Exception when communicating with the Polaris server. 
{e}{os.linesep}"
             )
 
-    @staticmethod

Review Comment:
   Refactoring these out is wise; this file has gotten really big



##########
client/python/cli/polaris_cli.py:
##########
@@ -119,101 +109,6 @@ def _try_print_exception(e):
                 f"Exception when communicating with the Polaris server. 
{e}{os.linesep}"
             )
 
-    @staticmethod

Review Comment:
   Refactoring these out is wise; this file has gotten really big when it 
should be a thin entry point into the CLI



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to