Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
milvus
milvus
提交
8465d03d
M
milvus
项目概览
milvus
/
milvus
11 个月 前同步成功
通知
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,发现更多精彩内容 >>
未验证
提交
8465d03d
编写于
9月 26, 2021
作者:
X
Xiaofan
提交者:
GitHub
9月 26, 2021
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Make tsafe close not elegant (#8582)
Signed-off-by:
N
xiaofan-luan
<
xiaofan.luan@zilliz.com
>
上级
d38adda8
变更
1
隐藏空白更改
内联
并排
Showing
1 changed file
with
20 addition
and
16 deletion
+20
-16
internal/querynode/tsafe.go
internal/querynode/tsafe.go
+20
-16
未找到文件。
internal/querynode/tsafe.go
浏览文件 @
8465d03d
...
...
@@ -34,7 +34,6 @@ func newTSafeWatcher() *tSafeWatcher {
func
(
watcher
*
tSafeWatcher
)
notify
()
{
if
len
(
watcher
.
notifyChan
)
==
0
{
watcher
.
notifyChan
<-
true
//log.Debug("tSafe watcher notify done")
}
}
...
...
@@ -70,8 +69,7 @@ type tSafe struct {
watcherList
[]
*
tSafeWatcher
tSafeChan
chan
tSafeMsg
tSafeRecord
map
[
UniqueID
]
Timestamp
// waitgroup for closing control
closeWg
sync
.
WaitGroup
isClose
bool
}
func
newTSafe
(
ctx
context
.
Context
,
channel
Channel
)
tSafer
{
...
...
@@ -90,17 +88,26 @@ func newTSafe(ctx context.Context, channel Channel) tSafer {
}
func
(
ts
*
tSafe
)
start
()
{
ts
.
closeWg
.
Add
(
1
)
go
func
()
{
defer
ts
.
closeWg
.
Done
()
for
{
select
{
case
<-
ts
.
ctx
.
Done
()
:
ts
.
tSafeMu
.
Lock
()
ts
.
isClose
=
true
log
.
Debug
(
"tSafe context done"
,
zap
.
Any
(
"channel"
,
ts
.
channel
),
)
for
_
,
watcher
:=
range
ts
.
watcherList
{
close
(
watcher
.
notifyChan
)
}
close
(
ts
.
tSafeChan
)
ts
.
tSafeMu
.
Unlock
()
return
case
m
:=
<-
ts
.
tSafeChan
:
case
m
,
ok
:=
<-
ts
.
tSafeChan
:
if
!
ok
{
// should not happen!!
return
}
ts
.
tSafeMu
.
Lock
()
ts
.
tSafeRecord
[
m
.
id
]
=
m
.
t
var
tmpT
Timestamp
=
math
.
MaxUint64
...
...
@@ -155,7 +162,13 @@ func (ts *tSafe) get() Timestamp {
func
(
ts
*
tSafe
)
set
(
id
UniqueID
,
t
Timestamp
)
{
ts
.
tSafeMu
.
Lock
()
defer
ts
.
tSafeMu
.
Unlock
()
if
ts
.
isClose
{
// should not happen if tsafe_replica guard correctly
log
.
Warn
(
"Try to set id with ts close "
,
zap
.
Any
(
"channel"
,
ts
.
channel
),
zap
.
Any
(
"it"
,
id
))
return
}
msg
:=
tSafeMsg
{
t
:
t
,
id
:
id
,
...
...
@@ -165,13 +178,4 @@ func (ts *tSafe) set(id UniqueID, t Timestamp) {
func
(
ts
*
tSafe
)
close
()
{
ts
.
cancel
()
// wait for all job done
ts
.
closeWg
.
Wait
()
ts
.
tSafeMu
.
Lock
()
defer
ts
.
tSafeMu
.
Unlock
()
for
_
,
watcher
:=
range
ts
.
watcherList
{
close
(
watcher
.
notifyChan
)
}
}
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录