Created
February 18, 2019 19:23
-
-
Save gabibeyer/345b9e60dbb86e4eb20e242d064ee7c2 to your computer and use it in GitHub Desktop.
Example concurrency bug found in project
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
package main | |
import ( | |
"fmt" | |
"math/rand" | |
"sync" | |
"time" | |
) | |
func doTask(fileName string) (string, error) { | |
r := rand.Intn(100) | |
if r%2 == 0 { | |
return "", fmt.Errorf("FAKE ERROR: %s", fileName) | |
} | |
return fileName, nil | |
} | |
func main() { | |
rand.Seed(time.Now().UnixNano()) | |
var wg sync.WaitGroup | |
workers := 4 | |
wg.Add(workers) | |
jobCh := make(chan string) | |
errCh := make(chan error) | |
for i := 0; i < workers; i++ { | |
go func() { | |
defer wg.Done() | |
for fileName := range jobCh { | |
in, err := doTask(fileName) | |
if err != nil { | |
errCh <- err | |
break | |
} | |
fmt.Println("string processed: ", in) | |
} | |
}() | |
} | |
testStrings := []string{"one", "two", "three", "four", "five", "six", "seven", "eight", "nine", "ten"} | |
var err error | |
for _, s := range testStrings { | |
select { | |
case jobCh <- s: | |
fmt.Println("adding job: ", s) | |
case err = <-errCh: | |
fmt.Println("error found", err) | |
break | |
} | |
} | |
close(jobCh) | |
wg.Wait() | |
} |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment