lidavidm commented on PR #4080: URL: https://github.com/apache/arrow-adbc/pull/4080#issuecomment-4044623129
@zeroshade @amoeba venvs don't work because we add the additional search path `<venv root>/etc/adbc/drivers`, and have no way to configure profiles. Options are: - Add the other search path; but then we search both paths for both manifests and profiles, and we risk confusing one for the other. - We could additionally require profiles to have a different extension. - Instead, we could add a second API function to set another search path. That's starting to get unwieldy. - We could ditch the API function, and instead use normal driver options to set additional search paths for drivers/manifests. -- 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]
