[GitHub] [arrow] bkietz commented on a diff in pull request #35565: GH-35498: [C++] Relax EnsureAlignment check in Acero from requiring 64-byte aligned buffers to requiring value-aligned buffers

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

bkietz commented on code in PR #35565:
URL: https://github.com/apache/arrow/pull/35565#discussion_r1196745002


##########
cpp/src/arrow/util/align_util.h:
##########
@@ -70,8 +70,26 @@ inline BitmapWordAlignParams BitmapWordAlign(const uint8_t* data, int64_t bit_of
 namespace util {
 
 // Functions to check if the provided Arrow object is aligned by the specified alignment
+
+/// \brief if this is specified in one of the CheckAlignment or EnsureAlignment functions
+/// then the funciton will ensure each buffer is suitably aligned for the data type of the

Review Comment:
   ```suggestion
   /// then the function will ensure each buffer is suitably aligned for the data type of the
   ```



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