[ 
http://issues.apache.org/jira/browse/IBATISNET-187?page=comments#action_12442777
 ] 
            
Gilles Bayon commented on IBATISNET-187:
----------------------------------------

Better

 ILoggerFactoryAdapter traceFA= TraceLoggerFA(); 
 ILog log = traceFA.GetLogger( typeof(LogManager) ); 
 log.Warn( "Unable to create instance of type " + 
setting.FactoryAdapterType.FullName + ". Using default logger.", ex ); 

> User is not notified of missing logging implementation file (e.g. 
> IBatisNet.Common.Logging.Log4Net)
> ---------------------------------------------------------------------------------------------------
>
>                 Key: IBATISNET-187
>                 URL: http://issues.apache.org/jira/browse/IBATISNET-187
>             Project: iBatis for .NET
>          Issue Type: Improvement
>            Reporter: Ron Grabowski
>         Assigned To: Gilles Bayon
>            Priority: Trivial
>
> The following code is executed when LogManager cannot load the specified 
> logging implementation (e.g. IBatisNet.Common.Logging.Log4net):
>  ILoggerFactoryAdapter defaultFactory = BuildDefaultLoggerFactoryAdapter();
>  ILog log = defaultFactory.GetLogger( typeof(LogManager) );
>  log.Warn( "Unable to create instance of type " + 
> setting.FactoryAdapterType.FullName + ". Using default logger.", ex );
> The default logger factory adapter is the NoOpLoggerFA which means the 
> message is swallowed and never shown to the user.
> We should write that message out so the user can see it. Maybe:
>  System.Diagnostics.Trace.WriteLine("Unable to create instance of type " + 
> setting.FactoryAdapterType.FullName + ". Using default logger.");

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to