milenkovicm commented on PR #18192: URL: https://github.com/apache/datafusion/pull/18192#issuecomment-3447579397
> > if the TaskContext and DecodeContext are reused to decode protos from multiple clients or even from one client sending plan after plan, is there a chance of id collision? > > Yes, my intention was that you create a new `DecodeContext` per plan that you decode. then public methods should consume (and invalidate) `DecodeContext` instead of passing is as a reference -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
