collado-mike commented on PR #589: URL: https://github.com/apache/polaris/pull/589#issuecomment-2599438624
This was a pretty major design decision, IMO. `CallContext` is an interface that allows for a lot of extension and flexibility in the runtime. This PR went from "Ready for review" to "merged" in 20 minutes with very little community feedback. There are a lot of implications for things like MDC variables in logs and other context variables that were being set at the start of the request and are no longer accessible to downstream components. I get that the thread local implementation wasn't loved by everyone, but to completely rip out the interface entirely is major. cc: @jbonofre @adutra @dennishuo @eric-maynard @flyrain @RussellSpitzer @takidau -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
