Enable Full-Text Search by Default #79
Replies: 3 comments
-
Another argument for:
And against:
Unfortunately, I don't have any real data on how developers use (or want to use) wa-sqlite, so it's hard to evaluate that trade off and I should probably default to not making things bigger and slower for the existing user base. Perhaps the way to go with this would be to include two .wasm builds (each with and without Asyncify), one with the current settings and an additional "kitchen sink" variation that favors features over size and more thorough testing. |
Beta Was this translation helpful? Give feedback.
-
Just to confirm: that is indeed the case. For the wasm build, since it's a new target platform for us, we've opted against including deprecated features by default (which includes, for example, the "double-quoted-string" support). |
Beta Was this translation helpful? Give feedback.
-
I added a dist-xl build target with additional features enabled, including FTS5. This target is not built (or tested) by default - instead I'll attach the artifacts produced to the release message. |
Beta Was this translation helpful? Give feedback.
-
This is just a suggestion, since there are tradeoffs.
The good:
sqlite3
. Since FTS 3, 4, 5 are enabled in the default build of sqlite most users are likely familiar with.Makefile
in order to use FTS features (i.e. via questions like Full text support #77)The bad:
What would need to be done:
In the Makefile, add new
WASQLITE_DEFINES
:Beta Was this translation helpful? Give feedback.
All reactions