search

duckdb / community-extensions

https://duckdb.org/community_extensions
143 stars 26 forks source link

httpserver extension #142

Closed lmangani closed 1 month ago

lmangani commented 1 month ago

Drafting a PR for the experimental DuckDB httpserver extension

carlopi commented 1 month ago

I think I fixed the problem you bumped on in CI, can you try merging with main? Thanks

lmangani commented 1 month ago

@carlopi not sure what's new but now docker commands are failing....

carlopi commented 1 month ago

That should also be fixed now, sorry for the inconvenience, can you give it another try? Thanks a lot

lmangani commented 1 month ago

Trying with review mode 👍

carlopi commented 1 month ago

@lmangani: I took the liberty of merging both minor changes, feel free to reopen a PR reverting them.

lmangani commented 1 month ago

Thanks @carlopi the changes are spot on and much appreciated. We'll update the extension ref again today to add the embedded UI and some other features being completed

carlopi commented 1 month ago

Should I block the publishing? That's easy on my side.

lmangani commented 1 month ago

If its fine to have a 0.0.1 published, it would help us troubleshoot a few real-life scenarios faster. I doubt anyone will use it other than for playing around and we'll do fast patches if we find hot issue 👍