You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "zanmato1984 (via GitHub)" <gi...@apache.org> on 2025/03/04 05:43:31 UTC

Re: [PR] GH-45652: [C++][Acero] Unify ConcurrentQueue and BackpressureConcurrentQueue API [arrow]

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

   > > Also, would you please write a small test case for the problem you are hitting?
   > 
   > No really sure what to test. IMHO ConcurrentQueue API is not well defined. Not that it causes issue in current state of the code but it leads to bad uses and bugs (like when I tried to reuse the component).
   > 
   > I think initially I did not describe this PR correctly. I created na [issue ](https://github.com/apache/arrow/issues/45652) with more details. I think it all boils down to confusing API in ConcurrentQueue and BackpressureConcurrentQueue.
   
   Thanks for creating the issue. I can add some test cases to this PR if you don't mind @gitmodimo .


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