Scala implicit ordering fun -- have 2 forms of sketch, defaulting to …
…the existing
@johnynek as we were discussing for the sketch method. More of a consideration than hard PR
looks good to me modulo the naming (I would use names for classes more specific to SketchJoin)
:+1: I like this. You can use the OrderedSerialization macros, not have an implicit conversion in scope and do a more efficient join at the same time. Seems like a win.
what do you think of this @rubanm @isnotinvain ? I like not having to set up an implicit conversion and the ability to use compiler generated serializers.
@johnynek @ianoc Sorry late to this. Just one unit test comment. LGTM!
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.