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