obelix74 commented on PR #3327:
URL: https://github.com/apache/polaris/pull/3327#issuecomment-3697337107

   > Thanks for the change @obelix74 !
   > 
   > took an initial passs, i do agree with @dimas-b on sending rest context as 
part of tag, can you explain how do you wanna use this ?
   
   I removed using `RestContext` and used `SLF4F#MDC` instead to get the 
request id in a commit that I pushed about 30 mins ago.  I am proposing using 
the request id (trace_id in the below picture) to tie in the request end to end 
from the client, all the way to the storage layer.  If the clients to the 
catalog are willing to share information about the SQLs executed, we can use 
the request id to tie it all together.
   
   Do I make sense?
   
   [
   <img width="696" height="454" alt="image" 
src="https://github.com/user-attachments/assets/740b4802-7e8d-49b1-8e97-955ee4d20cce";
 />
   ](url)


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