fix(papyrus_p2p_sync): retry upon class manager client error

[ShahakShama]

No description provided.

[reviewable-StarkWare]

This change is 

[noamsp-starkware]

Reviewed 1 of 2 files at r1, all commit messages.
Reviewable status: 1 of 2 files reviewed, 2 unresolved discussions (waiting on @ShahakShama)

---

crates/papyrus_p2p_sync/src/client/class.rs line 40 at r1 (raw file):

                // TODO(shahak): Verify class hash matches class manager response. report if not.
                // TODO(shahak): Try to avoid cloning. See if ClientError can contain the request.
                while let Err(err) = class_manager_client.add_class(class.clone()).await {

Add a timeout/max_num_attempts so this doesn't loop forever in case of an error.

---

crates/papyrus_p2p_sync/src/client/class.rs line 53 at r1 (raw file):

                // TODO(shahak): Test this flow.
                // TODO(shahak): Try to avoid cloning. See if ClientError can contain the request.
                while let Err(err) = class_manager_client

Same here

[ShahakShama]

Reviewable status: 1 of 2 files reviewed, 2 unresolved discussions (waiting on @noamsp-starkware)

---

crates/papyrus_p2p_sync/src/client/class.rs line 40 at r1 (raw file):

Previously, noamsp-starkware wrote…

Add a timeout/max_num_attempts so this doesn't loop forever in case of an error.

The idea is to loop forever

[noamsp-starkware]

Reviewable status: 1 of 2 files reviewed, all discussions resolved

[noamsp-starkware]

Reviewed 1 of 2 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ShahakShama)