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

*: report read statistics to pd #2307

Merged
merged 26 commits into from Sep 27, 2017

Conversation

Projects
None yet
3 participants
@nolouch
Contributor

nolouch commented Sep 18, 2017

report the read statistics.

@nolouch nolouch requested review from siddontang and zhangjinpeng1987 Sep 18, 2017

nolouch added some commits Sep 21, 2017

@nolouch nolouch changed the title from [DNM] *: report read statistics to pd to *: report read statistics to pd Sep 22, 2017

@siddontang

This comment has been minimized.

Show comment
Hide comment
@siddontang

siddontang Sep 26, 2017

Contributor

LGTM

But I suggest sending read statistics to the PD worker directly later.

PTAL @BusyJay @zhangjinpeng1987

Contributor

siddontang commented Sep 26, 2017

LGTM

But I suggest sending read statistics to the PD worker directly later.

PTAL @BusyJay @zhangjinpeng1987

nolouch added some commits Sep 26, 2017

fix

nolouch added some commits Sep 26, 2017

@siddontang

This comment has been minimized.

Show comment
Hide comment
@siddontang

siddontang Sep 27, 2017

Contributor

LGTM

CI failed @nolouch

Contributor

siddontang commented Sep 27, 2017

LGTM

CI failed @nolouch

@nolouch nolouch merged commit a551563 into master Sep 27, 2017

3 checks passed

ci/circleci Your tests passed on CircleCI!
Details
jenkins-ci-tikv/build Jenkins job succeeded.
Details
license/cla Contributor License Agreement is signed.
Details

@nolouch nolouch deleted the shuning/readbytes branch Sep 27, 2017

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