[ https://issues.apache.org/jira/browse/ARROW-16273?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17526712#comment-17526712 ]
Weston Pace commented on ARROW-16273: ------------------------------------- I have not been able to reproduce this and it appears the nightly valgrind is now passing. I'm not sure if some issue got fixed concurrently or if this is just flaky. > [C++] Valgrind error in arrow-compute-scalar-test > ------------------------------------------------- > > Key: ARROW-16273 > URL: https://issues.apache.org/jira/browse/ARROW-16273 > Project: Apache Arrow > Issue Type: Bug > Reporter: Weston Pace > Priority: Major > > Currently valgrind is failing earlier on the tpch-node-test and > hash-join-node-test. Once we fix those tests it seems the next error is this: > {noformat} > [ RUN ] TestStringKernels/0.Strptime > ==9928== Conditional jump or move depends on uninitialised value(s) > ==9928== at 0x411AEA2: arrow::TestInitialized(arrow::ArrayData const&) > (gtest_util.cc:682) > ==9928== by 0xAE1C79: arrow::compute::(anonymous > namespace)::ValidateOutput(arrow::ArrayData const&) (test_util.cc:287) > ==9928== by 0xAE23FC: arrow::compute::ValidateOutput(arrow::Datum const&) > (test_util.cc:320) > ==9928== by 0xAE4946: > arrow::compute::CheckScalarNonRecursive(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> > const&, > std::vector<arrow::Datum, std::allocator<arrow::Datum> > const&, arrow::Datum > const&, arrow::compute::FunctionOptions const*) (test_util.cc:80) > ==9928== by 0xAE63A4: > arrow::compute::CheckScalar(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, std::vector<arrow::Datum, > std::allocator<arrow::Datum> > const&, arrow::Datum, > arrow::compute::FunctionOptions const*) (test_util.cc:108) > ==9928== by 0xAE7E28: > arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, arrow::Datum, arrow::Datum, > arrow::compute::FunctionOptions const*) (test_util.cc:254) > ==9928== by 0xAE80D3: > arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > arrow::compute::FunctionOptions const*) (test_util.cc:260) > ==9928== by 0x9F783F: > arrow::compute::BaseTestStringKernels<arrow::StringType>::CheckUnary(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > arrow::compute::FunctionOptions const*) (scalar_string_test.cc:56) > ==9928== by 0xA2A62D: > arrow::compute::TestStringKernels_Strptime_Test<arrow::StringType>::TestBody() > (scalar_string_test.cc:1855) > ==9928== by 0x64974DC: void > testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, > void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2607) > ==9928== by 0x648E90C: void > testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, > void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2643) > ==9928== by 0x6469CDC: testing::Test::Run() (gtest.cc:2682) > ==9928== by 0x646A6FE: testing::TestInfo::Run() (gtest.cc:2861) > ==9928== by 0x646B0BD: testing::TestSuite::Run() (gtest.cc:3015) > ==9928== by 0x647B1DB: testing::internal::UnitTestImpl::RunAllTests() > (gtest.cc:5855) > ==9928== by 0x6498497: bool > testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, > bool>(testing::internal::UnitTestImpl*, bool > (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2607) > ==9928== by 0x648FAF9: bool > testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, > bool>(testing::internal::UnitTestImpl*, bool > (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2643) > ==9928== by 0x64796A8: testing::UnitTest::Run() (gtest.cc:5438) > ==9928== by 0x4204918: RUN_ALL_TESTS() (gtest.h:2490) > ==9928== by 0x420495B: main (gtest_main.cc:52) > ==9928== > { > <insert_a_suppression_name_here> > Memcheck:Cond > fun:_ZN5arrow15TestInitializedERKNS_9ArrayDataE > fun:_ZN5arrow7compute12_GLOBAL__N_114ValidateOutputERKNS_9ArrayDataE > fun:_ZN5arrow7compute14ValidateOutputERKNS_5DatumE > > fun:_ZN5arrow7compute23CheckScalarNonRecursiveERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERKSt6vectorINS_5DatumESaISA_EERKSA_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute11CheckScalarENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERKSt6vectorINS_5DatumESaIS8_EES8_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEENS_5DatumES7_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESt10shared_ptrINS_8DataTypeEES6_S9_S6_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute21BaseTestStringKernelsINS_10StringTypeEE10CheckUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEES9_St10shared_ptrINS_8DataTypeEES9_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute31TestStringKernels_Strptime_TestINS_10StringTypeEE8TestBodyEv > > fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc > > fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc > fun:_ZN7testing4Test3RunEv > fun:_ZN7testing8TestInfo3RunEv > fun:_ZN7testing9TestSuite3RunEv > fun:_ZN7testing8internal12UnitTestImpl11RunAllTestsEv > > fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc > > fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc > fun:_ZN7testing8UnitTest3RunEv > fun:_Z13RUN_ALL_TESTSv > fun:main > } > ==9928== Conditional jump or move depends on uninitialised value(s) > ==9928== at 0x411AEA2: arrow::TestInitialized(arrow::ArrayData const&) > (gtest_util.cc:682) > ==9928== by 0x411AEF3: arrow::TestInitialized(arrow::Array const&) > (gtest_util.cc:663) > ==9928== by 0xAE1E13: arrow::compute::(anonymous > namespace)::ValidateOutput(arrow::ChunkedArray const&) (test_util.cc:293) > ==9928== by 0xAE240E: arrow::compute::ValidateOutput(arrow::Datum const&) > (test_util.cc:323) > ==9928== by 0xAE753A: > arrow::compute::CheckScalar(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, std::vector<arrow::Datum, > std::allocator<arrow::Datum> > const&, arrow::Datum, > arrow::compute::FunctionOptions const*) (test_util.cc:167) > ==9928== by 0xAE7E28: > arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, arrow::Datum, arrow::Datum, > arrow::compute::FunctionOptions const*) (test_util.cc:254) > ==9928== by 0xAE80D3: > arrow::compute::CheckScalarUnary(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > arrow::compute::FunctionOptions const*) (test_util.cc:260) > ==9928== by 0x9F783F: > arrow::compute::BaseTestStringKernels<arrow::StringType>::CheckUnary(std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >, std::shared_ptr<arrow::DataType>, std::__cxx11::basic_string<char, > std::char_traits<char>, std::allocator<char> >, > arrow::compute::FunctionOptions const*) (scalar_string_test.cc:56) > ==9928== by 0xA2A62D: > arrow::compute::TestStringKernels_Strptime_Test<arrow::StringType>::TestBody() > (scalar_string_test.cc:1855) > ==9928== by 0x64974DC: void > testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, > void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2607) > ==9928== by 0x648E90C: void > testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, > void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2643) > ==9928== by 0x6469CDC: testing::Test::Run() (gtest.cc:2682) > ==9928== by 0x646A6FE: testing::TestInfo::Run() (gtest.cc:2861) > ==9928== by 0x646B0BD: testing::TestSuite::Run() (gtest.cc:3015) > ==9928== by 0x647B1DB: testing::internal::UnitTestImpl::RunAllTests() > (gtest.cc:5855) > ==9928== by 0x6498497: bool > testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, > bool>(testing::internal::UnitTestImpl*, bool > (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2607) > ==9928== by 0x648FAF9: bool > testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, > bool>(testing::internal::UnitTestImpl*, bool > (testing::internal::UnitTestImpl::*)(), char const*) (gtest.cc:2643) > ==9928== by 0x64796A8: testing::UnitTest::Run() (gtest.cc:5438) > ==9928== by 0x4204918: RUN_ALL_TESTS() (gtest.h:2490) > ==9928== by 0x420495B: main (gtest_main.cc:52) > ==9928== > { > <insert_a_suppression_name_here> > Memcheck:Cond > fun:_ZN5arrow15TestInitializedERKNS_9ArrayDataE > fun:_ZN5arrow15TestInitializedERKNS_5ArrayE > fun:_ZN5arrow7compute12_GLOBAL__N_114ValidateOutputERKNS_12ChunkedArrayE > fun:_ZN5arrow7compute14ValidateOutputERKNS_5DatumE > > fun:_ZN5arrow7compute11CheckScalarENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERKSt6vectorINS_5DatumESaIS8_EES8_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEENS_5DatumES7_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute16CheckScalarUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESt10shared_ptrINS_8DataTypeEES6_S9_S6_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute21BaseTestStringKernelsINS_10StringTypeEE10CheckUnaryENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEES9_St10shared_ptrINS_8DataTypeEES9_PKNS0_15FunctionOptionsE > > fun:_ZN5arrow7compute31TestStringKernels_Strptime_TestINS_10StringTypeEE8TestBodyEv > > fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc > > fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS_4TestEvEET0_PT_MS4_FS3_vEPKc > fun:_ZN7testing4Test3RunEv > fun:_ZN7testing8TestInfo3RunEv > fun:_ZN7testing9TestSuite3RunEv > fun:_ZN7testing8internal12UnitTestImpl11RunAllTestsEv > > fun:_ZN7testing8internal38HandleSehExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc > > fun:_ZN7testing8internal35HandleExceptionsInMethodIfSupportedINS0_12UnitTestImplEbEET0_PT_MS4_FS3_vEPKc > fun:_ZN7testing8UnitTest3RunEv > fun:_Z13RUN_ALL_TESTSv > fun:main > } > [ OK ] TestStringKernels/0.Strptime (45 ms) > {noformat} -- This message was sent by Atlassian Jira (v8.20.7#820007)