You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Weston Pace (Jira)" <ji...@apache.org> on 2022/04/22 22:48:00 UTC
[jira] [Commented] (ARROW-16273) [C++] Valgrind error in arrow-compute-scalar-test
[ 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)