You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "James E. King, III (JIRA)" <ji...@apache.org> on 2017/03/30 15:41:41 UTC

[jira] [Commented] (THRIFT-455) Compiler flag to make optional/required field modifiers compulsory

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

James E. King, III commented on THRIFT-455:
-------------------------------------------

THRIFT-4126 presents a php implementation change that does required field validation on serialization and deserialization for the binary protocol.  I started doing a little bit of work on THRIFT-4142 to change the cross and/or functional tests to verify behavior as well, and I'm concerned about the general lack of standardization in the area of required field handling, then I found this ticket.  It looks it will take a significant effort to properly handle required fields.

> Compiler flag to make optional/required field modifiers compulsory
> ------------------------------------------------------------------
>
>                 Key: THRIFT-455
>                 URL: https://issues.apache.org/jira/browse/THRIFT-455
>             Project: Thrift
>          Issue Type: New Feature
>          Components: Compiler (General)
>            Reporter: Bryan Duxbury
>
> if we are unwilling to globally require users specify "required" or "optional" when creating their structs, I think we should at least add a command line switch that causes us to enforce that behavior. It would certainly make my implementations more reliable, especially when I'm giving non-experts the tools to make their own IDLs.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)