Skip to content

Unify retry/error handling for content replay

Nicolas Dandrimont requested to merge generated-differential-D2782-source into generated-differential-D2782-target

This uses a custom wrapper exception and tenacity callbacks to log exceptions when the copy of a given content fails several times. This makes the consumer more robust (fewer crashes), which in turns allows fewer consumer rebalances, which finally drastically reduces the consumer bandwidth consumption.

At this point, the retry of "definitely" failed content replays needs to be handled separately.

Depends on !117 (closed)

Test Plan

one new tox test added for the retry behavior

Migrated from D2782 (view on Phabricator)

Merge request reports