zhengbuqian commented on PR #23491:
URL: https://github.com/apache/beam/pull/23491#issuecomment-1353903623
The unit test `testMultimapStateRemove` failed without commit `e1d765b`
because I made the assumption that the test framework will process the
following 4 elements in order, but seems that it's not the case, for example
`b, 33` may be the 1st/2nd/3rd/4th processed element. `testMultimapStateRemove`
was the only unit test that relies on such ordering and has been fixed, all
other tests verify the result only after all elements have been processed thus
have no issue.
```java
pipeline
.apply(
Create.of(
KV.of("hello", KV.of("a", 97)), KV.of("hello",
KV.of("a", 97)),
KV.of("hello", KV.of("a", 98)), KV.of("hello",
KV.of("b", 33))))
.apply(ParDo.of(fn));
```
--
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]
