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

Issue 9626044: code review 9626044: go.talks: add Advanced Go Concurrency Patterns (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 11 months ago by adg
Modified:
10 years, 11 months ago
Reviewers:
dave
CC:
Sameer, golang-dev
Visibility:
Public.

Description

go.talks: add Advanced Go Concurrency Patterns

Patch Set 1 #

Patch Set 2 : diff -r 7597f5c52a13 https://code.google.com/p/go.talks #

Patch Set 3 : diff -r 7597f5c52a13 https://code.google.com/p/go.talks #

Patch Set 4 : diff -r 5401b06d75c0 https://code.google.com/p/go.talks #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1456 lines, -0 lines) Patch
A 2013/advconc.slide View 1 2 1 chunk +384 lines, -0 lines 0 comments Download
A 2013/advconc/buffer.go View 1 1 chunk +43 lines, -0 lines 0 comments Download
A 2013/advconc/dedupermain.go View 1 1 chunk +41 lines, -0 lines 0 comments Download
A 2013/advconc/fakemain.go View 1 1 chunk +39 lines, -0 lines 0 comments Download
A 2013/advconc/fakemainnomerge.go View 1 1 chunk +33 lines, -0 lines 0 comments Download
A 2013/advconc/gopherhat.jpg View 1 Binary file 0 comments Download
A 2013/advconc/gopherrunning.jpg View 1 Binary file 0 comments Download
A 2013/advconc/gopherswim.jpg View 1 Binary file 0 comments Download
A 2013/advconc/gopherswrench.jpg View 1 Binary file 0 comments Download
A 2013/advconc/naivemain.go View 1 1 chunk +85 lines, -0 lines 0 comments Download
A 2013/advconc/naivemainnomerge.go View 1 1 chunk +30 lines, -0 lines 0 comments Download
A 2013/advconc/nilselect.go View 1 1 chunk +30 lines, -0 lines 0 comments Download
A 2013/advconc/pingpong1.go View 1 1 chunk +31 lines, -0 lines 0 comments Download
A 2013/advconc/pingpong2.go View 1 1 chunk +42 lines, -0 lines 0 comments Download
A 2013/advconc/pingpongdeadlock.go View 1 1 chunk +31 lines, -0 lines 0 comments Download
A 2013/advconc/pingpongpanic.go View 1 1 chunk +33 lines, -0 lines 0 comments Download
A 2013/advconc/race.out View 1 1 chunk +62 lines, -0 lines 0 comments Download
A 2013/advconc/race.png View 1 Binary file 0 comments Download
A 2013/advconc/reader/reader.go View 1 1 chunk +572 lines, -0 lines 0 comments Download

Messages

Total messages: 4
adg
Hello snanda@chromium.org (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.talks
10 years, 11 months ago (2013-05-21 22:26:07 UTC) #1
adg
*** Submitted as https://code.google.com/p/go/source/detail?r=1eecbdc1a1fc&repo=talks *** go.talks: add Advanced Go Concurrency Patterns R=snanda CC=golang-dev https://codereview.appspot.com/9626044
10 years, 11 months ago (2013-05-21 22:26:36 UTC) #2
dave_cheney.net
This broke the go.talks build, it isn't high priority. On Wed, May 22, 2013 at ...
10 years, 11 months ago (2013-05-21 22:53:45 UTC) #3
adg
10 years, 11 months ago (2013-05-21 23:18:10 UTC) #4
Can you please create an issue and assign it to sameer?


On 22 May 2013 08:52, Dave Cheney <dave@cheney.net> wrote:

> This broke the go.talks build, it isn't high priority.
>
> On Wed, May 22, 2013 at 8:26 AM,  <adg@golang.org> wrote:
> > *** Submitted as
> > https://code.google.com/p/go/source/detail?r=1eecbdc1a1fc&repo=talks ***
> >
> >
> > go.talks: add Advanced Go Concurrency Patterns
> >
> > R=snanda
> > CC=golang-dev
> > https://codereview.appspot.com/9626044
> >
> >
> > https://codereview.appspot.com/9626044/
> >
> >
> > --
> >
> > ---You received this message because you are subscribed to the Google
> Groups
> > "golang-dev" group.
> > To unsubscribe from this group and stop receiving emails from it, send an
> > email to golang-dev+unsubscribe@googlegroups.com.
> > For more options, visit https://groups.google.com/groups/opt_out.
> >
> >
>
Sign in to reply to this message.

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