Comment and blank line cleanups
@@ -263,7 +263,6 @@ public class Store extends AbstractIndexShardComponent implements Closeable, Ref
}
}
-
/**
* Renames all the given files from the key of the map to the
* value of the map. All successfully renamed files are removed from the map in-place.
@@ -389,7 +388,6 @@ public class Store extends AbstractIndexShardComponent implements Closeable, Ref
}
}
-
/**
* Reads a MetadataSnapshot from the given index locations or returns an empty snapshot if it can't be read.
*
@@ -597,7 +595,7 @@ public class Store extends AbstractIndexShardComponent implements Closeable, Ref
/**
* This method deletes every file in this store that is not contained in the given source meta data or is a
* legacy checksum file. After the delete it pulls the latest metadata snapshot from the store and compares it
- * to the given snapshot. If the snapshots are inconsistent an illegal state exception is thrown
+ * to the given snapshot. If the snapshots are inconsistent an illegal state exception is thrown.
*
* @param reason the reason for this cleanup operation logged for each deleted file
* @param sourceMetaData the metadata used for cleanup. all files in this metadata should be kept around.
@@ -641,9 +639,9 @@ public class Store extends AbstractIndexShardComponent implements Closeable, Ref
for (StoreFileMetaData meta : recoveryDiff.different) {
StoreFileMetaData local = targetMetaData.get(meta.name());
StoreFileMetaData remote = sourceMetaData.get(meta.name());
- // if we have different files the they must have no checksums otherwise something went wrong during recovery.
- // we have that problem when we have an empty index is only a segments_1 file then we can't tell if it's a Lucene 4.8 file
- // and therefore no checksum. That isn't much of a problem since we simply copy it over anyway but those files come out as
+ // if we have different files then they must have no checksums; otherwise something went wrong during recovery.
+ // we have that problem when we have an empty index is only a segments_1 file so we can't tell if it's a Lucene 4.8 file
+ // and therefore no checksum is included. That isn't a problem since we simply copy it over anyway but those files come out as
// different in the diff. That's why we have to double check here again if the rest of it matches.
// all is fine this file is just part of a commit or a segment that is different
@@ -676,7 +674,6 @@ public class Store extends AbstractIndexShardComponent implements Closeable, Ref
this.deletesLogger = deletesLogger;
}
-
@Override
public void close() throws IOException {
assert false : "Nobody should close this directory except of the Store itself";
