You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@thrift.apache.org by GitBox <gi...@apache.org> on 2020/11/06 07:34:56 UTC

[GitHub] [thrift] dcelasun commented on pull request #2181: THRIFT-5233: Handle I/O timeouts in go library

dcelasun commented on pull request #2181:
URL: https://github.com/apache/thrift/pull/2181#issuecomment-722927178


   > I suggest you take a look at the golang official net/http implementation before any furthur discussion
   
   If I may, "go read net/http" is not a productive attitude. You are arguing against a change that was already discussed, agreed to and merged. That's fine, it's even welcome. But the onus is on *you* to convince us there is a better way and that we should revert this. You didn't even address @fishy's first and second point.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org