search

databrickslabs / databricks-sdk-r

Databricks SDK for R (Experimental)
https://databrickslabs.github.io/databricks-sdk-r/
Apache License 2.0
19 stars 4 forks source link

Switch to httr2 #8

Open hadley opened 1 year ago

hadley commented 1 year ago

httr is mostly in maintenance mode, so I'd recommend using httr2 instead. It includes nice features for automatically retrying on failure, e.g. https://httr2.r-lib.org/reference/req_retry.html.

nfx commented 1 year ago

Thank you for the feature request! Currently, the team operates in a limited capacity, carefully prioritizing, and we cannot provide a timeline to implement this feature. Please make a Pull Request if you'd like to see this feature sooner, and we'll guide you through the journey.

nfx commented 1 year ago

one of the iterations was using httr2, though I found it difficult to inject headers to the request as well as using the VERB method https://github.com/databrickslabs/databricks-sdk-r/blob/main/R/api_client.R#L188-L217

hadley commented 1 year ago

It should be easier to do that with httr2 😄 You might have missed https://httr2.r-lib.org/reference/req_method.html. I can also take a look at a PR.

zacdav-db commented 1 year ago

I may be able to do this once I finish some other work given that brickster already uses httr2.