Add missing LINQ overloads for IRewritableQueryable#5
Conversation
There was a problem hiding this comment.
⚠️ Performance Alert ⚠️
Possible performance regression was detected for benchmark 'ExpressiveSharp Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.
| Benchmark suite | Current: 44c2f4b | Previous: 6808d2d | Ratio |
|---|---|---|---|
ExpressiveSharp.Benchmarks.ExpressionReplacerBenchmarks.Replace_Method |
3305.939254760742 ns (± 1490.9362570026249) |
2396.2288411458335 ns (± 597.6971388629727) |
1.38 |
ExpressiveSharp.Benchmarks.GeneratorBenchmarks.RunGenerator_Incremental_ExpressiveChange(ExpressiveCount: 1000) |
29223755.666666668 ns (± 939508.9888689092) |
12475252.375 ns (± 349659.53509311064) |
2.34 |
This comment was automatically generated by workflow using github-action-benchmark.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 62 out of 62 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/ExpressiveSharp/Extensions/RewritableQueryableLinqExtensions.cs
Outdated
Show resolved
Hide resolved
…rated interceptors
…able methods and chain continuity in EF Core operations
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 103 out of 103 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Introduce additional LINQ overloads to enhance the functionality of IRewritableQueryable, improving usability and flexibility in query operations.
This PR got out of hand and it also fixes the feature gaps that were left over with our EFCore integration. I think I tackled all gaps with a notable exception to
ExecuteUpdatesupport. We can address that as part of a separate PR