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...) > 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. > > >
