nealrichardson commented on pull request #8256:
URL: https://github.com/apache/arrow/pull/8256#issuecomment-725035841


   It's puzzling to me why that failure isn't on every build, but I can 
reproduce it locally. The issue is: 
   
   ```r
   min.Array <- function(..., na.rm = FALSE) {
     scalar_aggregate("min_max", ..., na.rm = na.rm)$GetFieldByName("min")
   }
   ```
   
   `scalar_aggregate("min_max", ...)` (aka compute::CallFunction) should return 
a `StructScalar` but instead it's returning just a `Scalar`, which doesn't have 
a `GetFieldByName` method.
   
   There is the Scalar/StructScalar handling in scalar.cpp 
https://github.com/apache/arrow/pull/8256/files#diff-15908f875c08c0199f6416241e0aa043509eed9741ba6e5bcb6113940ec61105
   
   But maybe (and on certain compilers only?) it's not getting picked up 
correctly in compute.cpp 
https://github.com/apache/arrow/pull/8256/files#diff-a138b87f0da58d824c72293eab62d9f352718dcb7f41a47ea7e1c3c84bbe27ddR136
 ?
   
   


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