Code Review Request: 7186555: (prefs) continual printing of BackingStoreException on console on Linux
Mandy Chung
mandy.chung at oracle.com
Tue May 21 08:35:44 UTC 2013
Kurchi,
The windows implementation also emits warning messages that should be
fixed up too since we don't want to emit anything to interfere the
application logging. The default logging configuration (default level
is INFO) should apply to the application logging and the runtime like
this case should have a different default level (e.g SEVERE) so that our
implementation can use the appropriate level rather than forcing to use
FINE etc. This is something for future enhancement (I'll file a RFE).
For this bug, have you considered setting the level to SEVERE and change
its default instead of changing to FINE?
Mandy
On 5/21/2013 9:21 AM, Mandy Chung wrote:
> On 5/21/2013 1:42 AM, Kurchi Hazra wrote:
>>
>> Hi,
>>
>> Please review this change to fix 718655. The bug complaints about
>> continuous BackingStoreExceptions
>> printed on the console, the user is just seeing logged warnings from
>> BackingStoreExceptions raised in periodic attempts to sync
>> preferences. Preferences use sun.util.logging.PlatformLogger.java
>> which is enabled by default, and has a default level of INFO.
>> Changing the logging in this file to FINE prevents the user from
>> seeing information that he/she is not interested in.
>> Bug: http://bugs.sun.com/view_bug.do?bug_id=7186555
>> Webrev:http://cr.openjdk.java.net/~khazra/7186555/webrev.00/
>>
>
> Looks good and L116 emits a INFO log message that should be changed to
> FINE too.
>
> Mandy
More information about the core-libs-dev
mailing list