[GitHub] [arrow] rtpsw commented on pull request #35541: GH-35498: [C++] Fix source node batch realignment

["rtpsw (via GitHub)" <gi...@apache.org>]

rtpsw commented on PR #35541:
URL: https://github.com/apache/arrow/pull/35541#issuecomment-1543748201

   I see a bunch of CI job ([here](https://github.com/apache/arrow/actions/runs/4945533592/jobs/8842394608?pr=35541), [here](https://github.com/apache/arrow/actions/runs/4945533592/jobs/8842394874?pr=35541), [here](https://github.com/apache/arrow/actions/runs/4945533592/jobs/8842395019?pr=35541) and [here](https://ci.appveyor.com/project/ApacheSoftwareFoundation/arrow/builds/47016479)) are failing due to OOM, presumably on realignment, but the allocation sizes are not large. More concerning is [this failure](https://github.com/apache/arrow/actions/runs/4945533592/jobs/8842394197?pr=35541), which shows an invalid alignment of 3. Presumably, the data type of the value being realigned is some kind of struct. @westonpace, what are your thoughts? Perhaps there is an API (or idiom) to get the byte width of the largest field of the data type, which I think would be the correct alignment in such a case?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org