Junfan Zhang

Results 434 comments of Junfan Zhang

![image](https://github.com/apache/incubator-uniffle/assets/8609142/853b8c8a-f1e3-4d9f-802c-de503dd0699f) I think we could remove this. @dingshun3016

This looks strange that it replace the partial blockMaxRetryTimes abilities and introduces the extra config option to meet something, but this change is not compatible with the previous logic.

> > This looks strange that it replace the partial blockMaxRetryTimes abilities and introduces the extra config option to meet something, but this change is not compatible with the previous...

If the range of [low-watermark, high-watermark] is large, this problem with this patch should still exist, right?

> > If the range of [low-watermark, high-watermark] is large, this problem with this patch should still exist, right? > > You are right, but if the range of [low-watermark,...

I hope we could reserve current jvm based hdfs impl, which is useful for those hadoop compatible filesystem.

> Any progress? ping ~ @EnricoMi @zuston LGTM . PTAL @rickyma

Merged. Thanks @EnricoMi @rickyma

Uniffle is another candidate. Do you mind integrating uniffle with blaze?