add finish to detectormanager span

[dinilatgit]

No description provided.

[codecov]

Codecov Report

Merging #765 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #765   +/-   ##
=======================================
  Coverage   88.65%   88.66%           
=======================================
  Files         171      171           
  Lines        3544     3545    +1     
  Branches      245      245           
=======================================
+ Hits         3142     3143    +1     
  Misses        320      320           
  Partials       82       82           

Impacted Files	Coverage Δ
...a/adaptivealerting/kafka/KafkaDetectorManager.java	89.10% <100.00%> (+0.10%)	⬆️

---

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 21a7742...7ff4c5a. Read the comment docs.

[dinilatgit]

The detector manager span is only to add tags to the complete trace which is created and managed by the library. The span is not designed to calculate trace in detector manager, that is done by the library interceptor class itself.

opentracing-contrib/java-kafka-client#77 has been logged to know if there is a way to provide tags to the library functions. Detector Manager span is a work around.

[zrashwani]

approved, by we should re-visit, and handles the limitation of library inability to add custom tags in better way