kou commented on code in PR #41163: URL: https://github.com/apache/arrow/pull/41163#discussion_r1562240202
########## cpp/src/arrow/util/config.h.cmake: ########## @@ -57,6 +57,7 @@ #cmakedefine ARROW_GCS #cmakedefine ARROW_HDFS #cmakedefine ARROW_S3 +#cmakedefine ARROW_TEST_MEMCHECK Review Comment: This will not be useful to third-party code but it helps us to know what macros are defined in a build. If we use `add_definitions(-DARROW_VALGRIND)`, we need to use `ninja -v` and find a `-DARROW_VALGRIND` from a long command line. -- 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]
