zhoufek commented on a change in pull request #15003:
URL: https://github.com/apache/beam/pull/15003#discussion_r652122871
##########
File path: sdks/python/apache_beam/transforms/ptransform_test.py
##########
@@ -476,6 +477,7 @@ def test_group_by_key(self):
result = pcoll | 'Group' >> beam.GroupByKey() | _SortLists
assert_that(result, equal_to([(1, [1, 2, 3]), (2, [1, 2]), (3, [1])]))
+ @skip('Behavior not available until Beam 2.33')
Review comment:
Sure, did it for both tests.
At least for the first test, there was no intent to allow a flag to override
the behavior, but in light of the discussion of `TriggerFn`, I've switched to
having `--allow_unsafe_triggers` also override it.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
