iemejia commented on pull request #11559:
URL: https://github.com/apache/beam/pull/11559#issuecomment-623940748
I see, thanks for the info @rehmanmuradali and @reuvenlax.
It seems this is only for `ParDo`, my point here is not to be annoying it is
that I can imagine in the future someone annotating their tests with `UsesKey`
just because the tests use a Key (like in KV) and I want to prevent this. What
about naming it `UsesKeyInParDo` ?
Also it seems then that the current documentation for the class is not ok
too, can you please also mention of the `OnWindowExpiration` and
`ProcessElement` case.
```java
/**
* Category tag for validation tests which use key. Tests tagged with {@link
UsesKey} should be run
* for runners which support key parameter in {@link OnTimer}.
*/
```
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.
For queries about this service, please contact Infrastructure at:
[email protected]
