Skip to content

Commit

Permalink
Replace error! with bail! in GlobalBarrierManagerContext for ea…
Browse files Browse the repository at this point in the history
…rly termination on fail

Signed-off-by: Shanicky Chen <[email protected]>
  • Loading branch information
shanicky committed Oct 24, 2024
1 parent f637d0f commit b655a66
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/meta/src/barrier/recovery.rs
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ use std::time::Duration;

use anyhow::{anyhow, Context};
use itertools::Itertools;
use risingwave_common::bail;
use risingwave_common::catalog::TableId;
use risingwave_common::config::DefaultParallelism;
use risingwave_common::hash::WorkerSlotId;
Expand All @@ -29,7 +30,7 @@ use risingwave_pb::stream_plan::{AddMutation, StreamActor};
use thiserror_ext::AsReport;
use tokio::time::Instant;
use tokio_retry::strategy::{jitter, ExponentialBackoff};
use tracing::{debug, error, info, warn, Instrument};
use tracing::{debug, info, warn, Instrument};

use super::{CheckpointControl, TracedEpoch};
use crate::barrier::info::{InflightGraphInfo, InflightSubscriptionInfo};
Expand Down Expand Up @@ -541,7 +542,7 @@ impl GlobalBarrierManagerContext {
info!("integrity check passed");
}
Err(_) => {
error!("integrity check failed");
bail!("integrity check failed");
}
}

Expand Down

0 comments on commit b655a66

Please sign in to comment.