Re: [DISCUSS] Race condition in ProtobufRpcEngine2

2022-03-02 Thread Vinayakumar B
Call.toString() was used only while sending the response, in the response processor thread. At this point, header would be already read and processed. So no chance of race conditions. This race condition was not introduced by the HADOOP-17046 ,

Re: [DISCUSS] Race condition in ProtobufRpcEngine2

2022-02-28 Thread Ayush Saxena
Hey Andras, I had a quick look at HADOOP-18143, the methods in question in ProtobufRpcEngine2 are identical to the ones in ProtobufRpcEngine. So, I am not very sure how the race condition doesn't happen in ProtobufRpcEngine. I have to debug and spend some more time, considering that I have

[DISCUSS] Race condition in ProtobufRpcEngine2

2022-02-28 Thread Gyori Andras
Hey everyone! We have started seeing test failures in YARN PRs for a while. We have identified the problematic commit, which is HADOOP-18082 , however, this change just revealed the race condition lying in ProtobufRpcEngine2 introduced in