sync before trimUnreferencedReaders to improve index preformance

server/src/main/java/org/elasticsearch/index/translog/Translog.java

@@ -1670,6 +1670,8 @@ public void rollGeneration() throws IOException {
      * required generation
      */
     public void trimUnreferencedReaders() throws IOException {
+        // move most of the data to disk to reduce the time the lock is held
+        sync();
         try (ReleasableLock ignored = writeLock.acquire()) {
             if (closed.get()) {
                 // we're shutdown potentially on some tragic event, don't delete anything
@@ -1697,6 +1699,7 @@ public void trimUnreferencedReaders() throws IOException {
                 // We now update the checkpoint to ignore the file we are going to remove.
                 // Note that there is a provision in recoverFromFiles to allow for the case where we synced the checkpoint
                 // but crashed before we could delete the file.
+                // sync at once to make sure that there's at most one unreferenced generation.
                 current.sync();
                 deleteReaderFiles(reader);
             }