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 md...@apache.org on 2017/03/27 15:34:45 UTC
svn commit: r1788957 -
/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java
Author: mduerig
Date: Mon Mar 27 15:34:44 2017
New Revision: 1788957
URL: http://svn.apache.org/viewvc?rev=1788957&view=rev
Log:
OAK-5991: FileStoreBuilder should print out a more helpful error message when buildReadOnly fails
Improved error message
Patch credits to Robert Munteanu
Modified:
jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java
Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java?rev=1788957&r1=1788956&r2=1788957&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/FileStoreBuilder.java Mon Mar 27 15:34:44 2017
@@ -350,7 +350,8 @@ public class FileStoreBuilder {
@Nonnull
public ReadOnlyFileStore buildReadOnly() throws InvalidFileStoreVersionException, IOException {
checkState(!built, "Cannot re-use builder");
- checkState(directory.exists() && directory.isDirectory());
+ checkState(directory.exists() && directory.isDirectory(),
+ "%s does not exist or is not a directory", directory);
built = true;
ReadOnlyRevisions revisions = new ReadOnlyRevisions(directory);
LOG.info("Creating file store {}", this);