Merge pull request #2371 from influxdb/2352

Don't set client to nil when closing broker
pull/2372/head
Jason Wilder 2015-04-21 15:37:40 -06:00
commit 9993c0dab3
2 changed files with 4 additions and 6 deletions

View File

@ -30,6 +30,7 @@
- [#2272](https://github.com/influxdb/influxdb/pull/2272): clustering: influxdb 0.9.0-rc23 panics when doing a GET with merge_metrics in a - [#2272](https://github.com/influxdb/influxdb/pull/2272): clustering: influxdb 0.9.0-rc23 panics when doing a GET with merge_metrics in a
- [#2350](https://github.com/influxdb/influxdb/pull/2350): Issue fix for :influxd -hostname localhost. - [#2350](https://github.com/influxdb/influxdb/pull/2350): Issue fix for :influxd -hostname localhost.
- [#2367](https://github.com/influxdb/influxdb/pull/2367): PR for issue #2350 - Always use localhost, not host name. - [#2367](https://github.com/influxdb/influxdb/pull/2367): PR for issue #2350 - Always use localhost, not host name.
- [#2371](https://github.com/influxdb/influxdb/pull/2371): Don't set client to nil when closing broker Fixes #2352
## v0.9.0-rc25 [2015-04-15] ## v0.9.0-rc25 [2015-04-15]

View File

@ -73,12 +73,9 @@ func (b *Broker) Close() error {
b.done = nil b.done = nil
} }
if b.client != nil { // since the client doesn't specify a Transport when created,
// since the client doesn't specify a Transport when created, // it will use the DefaultTransport.
// it will use the DefaultTransport. http.DefaultTransport.(*http.Transport).CloseIdleConnections()
http.DefaultTransport.(*http.Transport).CloseIdleConnections()
b.client = nil
}
return b.Broker.Close() return b.Broker.Close()
} }