[GitHub] [beam] AnandInguva commented on a change in pull request #16929: [BEAM-13709] Inconsistent behavior when parsing boolean flags across different APIs in Python SDK

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

AnandInguva commented on a change in pull request #16929:
URL: https://github.com/apache/beam/pull/16929#discussion_r820316824



##########
File path: sdks/python/apache_beam/options/pipeline_options.py
##########
@@ -234,12 +262,27 @@ def from_dictionary(cls, options):
 
     Returns:
       A PipelineOptions object representing the given arguments.
+
+    Note: If a boolean flag is True in the dictionary,
+          implicitly the method assumes the boolean flag is
+          specified as a command line argument. If the
+          boolean flag is False, this method simply discards
+          them.
+    Eg: {no_auth: True} is similar to python your_file.py --no_auth
+        {no_auth: False} is similar to python your_file.py.
     """
     flags = []
     for k, v in options.items():
       if isinstance(v, bool):

Review comment:
       Done




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