Added metrics to Smartling TMS sync

[maallen]

Adds latency metrics to the Smartling Third Party Sync to enable a better understanding of where performance bottlenecks may occur.

[maallen]

@aurambaj Just to note that there is some existing instrumentation on the overall push/pull methods using the @Timed annotation

@Timed("SmartlingSync.push")

however we can't use tags with this approach as the specific runtime repo/locale isn't available at compile time. With this in mind should I remove the above annotation and rename the new stats to match the old stat or leave them as they are in the PR at present?

https://github.com/box/mojito/pull/966/files#diff-debc760cec87b95894f514bc911c2692b8a928ac4052d0ad121a930f17d7127aR375

[aurambaj]

With this in mind should I remove the above annotation and rename the new stats to match the old stat or leave them as they are in the PR at present?

Good question. The annotation comes with 2 tags: method and class name. Those tags would be gone with the new code. Classname could be easily added back. The method not so much but I don't think we truly use those.

One thing i've been thinking was to change the aspect to support "tags extraction" from the method params, like what we do in PollableTask. That can also prevent some issues with "final" requirement.

With that said, it feels like we can reuse the old name and just drop those 2 tags since we have that in the metric name