jorisvandenbossche commented on PR #37436:
URL: https://github.com/apache/arrow/pull/37436#issuecomment-1698747941

   > Are these changes tested?
   
   For testing, you can certainly add a test that it raises an error on 
non-Windows platforms (the C++ function should already do that if the path 
cannot be configured but is determined at compile time)
   
   I would maybe also use this function ourselves in for example the Appveyor 
build, by putting the downloaded db somewhere else and pointing to that 
location?


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