Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
milvus
milvus
提交
c5ff8bf7
M
milvus
项目概览
milvus
/
milvus
大约 1 年 前同步成功
通知
261
Star
22476
Fork
2472
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
DevOps
流水线
流水线任务
计划
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
M
milvus
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
DevOps
DevOps
流水线
流水线任务
计划
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
流水线任务
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
未验证
提交
c5ff8bf7
编写于
12月 27, 2022
作者:
C
congqixia
提交者:
GitHub
12月 27, 2022
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Fix QueryNode may panics when stopped (#21406)
Signed-off-by:
N
Congqi Xia
<
congqi.xia@zilliz.com
>
上级
d16b7c3c
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
92 addition
and
0 deletion
+92
-0
internal/querynode/shard_node_detector.go
internal/querynode/shard_node_detector.go
+11
-0
internal/querynode/shard_node_detector_test.go
internal/querynode/shard_node_detector_test.go
+81
-0
未找到文件。
internal/querynode/shard_node_detector.go
浏览文件 @
c5ff8bf7
...
...
@@ -18,6 +18,7 @@ package querynode
import
(
"context"
"errors"
"sync"
"github.com/golang/protobuf/proto"
...
...
@@ -177,6 +178,11 @@ func (nd *etcdShardNodeDetector) handlePutEvent(e *clientv3.Event, collectionID,
idAddr
,
err
:=
nd
.
idAddr
()
if
err
!=
nil
{
if
errors
.
Is
(
err
,
context
.
Canceled
)
{
// session canceled, query node is stopping.
log
.
Warn
(
"EtcdNodeDetector id resolve failed"
,
zap
.
Error
(
err
))
return
}
log
.
Error
(
"Etcd NodeDetector session map failed"
,
zap
.
Error
(
err
))
panic
(
err
)
}
...
...
@@ -253,6 +259,11 @@ func (nd *etcdShardNodeDetector) handleDelEvent(e *clientv3.Event, collectionID,
}
idAddr
,
err
:=
nd
.
idAddr
()
if
err
!=
nil
{
if
errors
.
Is
(
err
,
context
.
Canceled
)
{
// session canceled, query node is stopping.
log
.
Warn
(
"EtcdNodeDetector id resolve failed"
,
zap
.
Error
(
err
))
return
}
log
.
Error
(
"Etcd NodeDetector session map failed"
,
zap
.
Error
(
err
))
panic
(
err
)
}
...
...
internal/querynode/shard_node_detector_test.go
浏览文件 @
c5ff8bf7
...
...
@@ -18,6 +18,7 @@ package querynode
import
(
"context"
"errors"
"fmt"
"path"
"strconv"
...
...
@@ -31,6 +32,8 @@ import (
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"go.etcd.io/etcd/api/v3/mvccpb"
clientv3
"go.etcd.io/etcd/client/v3"
"go.etcd.io/etcd/server/v3/etcdserver/api/v3client"
)
...
...
@@ -324,3 +327,81 @@ func TestEtcdShardNodeDetector_watch(t *testing.T) {
})
}
}
func
TestNodeDetectorHandleWithError
(
t
*
testing
.
T
)
{
t
.
Run
(
"unexpected error type"
,
func
(
t
*
testing
.
T
)
{
collectionID
:=
int64
(
1
)
replicaID
:=
int64
(
1001
)
nd
:=
&
etcdShardNodeDetector
{
idAddr
:
func
()
(
map
[
int64
]
string
,
error
)
{
return
nil
,
errors
.
New
(
"unexpected error"
)
},
}
replica
:=
&
querypb
.
Replica
{
ID
:
replicaID
,
CollectionID
:
collectionID
,
}
bs
,
err
:=
proto
.
Marshal
(
replica
)
require
.
NoError
(
t
,
err
)
assert
.
Panics
(
t
,
func
()
{
nd
.
handlePutEvent
(
&
clientv3
.
Event
{
Type
:
mvccpb
.
PUT
,
Kv
:
&
mvccpb
.
KeyValue
{
Value
:
bs
,
},
},
collectionID
,
replicaID
)
})
assert
.
Panics
(
t
,
func
()
{
nd
.
handleDelEvent
(
&
clientv3
.
Event
{
Type
:
mvccpb
.
DELETE
,
Kv
:
&
mvccpb
.
KeyValue
{
Value
:
bs
,
},
PrevKv
:
&
mvccpb
.
KeyValue
{
Value
:
bs
,
},
},
collectionID
,
replicaID
)
})
})
t
.
Run
(
"context canceled"
,
func
(
t
*
testing
.
T
)
{
collectionID
:=
int64
(
1
)
replicaID
:=
int64
(
1001
)
nd
:=
&
etcdShardNodeDetector
{
idAddr
:
func
()
(
map
[
int64
]
string
,
error
)
{
return
nil
,
context
.
Canceled
},
}
replica
:=
&
querypb
.
Replica
{
ID
:
replicaID
,
CollectionID
:
collectionID
,
}
bs
,
err
:=
proto
.
Marshal
(
replica
)
require
.
NoError
(
t
,
err
)
assert
.
NotPanics
(
t
,
func
()
{
nd
.
handlePutEvent
(
&
clientv3
.
Event
{
Type
:
mvccpb
.
PUT
,
Kv
:
&
mvccpb
.
KeyValue
{
Value
:
bs
,
},
},
collectionID
,
replicaID
)
})
assert
.
NotPanics
(
t
,
func
()
{
nd
.
handleDelEvent
(
&
clientv3
.
Event
{
Type
:
mvccpb
.
DELETE
,
Kv
:
&
mvccpb
.
KeyValue
{
Value
:
bs
,
},
PrevKv
:
&
mvccpb
.
KeyValue
{
Value
:
bs
,
},
},
collectionID
,
replicaID
)
})
})
}
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录