-
Notifications
You must be signed in to change notification settings - Fork 578
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
[Firestore] SQLite error: Cannot perform this operation because there is no current transaction #115
Comments
Sorry we missed this when it came in. We qualify every release, running an application against it, verifying that everything works as expected. Errors on startup should be exceptionally rare. There's a few avenues we can take to figure this out. The first thing to do is enable logging with Secondarily, looking at the code here: https://github.com/firebase/firebase-android-sdk/blob/master/firebase-firestore/src/main/java/com/google/firebase/firestore/local/SQLitePersistence.java#L164 it looks like it's possible that if there were an exception in Finally if nothing else comes up it's worth describing the environment in which you're running this. Android version, device type, etc. It seems doubtful that this will turn up anything obvious given what you're seeing, but who knows? |
I'm experiencing same issue -> http://crashes.to/s/1ef7aa3c95d |
@pamartineza While it's helpful to know that others are experiencing the problem, we haven't seen this issue ourselves, so if you're able to reliably reproduce this, performing the steps outlined above would be useful. |
@wilhuff we are unable to reproduce the crash in any of our Test devices, we are just seeing it on Crashlytics. I don't know if it can be related but I can only add that this started happening just after migrating to AndroidX and upgrading to Firestore to 17.1.3 |
Do either of you see any exceptions earlier in the log? From my reading, it appears that |
I can't reproduce the issue so I can't get logs, but looking at the crash report http://crashes.to/s/1ef7aa3c95d it seems the problem just happened when users upgraded the App, now that most of the user base has migrated to latest version crashes are remitting |
Finally we reproduce the error just access some sections in our app and then pressing the recent apps key at the bottom of the device (I attached the image just to be more clear). Also if we press it repeatedly. I attached again the log, we updated the component to 17.1.3 version to verify if the error was solved but we have the same result on both.
After this, when we try to open the app again it crash without doing any action. Device: Samsung SM-A520F |
This is just the stack trace which doesn't really shed any more light on the situation. We'll try the steps you're describing but in the meantime since you're able to reproduce, could you enable logging with |
@wilhuff I have already sent you a more detailed log to your email. Thanks |
Can those seeing this confirm:
And lastly, if you have a full log of a repro you can provide, if you haven't already, that would be very helpful. As above, if it contains sensitive info you can email it to either @wilhuf ([email protected]) or myself ([email protected]). |
@gsoltis @wilhuff I provided a few additional logs and a firebase db dump for this issue to @samtstern after hitting this error repeatedly in an app and stumbling upon a stackoverflow post where he was requesting information. Feel free to contact him for those logs. In my specific circumstances, the error seems to have gone away following a full day of upgrade pains, including disabling fabric.io/crashlytics and various updates:
It's hard for me to specify exactly what caused the crash, but it did return once more after deleting firebase from my app data, before disappearing following the more thorough upgrade above. I can't say yet if it's gone permanently though. |
Does the database backup you supplied demonstrate the issue? Or is it now fixed running the latest version against that database? My best guess for what is happening is as follows:
This scenario resulted in an unexpected schema, and an exception is being thrown by a query. Unfortunately, this exception is masked by our transaction handling code, so we cannot see from this log what is causing it. Newer versions ( |
I can say as much as i know the supplied database exhibited the issue on the previous version, but after the upgrade it seems to be gone. I have never to my knowledge reverted to an earlier version of the database. |
If it's working on the current version, there's probably no need to do the bisect. I will keep poking at it a bit on my end to see if I can find out what the underlying exception is. But, failing that, the latest version should report the actual exception so that if it surfaces again it should be much easier to identify. |
Alright, I'll leave it be then. :)
Den ons 23 jan. 2019 19:00Greg Soltis <[email protected]> skrev:
… If it's working on the current version, there's probably no need to do the
bisect. I will keep poking at it a bit on my end to see if I can find out
what the underlying exception is. But, failing that, the latest version
should report the actual exception so that if it surfaces again it should
be much easier to identify.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#115 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAbmSYCvIrabPg97DU2Jz5603Ze6aUIWks5vGKMhgaJpZM4YQO6V>
.
|
I'm going to mark this as closed, as I believe it is fixed in the most recent version. If anyone is still encountering this issue, please reopen. |
Environment
Android Studio version: 3.2.1
Firebase Component: Firestore
Component version: 17.1.2
Problem
Steps to reproduce:
Almost immediately, when connected to firestore realtime events, we get this:
We have been researching about the issue related with the internal error on firestore module related with SQLite but we found nothing (just clear data on the application each time we restart the app). Our first option to tackle the problem was problably have multiple instances of firestore but this was not the case.
We are trying other options, but at the moment to avoid the crash we disable the persistence through the following code:
Could you help us with the issue?
The text was updated successfully, but these errors were encountered: