-
Notifications
You must be signed in to change notification settings - Fork 439
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade split #253
base: master
Are you sure you want to change the base?
Upgrade split #253
Conversation
<< StatusCodeToString(m_status) << ", m_start_key " | ||
<< m_start_key << ", m_end_key " << m_end_key | ||
<< ", key_start " << key_start << ", key_end " << key_end; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
加上DebugString吧,防止乱码
@@ -134,6 +134,11 @@ bool TabletIO::Load(const TableSchema& schema, | |||
StatusCode* status) { | |||
{ | |||
MutexLock lock(&m_mutex); | |||
LOG(INFO) << __func__ << ", path " << path << ", status " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这些日志是必须的么?
kUnLoading = kTabletUnLoading, | ||
kUnLoading2 = kTabletUnLoading2 | ||
kUnLoading2 = kTabletUnLoading2, | ||
kFrozen = kTabletFrozen, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kUnLoading2
kFrozen
能加个注释不,分别代表什么状态
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kunloading2是原代码的状态,tablietio进行该卸载第二阶段,需要阻塞读写;kfrozen状态是卸载结束后tabletio进入的状态,表示该区间目前不操作;
&raw_split_key)) { | ||
VLOG(5) << "fail to find split key"; | ||
SetStatusCode(kTableNotSupport, status); | ||
&raw_split_key)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
缩进。。
@@ -2369,11 +2420,6 @@ void MasterImpl::UnloadTabletCallback(TabletPtr tablet, int32_t retry, | |||
ProcessOffLineTablet(tablet); | |||
TryLoadTablet(tablet); | |||
} else { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
else里面加个CHECK(false)吧。上面CHECK中的onsplit删掉
终于出现ts写坏meta表case了,感觉可以重启这货了 |
两年了。。。还能起得来么 |
分裂流程的代码结构非常稳定,基本没有更新 |
太有毅力了,简直我辈楷模呀,本渣自叹不如! 话说,感觉master修改meta表要“read-modify-wite”的事务才完全靠谱。。。 |
modify master && tabletserver 's split interface