Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Queue timeout before completed restoration #973

Closed
zgordon02 opened this issue Jul 10, 2024 · 21 comments
Closed

Queue timeout before completed restoration #973

zgordon02 opened this issue Jul 10, 2024 · 21 comments
Labels
bug Something isn't working 🚧 Work In Progress

Comments

@zgordon02
Copy link

zgordon02 commented Jul 10, 2024

  • analytics-react-native version:
    ^2.19.2
  • Integrations versions (if used):
    N/A
  • React Native version:
    0.72.1 AND 0.74.3
  • iOS or Android or both?
    BOTH

Screenshot 2024-07-10 at 1 04 04 PM

Steps to reproduce
Send various screen and track events

Here is an example screen event I am sending:
SCREEN event saved {"name": "TEST_SCREEN", "properties": {"_environment": "LOCAL", "_otaVersion": "N/A"}, "type": "screen"}

Expected behavior
No warning
Actual behavior
This warning appears multiple times per session. However, the events are shown properly in the segment console

@zgordon02 zgordon02 added the bug Something isn't working label Jul 10, 2024
@zgordon02
Copy link
Author

Important to note that I have left almost all options to their default:

    segmentClient.current = createClient({
      writeKey: environment?.segmentKey,
      trackAppLifecycleEvents: true,
    });

@mayfairr
Copy link

mayfairr commented Jul 13, 2024

++ Also getting this.

"@segment/analytics-react-native": "^2.17.0",

@umairmaqbool3
Copy link

I am also getting same warnings...
checkerror

@mcoeur
Copy link

mcoeur commented Jul 18, 2024

Same here with version 2.19.2.
Adding a comment to get notifications on this thread 👌

@seanadkinson
Copy link

Also seeing these warnings spewed in my console.

@MarkKravchuk
Copy link

Same here, any ideas how to suppress or silence these warnings?

@zgordon02
Copy link
Author

zgordon02 commented Jul 18, 2024

Same here, any ideas how to suppress or silence these warnings?

I have not tried this, but the code should be something like this:

import { LogBox } from 'react-native';
// ...
LogBox.ignoreLogs(["An internal error occurred: Error: Queue timeout before completed restoration"]);

That being said, I would not recommend ignoring the warning. Segment should fix it

@leethree
Copy link
Contributor

I think this is caused by this recent change #943

The QueueFlushingPlugin has a default timeout of 500ms and it will fail if it's not intialized within that time. But in our testing it sometimes takes 2 seconds for the entire thing to be ready. So it will block every events from being sent.

@zgordon02
Copy link
Author

So it will block every events from being sent.

FWIW I have not noticed events dropped while seeing this warning. It appears to be functioning normally

@anon-r-7
Copy link

Also getting this, following.

@CoryWritesCode
Copy link

also seeing this after upgrading

@itajenglish
Copy link

itajenglish commented Aug 11, 2024

+1

@DAVIS-PYTH
Copy link

Commenting to get updated

@itajenglish
Copy link

I think this has been fixed in the last release. I no longer these warnings anymore after updating.

@DAVIS-PYTH
Copy link

Thanks @itajenglish

installing version 2.19.4 fixed this for me.

PS: I reached out on LinkedIn

@zgordon02
Copy link
Author

Seeing that others have mentioned it's fixed in the latest release, I'm closing this issue.

@dhatGuy
Copy link

dhatGuy commented Aug 19, 2024

It might be related. After upgrading to 2.19.4, I keep getting this in the log

INFO Flush triggered but queue restoration and settings loading not complete. Flush will be retried.

image

@DAVIS-PYTH
Copy link

same here with @dhatGuy

@zgordon02
Copy link
Author

zgordon02 commented Aug 19, 2024

@dhatGuy I could repoen this issue, but that warning may be unrelated to this one. Can you create a new issue?

@dhatGuy
Copy link

dhatGuy commented Aug 19, 2024

@dhatGuy I could repoen this issue, but that warning may be unrelated to this one. Can you create a new issue?

Yes, I have
#989

@anon-r-7
Copy link

+1, also getting "Flush triggered but queue restoration and settings loading not complete. Flush will be retried" only after the upgrade.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🚧 Work In Progress
Projects
None yet
Development

No branches or pull requests