bdunlay commented on pull request #7939: URL: https://github.com/apache/arrow/pull/7939#issuecomment-673016929
For example the lines you pointed out to me: [src/arrow/util/trie.h](https://github.com/apache/arrow/blob/master/cpp/src/arrow/util/trie.h#L106-L108) If this is a debug build, the function is included in the header, and the assert is evaluated. If it's a release build, the function is omitted, and the assert is obviously never evaluated. But if you removed the macro completely, in a release build, the function exist but would no-op (assert disabled). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
