Github user NicoK commented on the issue:

    https://github.com/apache/flink/pull/4533
  
    ok, I fiddled with my remarks a bit and you are right for the problems of 
reducing the visibility, which is partly caused by the 
`CreditBasedClientHandler` not replacing `PartitionRequestClientHandler` yet.
    
    I'll send you my suggestions offline (as I can't attach them here) and they 
should fix all my last remarks.


---

Reply via email to