uber / cherami-client-go

81%

master: 83%

LAST BUILD ON BRANCH v2.6.0

branch: v2.6.0

CHANGE BRANCH

x

Reset

• v2.6.0
• master
• readme_build
• singleton-tchannel
• update_cthrift
• example
• fd-leak-fix
• v0.1.0
• reconfig_interval
• client_option
• processTimeMetric
• glideUp
• v0.2.0-rc1
• client_option_default
• hbClient
• v1.16.1-rc1
• v1.17.0-rc1
• graceful_drain
• v1.18.0-rc1
• stuck_read
• v1.19.0-rc0
• more_tags
• glide
• v1.20.0
• remove_childreporter
• fix_metric
• more_metrics
• v1.20.1
• pause
• v1.21.0-rc1
• v2.0.0
• v2.1.0
• aandrade.nack-delivery-when-closed
• v1_hotfix
• v1.21.0
• close-synchronous
• fix-double-close-wait
• release20160626
• v1.22.0
• v2.2.0
• tchannal_up
• v1.23.0
• t_up
• v2.3.0
• update_cherami_thrift_version
• v2.4.0
• v2.5.0
• cg_retry

Build # 516

Build Type

push

travis-ci

Committed by web-flow

Commit Message

Backoff retry is not needed for ReadConsumerGroupHosts() (#38)

Some users complained that Consumer.Open() does not respect the timeout setting in the client config. After digging I found that ReadConsumerGroupHosts() has a backoff retry wrapping the thrift call. For Open() such retry is not needed. The other use of ReadConsumerGroupHosts() is in reconfiguration which itself is a big retry loop. Therefore, we don't need to have backoff retry for ReadConsumerGroupHosts().

Run Details

233 of 288 relevant lines covered (80.9%)

2208.19 hits per line