Skip to content

Conversation

Chriztiaan
Copy link
Contributor

@Chriztiaan Chriztiaan commented Jan 9, 2025

Description

Updating the OPSqlite dependency to 11.2.13, this allows us to remove the ./ workaround needed for having multiple connections to the same DB (improved upstream).

While testing on a React Native project, using these two tests from RNQS - I found that the read and write locks weren't working as expected. The readLock and writeLock implementations are now correctly overriden for React Native.

Performance

Based on my tests with an iOS simulator and Android device there weren't astronomical performance gains.

iOS Simulator

Branch Iterations Time 1 (ms) Time 2 (ms)
Original 100k 35563.37 35752.51
Latest OPSQLITE 100k 34720.14 35457.24

Real Android Device

Branch Iterations Time 1 (ms) Time 2 (ms)
Original 100k 101075.12 101904.55
Latest OPSQLITE 100k 94097.53 96879.68

@changeset-bot
Copy link

changeset-bot bot commented Jan 9, 2025

🦋 Changeset detected

Latest commit: 61c0993

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@powersync/op-sqlite Patch
@powersync/react-native Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Chriztiaan Chriztiaan marked this pull request as ready for review January 9, 2025 14:35
rkistner
rkistner previously approved these changes Jan 9, 2025
# Conflicts:
#	demos/react-supabase-todolist/package.json
#	packages/web/package.json
#	pnpm-lock.yaml
…ja/powersync-js into chore/update-opsqlite-11.2.13
@Chriztiaan Chriztiaan merged commit b38bcdb into main Jan 10, 2025
6 checks passed
@Chriztiaan Chriztiaan deleted the chore/update-opsqlite-11.2.13 branch January 10, 2025 13:20
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