nevi-me commented on pull request #8200:
URL: https://github.com/apache/arrow/pull/8200#issuecomment-724493404


   > From what I have read in this PR, it looks good so far. Is there anything 
blocking it?
   
   @jorgecarleitao the actual integration tests not passing, are the blocker 😅
   
   There were 10 failures in the last run, logical equality related. It's 
actually odd, because on my local testing I get 120+ failures between Rust and 
the other langs (except C++, that passes).
   
   I'll monitor 
https://github.com/apache/arrow/pull/8200/checks?check_run_id=1378263093 in the 
background, if it passes; we can merge this (if you don't have any outstanding 
queries/concerns).
   
   I used your suggested implementation of logical equality 
(https://github.com/apache/arrow/pull/8200/commits/dde96c6d2628c7c0eedc4f7c15061e746b86fb36)


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


Reply via email to