[jira] [Commented] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163521#comment-14163521 ] Bret Curtis commented on THRIFT-2757: - I'm glad that we got it worked out! Cheers!

[jira] [Updated] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bret Curtis updated THRIFT-2757: Assignee: Konrad Grochowski (was: Bret Curtis) > TCompactProtocol and TJSONProtocol need boundary

[jira] [Assigned] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bret Curtis reassigned THRIFT-2757: --- Assignee: Bret Curtis (was: Konrad Grochowski) > TCompactProtocol and TJSONProtocol need bo

[jira] [Comment Edited] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163505#comment-14163505 ] Bret Curtis edited comment on THRIFT-2757 at 10/8/14 1:55 PM: -

[jira] [Commented] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163505#comment-14163505 ] Bret Curtis commented on THRIFT-2757: - Yeah, sys.maxint... doesn't exist in Python3.

[jira] [Commented] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163488#comment-14163488 ] Bret Curtis commented on THRIFT-2757: - If you don't like literals, maybe it is best t

[jira] [Commented] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163464#comment-14163464 ] Bret Curtis commented on THRIFT-2757: - 2) Where, I don't see that in the commit http

[jira] [Commented] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-08 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163443#comment-14163443 ] Bret Curtis commented on THRIFT-2757: - Great, thanks! I do however wonder about two

[jira] [Updated] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-02 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bret Curtis updated THRIFT-2757: Description: On the TCompactProtocol, using i16, i32, i64 there is currently no range checking hap

[jira] [Updated] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-02 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bret Curtis updated THRIFT-2757: Description: On the TCompactProtocol, using i16, i32, i64 there is currently no range checking hap

[jira] [Commented] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-02 Thread Bret Curtis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14156629#comment-14156629 ] Bret Curtis commented on THRIFT-2757: - My merge request can be found here: https://gi

[jira] [Created] (THRIFT-2757) TCompactProtocol and TJSONProtocol need boundary checking

2014-10-02 Thread Bret Curtis (JIRA)
Bret Curtis created THRIFT-2757: --- Summary: TCompactProtocol and TJSONProtocol need boundary checking Key: THRIFT-2757 URL: https://issues.apache.org/jira/browse/THRIFT-2757 Project: Thrift Issu