[jira] [Commented] (IMPALA-6996) PartitionedAggregationNode::Close() should not dump stack trace

2018-05-10 Thread Zoram Thanga (JIRA)
[ https://issues.apache.org/jira/browse/IMPALA-6996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16471506#comment-16471506 ] Zoram Thanga commented on IMPALA-6996: -- Dup'ing this to IMPALA-6997, as the narrowed

[jira] [Commented] (IMPALA-6996) PartitionedAggregationNode::Close() should not dump stack trace

2018-05-09 Thread Tim Armstrong (JIRA)
[ https://issues.apache.org/jira/browse/IMPALA-6996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16469237#comment-16469237 ] Tim Armstrong commented on IMPALA-6996: --- Just to add some context about why this is

[jira] [Commented] (IMPALA-6996) PartitionedAggregationNode::Close() should not dump stack trace

2018-05-09 Thread Tim Armstrong (JIRA)
[ https://issues.apache.org/jira/browse/IMPALA-6996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16469089#comment-16469089 ] Tim Armstrong commented on IMPALA-6996: --- +1, just avoiding unnecessary work seems l

[jira] [Commented] (IMPALA-6996) PartitionedAggregationNode::Close() should not dump stack trace

2018-05-08 Thread Dan Hecht (JIRA)
[ https://issues.apache.org/jira/browse/IMPALA-6996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16468259#comment-16468259 ] Dan Hecht commented on IMPALA-6996: --- Maybe we should make SetMemLimitExceeded() check i

[jira] [Commented] (IMPALA-6996) PartitionedAggregationNode::Close() should not dump stack trace

2018-05-08 Thread Zoram Thanga (JIRA)
[ https://issues.apache.org/jira/browse/IMPALA-6996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16468036#comment-16468036 ] Zoram Thanga commented on IMPALA-6996: -- Narrowed the scope to not dumping the stack