[ https://issues.apache.org/jira/browse/MESOS-4582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15135137#comment-15135137 ]
Marco Massenzio edited comment on MESOS-4582 at 2/5/16 10:17 PM: ----------------------------------------------------------------- I'm almost sure that duplicate keys are not legal JSON - worth checking the standard, but I'd be in favor of keeping the checks and throwing back a 406 (Bad Request). Incidentally, as almost *all* JSON libraries in most languages (I know of Java, Python, C++, Scala) model JSON documents with the {{map}} structure, it is virtually impossible (or, at best, extremely difficult) to generate a JSON document with duplicate keys (even assuming that such a thing is syntactically correct). If you want, I can look it up later this weekend and find out what the JSON standard says? Thanks for fixing it! was (Author: marco-mesos): I'm almost sure that duplicate keys are not legal JSON - worth checking the standard, but I'd be in favor of keepin the checks and throwing back a 406 (Bad Request). Incidentally, as almost *all* JSON libraries in most languages (I know of Java, Python, C++, Scala) model JSON documents with the {{map}} structure, it is virtually impossible (or, at best, extremely difficult) to generate a JSON document with duplicate keys (even assuming that such a thing is syntactically correct). If you want, I can look it up later this weekend and find out what the JSON standard says? Thanks for fixing it! > state.json serving duplicate "active" fields > -------------------------------------------- > > Key: MESOS-4582 > URL: https://issues.apache.org/jira/browse/MESOS-4582 > Project: Mesos > Issue Type: Bug > Affects Versions: 0.27.0 > Reporter: Michael Gummelt > Assignee: Michael Park > Priority: Blocker > Attachments: error.json > > > state.json is serving duplicate "active" fields in frameworks. See the > framework "47df96c2-3f85-4bc5-b781-709b2c30c752-0000" In the attached file -- This message was sent by Atlassian JIRA (v6.3.4#6332)