tvalentyn commented on a change in pull request #14815:
URL: https://github.com/apache/beam/pull/14815#discussion_r632849889
##########
File path: sdks/python/apache_beam/transforms/display.py
##########
@@ -137,9 +137,19 @@ def create_payload(dd):
except ValueError:
# Skip if the display data is invalid.
return None
- if 'value' not in display_data_dict or 'label' not in display_data_dict:
+
+ if 'key' not in display_data_dict or 'value' not in display_data_dict:
+ # Skipping DisplayData items that do not have a 'key' or 'value' set.
Review comment:
Ah, there is validation already here:
https://github.com/chamikaramj/beam/blob/5921a75c9166d524e3c0362a6c59837dcba15da5/sdks/python/apache_beam/transforms/display.py#L304-L327
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
