Hello Andrew Wong,

I'd like you to do a code review. Please visit

    http://gerrit.cloudera.org:8080/16169

to review the following change.


Change subject: WIP: lock manager batch release
......................................................................

WIP: lock manager batch release

before:
 Performance counter stats for './build/latest/bin/kudu tserver run -fs-wal-dir 
/tmp/ts':

         229884.12 msec task-clock                #    6.439 CPUs utilized
            269792      context-switches          #    0.001 M/sec
             46768      cpu-migrations            #    0.203 K/sec
           3182950      page-faults               #    0.014 M/sec
      946261367467      cycles                    #    4.116 GHz                
      (83.35%)
      176964710014      stalled-cycles-frontend   #   18.70% frontend cycles 
idle     (83.30%)
      111123086050      stalled-cycles-backend    #   11.74% backend cycles 
idle      (83.42%)
      834056713167      instructions              #    0.88  insn per cycle
                                                  #    0.21  stalled cycles per 
insn  (83.31%)
      148536649894      branches                  #  646.137 M/sec              
      (83.30%)
         577414668      branch-misses             #    0.39% of all branches    
      (83.34%)

     213.458498000 seconds user
      16.670046000 seconds sys

after:
Performance counter stats for './build/latest/bin/kudu tserver run -fs-wal-dir 
/tmp/ts':

         222268.41 msec task-clock                #    4.990 CPUs utilized
            283010      context-switches          #    0.001 M/sec
             50535      cpu-migrations            #    0.227 K/sec
           3330526      page-faults               #    0.015 M/sec
      918126009639      cycles                    #    4.131 GHz                
      (83.28%)
      148341591605      stalled-cycles-frontend   #   16.16% frontend cycles 
idle     (83.44%)
      110818610281      stalled-cycles-backend    #   12.07% backend cycles 
idle      (83.28%)
      822449096069      instructions              #    0.90  insn per cycle
                                                  #    0.18  stalled cycles per 
insn  (83.41%)
      146801150713      branches                  #  660.468 M/sec              
      (83.37%)
         585012592      branch-misses             #    0.40% of all branches    
      (83.22%)

     204.602454000 seconds user
      17.930236000 seconds sys

Change-Id: I3cb724e953ecdf188a35181c2f91b721b3416524
---
M src/kudu/tablet/lock_manager-test.cc
M src/kudu/tablet/lock_manager.cc
M src/kudu/tablet/lock_manager.h
M src/kudu/tablet/ops/write_op.cc
M src/kudu/tablet/tablet.cc
5 files changed, 135 insertions(+), 89 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/16169/1
--
To view, visit http://gerrit.cloudera.org:8080/16169
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3cb724e953ecdf188a35181c2f91b721b3416524
Gerrit-Change-Number: 16169
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Andrew Wong <andrew.w...@cloudera.com>

Reply via email to