Timur Kristóf
Timur Kristóf
@nwbrad No problem, we can keep this open for a few more weeks. In the meantime if any other user encounters the same or similar issue, they can chime in...
Can you indicate in the commit messages that these only affect the TypeScript binding and not the rest of `node-lmdb`? Thanks.
@degifted Thank you for the suggestion! What do you think, what exactly would be the performance benefit of this change?
@degifted What is the array for?
@degifted Ah, so you basically use the array instead of creating an object. I think I get it now. Maybe I can also suggest a better way of doing it.
@degifted Can you give me a performance test which shows the performance advantages of the array approach?
@degifted I'll look into your patches and merge those of them that I think would be useful to others too. > I also added UInt32 key type since existing Number...
@degifted What I meant by the performance test is: can you create a pull request with a test that I can run on my own machine?
Can you please give me a test case with which you can reproduce the problem?
@hanweifish I can't really tell what it is by just looking at that code fragment. Please give me a **full** test case which can reproduce the problem.