[
https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15083559#comment-15083559
]
ASF GitHub Bot commented on THRIFT-3521:
----------------------------------------
Github user bgould commented on the pull request:
https://github.com/apache/thrift/pull/772#issuecomment-169096832
I thought about that too but didn't have time to do it this morning and
just wanted to get a working build. I think that perhaps better than a regex
might be to actually parse the JSON string and check the values. I'll see if I
can get that working.
> TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)
> ----------------------------------------------------------------------
>
> Key: THRIFT-3521
> URL: https://issues.apache.org/jira/browse/THRIFT-3521
> Project: Thrift
> Issue Type: Bug
> Components: Java - Library
> Environment: OpenJDK 8
> Reporter: Benjamin Gould
> Assignee: Benjamin Gould
> Priority: Minor
> Fix For: 0.9.4
>
>
> TSimpleJSONProtocolTest has a couple of tests where structs are serialized,
> and the resultant JSON string is compared for equality with an expected value.
> Some of the structs' fields are sets, which do not ensure consistent
> ordering, causing this test to fail in environments where the "natural"
> ordering of the elements in the HashSet do not match the expected value.
> Suggested fix is to unset the fields that cause the inconsistent ordering and
> remove them from the expected value.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
