Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/8936 )
Change subject: IMPALA-3833: Fix invalid data handling in Sequence and RCFile
scanners
......................................................................
Patch Set 7:
I'm still seeing crashes when I loop this locally:
F0306 16:34:04.306205 29911 mem-pool.h:245] Check failed: size >= 0 (-9 vs. 0)
*** Check failure stack trace: ***
@ 0x3c32cbd google::LogMessage::Fail()
@ 0x3c34562 google::LogMessage::SendToLog()
@ 0x3c32697 google::LogMessage::Flush()
@ 0x3c35c5e google::LogMessageFatal::~LogMessageFatal()
@ 0x184eeb4 impala::MemPool::TryAllocate()
@ 0x1be5d08 impala::HdfsRCFileScanner::StartRowGroup()
@ 0x1be8032 impala::HdfsRCFileScanner::ProcessRange()
@ 0x296931a impala::BaseSequenceScanner::GetNextInternal()
@ 0x1bc71b2 impala::HdfsScanner::ProcessSplit()
@ 0x1b9f48c impala::HdfsScanNode::ProcessSplit()
@ 0x1b9e8b7 impala::HdfsScanNode::ScannerThread()
@ 0x1b9dd3f
_ZZN6impala12HdfsScanNode22ThreadTokenAvailableCbEPNS_17ThreadResourceMgr12ResourcePoolEENKUlvE_clEv
@ 0x1b9fcdd
_ZN5boost6detail8function26void_function_obj_invoker0IZN6impala12HdfsScanNode22ThreadTokenAvailableCbEPNS3_17ThreadResourceMgr12ResourcePoolEEUlvE_vE6invokeERNS1_15function_bufferE
@ 0x17f4ce0 boost::function0<>::operator()()
@ 0x1af63ff impala::Thread::SuperviseThread()
@ 0x1aff3cf boost::_bi::list5<>::operator()<>()
@ 0x1aff2f3 boost::_bi::bind_t<>::operator()()
@ 0x1aff2b6 boost::detail::thread_data<>::run()
@ 0x2dbc1aa thread_proxy
@ 0x7f5af8f926ba start_thread
@ 0x7f5af8cc841d clone
Picked up JAVA_TOOL_OPTIONS:
-agentlib:jdwp=transport=dt_socket,address=30000,server=y,suspend=n
Wrote minidump to
/home/tarmstrong/Impala/incubator-impala/logs/cluster/minidumps/impalad/31a8ae22-6a2a-4118-4ff4e1b6-be18ef7c.dmp
How about we leave the test disabled and move forward with the improvements,
then we can finish up the remaining fixes needed later?
--
To view, visit http://gerrit.cloudera.org:8080/8936
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic9cfc38af3f30c65ada9734eb471dbfa6ecdd74a
Gerrit-Change-Number: 8936
Gerrit-PatchSet: 7
Gerrit-Owner: Pranay Singh
Gerrit-Reviewer: Pranay Singh
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: anujphadke <[email protected]>
Gerrit-Comment-Date: Wed, 07 Mar 2018 01:24:21 +0000
Gerrit-HasComments: No