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


##########
sdks/python/apache_beam/yaml/yaml_mapping.py:
##########
@@ -467,7 +467,9 @@ def expand(self, pcoll):
           break
       else:
         raise ValueError(
-            f"No field name matches one of {self._ERROR_FIELD_NAMES}")
+            'The input to this transform does not appear to be an error '
+            "output.  Expected a schema'd input with a field named "

Review Comment:
   ```suggestion
               + "output.  Expected a schema'd input with a field named "
   ```



##########
sdks/python/apache_beam/yaml/yaml_mapping.py:
##########
@@ -467,7 +467,9 @@ def expand(self, pcoll):
           break
       else:
         raise ValueError(
-            f"No field name matches one of {self._ERROR_FIELD_NAMES}")
+            'The input to this transform does not appear to be an error '
+            "output.  Expected a schema'd input with a field named "

Review Comment:
   I think this is breaking checks (may need a formatter run after 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-unsubscr...@beam.apache.org

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

Reply via email to