opentracing-contrib / meta

A meta-repository for OpenTracing contributions
Apache License 2.0
35 stars 16 forks source link

request to transfer java-loggertracer under opentracing-contrib umbrella #14

Closed davidB closed 7 years ago

davidB commented 7 years ago

Hi,

I setup the project https://github.com/davidB/java-loggertracer, and would like to transfer it under opentracing-contrib umbrella.

Review and better name are welcome.

pavolloffay commented 7 years ago

Hi @davidB,

I had a quick look at it, correct me if I am wrong. It does not depend on any specific logging framework e.g. slf4j reporter can be moved to a separate module (with dep on jackson).

bhs commented 7 years ago

@davidB thanks for donating this :)

I think java-loggertracer is a reasonable, self-descriptive name. Since repos can always be renamed, I decided to carpe diem and created https://github.com/opentracing-contrib/java-loggertracer/invitations for you.

Please @-mention myself and @pavolloffay on your initial PR.

To be clear, you are an admin on this new repo and can run your own show there in terms of when merges happen, etc. Though I would recommend getting real / meaningful reviews at first while the groundwork is being established.

davidB commented 7 years ago

@bhs ,

About initial PR (migration), it's easier (for me) to push from my local clone of https://github.com/davidB/java-loggertracer to the new remote repo, than to create a PR. In fact, I don't know how to create a PR between 2 repos that have no fork relation. If you know a simple way to make it, tell me.

@pavolloffay ,

You're right since today, I introduce Reporter as abstraction to report to slf4j or other system (I'll probalbly provide a prometheus reporter). So each Reporter could be move to separated module, jar. But as starting point, I prefer to include Reporter and define its dependency in scope provided (except for jackson). It's simpler for developpement and not more works for user.

Thanks

davidB commented 7 years ago

@bhs ,

I 'll fork the new repo, and push to my fork.

davidB commented 7 years ago

Ah, I trashed my local and remote repo/copy :-(