DomGarguilo commented on PR #5719: URL: https://github.com/apache/accumulo/pull/5719#issuecomment-3062846945
Here is some updated testing with shell output and screenshots of the trace graphs in jaeger. Here is what I see in the shell now when running `trace on` and `trace off` ``` root@uno test2> insert row cf cq value323e2safsdffsdfs root@uno test2> trace on 2025-07-11T11:43:26,743 [trace.TraceUtil] INFO : Trace enabled in Accumulo: yes, OpenTelemetry instance: class io.opentelemetry.javaagent.instrumentation.opentelemetryapi.v1_27.ApplicationOpenTelemetry127, Tracer instance: class io.opentelemetry.javaagent.instrumentation.opentelemetryapi.trace.ApplicationTracer root@uno test2> scan row cf:cq [] value323e2safsdffsdfs root@uno test2> trace off 2025-07-11T11:43:29,172 [trace.TraceUtil] INFO : Trace enabled in Accumulo: no, OpenTelemetry instance: class io.opentelemetry.api.DefaultOpenTelemetry, Tracer instance: class io.opentelemetry.api.trace.DefaultTracer ``` It seems a bit verbose especially for `trace off` but this new output seems fine to me. Here is the graph view in jaeger of the operation above: <img width="3932" height="1438" alt="Screenshot from 2025-07-11 11-43-50" src="https://github.com/user-attachments/assets/0036cb25-88fb-4e3e-a97b-0992422aa0b7"; /> And here is another trace where I ran `createtable` in the shell: <img width="3847" height="668" alt="Screenshot from 2025-07-11 11-42-56" src="https://github.com/user-attachments/assets/88f0c7d6-625c-40e2-b575-6e13aff0960f"; /> The new code to add the command to the trace name is helpful. -- 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]
