[GitHub] [accumulo] hkeebler edited a comment on issue #454: Relocate accumulo-proxy to separate repository on its own release schedule

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

hkeebler edited a comment on issue #454: Relocate accumulo-proxy to separate repository on its own release schedule
URL: https://github.com/apache/accumulo/issues/454#issuecomment-499619776
 
 
   Running a verify gave me this `[ERROR] Exception is caught when Exception is not thrown in org.apache.accumulo.proxy.ProxyServer.updateRowsConditionally(String, Map) [org.apache.accumulo.proxy.ProxyServer] At ProxyServer.java:[line 2207] REC_CATCH_EXCEPTION`
   
   the code line is 
   ```java
   } catch (Exception e) {
         handleException(e);
         return null;
       }
   This apparently verifies okay in the accumulo project so why is it failing here - I'm not savvy with verify.  I changed the catch to the following and it verifies but not sure if you have better suggestions.  
   } catch (Throwable e) {
         handleException(new Exception(e));
         return null;
       }
   ```

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services