You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "David Li (Jira)" <ji...@apache.org> on 2022/06/27 19:12:00 UTC

[jira] [Resolved] (ARROW-16877) [C++] Valgrind failure (unintialized value) in arrow-compute-internals-test

     [ https://issues.apache.org/jira/browse/ARROW-16877?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Li resolved ARROW-16877.
------------------------------
    Fix Version/s: 9.0.0
       Resolution: Fixed

Issue resolved by pull request 13438
[https://github.com/apache/arrow/pull/13438]

> [C++] Valgrind failure (unintialized value) in arrow-compute-internals-test
> ---------------------------------------------------------------------------
>
>                 Key: ARROW-16877
>                 URL: https://issues.apache.org/jira/browse/ARROW-16877
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>            Reporter: David Li
>            Assignee: Raúl Cumplido
>            Priority: Major
>              Labels: Nightly, pull-request-available
>             Fix For: 9.0.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> Looks like GTest is trying to print an uninitalized unique_ptr.
> https://dev.azure.com/ursacomputing/crossbow/_build/results?buildId=27986&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=d9b15392-e4ce-5e4c-0c8c-b69645229181
> {noformat}
> 27/68 Test #28: arrow-compute-internals-test .............***Failed   15.30 sec
> ==11317== Memcheck, a memory error detector
> ==11317== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
> ==11317== Using Valgrind-3.18.1 and LibVEX; rerun with -h for copyright info
> ==11317==    by 0x1C31BF: void testing::internal::PrintTupleTo<std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >, 2ul>(std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > const&, std::integral_constant<unsigned long, 2ul>, std::ostream*) (gtest-printers.h:641)
> ==11317==    by 0x1C31F8: void testing::internal::PrintTupleTo<std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >, 3ul>(std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > const&, std::integral_constant<unsigned long, 3ul>, std::ostream*) (gtest-printers.h:641)
> ==11317==    by 0x1C3231: void testing::internal::PrintTupleTo<std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > >, 4ul>(std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > const&, std::integral_constant<unsigned long, 4ul>, std::ostream*) (gtest-printers.h:641)
> ==11317==    by 0x1C3285: void testing::internal::PrintTo<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<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::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > const&, std::ostream*) (gtest-printers.h:654)
> ==11317==    by 0x1C32AA: Print (gtest-printers.h:691)
> ==11317==    by 0x1C32AA: void testing::internal::UniversalPrint<std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<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::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > const&, std::ostream*) (gtest-printers.h:980)
> ==11317==    by 0x1C32E7: Print (gtest-printers.h:865)
> ==11317==    by 0x1C32E7: std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > testing::PrintToString<std::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<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::tuple<std::function<std::unique_ptr<arrow::compute::FunctionRegistry, std::default_delete<arrow::compute::FunctionRegistry> > ()>, std::function<int ()>, std::function<std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > > ()>, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > > const&) (gtest-printers.h:1018)
> ==11317==    by 0x1C4033: testing::internal::ParameterizedTestSuiteInfo<arrow::compute::TestRegistry>::RegisterTests() (gtest-param-util.h:590)
> ==11317==    by 0x6438DBC: testing::internal::ParameterizedTestSuiteRegistry::RegisterTests() (gtest-param-util.h:726)
> ==11317==    by 0x6445597: testing::internal::UnitTestImpl::RegisterParameterizedTests() (gtest.cc:2823)
> ==11317==    by 0x64558D3: testing::internal::UnitTestImpl::PostFlagParsingInit() (gtest.cc:5639)
> ==11317==    by 0x646C550: void testing::internal::InitGoogleTestImpl<char>(int*, char**) (gtest.cc:6646)
> ==11317==    by 0x64584C4: testing::InitGoogleTest(int*, char**) (gtest.cc:6664)
> ==11317==    by 0x4205956: main (gtest_main.cc:51)
> ==11317== 
> {
>    <insert_a_suppression_name_here>
>    Memcheck:Cond
>    fun:vfprintf
>    fun:vsnprintf
>    fun:snprintf
>    fun:_ZN7testing12_GLOBAL__N_126PrintByteSegmentInObjectToEPKhmmPSo
>    fun:_ZN7testing12_GLOBAL__N_124PrintBytesInObjectToImplEPKhmPSo
>    fun:_ZN7testing8internal20PrintBytesInObjectToEPKhmPSo
>    fun:PrintValue<std::function<std::unique_ptr<arrow::compute::FunctionRegistry>()> >
>    fun:_ZN7testing8internal17PrintWithFallbackISt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS6_EEvEEEEvRKT_PSo
>    fun:_ZN7testing8internal7PrintToISt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS6_EEvEEEEvRKT_PSo
>    fun:Print
>    fun:_ZN7testing8internal12PrintTupleToISt5tupleIJSt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS7_EEvEES3_IFivEES3_IFSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaISL_EEvEESL_EELm1EEEvRKT_St17integral_constantImXT0_EEPSo
>    fun:_ZN7testing8internal12PrintTupleToISt5tupleIJSt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS7_EEvEES3_IFivEES3_IFSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaISL_EEvEESL_EELm2EEEvRKT_St17integral_constantImXT0_EEPSo
>    fun:_ZN7testing8internal12PrintTupleToISt5tupleIJSt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS7_EEvEES3_IFivEES3_IFSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaISL_EEvEESL_EELm3EEEvRKT_St17integral_constantImXT0_EEPSo
>    fun:_ZN7testing8internal12PrintTupleToISt5tupleIJSt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS7_EEvEES3_IFivEES3_IFSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaISL_EEvEESL_EELm4EEEvRKT_St17integral_constantImXT0_EEPSo
>    fun:_ZN7testing8internal7PrintToIJSt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS6_EEvEES2_IFivEES2_IFSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaISK_EEvEESK_EEEvRKSt5tupleIJDpT_EEPSo
>    fun:Print
>    fun:_ZN7testing8internal14UniversalPrintISt5tupleIJSt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS7_EEvEES3_IFivEES3_IFSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaISL_EEvEESL_EEEEvRKT_PSo
>    fun:Print
>    fun:_ZN7testing13PrintToStringISt5tupleIJSt8functionIFSt10unique_ptrIN5arrow7compute16FunctionRegistryESt14default_deleteIS6_EEvEES2_IFivEES2_IFSt6vectorINSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEESaISK_EEvEESK_EEEESK_RKT_
>    fun:_ZN7testing8internal26ParameterizedTestSuiteInfoIN5arrow7compute12TestRegistryEE13RegisterTestsEv
>    fun:_ZN7testing8internal30ParameterizedTestSuiteRegistry13RegisterTestsEv
>    fun:_ZN7testing8internal12UnitTestImpl26RegisterParameterizedTestsEv
>    fun:_ZN7testing8internal12UnitTestImpl19PostFlagParsingInitEv
>    fun:_ZN7testing8internal18InitGoogleTestImplIcEEvPiPPT_
>    fun:_ZN7testing14InitGoogleTestEPiPPc
>    fun:main
> }
> Running main() from /build/cpp/googletest_ep-prefix/src/googletest_ep/googletest/src/gtest_main.cc
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.7#820007)