[GitHub] [hbase] ndimiduk commented on pull request #3754: HBASE-26361 Enable OpenTelemetry to be used from developer sandbox

ndimiduk commented on pull request #3754: URL: https://github.com/apache/hbase/pull/3754#issuecomment-946228343 Thanks for the reviews! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [hbase] ndimiduk commented on pull request #3754: HBASE-26361 Enable OpenTelemetry to be used from developer sandbox

ndimiduk commented on pull request #3754: URL: https://github.com/apache/hbase/pull/3754#issuecomment-946228257 Manual testing confirmed this works for both dev sandbox and bin-tgz. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [hbase] ndimiduk commented on pull request #3754: HBASE-26361 Enable OpenTelemetry to be used from developer sandbox

ndimiduk commented on pull request #3754: URL: https://github.com/apache/hbase/pull/3754#issuecomment-946177258 While creating the master patch, I noticed that the order of jshell configuration and otel configuration are swapped between the two branches. I've updated this patch so that

[GitHub] [hbase] ndimiduk commented on pull request #3754: HBASE-26361 Enable OpenTelemetry to be used from developer sandbox

ndimiduk commented on pull request #3754: URL: https://github.com/apache/hbase/pull/3754#issuecomment-944494472 > I think this is good if you test it locally and it could work. Please make sure that we do not miss the commit to master as the PR is for branch-2... @Apache9