There appears to be a memory leak issue

[zhuscat]

This is the minimal reproducible repo I created: https://github.com/zhuscat/test-nitro-sqlite

1. git clone https://github.com/zhuscat/test-nitro-sqlite
2. cd test-nitro-sqlite
3. npx expo prebuild --platform ios
4. xed ios
5. Edit Scheme -> Build Configuration -> Release
6. Build and then run the current scheme

I'm running it on my iPhone using Xcode with the Build Configuration set to Release.

I keep clicking "Create user database and insert 10000 records" and notice that memory usage keeps increasing. Even after I stop clicking, the memory doesn't decrease at all.

[zhuscat]

Also, I tried writing the same logic in a Swift app, and I didn't see any big memory spikes. The memory usage didn't keep going up, either.

[mrousavy]

Hey!

This doesn't 100% mean that there is a memory leak.
All this means is that neither Young GC or Old GC considers the allocated objects to be relevant for collection.

Whether that's because they have very complex cyclic references and are still being used somewhere, or because the objects don't properly implement getMemorySize() from Nitro, or because the RAM is so big (running on a real device or on a Mac?) that it can just grow without hitting GC.

if you reload the app (e.g. by pressing r in metro console), does the memory go down then? If no, then that's definitely a memory leak.

[mrousavy]

I think HybridNativeQueryResult would be a good candidate for implementing getMemorySize() from Nitro (see Hybrid Objects | memorySize), cc @chrispader