InternalEngine - only set dirty=false after a refresh happened

2014-03-31 17:35:48 +02:00 · 2014-03-31 17:35:48 +02:00 · 5553c383bc
parent 1aac41e0d1
commit 5553c383bc
1 changed files with 1 additions and 1 deletions
--- a/src/main/java/org/elasticsearch/index/engine/internal/InternalEngine.java
+++ b/src/main/java/org/elasticsearch/index/engine/internal/InternalEngine.java
@ -706,9 +706,9 @@ public class InternalEngine extends AbstractIndexShardComponent implements Engin
                // but, we want to make sure not to loose ant refresh calls, if one is taking time
                synchronized (refreshMutex) {
                    if (dirty || refresh.force()) {
-                        dirty = false;
                        boolean refreshed = searcherManager.maybeRefresh();
                        assert refreshed : "failed to refresh even though refreshMutex was acquired";
+                        dirty = false;
                    }
                }
            } catch (AlreadyClosedException e) {