BIND 10 trac2252, updated. d1b06874353849175446f37dfca5a95f66ce5093 [2252] moved timings of starting and stopping the statistics timer

[BIND 10 source code commits]

The branch, trac2252 has been updated
       via  d1b06874353849175446f37dfca5a95f66ce5093 (commit)
      from  ebc0eb94003fd33460070835dcb828638d9233aa (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit d1b06874353849175446f37dfca5a95f66ce5093
Author: Naoki Kambe <kambe at jprs.co.jp>
Date:   Mon Oct 1 14:17:41 2012 +0900

    [2252] moved timings of starting and stopping the statistics timer
    
    timer is started before XFRIN_IXFR_TRANSFER_SUCCESS and stopped after
    XFRIN_TRANSFER_SUCCESS

-----------------------------------------------------------------------

Summary of changes:
 src/bin/xfrin/xfrin.py.in |   13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/bin/xfrin/xfrin.py.in b/src/bin/xfrin/xfrin.py.in
index d88070a..c602803 100755
--- a/src/bin/xfrin/xfrin.py.in
+++ b/src/bin/xfrin/xfrin.py.in
@@ -930,15 +930,15 @@ class XfrinConnection(asyncore.dispatcher):
                 if not self.connect_to_master():
                     raise XfrinException('Unable to reconnect to master')
 
+            # start statistics timer
+            getattr(counter,
+                    'start_time_to_%s' % req_str.lower())(self._zone_name)
             logger.info(XFRIN_XFR_TRANSFER_STARTED, req_str, self.zone_str())
             self._send_query(self._request_type)
             # count (A|X)IXFR requests for statistics
             getattr(counter,
                     'inc_%sreq%s' % (req_str.lower(), self.get_ipver_str()))\
                     (self._zone_name)
-            # start statistics timer
-            getattr(counter,
-                    'start_time_to_%s' % req_str.lower())(self._zone_name)
             self.__state = XfrinInitialSOA()
             self._handle_xfrin_responses()
             # Depending what data was found, we log different status reports
@@ -964,6 +964,9 @@ class XfrinConnection(asyncore.dispatcher):
                             "%.3f" % self._transfer_stats.get_running_time(),
                             "%.f" % self._transfer_stats.get_bytes_per_second()
                            )
+            # stop start timer
+            getattr(counter,
+                    'stop_time_to_%s' % req_str.lower())(self._zone_name)
 
         except XfrinZoneUptodate:
             # Eventually we'll probably have to treat this case as a trigger
@@ -1000,10 +1003,6 @@ class XfrinConnection(asyncore.dispatcher):
             if ret == XFRIN_OK:
                 # count successful xfer requests
                 counter.inc_xfrsuccess(self._zone_name)
-                # stop timer
-                getattr(counter,
-                        'stop_time_to_%s' % req_str.lower())\
-                        (self._zone_name)
             elif ret == XFRIN_FAIL:
                 # count failed xfer requests
                 counter.inc_xfrfail(self._zone_name)