Skip to content

Add Linq2DB to benchmarks#20

Merged
PhenX merged 8 commits intomasterfrom
feature/add-linq2db-benchmark
May 24, 2025
Merged

Add Linq2DB to benchmarks#20
PhenX merged 8 commits intomasterfrom
feature/add-linq2db-benchmark

Conversation

@PhenX
Copy link
Copy Markdown
Owner

@PhenX PhenX commented May 21, 2025

No description provided.

@SebastianStehle
Copy link
Copy Markdown
Collaborator

SebastianStehle commented May 22, 2025

Why have you removed EFCore_SaveChanges. I think most people are interested in this.

Alos needs an update for MySQL

@PhenX
Copy link
Copy Markdown
Owner Author

PhenX commented May 22, 2025

Yeah, I hesitated to let it, but it takes so much time...
I'm also planning to add MySql too. Did you bench it ?

@PhenX
Copy link
Copy Markdown
Owner Author

PhenX commented May 22, 2025

I think I'll change N to 100 000, do you think it's enough ? results are pretty linear between 100K, 1M or 10M

@SebastianStehle
Copy link
Copy Markdown
Collaborator

I only added the benchmark, but had no patient to run it through. I would disable or reduce all the warmup stuff and just make one run, because it has very little bit benefit in this case. I would go to 1M

PhenX-Pro and others added 7 commits May 23, 2025 15:36
# Conflicts:
#	tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark/LibComparator.cs
#	tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark/PhenX.EntityFrameworkCore.BulkInsert.Benchmark.csproj
…ht with provider specific import and simplify benchmark
…g it right with provider specific import and simplify benchmark"

This reverts commit 0ba0264.
@PhenX PhenX marked this pull request as ready for review May 24, 2025 08:50
@SebastianStehle
Copy link
Copy Markdown
Collaborator

BulkExtensions work with MySQL. Why is there N/A? Otherwise: LGTM

@PhenX
Copy link
Copy Markdown
Owner Author

PhenX commented May 24, 2025

There is an error from the lib, I did not investigate further. Anyway we are still far ahead of others :)

@PhenX PhenX merged commit 1f21ee4 into master May 24, 2025
1 check passed
@SebastianStehle
Copy link
Copy Markdown
Collaborator

Yes. 90% is because you have decided to use the Reader and not data tables. Which was a very good decision. The rest is probably irrelevant.

@PhenX
Copy link
Copy Markdown
Owner Author

PhenX commented May 24, 2025

Yeah, but https://entityframework-extensions.net/ use data readers too, so that's weird. Anyway that's great! We'll try to keep the first place

@PhenX
Copy link
Copy Markdown
Owner Author

PhenX commented May 25, 2025

@SebastianStehle just to be sure about our results vs ef extensions, I made a fork of their benchmark ok .net fiddle
https://dotnetfiddle.net/P9kTIu

I'm reassured

image

@SebastianStehle
Copy link
Copy Markdown
Collaborator

Nice

@PhenX PhenX deleted the feature/add-linq2db-benchmark branch May 26, 2025 09:32
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.

3 participants