Thanks for the nice PR @clot27 !
IMO it covers the contents of #3509 pretty well and the overvall logic looks good :) I left some comments already though I'll ofc do a proper review again once it's out of draft status .

initial review. Will review again after tests are added, implementation looks ok to me

Thanks for the updates! This will come out great :) I left a bunch of new comments, but nothing major IMO.

more comments. But looking much more refined now 👍🏽

Thanks for the new updates! I left smaller comments :) Also the docs build is failing …

Regarding the unit tests on the update processors: TBH I don't see them going in a right direction and I'm not sure how I can explain better what should be covered. I will try and write some tests myself, but please do feel free to ask anything about that if you have questions :)

LGTM :) @harshil21 do you have anything to add?

After merge, the dev-wiki should be updated to

• elaborate on the new processor on the page on concurrency
• mention the BaseUpdateProcessor on the architecture page. This would be a good chance to convert the graphic to a mermaid diagram

final review. Looks much more refined and ready. Good job @clot27. Just some tiny errors caught:

nice upgrade!