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

Rok Mihevc commented on ARROW-5194:
-----------------------------------

This issue has been migrated to [issue 
#21670|https://github.com/apache/arrow/issues/21670] on GitHub. Please see the 
[migration documentation|https://github.com/apache/arrow/issues/14542] for 
further details.

> [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
>            Assignee: Guillaume Horel
>            Priority: Critical
>              Labels: pull-request-available
>             Fix For: 0.14.0
>
>          Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> 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
(v8.20.10#820010)

Reply via email to