jkff commented on a change in pull request #11406: [BEAM-9748] Move
Reparallelize transform to Reshuffle
URL: https://github.com/apache/beam/pull/11406#discussion_r408998366
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/Reshuffle.java
##########
@@ -65,6 +66,16 @@ private Reshuffle() {}
return new ViaRandomKey<>();
}
+ @Experimental
Review comment:
I think this (`withHint...`) is a great idea, thanks Luke. It also wouldn't
hurt to rerun the benchmarks (and to check them in - sadly I didn't do it back
then, and probably don't have them anymore, but it only took like 15 min to
write).
----------------------------------------------------------------
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]
With regards,
Apache Git Services
