wesm commented on code in PR #13521:
URL: https://github.com/apache/arrow/pull/13521#discussion_r916201356
##########
cpp/src/arrow/array/data.h:
##########
@@ -266,16 +266,19 @@ struct ARROW_EXPORT ArraySpan {
int64_t offset = 0;
BufferSpan buffers[3];
+ // 16 bytes of scratch space to enable this ArraySpan to be a view onto
+ // scalar values including binary scalars (where we need to create a buffer
+ // that looks like two 32-bit or 64-bit offsets)
+ uint8_t scratch_space[16];
Review Comment:
I'm making this aligned (by changing the type to be two uint64's)
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
