You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by fr...@apache.org on 2018/11/29 09:53:54 UTC
svn commit: r1847707 -
/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tooling/ConsistencyChecker.java
Author: frm
Date: Thu Nov 29 09:53:54 2018
New Revision: 1847707
URL: http://svn.apache.org/viewvc?rev=1847707&view=rev
Log:
OAK-7918 - Remove unused code
Modified:
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tooling/ConsistencyChecker.java
Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tooling/ConsistencyChecker.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tooling/ConsistencyChecker.java?rev=1847707&r1=1847706&r2=1847707&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tooling/ConsistencyChecker.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/tooling/ConsistencyChecker.java Thu Nov 29 09:53:54 2018
@@ -32,7 +32,6 @@ import java.util.LinkedHashSet;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;
-import java.util.Optional;
import java.util.Set;
import org.apache.jackrabbit.oak.api.Blob;
@@ -72,33 +71,6 @@ public abstract class ConsistencyChecker
}
- static class FoundRevision {
-
- private final String path;
-
- private final String revision;
-
- private final Long timestamp;
-
- private FoundRevision(String path, String revision, Long timestamp) {
- this.path = path;
- this.revision = revision;
- this.timestamp = timestamp;
- }
-
- public String getPath() {
- return path;
- }
-
- public Optional<String> getRevision() {
- return Optional.ofNullable(revision);
- }
-
- public Optional<Long> getTimestamp() {
- return Optional.ofNullable(timestamp);
- }
- }
-
protected void onCheckRevision(String revision) {
// Do nothing.
}
@@ -442,26 +414,6 @@ public abstract class ConsistencyChecker
return new Revision(entry.getRevision(), entry.getTimestamp());
}
- private List<FoundRevision> toFoundRevisionsList(List<PathToCheck> pathsToCheck) {
- List<FoundRevision> foundRevisions = new ArrayList<>(pathsToCheck.size());
- for (PathToCheck pathToCheck : pathsToCheck) {
- if (pathToCheck.journalEntry == null) {
- foundRevisions.add(new FoundRevision(pathToCheck.path, null, null));
- } else {
- foundRevisions.add(new FoundRevision(pathToCheck.path, pathToCheck.journalEntry.getRevision(), pathToCheck.journalEntry.getTimestamp()));
- }
- }
- return foundRevisions;
- }
-
- private Map<String, List<FoundRevision>> toFoundRevisionsMap(Map<String, List<PathToCheck>> pathsToCheck) {
- Map<String, List<FoundRevision>> foundRevisions = new HashMap<>();
- for (Entry<String, List<PathToCheck>> entry : pathsToCheck.entrySet()) {
- foundRevisions.put(entry.getKey(), toFoundRevisionsList(entry.getValue()));
- }
- return foundRevisions;
- }
-
/**
* Checks the consistency of a node and its properties at the given path.
*