Chen
Chen
> 可以写个Jersey这个RestClient的案例吗? I'm not familiar with the Rest Client, but [here](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/supported-libraries.md#libraries--frameworks) I see the official enhanced documentation. Maybe you can try it yourself?
> hi @chenmudu! does the new(-ish) https://github.com/open-telemetry/opentelemetry-demo repository address your thoughts above? LGMT. Thanks for remembering this issue.
Looks like a good suggestion, but there is some work involved in constructing the DAG. @shaohuading
> Generally look good to me. One problem is that can we keep consistency for the nouns? Something like logs, sometime it uses "日志", sometimes it uses "log". Same situation...
@JohnWu20 @lichunli-amzn Some minor bugs that were not easily detect have been fixed. If you have time, could you review it? Thanks a lot.
@JohnWu20 @lubingfeng @raptorsun Please review it again, if you have time. Thanks a lot.
> Direct PR is great, let's have at least one LGTM from Chinese speaker/reader contributor then we can merge (: Yeah, thanks. Waiting... 😀
> @chenmudu @lubingfeng @JohnWu20 @lichunli-amzn 我的修改建议都提完了。有劳各位再校对一下。 > @bwplotka My review is over. Now double-checking with other translators 👀 🎉 This update fixes everyone's suggestions. Thanks a lot. If there are...
> As we modify the original, I'll ping you folks to change the Chinese version too. Does that sound reasonable? Chinese version should change with the original version, I am...
LGTM, I think it's time for us to merge this pr. @ArthurSens There hasn't been any problem feedback for several days.