Skip to content

enh: refactor sqilte database to speed up while loading large dataset - #6266

Draft
shun2wang wants to merge 2 commits into
jasp-stats:developmentfrom
shun2wang:sqliteBetter
Draft

enh: refactor sqilte database to speed up while loading large dataset#6266
shun2wang wants to merge 2 commits into
jasp-stats:developmentfrom
shun2wang:sqliteBetter

Conversation

@shun2wang

@shun2wang shun2wang commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Improve the current database behavior with minimal workload. Now implement a chunked schema to store our tables. This avoids the hacky behavior of scaling SQLite's maximum column and significantly improves I/O performance for vary large datasets.I'm not sure if this is friendly to multiple datasets in the future, but it's not an issue in terms of I/O.

Befor:
image

image

@shun2wang
shun2wang marked this pull request as draft June 21, 2026 10:01
@shun2wang shun2wang changed the title enh: refactor sqilte database to speed up enh: refactor sqilte database to speed up for load large dataset Jun 21, 2026
@shun2wang shun2wang changed the title enh: refactor sqilte database to speed up for load large dataset enh: refactor sqilte database to speed up while loading large dataset Jun 22, 2026
@JorisGoosen

JorisGoosen commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@shun2wang dont work too much on this!

#5783 will require substantial changes to the database and Im getting near the end of it!
Unless you really think it won't conflict, but Ill leave the rebase to you then when the time comes.

@shun2wang

shun2wang commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

@shun2wang dont work too much on this!

#5783 will require substantial changes to the database and Im getting near the end of it! Unless you really think it won't conflict, but Ill leave the rebase to you then when the time comes.

If I look into #5783 its purpose is to load multiple datasets simultaneously, right? My PR aims to split and store a single large dataset, mainly to address the performance impact of manually enabling the maximum column limit in SQLite. I believe the goals are not conflicting.

I can rebase it on #5783,but maybe atfer #5783 be merged?

migrate legacy database to DataChunks from old version
@JorisGoosen

Copy link
Copy Markdown
Contributor

Splitting the tables might help, but I also think @RensDofferhoff is gearing up to replace sqlite entirely with something quicker.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants