hkeebler opened a new pull request #4: Fix #3 Updated the accumulo-proxy with latest from accumulo (pre-2.0) URL: https://github.com/apache/accumulo-proxy/pull/4 This builds and verifies with the updates from the accumulo project. The IT tests were moved over and execute. Not sure if more testing needs to be done. The couple of quirky things are: Added the Encoding exculsion to ProxyDurabilityIT test. This was not necessary in the accumulo project and I'm not sure how it was excluded there. ``` @SuppressFBWarnings(value = {"HARD_CODE_PASSWORD", "DM_DEFAULT_ENCODING"}, justification = "test password is okay and no check needed on encoding") ``` Added the Throwable in ProxyServer because it did not pass some code check. ``` catch (Throwable e) { handleException(new Exception(e)); ```
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
