Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1681)

Issue 7456049: code review 7456049: net: make TestConnAndListener less flaky (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 1 month ago by bradfitz
Modified:
11 years, 1 month ago
Reviewers:
mikio
CC:
golang-dev, minux1
Visibility:
Public.

Description

net: make some tests less flaky Fixes issue 4969

Patch Set 1 #

Patch Set 2 : diff -r 641f05de9e68 https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r 641f05de9e68 https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r f545866390ab https://go.googlecode.com/hg/ #

Patch Set 5 : diff -r f545866390ab https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -26 lines) Patch
M src/pkg/net/conn_test.go View 1 4 chunks +13 lines, -8 lines 0 comments Download
M src/pkg/net/protoconn_test.go View 1 2 3 6 chunks +18 lines, -18 lines 0 comments Download

Messages

Total messages: 9
bradfitz
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg/
11 years, 1 month ago (2013-03-04 17:23:42 UTC) #1
minux1
given that I couldn't make test TestTCPConnSpecificMethods pass on the netbsd/arm builder (see http://build.golang.org/log/67d1c8aec152c5d1c23923fba2aa8f254ff3f83b), could ...
11 years, 1 month ago (2013-03-04 18:28:26 UTC) #2
bradfitz
Hello golang-dev@googlegroups.com, minux.ma@gmail.com (cc: golang-dev@googlegroups.com), Please take another look.
11 years, 1 month ago (2013-03-04 19:14:33 UTC) #3
bradfitz
On Mon, Mar 4, 2013 at 10:28 AM, <minux.ma@gmail.com> wrote: > given that I couldn't ...
11 years, 1 month ago (2013-03-04 19:14:40 UTC) #4
minux1
LGTM. Please make the description more general, perhaps just "net: make non-timeout tests less flaky".
11 years, 1 month ago (2013-03-04 19:44:59 UTC) #5
minux1
On Tue, Mar 5, 2013 at 3:14 AM, Brad Fitzpatrick <bradfitz@golang.org> wrote: > On Mon, ...
11 years, 1 month ago (2013-03-04 19:46:39 UTC) #6
bradfitz
*** Submitted as https://code.google.com/p/go/source/detail?r=e39b9fa789ca *** net: make some tests less flaky Fixes issue 4969 R=golang-dev, ...
11 years, 1 month ago (2013-03-04 19:55:31 UTC) #7
mikio
phew, thanks. did i make a crosstalk situation by deadlines?
11 years, 1 month ago (2013-03-04 23:48:45 UTC) #8
minux1
11 years, 1 month ago (2013-03-04 23:50:25 UTC) #9
On Tue, Mar 5, 2013 at 7:48 AM,  <mikioh.mikioh@gmail.com> wrote:
> phew, thanks.
> did i make a crosstalk situation by deadlines?
no, it's just some builders are really slow.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b