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

Posted to github@arrow.apache.org by "WillAyd (via GitHub)" <gi...@apache.org> on 2023/06/23 21:32:00 UTC

[GitHub] [arrow-adbc] WillAyd commented on pull request #844: build(c): force C++11 for drivers for R's sake

WillAyd commented on PR #844:
URL: https://github.com/apache/arrow-adbc/pull/844#issuecomment-1605004614

   Guess all the utils stuff is complaining about NanoArrow's `GetInt` method returning a 64 bit integer, while some of the fields we are assigning to are 16/32. I can add some more safety checks there in a separate PR if it helps


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