At the cluster side, if the client does not pass any trace info to the server, we will start a new root span to do tracing.
So there is no problem for an hbase2 client talking with hbase3 cluster. Stack <[email protected]> 于2021年4月19日周一 下午11:53写道: > Nice work here Duo. I think we should get this in. > > Any new pictures of traces to entice? > > I don't think this will get in the way of an hbase2 client talking to an > hbase3 cluster; what do you think? > > S > > On Thu, Apr 8, 2021 at 9:24 PM 张铎(Duo Zhang) <[email protected]> > wrote: > > > Hi, all the sub tasks for HBASE-22120 has been resolved. > > > > And in https://issues.apache.org/jira/browse/HBASE-23762, I added a > simple > > documentation on how to enable tracing in HBase. > > > > PTAL at the big PR > > > > https://github.com/apache/hbase/pull/2901 > > > > If no big concerns, I will start a vote later to merge the branch back to > > master. > > > > Thanks. > > > > > > > > 张铎(Duo Zhang) <[email protected]> 于2021年2月27日周六 下午5:56写道: > > > > > > https://github.com/open-telemetry/opentelemetry-java/releases/tag/v1.0.0 > > > > > > 1.0.0 is out finally. Let me start working. > > > > > > 张铎(Duo Zhang) <[email protected]> 于2021年2月19日周五 下午1:50写道: > > > > > >> > > >> > > >> Stack <[email protected]> 于2021年2月19日周五 上午9:17写道: > > >> > > >>> On Sun, Feb 14, 2021 at 7:26 AM 张铎(Duo Zhang) <[email protected] > > > > >>> wrote: > > >>> > > >>> > Some updates here, I've successfully set up a cluster with tracing > > >>> enabled > > >>> > in my company. > > >>> > > > >>> > > > >>> Hurray! > > >>> > > >>> > > >>> > But the problem is that, during opentelemetry 0.14 -> 0.15, the API > > has > > >>> > been changed a lot, so we need to modify our code accordingly. > > >>> > > > >>> > > > >>> Is the incompatible change in API allowed because they are not at > 1.0.0 > > >>> do > > >>> you think? > > >>> > > >>> (Wondering how 'stable' 1.0.0 API will be when it comes out...) > > >>> > > >> They are publishing new releases very quickly, now it is 0.17.0, and > > >> there are plenty of breaking changes in every release... > > >> Seems their goal is to make a clean API when 1.0.0 is out... > > >> > > >>> > > >>> > > >>> > > >>> > The good news is that the opentelemetry community kept publishing > new > > >>> > releases quickly recently, so the final 1.0.0 RC is likely to come > > >>> soon. > > >>> > > > >>> > Will keep an eye on the new releases, and once it is done, will > open > > >>> new > > >>> > issues to modify our code base and finish HBASE-22120. > > >>> > > > >>> > > > >>> Thanks, > > >>> S > > >>> > > >>> > > >>> > > >>> > Thanks. > > >>> > > > >>> > Sean Busbey <[email protected]> 于2021年1月23日周六 上午2:40写道: > > >>> > > > >>> > > One of the big gaps in our tracing stuff to date is a lack of > > >>> > > documentation. Without some guidance to show folks how to use the > > >>> > > tracing, it was always unclear how broken or working our tracing > > was > > >>> > > in htrace. I'd like to see some kind of docs in the feature > branch > > >>> > > prior to merge. > > >>> > > > > >>> > > On Thu, Jan 21, 2021 at 11:43 PM Duo Zhang <[email protected]> > > >>> wrote: > > >>> > > > > > >>> > > > In HBASE-22120, I tried to use OpenTelemetry to replace HTrace > in > > >>> > HBase, > > >>> > > > and it is basically done. > > >>> > > > > > >>> > > > https://github.com/apache/hbase/pull/2901 > > >>> > > > > > >>> > > > I've filed HBASE-25526 for later improvement. > > >>> > > > > > >>> > > > The remaining works for HBASE-22120 are > > >>> > > > 1. Enabling trace on a real cluster, to test whether the whole > > flow > > >>> > > works. > > >>> > > > 2. Test the performance impact. > > >>> > > > 3. Documentation. > > >>> > > > > > >>> > > > Once the above works are done and there are no big concerns, I > > will > > >>> > > start a > > >>> > > > vote to merge HBASE-22120 back to master. > > >>> > > > > > >>> > > > Thanks. > > >>> > > > > >>> > > > >>> > > >> > > >
