Increase string pool limit #12
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Wow, lots of action in the CSV Battle Royale recently!
Looks like Cursively took the crown by using my own tricks against me. I think that I can take it back by increasing the string pool limit... at least it does on my machine. I had landed on 64 just by eye-balling the contents of the test csv, but it turns out that there are some larger strings in there too.
Are you regretting putting these benchmarks together yet?! I'm sure its a pain to be rerunning these and updating the blog, so don't bother unless/until some other interesting contenders appear.