RESTier icon indicating copy to clipboard operation
RESTier copied to clipboard

Batchhandler results in an error when doing bulk updates.

Open MitchKuijpers opened this issue 1 year ago • 1 comments

Issues

*This pull request fixes issue #760

Description

Concurrent dictionary to resolve the crashes.

Checklist (Uncheck if it is not completed)

  • [ ] Test cases added
  • [ ] Build and test with one-click build and test script passed

Additional work necessary

Hopefully It won't have more concurrency errors. The default method doesn't resolve the requests all at the same time, and I don't believe it is tested properly after this change.

MitchKuijpers avatar Jul 04 '24 13:07 MitchKuijpers

@microsoft-github-policy-service agree company="Nocore"

MitchKuijpers avatar Jul 04 '24 14:07 MitchKuijpers

Great catch, thanks!

robertmclaws avatar Jul 06 '24 22:07 robertmclaws

Fixed here: https://www.nuget.org/packages/Microsoft.Restier.AspNetCore/1.1.1-CI-20240706-232326

LMK if there are any issues, otherwise an RC build will go out next week.

robertmclaws avatar Jul 06 '24 23:07 robertmclaws