You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@avro.apache.org by GitBox <gi...@apache.org> on 2023/01/17 09:34:17 UTC

[GitHub] [avro] clesaec commented on pull request #2016: AVRO-3691:[maven build]Add nondex:2.1.1 to maven build section

clesaec commented on PR #2016:
URL: https://github.com/apache/avro/pull/2016#issuecomment-1385097100

   The issue comes from that [the tested java class "Company" use HashMap](https://github.com/apache/avro/blob/master/lang/java/avro/src/test/java/org/apache/avro/reflect/TestNonStringMapKeys.java#L364-L373), which does not guaranted to keep insert ordering. So replacing HashMap by LinkedHashMap (and same for caller) should solve it (and this also shows utility of nondex which found a bug in unit test code :))


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@avro.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org