search

fullstackhero / blazor-starter-kit

Clean Architecture Template for Blazor WebAssembly Built with MudBlazor Components.
MIT License
3.48k stars 730 forks source link

Should GetAllAsync() include AsNoTracking? #363

Open dotnetshadow opened 2 years ago

dotnetshadow commented 2 years ago

Is your feature request related to a problem? Please describe. I noticed that you add AsNoTracking() on GetPagedResponseAsync() but it's not on the GetAllAsync() is there a reason for this?

Describe the solution you'd like Just wondering if it's an oversight that it's not added?

nameofSEOKWONHONG commented 1 year ago

If it's a select statement, it's a good idea to include it.

Here's why.

  1. Select statement performance has been improved.

  2. Avoiding multiprocessing errors in case of Singleton.

There may be problems saving nested hierarchies.