grpc_logrus multithreading concerns about go-grpc-middleware

[mwitkow]

The log entry is then cloned every time on new request and the clone is put inside the Context variable of the request. That request is then handled by the dispatching goroutine. Most request handling is done in a single synchronous block, and unless someone dispatches more goroutines from it, it is safe.

Any concerns around that?

[pkartner]

Has this been a conscious decision? It is not uncommon to open new goroutines in a single request. Like you said most request handling is done in a single synchronous block. Logging in this way would force you to adopt two logging approaches, one for in the main request thread and one for the child threads.

[mwitkow]

Yes, this was a concious decision. In our codebase, whenever we fan out to sub to go routines (rare) we make sure that we clone the relevant objects and pass them explicitly.

Would you like to contribute documentation clarification around this?

[bwplotka]

This is pretty old and we know have v2 code with different structure. Let us know if this is still important, we can reopen.