-
Notifications
You must be signed in to change notification settings - Fork 6
PRAGMA page_size has no effect, and SQLITE_DEFAULT_PAGE_SIZE 512 has no effect. #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
This is for the Arduino environment, not the ESP-IDF environment. This repository is for the ESP-IDF environment. Your PR to fix this issue is welcome. |
You mentioned your fork is only optimised for the building procedure for IDF.
It now looks like the library is poorly ported to IDF. |
The original is here. We believe this fork functions exactly the same as the original. Please check if this issue exists in the original. |
The original is not broken; the fork is. This has no effect: https://github.com/nopnop2002/esp32-idf-sqlite3/blame/a27228d70b9bcf92149a28e7ea490661f9a9ca4d/components/esp32-idf-sqlite3/config_ext.h#L23 The original: siara-cc/esp32_arduino_sqlite3_lib#92 (comment) |
siara-cc/esp32_arduino_sqlite3_lib#92 (comment) This is NOT original. The original of this fork is here. |
Here is the original maintainer Please read carefully. This directive |
The original version of this fork can be found here. No code has been changed from the original. I only added an example and changed Make to Cmake.
This directive SQLITE_DEFAULT_PAGE_SIZE has no effect in ORIGINAL. |
So, updating the README file will help people understand the library's supported level. |
I added this to README.
|
Hello,
As described at this issue post: siara-cc/esp32_arduino_sqlite3_lib#92 (comment)
I'm trying to reduce SQLite's memory footprint by lowering the page size, but this command or definition has no effect.
The author of the original lib says he changed page_size some time ago: siara-cc/esp32_arduino_sqlite3_lib#92 (comment)
The text was updated successfully, but these errors were encountered: