Closed manojbalaji1 closed 4 years ago
In certain scenarios, druid url's are accessible explicitly through proxies, so added proxies as input parameter for connection which is passed to requests.post in Cursor
@donbowman @mistercrunch can you please review this PR?
pre-commit install as noted in README should help
pre-commit install
Trailing coma + rebase, otherwise LGTM
Completed requested changes
In certain scenarios, druid url's are accessible explicitly through proxies, so added proxies as input parameter for connection which is passed to requests.post in Cursor
@donbowman @mistercrunch can you please review this PR?