glitch commented on issue #289: ACCUMULO-4677 Sanitizing PathParam values in REST-based Monitor URL: https://github.com/apache/accumulo/pull/289#issuecomment-345890219 ~~Ah, I got this to break locally with the right mvn incantation (was running the sunny day tests instead of the mvn clean verify -DskipITs that travis is running). I'll see if I can get a workaround over Thanksgiving holiday; otherwise I can add an Ignore annotation or gut the PowerMock test.~~ I noticed that my branch is pretty far behind master at this point and it looks like the real issue is that the Table.ID inner class actually changed (ACCUMULO-4681). Previously it has a public constructor and now it appears to have a private one hence the test failure. Those changes looked like they were merged a couple of weeks after my original PR was posted. I'll take a look over Thanksgiving and see if I can get this working again. As it stands it should not be merged.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
