fix: do not ignore failed persist tasks
I'm seeing some panics in our test bench, but it the ingester happily
continues and thinks it persisted tasks even though it didn't. Let's at
least bail out if a persist task fails.
@@ -253,7 +253,10 @@ impl LifecycleManager {
if !persist_tasks.is_empty() {
let persists = futures::future::join_all(persist_tasks.into_iter());
- persists.await;
+ let results = persists.await;
+ for res in results {
+ res.expect("not aborted").expect("task finished");
+ }
// for the sequencers that had data persisted, update their min_unpersisted_sequence_number to
// either the minimum remaining in everything that didn't get persisted, or the highest
