I took care of this - it's now more informative and in debug mode

Carsten

2012/11/26 Felix Meschberger <[email protected]>:
> Hi,
>
> Am 22.11.2012 um 16:18 schrieb <[email protected]> 
> <[email protected]>:
>
>> Author: bdelacretaz
>> Date: Thu Nov 22 15:18:04 2012
>> New Revision: 1412583
>>
>> URL: http://svn.apache.org/viewvc?rev=1412583&view=rev
>> Log:
>> SLING-2674 - take new run mode options into account when starting on an 
>> older Sling instance
>>
>> Modified:
>> +        logger.info("isInstall={}, isUpgrade={}", isInstall, isUpgrade);
>
> I think that this message is confusing: I suggest to beef it up with more 
> prose to indicate, what exactly is being logged. Also, I am not sure, whether 
> using INFO level is appropriate...
>
> Regards
> Felix
>



-- 
Carsten Ziegeler
[email protected]

Reply via email to