Skip to content

Commit

Permalink
fix: flowgraph crash when channel releasing (milvus-io#37285)
Browse files Browse the repository at this point in the history
issue: milvus-io#37284

Signed-off-by: chyezh <chyezh@outlook.com>
  • Loading branch information
chyezh authored Oct 31, 2024
1 parent f13faa3 commit 448cc08
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion internal/datanode/channel/channel_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -75,10 +75,11 @@ func NewChannelManager(pipelineParams *util.PipelineParams, fgManager pipeline.F
if pipelineParams.CompactionExecutor != nil {
pipelineParams.CompactionExecutor.DiscardPlan(channelName)
}
// RemoveFlowgraph before remove WriteBuffer, otherwise panics will happen.
fgManager.RemoveFlowgraph(channelName)
if pipelineParams.WriteBufferManager != nil {
pipelineParams.WriteBufferManager.RemoveChannel(channelName)
}
fgManager.RemoveFlowgraph(channelName)
},

closeCh: lifetime.NewSafeChan(),
Expand Down

0 comments on commit 448cc08

Please sign in to comment.