Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

Sort keys by type, same as IndexedDB Second Edition #185

Closed
wants to merge 1 commit into from

Conversation

vweevers
Copy link
Member

NB. This has a performance impact on all operations (any type of read or write) but I'm not doing benchmarks yet, because I strongly believe we must first make a collective decision on the desired behavior, then optimize for that. I'm opening a separate ticket for that discussion.


As for this PR: it's feature-complete. I wrote the comparator by following IDB specs and copied tests from level-js to verify it.

@vweevers vweevers added the semver-major Changes that break backward compatibility label Jun 21, 2019
@vweevers vweevers mentioned this pull request Jun 21, 2019
cmp.js Show resolved Hide resolved
@vweevers
Copy link
Member Author

If someone wants this functionality, please speak up now, because I'm proposing to drop key types other than strings and buffers entirely: #186 (comment).

@vweevers vweevers closed this Aug 10, 2019
@vweevers vweevers deleted the idb-sort-order branch August 10, 2019 07:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
semver-major Changes that break backward compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants