Thomas Marshall has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11650 )

Change subject: IMPALA-7272: Fix crash in StringMinMaxFilter
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11650/1/be/src/runtime/mem-pool.h
File be/src/runtime/mem-pool.h:

http://gerrit.cloudera.org:8080/#/c/11650/1/be/src/runtime/mem-pool.h@260
PS1, Line 260:   uint8_t* ALWAYS_INLINE Allocate(int64_t size, int alignment) 
noexcept {
> Good to hear that DFAKE_SCOPED_LOCK() is catching the problem.
After digging in more, it looks like the problem is actually related to codegen 
- even correct uses of MemPool with the DFAKE_MUTEX from a codegen'd function 
cause crashes and disabling codegen fixes the crashes.

I'm not sure what's going on - there isn't any sort of error and the backtraces 
from the crashes are useless - but I'm continuing to dig in. Any thoughts on 
what might be happening?



--
To view, visit http://gerrit.cloudera.org:8080/11650
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I751cad7e6b75c9d95d7ad029bbd1e52fe09e8a29
Gerrit-Change-Number: 11650
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Marshall <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Thomas Marshall <[email protected]>
Gerrit-Comment-Date: Thu, 11 Oct 2018 23:17:45 +0000
Gerrit-HasComments: Yes

Reply via email to