Mridul M
Mridul M
@wankunde, please ping me when you are done with the updates to the PR. Thanks
I just merged #39316 to unblock this PR, can you update to latest please ? Thx
The tests are failing @wankunde, though I dont think it is due to your PR. Can you please take a look ? And retrigger it if it is unrelated ?...
Merged to master. Thanks for fixing this @wankunde ! Thanks for the reviews @otterc, @akpatnam25, @yabola :-)
+CC @Ngone51 , @HyukjinKwon
QQ: Why is this PR targeting 3.3 and not master ?
Can you fix the linter error @rmcyang ? ` src/main/java/org/apache/spark/network/shuffle/RemoteBlockPushResolver.java:[1281] (sizes) LineLength: Line is longer than 100 characters (found 104).`
Merged to master. Thanks for working on this @rmcyang ! Thanks for the reviews @zhouyejoe, @otterc :-)
I have not looked into this in great detail, but shouldn't cogroup, etc (anything using multiple rdd's) validate the partitioners are equal before comparing partition index, etc ? Or is...
To give an example of what I was referring to - take a look at cogroup in IndexedRDDFunctions, self and other IndexedRDDFunctions need not be partitioned by the same partitioner...