[
https://issues.apache.org/jira/browse/THRIFT-815?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12895107#action_12895107
]
Jordan commented on THRIFT-815:
-------------------------------
Jake,
I'll have to check, but I remember the symptoms of the problem with lists of
objects, was that the results would have maybe one or two valid objects, and
the rest were missing. Or even worse, and object would be duplicated in the
place of all of the objects, and the others would be missing. I recall that
testInsanity didn't check these edge cases (unless you fixed it.)
I'm not currently able to get the most recent Thrift to build on my desktop,
but I'll see if I can pinpoint where the test is lacking.
> Deserialization of lists is critically broken.
> ----------------------------------------------
>
> Key: THRIFT-815
> URL: https://issues.apache.org/jira/browse/THRIFT-815
> Project: Thrift
> Issue Type: Bug
> Components: Compiler (JavaScript)
> Affects Versions: 0.4
> Reporter: Jordan
> Assignee: T Jake Luciani
> Priority: Critical
> Fix For: 0.4
>
> Attachments: 813-1.patch, test.html, THRIFT-815_thrift.js.patch
>
>
> Edit the test code that comes with the js language target:
> var list = [1,2,3];
> var ret = client.testList(list);
> debugger;
> ret comes back as [3,3,3] when it should be echoed back as [1,2,3]
> The test case never caught this because it only verified the size, and not
> the contents of the returned array.
> I cannot find an immediate workaround, but I will try wrapping it in a dummy
> type like ListServiceResponse.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
