[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15083540#comment-15083540 ]
ASF GitHub Bot commented on THRIFT-3521: ---------------------------------------- Github user nsuke commented on the pull request: https://github.com/apache/thrift/pull/772#issuecomment-169092979 @bgould, thanks for looking into this. Would it be tedious and/or complicate the code if we match against regex instead of removing them ? > 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)