[GitHub] [flink] azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools

2019-10-21 Thread GitBox
azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools URL: https://github.com/apache/flink/pull/9693#issuecomment-544589920 Thanks @carp84 , merging it when CI is green This is

[GitHub] [flink] azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools

2019-10-21 Thread GitBox
azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools URL: https://github.com/apache/flink/pull/9693#issuecomment-544511037 Thanks for the review @carp84 I addressed the comments and introduced `MemoryReservationException` for

[GitHub] [flink] azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools

2019-10-15 Thread GitBox
azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools URL: https://github.com/apache/flink/pull/9693#issuecomment-542100750 Thanks for the reviews @xintongsong @KarmaGYZ I removed the pre-allocation left-overs, I guess got lost during

[GitHub] [flink] azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools

2019-10-14 Thread GitBox
azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools URL: https://github.com/apache/flink/pull/9693#issuecomment-541687094 Thanks for the review @xintongsong I addressed the comments

[GitHub] [flink] azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools

2019-09-20 Thread GitBox
azagrebin commented on issue #9693: [FLINK-13984] Separate on-heap and off-heap managed memory pools URL: https://github.com/apache/flink/pull/9693#issuecomment-533599524 Thanks for the review @KarmaGYZ I have updated the PR.

[GitHub] [flink] azagrebin commented on issue #9693: Flink 13984

2019-09-16 Thread GitBox
azagrebin commented on issue #9693: Flink 13984 URL: https://github.com/apache/flink/pull/9693#issuecomment-531803716 cc @xintongsong This is an automated message from the Apache Git Service. To respond to the message,