bkietz edited a comment on pull request #8256:
URL: https://github.com/apache/arrow/pull/8256#issuecomment-725059297


   Sorry, my fault: the problem is I've moved the trait to arrow_exports.h so 
that arrowExports.cpp could access them because I was only thinking of the 
as_sexp usage. However compute.cpp can't always access them and so from_datum 
breaks. The solution is to ensure both the forward declarations and the traits 
are visible to all c++ code. The most straightforward way to accomplish that 
would be: inline arrow_exports.h into arrow_types.h (also entails moving the 
`r6_class_name` trait there)


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to