xavifeds8 commented on PR #2394: URL: https://github.com/apache/phoenix/pull/2394#issuecomment-4108588335
@stoty , @virajjasani Can you please review on this approach for the migration of htrace into opentelemetry. This is a initial migration PR in the Future PR's i will be working on. 1. Depreciation TRACE ON, TRACE OFF through SQL Statement. 2. Adding in more Semantic Attributes in tracing through opentemetry-semconv 3. Adding in the test cases. 4. Standarising the span names as per Otel conventions. 5. Updating the https://phoenix.apache.org/tracing.html website. setup gist: [<script src="https://gist.github.com/xavifeds8/b7e8bcb7707844a1e106686559f04ccd.js";></script>](https://gist.github.com/xavifeds8/b7e8bcb7707844a1e106686559f04ccd) Please tell me if the approach seems good to you or you see any issues in the approach. Any suggestions or improvement in the approach are welcome. -- 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 specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
