Mridul M

Results 143 comments of 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 :-)

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...