[jira] [Commented] (ARROW-5194) [C++][Plasma] TEST(PlasmaSerialization, GetReply) is failing

["Wes McKinney (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARROW-5194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16826064#comment-16826064 ] 

Wes McKinney commented on ARROW-5194:
-------------------------------------

Would you like to submit a pull request?

> [C++][Plasma] TEST(PlasmaSerialization, GetReply) is failing
> ------------------------------------------------------------
>
>                 Key: ARROW-5194
>                 URL: https://issues.apache.org/jira/browse/ARROW-5194
>             Project: Apache Arrow
>          Issue Type: Test
>          Components: C++ - Plasma
>    Affects Versions: 0.13.0
>            Reporter: Guillaume Horel
>            Priority: Critical
>
> I believe the [assertions|https://github.com/apache/arrow/blob/master/cpp/src/plasma/test/serialization_tests.cc#L191] here are incorrect. You can't use memcmp to compare structs
>  On my machine (64bits linux) int are 4 bytes, but the struct fields are 8 bytes aligned, so this line fails:
> ASSERT_EQ(memcmp(&plasma_objects[object_ids[0]], &plasma_objects_return[0],
>  sizeof(PlasmaObject)),
>  0);
> even though the struct contents are the same.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)