[jira] [Commented] (FLINK-9736) Potential null reference in KeyGroupPartitionedPriorityQueue#poll()

2018-07-05 Thread Stefan Richter (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-9736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16533515#comment-16533515 ] Stefan Richter commented on FLINK-9736: --- [~sihuazhou] Is correct, this heap is always initialized

[jira] [Commented] (FLINK-9736) Potential null reference in KeyGroupPartitionedPriorityQueue#poll()

2018-07-05 Thread Sihua Zhou (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-9736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16533506#comment-16533506 ] Sihua Zhou commented on FLINK-9736: --- Hi [~yuzhih...@gmail.com] AFAIK, {{heapOfKeyGroupHeaps}} will

[jira] [Commented] (FLINK-9736) Potential null reference in KeyGroupPartitionedPriorityQueue#poll()

2018-07-04 Thread Ted Yu (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-9736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16533106#comment-16533106 ] Ted Yu commented on FLINK-9736: --- bq. we never call `poll()` on it. What happens to the {{peek}} call when

[jira] [Commented] (FLINK-9736) Potential null reference in KeyGroupPartitionedPriorityQueue#poll()

2018-07-04 Thread Sihua Zhou (JIRA)
[ https://issues.apache.org/jira/browse/FLINK-9736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16532726#comment-16532726 ] Sihua Zhou commented on FLINK-9736: --- Hi [~yuzhih...@gmail.com] I think this might look like a non