sahib opened a new pull request, #323:
URL: https://github.com/apache/arrow-go/pull/323

   ### Rationale for this change
   
   I've tried to run a pqarrow based writer/reader on a RaspberryPi. This lead 
to an almost immediate crash however, due to an unaligned struct member in 
`ipc.Reader`.
   
   I did not check yet if there are other structs that might have similar 
issues. It would probably be nice to 
   
   ### What changes are included in this PR?
   
   Swapping the struct field order is enough to fix the issue. See also the 
link in the diff.
   
   ### Are these changes tested?
   
   Yes, no additional unit tests due to hardware constraints, but my program 
runs through now.
   
   ### Are there any user-facing changes?
   
   Less crashing?
   
   ### Other note
   
   I noticed that this will also crash if I do not pass the `noasm` build tag, 
which is probably also a little unfortunate. This seems to be because the 
`funclist` in `parquet/internal/bmi/bmi.go` is empty. If you want I can add a 
default, so the funclist is always initialized with the fallback go 
implementation, unless overwritten.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to