[GitHub] [beam] tvalentyn commented on a diff in pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

[GitBox <gi...@apache.org>]

tvalentyn commented on code in PR #22077:
URL: https://github.com/apache/beam/pull/22077#discussion_r909669404


##########
sdks/python/apache_beam/options/pipeline_options.py:
##########
@@ -971,6 +971,20 @@ def _add_argparse_args(cls, parser):
             'override. Values can be either a labeled level or a number '
             '(See https://docs.python.org/3/library/logging.html#levels). '
             'Default log level is INFO.'))
+    parser.add_argument(
+        '--sdk_harness_log_level_overrides',
+        action='append',
+        default=None,
+        help=(
+            'Controls the log levels for specifically named loggers. The '
+            'expected format is a json string: {\"module\":\"log_level\",...}. '
+            'For example, by specifying the value "{\"a.b.c\":\"DEBUG\"}, '
+            'the logger underneath the module "a.b.c" will be configured to '
+            'output logs at the DEBUG level. Similarly, by specifying the '
+            'value {"a.b.c":"WARNING"} all loggers underneath the "a.b.c" '

Review Comment:
   is escaping quotes necessary here as well?



-- 
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: github-unsubscribe@beam.apache.org

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