We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
firebase-tools: all
Platform: all
In the export code for Firestore we delete all the existing export data before the new data is written:
firebase-tools/src/emulator/controller.ts
Line 782 in 0641bd2
This can cause data loss if the export fails. It can also cause issues like #3487 where a clean shutdown puts the emulator in a bad state.
Instead we should:
tmp
Hard to reproduce, but see above.
Never lose data unless there is better data.
See above.
The text was updated successfully, but these errors were encountered:
samtstern
Successfully merging a pull request may close this issue.
[REQUIRED] Environment info
firebase-tools: all
Platform: all
[REQUIRED] Test case
In the export code for Firestore we delete all the existing export data before the new data is written:
firebase-tools/src/emulator/controller.ts
Line 782 in 0641bd2
This can cause data loss if the export fails. It can also cause issues like #3487 where a clean shutdown puts the emulator in a bad state.
Instead we should:
tmp
directorytmp
data[REQUIRED] Steps to reproduce
Hard to reproduce, but see above.
[REQUIRED] Expected behavior
Never lose data unless there is better data.
[REQUIRED] Actual behavior
See above.
The text was updated successfully, but these errors were encountered: