Skip to content

Commit

Permalink
fix(meta): handle case where progress is done (#12631)
Browse files Browse the repository at this point in the history
  • Loading branch information
kwannoel authored Oct 5, 2023
1 parent e365f2f commit 8195963
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/meta/src/barrier/progress.rs
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,7 @@ impl Progress {
match self.states.remove(&actor).unwrap() {
ChainState::Init => {}
ChainState::ConsumingUpstream(_, old_consumed_rows) => {
if !matches!(new_state, ChainState::Done(_)) {
self.consumed_rows -= old_consumed_rows;
}
self.consumed_rows -= old_consumed_rows;
}
ChainState::Done(_) => panic!("should not report done multiple times"),
};
Expand Down

0 comments on commit 8195963

Please sign in to comment.