Skip to content

Commit

Permalink
fix: SyncSegments rpc always failed (#38578)
Browse files Browse the repository at this point in the history
miss the patch due to code branching
previous pr: #38032

Signed-off-by: Congqi Xia <[email protected]>
Co-authored-by: Wei Liu <[email protected]>
  • Loading branch information
congqixia and weiliu1031 authored Dec 19, 2024
1 parent b537a72 commit 3d360c0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal/datacoord/session/datanode_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -229,8 +229,8 @@ func (c *DataNodeManagerImpl) SyncSegments(ctx context.Context, nodeID int64, re
zap.Int64("nodeID", nodeID),
zap.Int64("planID", req.GetPlanID()),
)
ctx, cancel := context.WithTimeout(context.Background(), paramtable.Get().DataCoordCfg.CompactionRPCTimeout.GetAsDuration(time.Second))
cli, err := c.getClient(ctx, nodeID)
childCtx, cancel := context.WithTimeout(context.Background(), paramtable.Get().DataCoordCfg.CompactionRPCTimeout.GetAsDuration(time.Second))
cli, err := c.getClient(childCtx, nodeID)
cancel()
if err != nil {
log.Warn("failed to get client", zap.Error(err))
Expand Down

0 comments on commit 3d360c0

Please sign in to comment.