You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/28 14:41:33 UTC

[GitHub] [beam] Abacn opened a new pull request, #22077: Add sdk_harness_log_level_overrides option for python sdk

Abacn opened a new pull request, #22077:
URL: https://github.com/apache/beam/pull/22077

   Fixes #18222
   
   **Please** add a meaningful description for your change here
   
   ------------------------
   
   Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
   
    - [ ] [**Choose reviewer(s)**](https://beam.apache.org/contribute/#make-your-change) and mention them in a comment (`R: @username`).
    - [ ] Mention the appropriate issue in your description (for example: `addresses #123`), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment `fixes #<ISSUE NUMBER>` instead.
    - [ ] Update `CHANGES.md` with noteworthy changes.
    - [ ] If this contribution is large, please file an Apache [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   See the [Contributor Guide](https://beam.apache.org/contribute) for more tips on [how to make review process smoother](https://beam.apache.org/contribute/#make-reviewers-job-easier).
   
   To check the build health, please visit [https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md](https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md)
   
   GitHub Actions Tests Status (on master branch)
   ------------------------------------------------------------------------------------------------
   [![Build python source distribution and wheels](https://github.com/apache/beam/workflows/Build%20python%20source%20distribution%20and%20wheels/badge.svg?branch=master&event=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Build+python+source+distribution+and+wheels%22+branch%3Amaster+event%3Aschedule)
   [![Python tests](https://github.com/apache/beam/workflows/Python%20tests/badge.svg?branch=master&event=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Python+Tests%22+branch%3Amaster+event%3Aschedule)
   [![Java tests](https://github.com/apache/beam/workflows/Java%20Tests/badge.svg?branch=master&event=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Java+Tests%22+branch%3Amaster+event%3Aschedule)
   
   See [CI.md](https://github.com/apache/beam/blob/master/CI.md) for more information about GitHub Actions CI.
   


-- 
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

[GitHub] [beam] github-actions[bot] commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1170176457

   [The Workflow run](https://github.com/apache/beam/actions/runs/2584123859) is cancelling this PR. It is an earlier duplicate of 2083803 run.


-- 
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

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

Posted by 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

[GitHub] [beam] github-actions[bot] commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168992275

   Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control


-- 
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

[GitHub] [beam] asf-ci commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
asf-ci commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168816071

   Can one of the admins verify this patch?


-- 
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

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

Posted by GitBox <gi...@apache.org>.
tvalentyn commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1171618029

   Thanks a lot, @Abacn , very nice.


-- 
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

[GitHub] [beam] tvalentyn merged pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
tvalentyn merged PR #22077:
URL: https://github.com/apache/beam/pull/22077


-- 
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

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

Posted by GitBox <gi...@apache.org>.
Abacn commented on code in PR #22077:
URL: https://github.com/apache/beam/pull/22077#discussion_r909671866


##########
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:
   yeah, not necessary. Copied from java sdk and found I even missed a quote mark. Will fix



-- 
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

[GitHub] [beam] asf-ci commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
asf-ci commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168816065

   Can one of the admins verify this patch?


-- 
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

[GitHub] [beam] asf-ci commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
asf-ci commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168816087

   Can one of the admins verify this patch?


-- 
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

[GitHub] [beam] Abacn commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
Abacn commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168889877

   R: @tvalentyn


-- 
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

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

Posted by GitBox <gi...@apache.org>.
tvalentyn commented on code in PR #22077:
URL: https://github.com/apache/beam/pull/22077#discussion_r909672769


##########
sdks/python/apache_beam/runners/worker/sdk_worker_main.py:
##########
@@ -268,6 +271,38 @@ def _get_log_level_from_options_dict(options_dict: dict) -> int:
   return log_level
 
 
+def _set_log_level_overrides(options_dict: dict) -> None:
+  """Set module log level overrides from options dict's entry
+  `sdk_harness_log_level_overrides`.
+  """
+  option_raw = options_dict.get('sdk_harness_log_level_overrides', None)
+
+  if option_raw is None:
+    return
+
+  parsed_overrides = {}
+
+  try:
+    # parsing and flatten the appended option
+    deserialized = [json.loads(line) for line in option_raw]
+    for line in deserialized:
+      parsed_overrides.update(line)
+  except Exception:
+    _LOGGER.error(
+        "Unable to parse sdk_harness_log_level_overrides. "

Review Comment:
   Maybe print the line that cannot be parsed here?



##########
sdks/python/apache_beam/runners/worker/sdk_worker_main.py:
##########
@@ -268,6 +271,38 @@ def _get_log_level_from_options_dict(options_dict: dict) -> int:
   return log_level
 
 
+def _set_log_level_overrides(options_dict: dict) -> None:
+  """Set module log level overrides from options dict's entry
+  `sdk_harness_log_level_overrides`.
+  """
+  option_raw = options_dict.get('sdk_harness_log_level_overrides', None)
+
+  if option_raw is None:
+    return
+
+  parsed_overrides = {}
+
+  try:
+    # parsing and flatten the appended option
+    deserialized = [json.loads(line) for line in option_raw]
+    for line in deserialized:
+      parsed_overrides.update(line)
+  except Exception:
+    _LOGGER.error(
+        "Unable to parse sdk_harness_log_level_overrides. "
+        "Log level overrides won't take effect.")
+    return
+
+  for module_name, log_level in parsed_overrides.items():
+    try:
+      logging.getLogger(module_name).setLevel(log_level)
+    except Exception as e:
+      # Never crash the worker when exception occurs during log level setting
+      # but logging the error.
+      _LOGGER.error(
+          "Error occurs when setting log level for %s: %s", module_name, e)

Review Comment:
   ```suggestion
             "Error occurred when setting log level for %s: %s", module_name, e)
   ```



##########
sdks/python/apache_beam/runners/worker/sdk_worker_main_test.py:
##########
@@ -121,6 +124,52 @@ def test__get_log_level_from_options_dict(self):
       self.assertEqual(
           sdk_worker_main._get_log_level_from_options_dict(case), expected)
 
+  def test__set_log_level_overrides(self):
+    test_cases = [
+
+        ([], {}), # not provided, as a smoke test
+        (
+            # single overrides
+            ['{"a.b":"DEBUG","c.d":"INFO"}'],
+            {"a.b": logging.DEBUG, "a.b.f": logging.DEBUG, "c.d": logging.INFO}
+        ),
+        (
+            # multiple overrides
+            ['{"a.b":"DEBUG"}', '{"c.d":"ERROR","c.d.e":15}'],
+            {"a.b": logging.DEBUG, "a.b.f": logging.DEBUG, "c.d": logging.ERROR,
+             "c.d.e": 15, "c.d.f": logging.ERROR}
+        )
+    ]
+    for case, expected in test_cases:
+      # Set log level overrides for this fake package name.
+      # This avoids pollutes real logging.

Review Comment:
   Is it to just to avoid side-effects between `single` and `multiple` scenarios? 



##########
sdks/python/apache_beam/runners/worker/sdk_worker_main_test.py:
##########
@@ -121,6 +124,52 @@ def test__get_log_level_from_options_dict(self):
       self.assertEqual(
           sdk_worker_main._get_log_level_from_options_dict(case), expected)
 
+  def test__set_log_level_overrides(self):
+    test_cases = [
+
+        ([], {}), # not provided, as a smoke test
+        (
+            # single overrides
+            ['{"a.b":"DEBUG","c.d":"INFO"}'],
+            {"a.b": logging.DEBUG, "a.b.f": logging.DEBUG, "c.d": logging.INFO}
+        ),
+        (
+            # multiple overrides
+            ['{"a.b":"DEBUG"}', '{"c.d":"ERROR","c.d.e":15}'],
+            {"a.b": logging.DEBUG, "a.b.f": logging.DEBUG, "c.d": logging.ERROR,
+             "c.d.e": 15, "c.d.f": logging.ERROR}
+        )
+    ]
+    for case, expected in test_cases:
+      # Set log level overrides for this fake package name.
+      # This avoids pollutes real logging.

Review Comment:
   ```suggestion
         # This avoids polluting real logging.
   ```



-- 
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

[GitHub] [beam] asf-ci commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
asf-ci commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168816085

   Can one of the admins verify this patch?


-- 
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

[GitHub] [beam] github-actions[bot] commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1170175556

   [The Workflow run](https://github.com/apache/beam/actions/runs/2584122110) is cancelling this PR. It is an earlier duplicate of 1729654 run.


-- 
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

[GitHub] [beam] asf-ci commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
asf-ci commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168816066

   Can one of the admins verify this patch?


-- 
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

[GitHub] [beam] codecov[bot] commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
codecov[bot] commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1168848425

   # [Codecov](https://codecov.io/gh/apache/beam/pull/22077?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#22077](https://codecov.io/gh/apache/beam/pull/22077?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (bb1bbe1) into [master](https://codecov.io/gh/apache/beam/commit/141e95074370e2e1dde40605d05e653503768e14?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (141e950) will **increase** coverage by `0.00%`.
   > The diff coverage is `100.00%`.
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #22077   +/-   ##
   =======================================
     Coverage   74.00%   74.00%           
   =======================================
     Files         703      703           
     Lines       92936    92955   +19     
   =======================================
   + Hits        68774    68793   +19     
     Misses      22896    22896           
     Partials     1266     1266           
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | python | `83.59% <100.00%> (+<0.01%)` | :arrow_up: |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more.
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/22077?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [...dks/python/apache\_beam/options/pipeline\_options.py](https://codecov.io/gh/apache/beam/pull/22077/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vb3B0aW9ucy9waXBlbGluZV9vcHRpb25zLnB5) | `94.96% <100.00%> (+0.01%)` | :arrow_up: |
   | [...thon/apache\_beam/runners/worker/sdk\_worker\_main.py](https://codecov.io/gh/apache/beam/pull/22077/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvc2RrX3dvcmtlcl9tYWluLnB5) | `78.36% <100.00%> (+2.54%)` | :arrow_up: |
   | [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/22077/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kaXJlY3QvdGVzdF9zdHJlYW1faW1wbC5weQ==) | `93.28% <0.00%> (-0.75%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/22077/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.30% <0.00%> (-0.25%)` | :arrow_down: |
   | [...ks/python/apache\_beam/runners/worker/sdk\_worker.py](https://codecov.io/gh/apache/beam/pull/22077/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvc2RrX3dvcmtlci5weQ==) | `88.94% <0.00%> (-0.16%)` | :arrow_down: |
   | [...eam/runners/portability/fn\_api\_runner/execution.py](https://codecov.io/gh/apache/beam/pull/22077/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2V4ZWN1dGlvbi5weQ==) | `93.08% <0.00%> (+0.64%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/22077?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/22077?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [141e950...bb1bbe1](https://codecov.io/gh/apache/beam/pull/22077?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
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

[GitHub] [beam] Abacn commented on pull request #22077: Add sdk_harness_log_level_overrides option for python sdk

Posted by GitBox <gi...@apache.org>.
Abacn commented on PR #22077:
URL: https://github.com/apache/beam/pull/22077#issuecomment-1171450768

   PTAL


-- 
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