You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "damccorm (via GitHub)" <gi...@apache.org> on 2023/09/06 20:16:46 UTC

[GitHub] [beam] damccorm commented on a diff in pull request #28334: Interpret yaml mapping elements as rows.

damccorm commented on code in PR #28334:
URL: https://github.com/apache/beam/pull/28334#discussion_r1317784767


##########
sdks/python/apache_beam/yaml/yaml_provider.py:
##########
@@ -438,7 +447,7 @@ def _parse_window_spec(spec):
   return InlineProvider(
       dict({
           'Create': lambda elements,
-          reshuffle=True: beam.Create(elements, reshuffle),
+          reshuffle=True: beam.Create(dicts_to_rows(elements), reshuffle),

Review Comment:
   Are we tracking work to add a docs section for these built in transforms? This is the type of behavior that would be good to document (with an example), but it doesn't seem like a structure exists for that yet.



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