paleolimbot opened a new pull request, #39969:
URL: https://github.com/apache/arrow/pull/39969

   
   ### Rationale for this change
   
   The integration tests and documentation build is failing
   
   ### What changes are included in this PR?
   
   Instead of relying on how reticulate converts an R external pointer, use a 
Python integer instead. We can't use an R integer (because they're only 32 
bits); we can't use an R double (because the static cast to/from uintptr_t is a 
bit iffy); however, we can use Python to convert a string to Python integer. 
This is probably how I should have written it the first time but it didn't 
occur to me at the time.
   
   ### Are these changes tested?
   
   Yes, covered by existing tests.
   
   ### Are there any user-facing changes?
   
   No


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to