-
Notifications
You must be signed in to change notification settings - Fork 893
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
[URGENT] Performance bug in Edge, Firefox #461
Comments
updated the sample stripping irrelevant code. |
I've also seen some serious performance issues with Edge also. Sometimes it is on par with Chrome (so almost instantaneous), and other times it is anywhere from 30s to 2min to forever. Have not tested within Firefox. Chrome works and behaves perfectly. I suspect this has started more recently (in the past few months), either just due to the type of queries I've used more recently, or due to firestore/edge updates. I don't have the time to investigate which atm unfortunately. |
For whoever this may help..
Since I'm running a very simple build of webpack. I'm suspecting there's some problem with the node build of firebase which causes this issue. |
FYI- This is on our radar (I've been working with support on your case) and we are looking into it. Regarding the support request from 3 months ago, there was a specific issue with Edge that we addressed in 4.10.0 (https://firebase.google.com/support/release-notes/js#4.10.0). I'm unsure if this new issue is related and it seems to be much harder to reproduce. In the repro I have, it only happens around 1 out of 5 times and as you noticed it only happens after compiling with webpack, which suggests it may be some sort of minification bug (FYI- the node build wouldn't work at all in a browser, so that's not the issue). If anybody has a minimal (without webpack) or more consistent repro (happens every time), that would be extremely helpful. Else, I'm looking into it, but it's going to take some time to isolate. |
The repro I provided you runs webpack in the minimal way, which only bundle the files, but without actual code transformation, so there's no minification here. |
Ahh, thanks for pointing that out. I hadn't looked at your latest repro yet (I had independently found that using the CDN assets from gstatic didn't reproduce the problem). That is indeed interesting. I think when you say "nodejs" you're probably generically referring to our npm package, but it actually contains both a browser version and a nodejs version:
And the two are quite different. Using the nodejs code in a browser wouldn't work at all. :-) As for the difference between the npm package code and the CDN code, that is a good question. They come from the same sources, but the CDN code is bundled and minified, etc. I'll continue to investigate. Thanks. |
I am also running into this issue with an updated version of firebase. Using the vanilla create-react-app, which does run webpack. This issue occurs for me every time in edge. https://youtu.be/lyPEY_F9nEs Locally the call times out. When deployed here it takes about thirty seconds. This is the call that is hanging for me:
Let me know if you'd like to see more code for a repro. |
So I think I figured out what's going on. This is the same bug that we investigated / fixed before (fixed by #509), but we didn't release a new version of @firebase/webchannel-wrapper... so if you're building with webpack you didn't get the fix. But if you used the prebuilt client (firebase-firestore.js) you did get the fix. I'll work with @jshcrowthe to get this resolved. |
One note though: The fix that was made is only applicable to Edge. If anybody is seeing issues with Firefox, please open a separate issue. |
Awesome! Thx for all of your hard work! |
Thanks so much for working through this! This is quite a big bug that I just stumbled upon when testing my application on Edge. Let me know if I can do anything to help 👍 |
You're welcome. :-) We believe everything should be fixed in our next release, targeted for Thursday. You can |
I caught the issue on Firefox by suprise while testing the bug on browserstack. I'll run more tests and will close the issue if nothing else comes up. |
Please wait until the new release is published :) |
edge 15-16: pass; Since the latest version of firefox are passing the test will consider the issue to be fixed. |
A similar problem is existing in Internet Explorer 11. |
This issue has been closed for months and I'm not aware of any active IE or Edge performance issues being tracked. Please open a new issue with as much detail as possible about what you're expecting and what you're seeing. Thanks! |
Querying firestore collections and documents in Edge browser is either instant or with a 30s delay about every other time you load the app; Firefox fails completely.
[REQUIRED] Describe your environment
[REQUIRED] Describe the problem
Steps to reproduce:
match /channels/{document=**} { allow read; }
Relevant Code:
https://stackblitz.com/edit/angular-xldoe5
The text was updated successfully, but these errors were encountered: