Wes McKinney created ARROW-2840:

             Summary: [C++] See if stream alignment logic can be simplified
                 Key: ARROW-2840
                 URL: https://issues.apache.org/jira/browse/ARROW-2840
             Project: Apache Arrow
          Issue Type: Improvement
          Components: C++
            Reporter: Wes McKinney
             Fix For: 0.10.0

There were a couple patches related to 64-byte alignment for tensor messages at 
the beginning of April that I've been concerned about: 



In my opinion, the alignment issue should be encapsulated in the protocol if 
possible; to have argument that toggles whether the file is to be aligned after 
reading the metadata seems wrong to me. The bytes written for alignment 
purposes should be accounted for in the metadata size prefix in the IPC message.

Since there are unit tests for this, I'd like to take a look and see if I can 
improve this without breaking the tests.

This message was sent by Atlassian JIRA

Reply via email to