[GitHub] zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow

2018-11-27 Thread GitBox
zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow URL: https://github.com/apache/flink/pull/6829#issuecomment-442104761 Thanks for merging @pnowojski .

[GitHub] zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow

2018-11-26 Thread GitBox
zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow URL: https://github.com/apache/flink/pull/6829#issuecomment-441663437 Thanks for reviews and benchmark verifies! 

[GitHub] zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow

2018-11-22 Thread GitBox
zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow URL: https://github.com/apache/flink/pull/6829#issuecomment-441064783 @pnowojski , thanks for your reviews! I submitted the hotfix commit for

[GitHub] zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow

2018-11-21 Thread GitBox
zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow URL: https://github.com/apache/flink/pull/6829#issuecomment-440616620 cc @pnowojski @NicoK

[GitHub] zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow

2018-10-31 Thread GitBox
zhijiangW commented on issue #6829: [FLINK-10367][network] Introduce NotificationResult for BufferListener to solve recursive stack overflow URL: https://github.com/apache/flink/pull/6829#issuecomment-434916161 @NicoK , do you think this pr is necessary to be covered in flink-1.7?