Skip to content

Switch build from SQLite to DuckDB for penguins DB#7

Open
Pavan0277 wants to merge 1 commit intogvwilson:mainfrom
Pavan0277:feat/duckdb-switch
Open

Switch build from SQLite to DuckDB for penguins DB#7
Pavan0277 wants to merge 1 commit intogvwilson:mainfrom
Pavan0277:feat/duckdb-switch

Conversation

@Pavan0277
Copy link

  1. Replace sqlite3 CLI with duckdb in common.mk for in-repo query targets.
  2. Add create_penguins_duckdb.py to generate DuckDB-format db/penguins.db.
  3. command in ./Makefile that generates db/penguins.db

@Pavan0277
Copy link
Author

Hi @gvwilson, could you please review this PR when you get a chance and merge it if everything looks good? Thanks!

@gvwilson gvwilson self-requested a review August 16, 2025 14:23
@gvwilson gvwilson added the change cleanup label Aug 16, 2025
@Pavan0277 Pavan0277 removed their assignment Aug 16, 2025
@gvwilson
Copy link
Owner

Thanks @Pavan0228 - I'll try it out and leave feedback this week.

@gvwilson
Copy link
Owner

Hi @Pavan0228 - after looking through this and thinking about how many other things would have to change, I've decided to stick with SQLite for now, and to create another short lesson on DuckDB similar to the one on PostgreSQL. I'll keep this PR open for reference, and may merge it into the branch where I do the DuckDB lesson so that your contribution shows up in the Git history. Thanks again for creating it - I appreciate it a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments