Right yes. All good then!

On Sat, Jan 30, 2016 at 2:18 PM, Alex Parvulescu
<alex.parvule...@gmail.com> wrote:
> you missed a couple of lines up:
>
>> -            System.out.println("Debug " + args[0]);
>> +           System.out.println("Debug " + file);
>
>
> On Fri, Jan 29, 2016 at 5:41 PM, Julian Sedding <jsedd...@gmail.com> wrote:
>
>> > + System.out.println("Debug " + file);
>>
>> Is this on purpose or an oversight?
>>
>> Regards
>> Julian
>>
>> On Thu, Jan 28, 2016 at 10:56 AM,  <alexparvule...@apache.org> wrote:
>> > Author: alexparvulescu
>> > Date: Thu Jan 28 09:56:31 2016
>> > New Revision: 1727297
>> >
>> > URL: http://svn.apache.org/viewvc?rev=1727297&view=rev
>> > Log:
>> > OAK-3928 oak-run debug should use a read-only store
>> >
>> > Modified:
>> >
>>  
>> jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/run/Main.java
>> >
>> > Modified:
>> jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/run/Main.java
>> > URL:
>> http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/run/Main.java?rev=1727297&r1=1727296&r2=1727297&view=diff
>> >
>> ==============================================================================
>> > ---
>> jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/run/Main.java
>> (original)
>> > +++
>> jackrabbit/oak/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/run/Main.java
>> Thu Jan 28 09:56:31 2016
>> > @@ -839,12 +839,9 @@ public final class Main {
>> >              System.exit(1);
>> >          } else {
>> >              // TODO: enable debug information for other node store
>> implementations
>> > -            System.out.println("Debug " + args[0]);
>> >              File file = new File(args[0]);
>> > -            FileStore store = newFileStore(file)
>> > -                .withMaxFileSize(256)
>> > -                .withMemoryMapping(false)
>> > -                .create();
>> > +            System.out.println("Debug " + file);
>> > +            ReadOnlyStore store = new ReadOnlyStore(file);
>> >              try {
>> >                  if (args.length == 1) {
>> >                      debugFileStore(store);
>> >
>> >
>>

Reply via email to