You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Ashutosh Chauhan (JIRA)" <ji...@apache.org> on 2012/06/07 23:52:23 UTC
[jira] [Created] (THRIFT-1625) Null handling in TBinaryProtocol
Ashutosh Chauhan created THRIFT-1625:
----------------------------------------
Summary: Null handling in TBinaryProtocol
Key: THRIFT-1625
URL: https://issues.apache.org/jira/browse/THRIFT-1625
Project: Thrift
Issue Type: Bug
Components: Java - Library
Affects Versions: 0.8, 0.7, 0.9
Reporter: Ashutosh Chauhan
As pointed out here: https://issues.apache.org/jira/browse/HIVE-2800?focusedCommentId=13291334&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13291334 thrift doesn't check for nulls in payload and will throw NPE if data has null in it.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (THRIFT-1625) Null handling in TBinaryProtocol
Posted by "Ashutosh Chauhan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/THRIFT-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13291351#comment-13291351 ]
Ashutosh Chauhan commented on THRIFT-1625:
------------------------------------------
If its by design and applications are expected never to serialize nulls, feel free to close the ticket as invalid. Wanted to make sure this is intentional.
> Null handling in TBinaryProtocol
> --------------------------------
>
> Key: THRIFT-1625
> URL: https://issues.apache.org/jira/browse/THRIFT-1625
> Project: Thrift
> Issue Type: Bug
> Components: Java - Library
> Affects Versions: 0.7, 0.8, 0.9
> Reporter: Ashutosh Chauhan
>
> As pointed out here: https://issues.apache.org/jira/browse/HIVE-2800?focusedCommentId=13291334&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13291334 thrift doesn't check for nulls in payload and will throw NPE if data has null in it.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Resolved] (THRIFT-1625) Null handling in TBinaryProtocol
Posted by "Ashutosh Chauhan (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/THRIFT-1625?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ashutosh Chauhan resolved THRIFT-1625.
--------------------------------------
Resolution: Won't Fix
Sushanth pointed out this thread : http://www.mail-archive.com/thrift-dev@incubator.apache.org/msg01185.html So consensus seems to be that its not allowed to send null values in containers and user should do null checks to prevent attempting to serialize nulls.
Resolving as Won't Fix.
> Null handling in TBinaryProtocol
> --------------------------------
>
> Key: THRIFT-1625
> URL: https://issues.apache.org/jira/browse/THRIFT-1625
> Project: Thrift
> Issue Type: Bug
> Components: Java - Library
> Affects Versions: 0.7, 0.8, 0.9
> Reporter: Ashutosh Chauhan
>
> As pointed out here: https://issues.apache.org/jira/browse/HIVE-2800?focusedCommentId=13291334&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13291334 thrift doesn't check for nulls in payload and will throw NPE if data has null in it.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira