You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2014/11/05 05:48:40 UTC

[jira] [Commented] (THRIFT-2720) nodejs - Error with enum generation

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

Hudson commented on THRIFT-2720:
--------------------------------

SUCCESS: Integrated in Thrift #1331 (See [https://builds.apache.org/job/Thrift/1331/])
THRIFT-2720 nodejs - Error with enum generation (roger: rev a9ddab5fa791b223f526e07a9530fd292ff96646)
* compiler/cpp/src/generate/t_js_generator.cc


> nodejs - Error with enum generation
> -----------------------------------
>
>                 Key: THRIFT-2720
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2720
>             Project: Thrift
>          Issue Type: Bug
>          Components: Node.js - Compiler
>    Affects Versions: 0.9.2, 1.0
>            Reporter: Rodolfo Ochoa
>            Assignee: Roger Meier
>             Fix For: 0.9.2
>
>
> the issue is on node.js, an enum is generated like this:
> ttypes.TPolicy = {
>  '1' : 'NO_SYNC',
>  'NO_SYNC' : 1,
>  '2' : 'SYNC',
>  'SYNC' : 2,
>  '3' : 'WRITE_NO_SYNC',
>  'WRITE_NO_SYNC' : 3
> };
> correct is:
> ttypes.TPolicy = {
>  'NO_SYNC' : 1,
>  'SYNC' : 2,
>  'WRITE_NO_SYNC' : 3
> };
> from the .thrift file:
> enum TPolicy
> {
>     NO_SYNC       = 1,
>     SYNC          = 2,
>     WRITE_NO_SYNC = 3
> }



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