claudevdm commented on code in PR #36149:
URL: https://github.com/apache/beam/pull/36149#discussion_r2349407615
##########
sdks/python/apache_beam/transforms/ptransform_test.py:
##########
@@ -728,7 +728,10 @@ def test_flatten_one_single_pcollection(self):
param(compat_version="2.66.0"),
])
@pytest.mark.it_validatesrunner
+ @pytest.mark.uses_dill
def test_group_by_key_importable_special_types(self, compat_version):
+ pytest.importorskip("dill")
Review Comment:
Can we move this to the "if compat_version" block? Dill is only used if the
compat version flag is set, otherwise the test should not be skipped.
--
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]
