[ 
https://issues.apache.org/jira/browse/THRIFT-2968?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14306062#comment-14306062
 ] 

Randy Abernethy commented on THRIFT-2968:
-----------------------------------------

Hey Andrew,

As you note this patch does not apply yet. I just looked ti over though and 
want to flag the fact that as it stands the patch introduces a lot of white 
spaces errors. I bring this up not to nit pick but because each white space 
error is a diff and it takes everyone longer to figure out where the actual 
code changes are. The attached patch has 10 hunks, 8 of which are white space 
errors. 

-Randy

> nodejs: http connection socket timeout option
> ---------------------------------------------
>
>                 Key: THRIFT-2968
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2968
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Node.js - Library
>            Reporter: Andrew de Andrade
>            Priority: Trivial
>         Attachments: 0001-nodejs-http-socket-timeout.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to