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


##########
client/python/cli/options/parser.py:
##########
@@ -57,22 +57,24 @@ class Parser(object):
         ),
         Argument(Arguments.PROFILE, str, hint="profile for token-based 
authentication"),
         Argument(Arguments.PROXY, str, hint="proxy URL"),
+        Argument(Arguments.DEBUG, bool, hint="Enable debug mode"),

Review Comment:
   Yeah, just show the request. This isn't a blocker, I'm just wondering if it 
might be useful because 
   1. dry-run would be very useful for testing
   2. the current mode ("debug"?) is a little fragile because it 
non-transactionally does the printing and executes the command



-- 
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: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to