mrproliu commented on issue #8164:
URL: https://github.com/apache/skywalking/issues/8164#issuecomment-974823162


   > For what you propose, we need to cover `sw8`, `sw8-x` and 
`sw8-correlation` as keys, even in the SQL comment.
   
   Yes, all these keys should be covert and put into the SQL comment.
   
   > 
   > Meanwhile, how to put in the comment isn't a protocol level definition. 
But, considering this injection will effect existing standard SQL plugins, it 
is better to provide a new configuration at JDBC plugin level to activate this 
and build this format.
   
   That's true. we need to create a new configuration item to activate them. I 
think `plugin.jdbc.is_propagating_context` is fine, it mean should we need to 
add the tracing context to the SQL, default is `false`.
   


-- 
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]


Reply via email to