TheNeuralBit commented on a change in pull request #17026:
URL: https://github.com/apache/beam/pull/17026#discussion_r838931448
##########
File path: sdks/python/apache_beam/dataframe/frames_test.py
##########
@@ -131,7 +137,12 @@ def _run_test(
This option should NOT be set to False in tests added for new
operations if at all possible. Instead make sure the new operation
produces the correct proxy. This flag only exists as an escape
hatch
- until existing failures can be addressed (BEAM-12379)."""
+ until existing failures can be addressed (BEAM-12379).
+ check_dtypes (bool): Whether or not to check the dtypes of the values.
+ The default is True. This option should be set to False for
+ non-deferred operations, where the dtype of the values in the proxy
+ are not known ahead of time.
Review comment:
Awesome, thanks!
--
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]
