Skip to content

Conversation

@nigrosimone
Copy link
Contributor

No description provided.

@krausest
Copy link
Owner

Hi Nigro, I'm not going to merge it as it doesn't have any measurable impact on the results (optimized is your version with id++ inlined and the preallocated array);
Screenshot 2025-05-10 at 20 29 30
Neither create rows (1k and 10k) nor update rows were not faster. Those are the benchmarks which should be impacted most by the change.

@krausest krausest closed this May 10, 2025
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.

2 participants