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

Adjust revs_limit in existing test to reproduce RangeError #8396

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jlami
Copy link
Contributor

@jlami jlami commented Oct 26, 2021

This should fail for indexeddb on the following test:
test.bulk_docs.js-local
2839 implement revs_limit

@jlami
Copy link
Contributor Author

jlami commented Oct 27, 2021

Hmmm, I guess this only fails on Windows Chrome.
After checking Firefox I see that it has no problems on Windows.

I'm not sure if reporting this as a bug on Chrome is worth it, but here is the bug tracker: https://bugs.chromium.org/p/chromium/issues/detail?id=1263612

@jlami
Copy link
Contributor Author

jlami commented Oct 28, 2021

The Chromium issue got tagged Wontfix, so we might have to decide if we want to do something about this on our side. I'll make a separate issue for that.

@jlami jlami closed this Oct 28, 2021
Linux does not have a problem with 1000 revisions, let's see if 4900 will work
@jlami
Copy link
Contributor Author

jlami commented Oct 28, 2021

Test if higher revision count will trigger this.

@jlami jlami reopened this Oct 28, 2021
@jlami
Copy link
Contributor Author

jlami commented Oct 28, 2021

The pouchdb-server failure is unexpected, but the change to use 4900 as revs_limit reproduces the RangeError now on both Firefox and Chrome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants