zanmato1984 commented on PR #45674:
URL: https://github.com/apache/arrow/pull/45674#issuecomment-2723908097

   ```diff
   --git a/cpp/src/arrow/acero/concurrent_queue_internal.h 
b/cpp/src/arrow/acero/concurrent_queue_internal.h
   index a751db7026..40c7857d2a 100644
   --- a/cpp/src/arrow/acero/concurrent_queue_internal.h
   +++ b/cpp/src/arrow/acero/concurrent_queue_internal.h
   @@ -20,7 +20,11 @@
    #include <condition_variable>
    #include <mutex>
    #include <queue>
   +
    #include "arrow/acero/backpressure_handler.h"
   +#include "arrow/util/macros.h"
   +
   +#include MUTEX
    
    namespace arrow::acero {
    
   --git a/cpp/src/arrow/util/macros.h b/cpp/src/arrow/util/macros.h
   index af29fd636b..f3b36495d8 100644
   --- a/cpp/src/arrow/util/macros.h
   +++ b/cpp/src/arrow/util/macros.h
   @@ -129,6 +129,16 @@
    
    #endif  // ifndef NULLPTR
    
   +#ifndef MUTEX
   +
   +#  ifdef __cplusplus_cli
   +#    error "<mutex> not supported in C++/CLI"
   +#  else
   +#    define MUTEX <mutex>
   +#  endif
   +
   +#endif  // ifndef MUTEX
   +
    // ----------------------------------------------------------------------
    
    // clang-format off
   ```
   
   Hi @kou , does the workaround above make sense?


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

Reply via email to