You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Antoine Pitrou (Jira)" <ji...@apache.org> on 2022/09/21 08:19:00 UTC
[jira] [Resolved] (ARROW-17686) [C++] AsofJoinBasicParams has no gtest printer defined, leading to valgrind errors
[ https://issues.apache.org/jira/browse/ARROW-17686?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Antoine Pitrou resolved ARROW-17686.
------------------------------------
Fix Version/s: 10.0.0
Resolution: Fixed
Issue resolved by pull request 14172
[https://github.com/apache/arrow/pull/14172]
> [C++] AsofJoinBasicParams has no gtest printer defined, leading to valgrind errors
> ----------------------------------------------------------------------------------
>
> Key: ARROW-17686
> URL: https://issues.apache.org/jira/browse/ARROW-17686
> Project: Apache Arrow
> Issue Type: Bug
> Components: C++
> Reporter: Weston Pace
> Assignee: Percy Camilo TriveƱo Aucahuasi
> Priority: Major
> Labels: pull-request-available
> Fix For: 10.0.0
>
> Attachments: valgrind.txt
>
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> Valgrind is currently failing on arrow-compute-asof-join-node-test: https://dev.azure.com/ursacomputing/crossbow/_build/results?buildId=34147&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=d9b15392-e4ce-5e4c-0c8c-b69645229181&l=4118
> The issue appears to be that AsofJoinBasicParams has no gtest printer and so gtest is using the default universal-printer which doesn't really play well with valgrind. We should add a custom PrintTo method for AsofJoinBasicParams per: https://github.com/google/googletest/blob/main/googletest/include/gtest/gtest-printers.h
--
This message was sent by Atlassian Jira
(v8.20.10#820010)