Skip to content

Upgrade split #253

Open
caijieming-ng wants to merge 12 commits intobaidu:masterfrom
caijieming-ng:CodeMerge
Open

Upgrade split #253
caijieming-ng wants to merge 12 commits intobaidu:masterfrom
caijieming-ng:CodeMerge

Conversation

@caijieming-ng
Copy link
Collaborator

modify master && tabletserver 's split interface

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

加上DebugString吧,防止乱码

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这些日志是必须的么?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kUnLoading2 kFrozen能加个注释不,分别代表什么状态

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kunloading2是原代码的状态,tablietio进行该卸载第二阶段,需要阻塞读写;kfrozen状态是卸载结束后tabletio进入的状态,表示该区间目前不操作;

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

缩进。。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants