damccorm commented on code in PR #30085:
URL: https://github.com/apache/beam/pull/30085#discussion_r1464966095
##########
sdks/python/apache_beam/ml/transforms/base_test.py:
##########
@@ -443,6 +442,26 @@ def test_handler_on_multiple_columns(self):
equal_to(expected_data),
)
+ def test_handler_on_columns_not_exist_in_input_data(self):
+ data = [
+ {
+ 'x': "Hello world", 'y': "Apache Beam"
Review Comment:
Yeah, at least for now I think we should throw and be consistent (and
ideally document that behavior).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
