zeroshade commented on PR #3320:
URL: https://github.com/apache/arrow-adbc/pull/3320#issuecomment-3211532970

   @paleolimbot How's the updated version? 
   
   I've put the checks for `CONDA_PREFIX` behind the existence of the 
`CONDA_BUILD` env var at compile time (since it seems all conda build utilities 
will set that env var during build) in both the C and Rust driver managers. 
   
   I've then added the `additional_search_path_list` argument to 
`AdbcFindLoadDriver` and added an appropriate setter for `AdbcDatabase` to 
manipulate it. Finally, I updated the python driver manager to check if it's in 
a virtual environment, and if it is it will add the virtual environment prefix 
+'/etc/adbc' to the search path list. (And updated the R implementation 
accordingly).
   
   Thoughts?
   
   @lidavidm @ianmcook @amoeba Did I cover everything?


-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to