[GitHub] [accumulo] EdColeman commented on pull request #2254: Replacing MockReateLimiter with mocking object to improve test design

[GitBox <gi...@apache.org>]

EdColeman commented on pull request #2254:
URL: https://github.com/apache/accumulo/pull/2254#issuecomment-913863466


   The build failed because of formatting issues - as said earlier, if you build locally it will automatically reformat the code.  Most of the QA checks and your tests can be run using:
   
   ```mvn clean verify javadoc:jar -Psec-bugs -DskipITs=true -Dtest=RateLimitedInputStreamTest,RateLimitedOutputStreamTest```
   
   Some of these checks will add to the build time so you might want to run the full set when you are ready to commit.  The original suggestion:
   
   ```mvn clean verify -DskipTests```
   
   Should correct your current issue.  I was able to get the full set to pass from your current branch.
   


-- 
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: notifications-unsubscribe@accumulo.apache.org

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