Skip to content
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.

Sign up for GitHub

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

Jump to bottom

panic when num conns == 0 #32

Merged
merged 1 commit into from
May 5, 2014
Merged

panic when num conns == 0 #32

merged 1 commit into from
May 5, 2014

Conversation

mreiferson
Copy link
Member 

panic: invalid argument to Intn

goroutine 62 [running]:
runtime.panic(0x846860, 0xc2108b9fc0)
        /usr/local/go/src/pkg/runtime/panic.c:266 +0xb6
math/rand.(*Rand).Intn(0xc21000a160, 0x0, 0x1)
        /usr/local/go/src/pkg/math/rand/rand.go:89 +0x65
math/rand.Intn(0x0, 0xc21012f120)
        /usr/local/go/src/pkg/math/rand/rand.go:155 +0x34
github.com/hailocab/go-nsq.(*Reader).rdyLoop(0xc2100d3200)
        /mnt/tmp/EO6AEmKY_hailocab-go-login-service-ac74f36abe66/src/github.com/hailocab/go-nsq/reader.go:660 +0xf9
created by github.com/hailocab/go-nsq.NewReader
        /mnt/tmp/EO6AEmKY_hailocab-go-login-service-ac74f36abe66/src/github.com/hailocab/go-nsq/reader.go:193 +0x3ec

@mreiferson mreiferson added the bug label May 5, 2014
@mreiferson
Copy link
Member Author

RFR @jehiah @jphines

jehiah added a commit that referenced this pull request May 5, 2014
@jehiah
Merge pull request #32 from mreiferson/panic_32
8d9a71e 
panic when num conns == 0
@jehiah jehiah merged commit 8d9a71e into nsqio:master May 5, 2014
@mreiferson mreiferson deleted the panic_32 branch May 5, 2014 15:21
amosir pushed a commit to amosir/go-nsq that referenced this pull request Jan 16, 2024
@mreiferson
Merge pull request nsqio#32 from kev1n80/inaccurate_depth_read_error_29
58c8e08 
Inaccurate depth count after readError on "current" file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mreiferson @jehiah