apm-agent-java icon indicating copy to clipboard operation
apm-agent-java copied to clipboard

Remove unused API and resolve naming ambiguities

Open raphw opened this issue 1 year ago • 1 comments

This PR makes a clear distinction between API and implementation, and cleans up a bit. For this:

  • The internal Tracer API is removed.
  • Implementation versions of API types are suffixed Impl.
  • The ServiceAwareTracer API is consolidated with the Tracer API.
  • Minor clean up of dependency imports.

raphw avatar Mar 13 '24 19:03 raphw

👋 @raphw Thanks a lot for your contribution!

It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it.

Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it.

github-actions[bot] avatar Mar 13 '24 19:03 github-actions[bot]

Hi! We just realized that we haven't looked into this issue in a while. We're sorry! We're labeling this issue as stalled to make it hit our filters and make sure we get back to it in as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

botelastic[bot] avatar May 12 '24 20:05 botelastic[bot]

note approved but failing

jackshirazi avatar Jun 13 '24 21:06 jackshirazi

My bad. Non-trivial merge went too quick. It should work now!

raphw avatar Jun 16 '24 05:06 raphw

run docs-build

jackshirazi avatar Jun 17 '24 11:06 jackshirazi