Feat/remove nodesids map#1597
Conversation
Docs: banner we-are-hiring (handsontable#1595) * Add We are hiring banner to the docs page * Add We are hiring information to the readme file
|
Hi @Krizz! 👋 Thank you so much for taking the time to work on this PR and for signing the CLA! This is a really valuable contribution. I really appreciate the effort you put into this refactoring across multiple files. |
|
@Krizz thank you for your contribution. I'm merging this code to a feature branch to fix failing unit tests. Most likely, it will be included in the upcoming release |
fcb29f0
into
handsontable:feature/issue-1602
@sequba Thank you very much for taking care of this! I was not really looking forward fixing the unit tests and that's why the pull request stayed as draft. |
|
Hi @Krizz Thank you again for your contribution. We just released HyperFormula v3.2.0, which solves this issue. Here's the pull request if you'd like to dig deeper. |
Context
With large datasets like NY 2015 Tree Census nodeIds Map will exceed the maximum size. This will directly store the _graphId to node without the Map.
How did you test your changes?
Imported NY 2015 Street Tree Census dataset into hyperformula
Types of changes
Checklist: