AnandInguva commented on code in PR #30085:
URL: https://github.com/apache/beam/pull/30085#discussion_r1464236982
##########
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:
tensorflow-transform throws the error for this kind of example. Maybe we
should follow same pattern? may be not? let me think about it.
```
data = [
{
'x': 'foo', 'y': 'bar', 'z': 'baz'
},
{
'x': 'foo2', 'y': 'bar2'
}
]
```
--
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]
