Mridul M

Results 143 comments of Mridul M

Or, we can use what `RDD.coalesce` does currently.

+CC @dongjoon-hyun, @HyukjinKwon who might be able to review this better than me.

Please don't force push the code, unless it is to resolve conflicts/etc which can be handled otherwise - it makes reviewing what changed much harder

Can you take a look at the test failures @thejdeep ? They look relevant.

I am not sure what has changed since last review due to the force pushes - please do not force push, particularly for large PR's.

Looks good to me, will wait for successful build before merging.

Merged to master. Thanks for working on this @thejdeep ! Thanks for the reviews @otterc, @zhouyejoe, @gengliangwang :-)