Code review comment for lp:~axwalk/juju-core/apiclient-open-parallel

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Reviewers: mp+213401_code.launchpad.net,

Message:
Please take a look.

Description:
state/api: dial info.Addrs in parallel

This change is to dial all addresses in
parallel. The first one that successfully
connects is used.

https://code.launchpad.net/~axwalk/juju-core/apiclient-open-parallel/+merge/213401

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/82450043/

Affected files (+135, -25 lines):
   A [revision details]
   M state/api/apiclient.go
   A state/api/apiclient_test.go

« Back to merge proposal