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 also 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.  Previously it has a public constructor and now it appears to have a 
private one hence the test failure.

----------------------------------------------------------------
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

Reply via email to