[ 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)