iChauster commented on code in PR #13366:
URL: https://github.com/apache/arrow/pull/13366#discussion_r910402222


##########
cpp/src/arrow/util/benchmark_util.h:
##########
@@ -15,10 +15,24 @@
 // specific language governing permissions and limitations
 // under the License.
 
+#include <condition_variable>
+#include <mutex>

Review Comment:
   @westonpace , sorry if this is trivial, but I'm having some trouble here.
   
   I committed my attempt -- it seems like the compiler is having trouble 
finding third party dependencies imported in `benchmark_util.h` (like 
benchmarks, gtest) after including `benchmark_util.cc` in the 
`src/arrow/CMakeLists.txt`. (the edit in cmake is currently commented out)
   
   I'm not too familiar with how to specify that those are needed in `CMake`, 
and if that would be the right solution.
   
   Let me know if I'm missing a step!



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

Reply via email to