flushEvents CPU drain

[cvb941]

Version

3.11.3

Steps to Reproduce

Run on emulator, Initialize the SDK.

Expected Result

Minimal CPU usage

Actual Result

Function flushEvents is called in a loop forever. CPU usage gets high and triggers a lot of GCs.

[cvb941]

The problem is here:

posthog-android/posthog/src/main/java/com/posthog/internal/PostHogSendCachedEventsIntegration.kt

Line 174 in 6185d73

val event = config.serializer.deserialize<PostHogEvent?>(theInputStream.reader().buffered())

listFiles contains only a replay-queue file, which fails to get deserialized into a PostHogEvent (result is null) which causes the later events.isNotEmpty() if statement to not pass thus never deleting the file in listFiles

[marandaneto]

Hello @cvb941 thanks for the report.
The event should be serializable though, very strange that it does not, can you send me a copy of the file in the disk so I can try this out as well?

[cvb941]

I checked the file, and it is empty (0 bytes)

[marandaneto]

I checked the file, and it is empty (0 bytes)

can you reproduce to actually write this file again as empty 0 bytes? if the bug is in the serialization, this should be addressed first.

[cvb941]

I have no idea how that happened. I just turned on my app in the emulator and noticed the cpu usage. After I deleted the empty file it didn't come back and the issue is not there.

[marandaneto]

I have no idea how that happened. I just turned on my app in the emulator and noticed the cpu usage. After I deleted the empty file it didn't come back and the issue is not there.

I tried to reproduce this issue and it does not happen at all.
#232 should prevent that at least