[GitHub] brooklyn-server pull request #833: Fix PeriodicEffectorPolicy

[grkvlt]

Github user grkvlt commented on a diff in the pull request:

    https://github.com/apache/brooklyn-server/pull/833#discussion_r139935476
  
    --- Diff: 
policy/src/main/java/org/apache/brooklyn/policy/action/AbstractScheduledEffectorPolicy.java
 ---
    @@ -233,8 +269,8 @@ public synchronized void run() {
                 Object result = entity.invoke(effector, 
resolved).getUnchecked();
                 LOG.debug("{}: Effector {} returned {}", new Object[] { this, 
effector.getName(), result });
             } catch (RuntimeInterruptedException rie) {
    -            Thread.interrupted();
    -            // TODO sometimes this seems to hang the executor?
    +            // Gracefully stop
    +            Thread.currentThread().interrupt();
    --- End diff --
    
    Thanks, I couldn't remember the right way to handle this.


---