jsonb instead of text for json value (copy)

opral / lix-sdk

1 stars 0 forks source link

jsonb instead of text for json value (copy) #46

Open samuelstroschein opened 3 weeks ago

samuelstroschein commented 3 weeks ago

Context

For simplicity reasons we choose TEXT as type for JSON columns so that JSON.stringify() works instead of something like https://github.com/kysely-org/kysely/issues/209#issuecomment-1306187965

Proposal

Implement JSON columns as JSONB for efficiency.

samuelstroschein commented 3 days ago

The inlang SDK has a rough implementation that could work for lix too MESDK-173.

If the inlang SDK plugin works for lix as well, we should choose it and close this ticket. As long as we use BLOB for the columns, a few bugs here and there with parsing/serialization are OK because they don't lead to a breaking change.