Re: svn commit: r1508445 - in /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr: RepositoryImpl.java SessionContext.java SessionImpl.java

2013-07-30 Thread Jukka Zitting
Hi, On Tue, Jul 30, 2013 at 5:34 PM, Michael Dürig wrote: > On 30.7.13 16:24, Jukka Zitting wrote: >> The user already knows the settings as they're the one providing the >> information (or using the default). > > He knows what he passed in. That might be different from what is being > applied. S

Re: svn commit: r1508445 - in /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr: RepositoryImpl.java SessionContext.java SessionImpl.java

2013-07-30 Thread Michael Dürig
On 30.7.13 16:24, Jukka Zitting wrote: Hi, On Tue, Jul 30, 2013 at 5:15 PM, Michael Dürig wrote: On 30.7.13 16:09, Jukka Zitting wrote: On Tue, Jul 30, 2013 at 4:56 PM, wrote: +log.info(); These should become log.debug(...) as soon as the code stabilizes. I don't

Re: svn commit: r1508445 - in /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr: RepositoryImpl.java SessionContext.java SessionImpl.java

2013-07-30 Thread Jukka Zitting
Hi, On Tue, Jul 30, 2013 at 5:15 PM, Michael Dürig wrote: > On 30.7.13 16:09, Jukka Zitting wrote: >> On Tue, Jul 30, 2013 at 4:56 PM, wrote: >>> +log.info(); >> >> These should become log.debug(...) as soon as the code stabilizes. > > I don't agree. As long as this is user

Re: svn commit: r1508445 - in /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr: RepositoryImpl.java SessionContext.java SessionImpl.java

2013-07-30 Thread Michael Dürig
On 30.7.13 16:09, Jukka Zitting wrote: Hi, On Tue, Jul 30, 2013 at 4:56 PM, wrote: +log.info(); These should become log.debug(...) as soon as the code stabilizes. I don't agree. As long as this is user configurable we should make that information available in the lo

Re: svn commit: r1508445 - in /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr: RepositoryImpl.java SessionContext.java SessionImpl.java

2013-07-30 Thread Jukka Zitting
Hi, On Tue, Jul 30, 2013 at 4:56 PM, wrote: > +log.info(); These should become log.debug(...) as soon as the code stabilizes. BR, Jukka Zitting