Improved graceful shutdown - Collector

[jpkrohling]

Which problem is this PR solving?

• The Collector parts of Improved graceful shutdown - Agent #2031
• See Improved graceful shutdown - Agent #2031 (review) for context

Resolves #295

[codecov]

Codecov Report

Merging #2076 into master will increase coverage by <.01%.
The diff coverage is 57.69%.

@@            Coverage Diff             @@
##           master    #2076      +/-   ##
==========================================
+ Coverage   96.31%   96.31%   +<.01%     
==========================================
  Files         214      214              
  Lines       10532    10540       +8     
==========================================
+ Hits        10144    10152       +8     
  Misses        329      329              
  Partials       59       59

Impacted Files	Coverage Δ
cmd/collector/app/server/test.go	33.33% <0%> (-16.67%)	⬇️
cmd/collector/app/span_handler_builder.go	100% <100%> (ø)	⬆️
cmd/collector/app/span_processor.go	100% <100%> (ø)	⬆️
cmd/collector/app/collector.go	72.41% <43.75%> (-1.4%)	⬇️
...lugin/sampling/strategystore/adaptive/processor.go	100% <0%> (+0.79%)	⬆️
cmd/query/app/server.go	94.52% <0%> (+2.73%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5036f4...f686434. Read the comment docs.

[yurishkuro]

did you mean to do git checkout master cmd/agent?

[jpkrohling]

This is on top of #2031. Once that one is merged, this will get rebased and should show only the collector changes.

[annanay25]

Should this be a factory? zipkinSpansHandler, jaegerBatchesHandler, grpcHandler can implement Factory.CreateHandler()

[jpkrohling]

Good question, probably as part of another PR.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (master@043b1ae). Click here to learn what that means.
The diff coverage is 57.69%.

@@            Coverage Diff            @@
##             master    #2076   +/-   ##
=========================================
  Coverage          ?   96.33%           
=========================================
  Files             ?      214           
  Lines             ?    10537           
  Branches          ?        0           
=========================================
  Hits              ?    10151           
  Misses            ?      328           
  Partials          ?       58

Impacted Files	Coverage Δ
cmd/collector/app/server/test.go	33.33% <0%> (ø)
cmd/collector/app/span_handler_builder.go	100% <100%> (ø)
cmd/collector/app/span_processor.go	100% <100%> (ø)
cmd/collector/app/collector.go	72.41% <43.75%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 043b1ae...c7b0a5b. Read the comment docs.

[jpkrohling]

@annanay25, @yurishkuro is this ready to be merged once 1.17 is out?

[yurishkuro]

I haven't reviewed this one. Let's merge the agent change and rebase this PR.