6 Dec
2023
6 Dec
'23
10:46 p.m.
I think that's usually the case. The timeout was originally 1 ms and I didn't encounter obvious issues while testing (of course my testing was also necessarily limited). I guess the question is whether trying to go with this and possibly finding out the hard way is what we want to do for this one.
Perhaps this is rash of me, but the way I see it, if we don't find out the hard way, we're probably not going to find out at all. If there's a clear benefit on the other side, and the patch is bisectable, and we've tested some applications (especially if we know applications that are sensitive to this kind of thing) then I'm inclined to accept it.