Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

duckdb/pg_duckdb

Y-- created a review comment on a pull request on duckdb/pg_duckdb
Yep it is a no-op, cf. [here](https://github.com/duckdb/duckdb/blob/52b43b166091c82b3f04bf8af15f0ace18207a64/src/main/extension/extension_install.cpp#L538-L548)

View on GitHub

Y-- created a review on a pull request on duckdb/pg_duckdb

View on GitHub

JelteF created a review comment on a pull request on duckdb/pg_duckdb
Is `INSTALL` an actual no-op if the extension is already installed? Or does it do something semi-expensive like network requests, which would be good to avoid on every DB initialization.

View on GitHub

JelteF created a review on a pull request on duckdb/pg_duckdb

View on GitHub

bleskes pushed 1 commit to readme_tweaks duckdb/pg_duckdb
  • Update README.md feedback f9c461a

View on GitHub

bleskes pushed 1 commit to readme_tweaks duckdb/pg_duckdb

View on GitHub

Y-- created a comment on an issue on duckdb/pg_duckdb
Yeah exactly.

View on GitHub

Y-- pushed 1 commit to yl/reload-extensions duckdb/pg_duckdb
  • Update pgduckdb_duckdb.cpp d1c370f

View on GitHub

JelteF created a review comment on a pull request on duckdb/pg_duckdb
nit: This part of the query is now missing some indentation that it had before. I think it looked better with the indentation.

View on GitHub

JelteF created a review comment on a pull request on duckdb/pg_duckdb
```suggestion pg_duckdb relies on DuckDB's vectorized execution engine to read and write data to object storage bucket (AWS S3, Cloudflare R2, or Google GCS) and/or MotherDuck. The follow two sect...

View on GitHub

JelteF created a review on a pull request on duckdb/pg_duckdb
Some small comments but other than that LGTM.

View on GitHub

JelteF created a review on a pull request on duckdb/pg_duckdb
Some small comments but other than that LGTM.

View on GitHub

Y-- pushed 1 commit to yl/reload-extensions duckdb/pg_duckdb
  • Update pgduckdb_duckdb.cpp 84720dc

View on GitHub

Y-- created a review comment on a pull request on duckdb/pg_duckdb
In the test we insert rows directly in `extensions` table, so we don't run the `install_extension` function. I hesitated using the function or making sure that no matter what we install the extensi...

View on GitHub

JelteF created a comment on an issue on duckdb/pg_duckdb
You mean allocate the `values` and `nulls` arrays once? Yeah that seems like an easy perf improvement.

View on GitHub

Y-- created a review on a pull request on duckdb/pg_duckdb

View on GitHub

JelteF created a review comment on a pull request on duckdb/pg_duckdb
Interesting, was this not needed before?

View on GitHub

JelteF created a review on a pull request on duckdb/pg_duckdb

View on GitHub

JelteF created a review comment on a pull request on duckdb/pg_duckdb
Same for a few lines below

View on GitHub

JelteF created a review on a pull request on duckdb/pg_duckdb

View on GitHub

JelteF created a review comment on a pull request on duckdb/pg_duckdb
nit: why swap this around? It has no functional difference here right? I checked to see if this is common duckdb style, and generally it seems postfix is preferred: ``` duckdb on  main via △ v3....

View on GitHub

JelteF created a review on a pull request on duckdb/pg_duckdb

View on GitHub

JelteF deleted a branch duckdb/pg_duckdb

fix-mistake-using-wrong-index

JelteF pushed 1 commit to main duckdb/pg_duckdb
  • Lookup is_null correctly in case of pushed down filters (#322) While running TPCDS queries a few of them crashed Pos... 76872f1

View on GitHub

JelteF closed a pull request on duckdb/pg_duckdb
Lookup is_null correctly in case of pushed down filters
While running TPCDS queries a few of them crashed Postgres when using `duckdb.force_execution = true`. It turns out that happened because it would look at the NULL value for the wrong field. This f...
Y-- created a comment on an issue on duckdb/pg_duckdb
Great points. I actually had started re-working the code in here https://github.com/duckdb/pg_duckdb/compare/main...yl/custom-allocator as I had some of the same concerns. (note that this branch wa...

View on GitHub

Y-- created a review on a pull request on duckdb/pg_duckdb

View on GitHub

Y-- pushed 1 commit to yl/reload-extensions duckdb/pg_duckdb

View on GitHub

Y-- pushed 7 commits to yl/reload-extensions duckdb/pg_duckdb
  • Rename duckdb.execution to duckdb.force_execution (#314) Many people won't need to turn this setting on anymore now ... 379a0ff
  • README: Add shared_preload_libraries requirement (#319) We were missing this in the installation steps. Fixes #31... 0c2df0d
  • Don't crash if query throws 1059d77
  • Load extensions when table is updated 5f54b8d
  • Don't re-create connection 8db4ecb
  • Ignore jemalloc extension f6ec22b
  • PR Feedbacks 1eebeba

View on GitHub

Y-- created a review comment on a pull request on duckdb/pg_duckdb
Yeah - I actually wanted to make it so if one fails it doesn't arbitrarily fails all the following ones but didn't finish. I might do that in a different PR, but will revert in the meantime.

View on GitHub

Load more