You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Unable to change page size in latest version #1021
Changing the page size has no effect when done either in the GUI or in SQL query window. Database is brand new, no tables yet. Have tried with all different journal modes too, no change.
@jjxtra Thanks for pointing this out! 😄 I believe this issue is fixed now. Can you download tomorrow's nightly build (or any later version) and double check that it's working for you?
Unfortunately the latest nightly build as of this morning is limiting the page size to 4096. So now when I run the two lines from the top of this issue, it gives 4096 as the page size.
@jjxtra Thanks for testing this so far. I've just made an extra change that should allow you to also change the page size from the SQL editor. Are you ok to double check that too using the next nightly build? 😄
Thanks for testing, @jjxtra, and sorry for the long delay!
I've just added an additional patch which you can test by downloading tomorrow's nightly build. With this it should work just fine using the SQL editor. However, you need to set the page size and, directly after that, vacuum the database, like this:
Activity
justinclift commentedon May 24, 2017
Oh that's different. We'll definitely need to look into that.
Thanks for pointing it out @jjxtra. 😄
Allow changing the page size pragma using the UI
MKleusberg commentedon Jun 2, 2017
@jjxtra Thanks for pointing this out! 😄 I believe this issue is fixed now. Can you download tomorrow's nightly build (or any later version) and double check that it's working for you?
jjxtra commentedon Jun 3, 2017
Absolutely, thanks for doing this. I had briefly taken a gander in the code but didn't see an obvious spot for the fix.
jjxtra commentedon Jun 12, 2017
Unfortunately the latest nightly build as of this morning is limiting the page size to 4096. So now when I run the two lines from the top of this issue, it gives 4096 as the page size.
justinclift commentedon Jun 12, 2017
Gaaah. Ok so we're in the right area of things, but still aren't quite over the finish line. 😉
MKleusberg commentedon Jun 17, 2017
Hmm, the limit should be 65536. I have no idea where the 4096 value might come from.
That said, my fix only changes the GUI pragma editor, not the SQL execution bit. Have you tried the GUI editor yet?
I'll also put a reference to issue #966 here. That issue is about not being able to edit pragmas from the SQL editor.
jjxtra commentedon Jun 17, 2017
This does work using the GUI.
Allow setting PRAGMAs from the Execute SQL tab
MKleusberg commentedon Jun 30, 2017
@jjxtra Thanks for testing this so far. I've just made an extra change that should allow you to also change the page size from the SQL editor. Are you ok to double check that too using the next nightly build? 😄
jjxtra commentedon Jul 1, 2017
Will try it tomorrow
jjxtra commentedon Jul 3, 2017
Unfortunately the pragma has no effect, page size remains previous value.
justinclift commentedon Jul 18, 2017
Damn, still needs more effort then to get it working.☹️
Thanks for testing @jjxtra. 😄
Allow VACUUM command in Execute SQL tab
MKleusberg commentedon Aug 11, 2017
Thanks for testing, @jjxtra, and sorry for the long delay!
I've just added an additional patch which you can test by downloading tomorrow's nightly build. With this it should work just fine using the SQL editor. However, you need to set the page size and, directly after that, vacuum the database, like this:
Also see here: https://www.sqlite.org/pragma.html#pragma_page_size
The GUI editor for pragmas does this automatically, with the SQL editor you'll need to do it yourself 😉
justinclift commentedon Nov 1, 2017
@jjxtra Do you have time/interest to try the latest nightly build and the PRAGMA+VACUUM thing (as per @MKleusberg's comment above)? 😄
jjxtra commentedon Nov 2, 2017
We can close this, it's all working great!
justinclift commentedon Nov 2, 2017
Awesome @jjxtra, thanks for checking. 😄