[GitHub] [accumulo] dlmarion opened a new issue, #3240: LowMemoryDetector check incorrect

["dlmarion (via GitHub)" <gi...@apache.org>]

dlmarion opened a new issue, #3240:
URL: https://github.com/apache/accumulo/issues/3240

   [This](https://github.com/apache/accumulo/blob/f0555f207e70861072eebcf482efce1a1bab1fba/server/base/src/main/java/org/apache/accumulo/server/mem/LowMemoryDetector.java#L92) conditional needs to be modified to:
   
   ```
         if (isEnabled && runningLowOnMemory) {
           action.execute();
           return true;
         }
   ```
   
   The way that it's currently coded the action will always be executed, and that's not what we want.


-- 
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.apache.org

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

[GitHub] [accumulo] dlmarion closed issue #3240: LowMemoryDetector check incorrect

["dlmarion (via GitHub)" <gi...@apache.org>]

dlmarion closed issue #3240: LowMemoryDetector check incorrect
URL: https://github.com/apache/accumulo/issues/3240


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