EF Core Performance Optimization Challenge | 233x FASTER

แชร์
ฝัง
  • เผยแพร่เมื่อ 29 ก.ย. 2024

ความคิดเห็น • 224

  • @MilanJovanovicTech
    @MilanJovanovicTech  2 ปีที่แล้ว +11

    Want to master Clean Architecture? Go here: bit.ly/3PupkOJ
    Want to unlock Modular Monoliths? Go here: bit.ly/3SXlzSt

  • @thecodeman_
    @thecodeman_ 2 ปีที่แล้ว +44

    Hey MIlan. Thank you for solving my EF Core performance optimization challenge. You did a great job and explained it even better in the video. 😃

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +3

      Thanks for creating the challenge. I'm already looking forward to the next one 😁

    • @justind6983
      @justind6983 2 ปีที่แล้ว

      where can we see the challenges?

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +3

      @@justind6983 Check the video description

  • @amitsinghrawat4760
    @amitsinghrawat4760 2 ปีที่แล้ว +1

    Your content is great and it relates to me.

  • @ramytawfik9168
    @ramytawfik9168 ปีที่แล้ว +1

    Why removing includes do not yield any significant improvements?

  • @viveks9711
    @viveks9711 ปีที่แล้ว

    In select getting allbook can we use extension method to get books list? somewhere i tried but navigation property not working in extension method.

  • @anthonylerouge9900
    @anthonylerouge9900 2 ปีที่แล้ว +11

    Just a small note to everyone, may be it is obvious for most but always use the Async overload as much as possible (at least for all microsoft frameworks and libraries, for 3rd parts, it depends) => async/await end-to-end
    It may not improve the performance (depend on a lot of things) but it is much better to manage the connections and anything cpu related

  • @leulgirma
    @leulgirma 2 ปีที่แล้ว +16

    Great video, more challenges like this would really help

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      Thanks! I'll see if I can give Stefan some ideas.

  • @Javilingys
    @Javilingys ปีที่แล้ว +1

    Can you explain please 6 stet: Filtered Include? I don't understand why it faster?

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว +1

      Actually, that's a slight mistake if you read the comments.
      The filtered include doesn't work with projections, which I wasn't aware of.
      So the reason is faster is that we end up filtering in-memory...

  • @philmontewelde6823
    @philmontewelde6823 2 ปีที่แล้ว +5

    Excellent video. I did not know about include statements with filters and the significant performance improvement

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      Note that they only work when loading actual entities. In my example, it won't work properly because I have a projection.

  • @nilesh-gule
    @nilesh-gule 2 ปีที่แล้ว +7

    This is an amazing video. I loved the way you went about deciphering each layer of the query and combining the changes with the benchmark results. Thanks for sharing such valuable insights 👍

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      You're welcome, Nilesh! I want to bring as much value as possible with my videos

  • @smookkee1
    @smookkee1 ปีที่แล้ว +1

    Hello from russia. Thx for usefull videos. Think i'm ready to start your course this fall

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      Sending my regards from 🇷🇸 :)
      Be sure to check with your bank if Teachable.com is supported for payments. I've had a few people from Russia complaining that their payments failed. And of course I'm available if you have any questions about the course, you can always send me an email.

  • @hailuong9295
    @hailuong9295 ปีที่แล้ว +1

    man i want to know who the maniac coding like this, seeing ToList() being put everywhere almost make me got a stroke

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      It's on purpose to make the original query extra slow, relax 🤣

  • @muttBunch
    @muttBunch 9 หลายเดือนก่อน +1

    What if by chance you wanted to load all records from a table? I’m only asking because I have event logs stored in a database and it takes 500 years to load them all 😢

    • @MilanJovanovicTech
      @MilanJovanovicTech  9 หลายเดือนก่อน +1

      Load them in batches with a few threads?

    • @muttBunch
      @muttBunch 9 หลายเดือนก่อน

      @@MilanJovanovicTech thank you Milan, I’m looking into how to do that. Thanks :)

  • @svorskemattias
    @svorskemattias ปีที่แล้ว +1

    How about compiling the query?

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      Didn't try that as I wasn't aware of the feature at the time, but it adds quite a bit of a performance boost on top

  • @alexandrohdez3982
    @alexandrohdez3982 ปีที่แล้ว +1

    Thank you! I hate ORM .. prefer mappers 😉

  • @marcioduarte2906
    @marcioduarte2906 ปีที่แล้ว +1

    You could still stay within the rules, use EF and use plain sql queries. With that, got over 306%

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      Not possible before EF Core 8

    • @marcioduarte2906
      @marcioduarte2906 ปีที่แล้ว

      ​@@MilanJovanovicTechOf course it is.
      You could use something like:
      using (var command = dbContext.Database.GetDbConnection().CreateCommand())
      ...
      Then perform an Execute on your sql query.
      Its technically, still entity framework ;)

  • @PelFox
    @PelFox 2 ปีที่แล้ว +4

    Would be interesting to see how fast the Dapper alternative is, 4ms is still very fast. So its a matter if its worth giving up on strongly typing to use raw sql to gain something like 1 or 2 ms.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      I'll benchmark it to see, but I suspect the same as you. To improvement will be nothing compared to the roundtrip cost to an actual database.

  • @complicated2359
    @complicated2359 ปีที่แล้ว +1

    There is always the case with frameworks to optimize them. Because of it hard to maintain them in big projects.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      Not really when you know what you're doing

    • @complicated2359
      @complicated2359 ปีที่แล้ว

      @@MilanJovanovicTech Sadly, it is not what you are doing the problem, it is what you have to do within framework to do what you need. Framework mostly is about constraints, especially ef core, not to mention it took ef team years to made it to run arbitrary sql.

  • @amantinband
    @amantinband 2 ปีที่แล้ว +5

    This was fun 🔥

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      You should try it also, maybe you have a better idea 🦄

  • @DeltaZe1
    @DeltaZe1 2 ปีที่แล้ว +3

    Should be empty list, no? Because how can an author aged 27 have books published before year 1900?

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +3

      Leaves this world at 27 years of age, and was born in the 1800? 😅

    • @Dragonet17
      @Dragonet17 2 ปีที่แล้ว +1

      Magic!

  • @microtech2448
    @microtech2448 ปีที่แล้ว +1

    Hi, can you please show an example of calling ef read and save operations from parallel.foreach which does not crash due to multiple threads?

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      Just create a separate database context inside of each iteration of the loop, that way you won't run into that issue.

    • @microtech2448
      @microtech2448 ปีที่แล้ว

      @@MilanJovanovicTech I am creating dbcontext object using scope factory but it still fails. Do you have any sample code to share? Thank you.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      ​@@microtech2448 Well, if it's the same scope it's going to be the same DbContext.
      Just do something like:
      Parallel.For(() =>
      {
      using var ctx = new MyDbContext();
      ...
      });

    • @microtech2448
      @microtech2448 ปีที่แล้ว

      @@MilanJovanovicTech thanks for sharing it but I wanna use dependency injection. From parallel.foreach I would want to call business layer which would have ef core operations. Also, I wanna avoid the use of new keyword as much as possible.

  • @marinero.bengali2
    @marinero.bengali2 2 ปีที่แล้ว +3

    Really good explanation!!, it's pretty common to find this kind of issues when working on legacy code, and just by making this small changes you get a boost of performance that will be noticed by the users. Sometimes is more complex, other not so much, but is always a good place to start.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      Thank you. The original example is intentionally bad, but as you said it can be common in legacy code or with beginners. Luckily, fixing it is easier.

  • @vesnx
    @vesnx ปีที่แล้ว +1

    I guess you ran into a EF bug, perhaps even a major one. if you need to reboot and re-run your tests to get back 50x times then perhaps you should open a bug somewhere as i am sure you are having a massive issue that should not go in production, if anything having cashed data in the down-stream like sql cashing etc. should improve your data the more often you load it. if I loose ~1/6th of my performance after executing my code I would not think this was production ready. I would open a defect with Microsoft with your findings

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      Actually no, the regression was because we moved the filtering of books to the database, and there are no indexes in place.

    • @vesnx
      @vesnx ปีที่แล้ว

      @@MilanJovanovicTech things do not tend to get slower over time when there are no indexes, if you added indexes and that's why you got faster you should mention it that the difference was due to the database changes and not the EF code.
      If that's the challenge why not put the database on a partitioning scheme and then the query would have taken a micro second. Nothing to do with the CODING challenge but hell, if we are cheating then lets cheat to win ;-)

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      @@vesnx So now I'm cheating? 😂

    • @vesnx
      @vesnx ปีที่แล้ว +1

      @@MilanJovanovicTech it's a coding challenge, you are not showing you are altering the play ground. What do you think, update the video and show you altered the database or update the video and only show the C# changes, the way you do it now is not reproduceable by any one that follows your shows.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      @@vesnx But I didn't alter the database 🤦‍♂️

  • @mysteriouslyhandmade
    @mysteriouslyhandmade 2 ปีที่แล้ว +3

    try this same query in .NET 7. It's superemely fast

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      You should try, and submit a PR to the challenge 😁

  • @luan_maik
    @luan_maik 2 ปีที่แล้ว +2

    Maybe it could improve the performance a bit by using:
    b.Published < new DateTime(1900,1,1)
    to avoid fullscan on Book table, because the index was applied to date and not just year

  • @vladdorokh7466
    @vladdorokh7466 ปีที่แล้ว +2

    Hi, Milan. It's a really good video with cool explanation of EF queries. I so appreciate it. And I really like your vs studio theme, especially code colors. Could you tell me how to set this up, please?

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      1. Install ReSharper (it's a paid tool 😔)
      2. Use the ReSharper Dark theme

  • @Busk1
    @Busk1 2 ปีที่แล้ว +2

    Great challenge, didn't know about this
    so i used your code as a base to improve the perfomance
    i used "compiled queries"
    this even improve the memory allocation in a 50% and the perfomance from 191x to 203x faster (didnt reach to 233x in my pc)

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +5

      I wasn't aware of compiled queries when making the video. So that was a missed opportunity on my end. Gonna make a separate video about it 😁

  • @peterk4694
    @peterk4694 3 หลายเดือนก่อน

    How did you configure BenchmarkDotNet to show the Faster calculation column?

    • @MilanJovanovicTech
      @MilanJovanovicTech  3 หลายเดือนก่อน

      There's a Ratio setting or something, can't recall exactly. Check this: davecallan.com/how-to-set-the-ratio-column-style-in-benchmarkdotnet-results/

  • @AsemAlexanDer
    @AsemAlexanDer ปีที่แล้ว +1

    wow it's amazing

  • @angelldark6426
    @angelldark6426 ปีที่แล้ว +1

    I'm Beginner 😢

  • @Silentsouls
    @Silentsouls 2 ปีที่แล้ว +2

    Fitering the books on year becore counting them , can give a differend ordering, due to all the books before 1900 not being counted anymore.
    Performance yes, differend results as well.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      Note that these are different columns!

    • @Silentsouls
      @Silentsouls 2 ปีที่แล้ว +1

      @@MilanJovanovicTech Right its not a count, but just a number.
      I missed that. only listened to my brain.

  • @tobiasj8019
    @tobiasj8019 ปีที่แล้ว +1

    Great Job!

  • @rakeshkrishnan7729
    @rakeshkrishnan7729 ปีที่แล้ว +2

    wow you have got such a great ability to explain things very clearly !!! subscribed 👍

  • @tasin5541
    @tasin5541 ปีที่แล้ว +1

    Faced a similar issue when we tested on prod db. We had an SLA of 2 seconds but the query was taking more than 1 minute. We had to migrate to Stored Procedure and use multiple temp tables to reach the SLA.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      2 seconds can be quite hard for some heavy queries

  • @dmitryc2029
    @dmitryc2029 ปีที่แล้ว +1

    So, the main idea is to squeeze all the work out of EF core and put it into the SQL server, isn't it?:) It's a junior level optimization.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      As long as the video helps someone, I don't mind what you call it 😁

  • @davidemmanuel3001
    @davidemmanuel3001 2 ปีที่แล้ว +2

    Great video. What's that tool you use to measure and display the linq performance?

  • @ajirahman5289
    @ajirahman5289 ปีที่แล้ว +1

    Thanks...

  • @a_ryota
    @a_ryota 2 ปีที่แล้ว +1

    Filtering in include doesn't work everywhere, I had several cases where it still return all joined objects and I had to add filtering in where clause. Also there is an option to use AsNoTrackingWithIdentityResolution bacause it'll create less linked objects in memory. It might be slower than AsNoTracking but you''ll waste more time for garbage collection to clear all that mess. And you might face equality problems if don't use AsNoTrackingWithIdentityResolution

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +2

      That's a fair point, Andrew. In my example, Filtered Include did not work at all. I found that out after recording the video unfortunately. Because it won't work with projections.

  • @solidusflux
    @solidusflux ปีที่แล้ว +1

    When I looked at the query results, there were books published after 1900. I don't think the filtered include is working. It may be faster because there is no filter at all in the SQL.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว +1

      Indeed that is the case Christopher. I pointed it out in a few comments.
      It's not working because we have a projection, that's why.
      In the solution I submitted in the end, I just filtered the books in-memory. It was only slightly slower than in the video.

  • @AlFasGD
    @AlFasGD 2 ปีที่แล้ว +1

    There is great explanation of common fallacies when building queries, and I used most of these when I improved a db query for just 5000 records from 3 minutes down to 0.5 seconds
    But I still fail to feel that you know 100% what you're doing in all of those areas, for instance you failed to explain why we got a performance regression from an optimization change you made that you kept later on, and you also account for little adjustments in the ratio without accounting for the average error induced in benchmarking
    For instance, seeing an improvement from 179.812x to 181.006x is only 0.664% better, which could very well be within the error of the results you get (at 9:52 it shows that the error of your result is 0.265% with the stddev being 0.247%), including potentially hidden error in the measurements like background tasks or whatnot
    While this is not microbenchmarking as we're talking in the scale of milliseconds and not micro or below, I still fail to see some "progressions" as minor as this one as absolute wins, especially in the context of database querying which involves connectivity and could perform more inconsistently than you can imagine

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Noted! I'm curious what you did to go from 3minutes to 0.5 seconds?

    • @AlFasGD
      @AlFasGD 2 ปีที่แล้ว

      I think I removed an Include, used Contains inside a Where when there was a hand-written loop that was performing a join with multiple queries, added indexing based on date and pagination of the queried results

  • @ugochukwuumerie6378
    @ugochukwuumerie6378 2 ปีที่แล้ว +2

    Thanks for this wonderfully explained video. Very useful

  • @dotnetdevni
    @dotnetdevni 2 ปีที่แล้ว +1

    But thats not the point removing the includes a developer could require those to make joins of data. I dont really think removing them is the right answer

  • @recycledsoldier
    @recycledsoldier ปีที่แล้ว +1

    Love your videos. One of the patterns I've used very commonly when dealing with returning complex datasets is actually pulling the root objects from the database first, in this case Authors. And then doing a follow-up query to pull additional metadata based on the filtered Authors.
    For instance, I have a search api that returns products and child products (there's only 2 tiers) but there are 50,000 products to search. It's a paginated list so I pre-filter the parent products and execute the statement. Then I execute a child query specifically for the returned results.
    It's not always a performance boost, but I find on very large datasets with a lot of metadata properties that need to come from the database, it can be faster to pull and then combine in memory.

  • @TheMiamiVice27
    @TheMiamiVice27 2 ปีที่แล้ว +1

    Dolazim iz istog svijeta kao i ti Milane :) Kad mi je ovaj video iskocio i kad sam pogledao na prvu "neoptimiziran" query, iskreno sam se zapitao koja je j***na budala napisala ovako los LINQ upit :) Ali valjda je namjerno to napravljeno kako bi se pokazalo da je cesto lako ubrzati query i to na nacin da se reorganizira malo query.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Tako je, namerno je lose napravljeno u startu 😅

  • @rasmadrak
    @rasmadrak 2 ปีที่แล้ว +1

    It could possibly be optimized slightly more by not fetching the country and age, since those are hardcoded at Serbia and 27 in this case.
    But most likely that would have a very minor speed boost. :P

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      The filtering at database level will likely cost more, I doubt it will make much difference. But a fun idea 😁

  • @5abi
    @5abi ปีที่แล้ว +1

    Hey Milan, very well explained. One small query. How did you get that performance matrix console window results?

  • @abdulatifrasulov7291
    @abdulatifrasulov7291 2 ปีที่แล้ว +1

    Thanks for video. Great job. Could you recommend some books to read for deep learning Entity Framework

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Take a look at "Entity Framework Core in Action"

  • @dagtveitgmail
    @dagtveitgmail ปีที่แล้ว +1

    I think you should try this query with a window query with rank function. It is possible that your data set may to small for this to be effective. but given a lot of data. window querys usualy perform better if you need to rank or order etc. this can be done with using linq2db efcore extension

  • @SandeepKumar-tc2vz
    @SandeepKumar-tc2vz ปีที่แล้ว +1

    I am a .NET developer , I like your concept. It is helpful really great job

  • @10199able
    @10199able 2 ปีที่แล้ว +1

    (surprised pikachu) You can filter inside Include statement??

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Yes, but it only works when you actually load the entities from the database.
      I kind of f-ed up in that regard.

  • @fazainlogicel
    @fazainlogicel 11 หลายเดือนก่อน +1

    who did too many times write tolist in query? this is not optimization dude i don't think that no one write queries like that

  • @KenzoArts
    @KenzoArts ปีที่แล้ว +1

    Hello Milan,
    Thank you for the challenge but my EF Core skills are still at beginner level
    and I am learning a lot from you!
    I would like to know if you can point me to some good resources about Change Tracker
    i kind of don't grasp its utility in regard to DB Context.
    Thank you for sharing your knowledge, it is highly appreciated.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว +1

      I don't think you need to dive deep into change tracker, other than high level understanding. I would look at Microsoft docs.

    • @KenzoArts
      @KenzoArts ปีที่แล้ว

      @@MilanJovanovicTech Thanks brother

  • @bengi8406
    @bengi8406 ปีที่แล้ว +1

    Hi Milan great video, can you do in future comparation of dapper and entity framework speed of certain query executions

  • @longuinni
    @longuinni 2 ปีที่แล้ว +1

    Really nice video. The next step is to implement specification pattern to make the query easy to maintain and unit test.
    ps: you can add AsNoTracking() too

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +2

      AsNoTracking won't have an effect when we return a projection, don't forget.

  • @liveabhishekshukla
    @liveabhishekshukla 2 ปีที่แล้ว +2

    Nice video... Curious what will be time for Dapper with raw sql

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Probably a bit faster, but nothing drastic. I can even check!

    • @etechguide7251
      @etechguide7251 2 ปีที่แล้ว

      @@MilanJovanovicTech What about using Dapper with stored procedure??

  • @DanielEkeroth
    @DanielEkeroth 2 ปีที่แล้ว +1

    Managed 253.92x improvement in speed and 1533x memory usage reduction :)

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Wow, fantastic Daniel! Great job 😁
      What did you do?

  • @zavjalovp
    @zavjalovp ปีที่แล้ว +1

    If you add "covered indexes" you will get a bit better result ))

  • @kjbetz
    @kjbetz ปีที่แล้ว +2

    Great video and explanations! Well done!

  • @DotnetDummies
    @DotnetDummies ปีที่แล้ว +1

    Awesome video. The way you explained is very clear and feel like everyone was waiting eagerly for climax.

  • @prasadmadushan
    @prasadmadushan ปีที่แล้ว +1

    You nailed it Milan, I love how your explanation with benchmark. Im sure ppl out there will get benefit of that

  • @farzaibsheikh9764
    @farzaibsheikh9764 2 ปีที่แล้ว +1

    i couldn't find ratio on benchmarking when i do i couldn't figure out how can i compare like yours how many times its faster can you tell me the process of this ??

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Take a look at this:
      benchmarkdotnet.org/articles/configs/configs.html

    • @farzaibsheikh9764
      @farzaibsheikh9764 ปีที่แล้ว +1

      @@MilanJovanovicTech thank you i got it :)

  • @grigorsargsyan8363
    @grigorsargsyan8363 8 หลายเดือนก่อน

    Thank you for this amazing playlist. Hello from Armenia

    • @MilanJovanovicTech
      @MilanJovanovicTech  8 หลายเดือนก่อน +1

      Thank you too!

    • @grigorsargsyan4506
      @grigorsargsyan4506 8 หลายเดือนก่อน

      @@MilanJovanovicTech I will wait new amazing videos )

  • @kodindoyannick5328
    @kodindoyannick5328 8 หลายเดือนก่อน +1

    Thanks you for all. I take much pleasure to learn new thing from you.

  • @jamestheloyalkingsfan1103
    @jamestheloyalkingsfan1103 ปีที่แล้ว +1

    Love the video, hearing you think it through reinforces my understanding of building queries

  • @joshem32
    @joshem32 2 ปีที่แล้ว +1

    in your experience, do you find yourself using EF more than raw SQL?

  • @stephenraphael6911
    @stephenraphael6911 ปีที่แล้ว +1

    Awesome tutorial with a lot of useful efcore information. Thanks a lot!

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      Glad you liked it, I'm making some more EF Core videos

  • @wonpaper
    @wonpaper ปีที่แล้ว +1

    This is a very good post and video. Subscribed right now. Thanks. More videos please.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว +1

      Thanks a lot! I have plenty more on the channel you can watch, and lot coming in the future also 😁

    • @wonpaper
      @wonpaper ปีที่แล้ว +1

      @@MilanJovanovicTech

  • @Un4GivNX
    @Un4GivNX 2 ปีที่แล้ว +1

    As a video game dev, we strictly prohibit the usage of LINQ because they are notoriously slow and generates TONS of garbage to collect.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      What do you do instead?

    • @Un4GivNX
      @Un4GivNX 2 ปีที่แล้ว

      @@MilanJovanovicTech prealloc and re-use list, hashset, some dictionary, and mostly achieve all that using for-loops.

    • @stepbackandthink
      @stepbackandthink ปีที่แล้ว

      .NET 7 should fix that.

  • @arsman_ahmad
    @arsman_ahmad 2 ปีที่แล้ว +1

    Hi Milan, thank you for this. You really explained very well.

  • @ardavaztterterian1211
    @ardavaztterterian1211 2 ปีที่แล้ว +2

    I've been following you on LinkedIn for a while and happy to see you making such videos. This was fun. Keep rocking!

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Thanks a lot! I'm glad you are finding my content useful 😁

  • @pedrocano3798
    @pedrocano3798 2 ปีที่แล้ว +1

    Hi Milan, a question. How may I make a dynamic Select statement using entity framework?. I'll put you in context. I want to select only the columns that I pass to my method

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      You have to write code that will generate the select statement. Not a trivial thing.

    • @11r3start11
      @11r3start11 2 ปีที่แล้ว

      You can make your own using Expression API

    • @longuinni
      @longuinni 2 ปีที่แล้ว

      OData can do that!! It's pretty easy to implement.

  • @mikaelsofi3444
    @mikaelsofi3444 ปีที่แล้ว +1

    Wow really nice video. Would have been cool to see the raw SQL query as well.

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      It would definitely be the fastest version

    • @mikaelsofi3444
      @mikaelsofi3444 ปีที่แล้ว +1

      @@MilanJovanovicTech I was expressing my self poorley. I meant the raw SQL query generated by LINQ on every benchmark test as an reference to what changed in the query. Although you where good at explaining.

  • @nileshtiwalkar9437
    @nileshtiwalkar9437 2 ปีที่แล้ว +1

    Thanks Milan, Very nicely explained.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Do you have any idea how we can make it even faster?

  • @XinoMbr
    @XinoMbr 4 หลายเดือนก่อน

    Amazing! thank you! 😁

    • @MilanJovanovicTech
      @MilanJovanovicTech  4 หลายเดือนก่อน +1

      No problem 😊

    • @XinoMbr
      @XinoMbr 4 หลายเดือนก่อน

      @MilanJovanovicTech could I ask you to bench the same tests on .net 4.5 , in the same host, so we can compare the performance between 4.5, or 4.8 vs .net core 6, 7 ,8 . The latest, something like it? 😁. That will be a nice video comparison for all community. Tô show if only the changing of versions bring some performance

  • @NG-tn8cn
    @NG-tn8cn ปีที่แล้ว +1

    Thanks Milan, stay strong !!

  • @vedadherceglija1529
    @vedadherceglija1529 2 ปีที่แล้ว +1

    Great video, thank you Milan

  • @Dragonet17
    @Dragonet17 2 ปีที่แล้ว +1

    Thanks for your videos ! Very nice topic. I created the same query improvment before watching this video, thanks for being able to checking the solution
    and one more thing - every string in Db has max length ( nvarchar(max)) - for me changing this, it's kind of optimazition too, first of all it will be affect to memory allocation but of course change execution time of first query (not-optimised).
    All the best and see you in next video ;)

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Did you try Compiled Queries?

    • @Dragonet17
      @Dragonet17 2 ปีที่แล้ว

      @@MilanJovanovicTech , I didn't

    • @Dragonet17
      @Dragonet17 2 ปีที่แล้ว +1

      @@MilanJovanovicTech I read about it and used it, execution time is almost the same, but memory allocation is less by 30%, thanks for tip

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      @@Dragonet17 I actually saw a nice improvement in execution time. Interesting.

    • @Dragonet17
      @Dragonet17 2 ปีที่แล้ว +1

      @@MilanJovanovicTech using compiled queries or something else ?

  • @mariumbegum7325
    @mariumbegum7325 ปีที่แล้ว +1

    great video!!

  • @hazemal-takleh3703
    @hazemal-takleh3703 2 ปีที่แล้ว +1

    Awesome video!

  • @mohamedroshdy4916
    @mohamedroshdy4916 2 ปีที่แล้ว +1

    Good Work🥰

  • @JedaiasRodrigues7
    @JedaiasRodrigues7 9 หลายเดือนก่อน

    LGTM!

  • @willinton06
    @willinton06 2 ปีที่แล้ว +1

    The include filter was good tho

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      The worst part is - it didn't work 😅

    • @willinton06
      @willinton06 2 ปีที่แล้ว

      @@MilanJovanovicTech It didn’t? As in the data returned was wrong? That’s worrying

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      @@willinton06 It's how it works, actually. Includes don't make a difference with projections. I wasn't aware of that recording.
      And there was one performance improvement left which I also didn't know about: Compiled Queries.
      Gotta make a separate video about that.

    • @pavelstafl6043
      @pavelstafl6043 2 ปีที่แล้ว +1

      Yay!

  • @sandordaemen
    @sandordaemen 2 ปีที่แล้ว +2

    How is it that when you apply the filtered include to the query, the query performs better? I always thought that when using a custom projection (as is the case here), EF ignores the calls to `Include()`

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      You are totally correct there! It was missed by me when recording the video. I did however fix in the PR on GitHub

  • @erpepl
    @erpepl 2 ปีที่แล้ว +1

    Strange video thumbnail, are you 3?

  • @XELTRA1
    @XELTRA1 ปีที่แล้ว

    Sir, i want to know, why this video not same from source code? , like line 104 , in source code,GetAuthors_Optimized() was disabled and different line, i want to know how many you edited from source code, thank's

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      I may have made the code a little prettier before starting recording

    • @XELTRA1
      @XELTRA1 ปีที่แล้ว

      @@MilanJovanovicTech Sir, do you have the tutorial manual config like your video , i saw ratio value is not 1.00 but baseline and 277 faster, how can make like this?

  • @levkirichuk
    @levkirichuk 2 ปีที่แล้ว +4

    Thanks a lot for that cool introduction. Some additional improvements that may have a place:
    1. To use Enum instead of "Serbia" like Country.Serbia. That will give faster code changes in the future.
    2. Use smaller queries and return IQuerytable object. That flow will give us more flexibility in the code and in this way code will be more readable and understandable.
    3. Use instead of ToList -> ToListAsync and return the async task. That will provide you more performance for 100 and more concurrent calls.
    I like your's improvements and your explanation that was great.

    • @arsman_ahmad
      @arsman_ahmad 2 ปีที่แล้ว +1

      Very well addition.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +1

      1. Good point
      2. Make sense in some situations
      3. It wouldn't make much difference for the benchmark

  • @techpc5453
    @techpc5453 ปีที่แล้ว +1

  • @EnerJk
    @EnerJk ปีที่แล้ว +1

    Hi Milan, I think you can change .Take(2) row before select DTO model. Because if you need just specific 2 rows and then you fetch 2 rows from DB after that you can use select method. Probably performance much better than before. By the way thanks for video :)

    • @MilanJovanovicTech
      @MilanJovanovicTech  ปีที่แล้ว

      How do you think my version works?
      It loads just 2 rows from the DB.

    • @EnerJk
      @EnerJk ปีที่แล้ว

      @@MilanJovanovicTech I checked again you are right, my mistake sorry :)

    • @MilanJovanovicTech
      @MilanJovanovicTech  หลายเดือนก่อน

      Take(2) is still executed on the DB

  • @anomalii7720
    @anomalii7720 2 ปีที่แล้ว

    It is time Microsoft drops EF.

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว

      Never 😁

    • @anomalii7720
      @anomalii7720 2 ปีที่แล้ว

      @@MilanJovanovicTech Learn ADO (1 hour ?) and you can do much more than with EF and it is always faster than EF because EF depends on ADO. So why does EF exist?
      EF is ten years now? And still it has problems.
      Dump EF :-)
      I did the first 3 editions of EF and after that i was done with it.
      Take it from me: stop using it.

  • @ferdicuyar814
    @ferdicuyar814 ปีที่แล้ว

    ok here is my optimisation idea; db.tables.where(filters).include(xxx).theninclude(xxx).include(xxx).theninclude(xxxx).tolist(); this will be far more faster than db.tables..include(xxx).theninclude(xxx).include(xxx).theninclude(xxxx).where(xxX).tolist()

  • @Andruuid
    @Andruuid 2 ปีที่แล้ว +3

    why not just make a stored procedure and call it? this code seems so ugly..

    • @MilanJovanovicTech
      @MilanJovanovicTech  2 ปีที่แล้ว +5

      Did you not hear this was a coding challenge at the start?

    • @MrKlarthums
      @MrKlarthums 2 ปีที่แล้ว +2

      If you don't use Linq and instead use SQL everyday, the code is going to look ugly (or vice-versa). It's just a matter of specialized familiarity and bias.

    • @Andruuid
      @Andruuid 2 ปีที่แล้ว +1

      @@MilanJovanovicTech ah, youre right ;-)