BIND 10 #2689: Investigate work needed for removing threads from stats tests

BIND 10 Development do-not-reply at isc.org
Fri Feb 22 18:43:11 UTC 2013


#2689: Investigate work needed for removing threads from stats tests
-------------------------------------+-------------------------------------
            Reporter:  jelte         |                        Owner:
                Type:  task          |  jinmei
            Priority:  medium        |                       Status:
           Component:  statistics    |  reviewing
            Keywords:                |                    Milestone:
           Sensitive:  0             |  Sprint-20130305
         Sub-Project:  DNS           |                   Resolution:
Estimated Difficulty:  5             |                 CVSS Scoring:
         Total Hours:  0             |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Replying to [comment:6 jelte]:
 > I have pushed a few trivial typo fixes in comments.
 >
 > I suspect a number of these tests are still quite fragile in terms of
 refactoring (or worse, might miss failures due to external refactoring),
 but in general I think this is a very good step forwards, and have no
 code-wise issues with the branch, so please go ahead and merge.
 >
 > big +1 on the unrelated fix btw, awesome.

 Thank for the review.

 Unfortunately I noticed it caused non trivial regression on merge
 due to recent changes to `ModuleCCSession`.  So I first merge master
 into the branch and fixed the issues.  I believe these are mostly
 straightforward, but they are still beyond a trivial level and I think
 need another round of review.  Could you check them on the same
 trac2689 branch?  git diff 808dcdccc72caa04645c15af4493a76bacbd838f
 should give the diff to review.

-- 
Ticket URL: <http://bind10.isc.org/ticket/2689#comment:7>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list