Apologies, I didn't intend to remove the log4j.xml file in this commit.

I do however strongly object to its inclusion. The files existence
will cause Qpid end users a lot of problems.

Specifically the logging that the end user gets will depend on their
classpath... a very poor solution IMO.

I shall restore the file for now, but I would like to reach a
understanding as to why it was included before we do a release of the
Java client libraries.

Regards

Martin

2009/10/21  <[email protected]>:
> Author: ritchiem
> Date: Wed Oct 21 14:04:00 2009
> New Revision: 828019
>
> URL: http://svn.apache.org/viewvc?rev=828019&view=rev
> Log:
> Update PropertiesFileInitialContextFactory to give more details on error.
>
> Removed:
>    qpid/trunk/qpid/java/client/src/main/java/log4j.xml
> Modified:
>    
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/jndi/PropertiesFileInitialContextFactory.java
>
> Modified: 
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/jndi/PropertiesFileInitialContextFactory.java
> URL: 
> http://svn.apache.org/viewvc/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/jndi/PropertiesFileInitialContextFactory.java?rev=828019&r1=828018&r2=828019&view=diff
> ==============================================================================
> --- 
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/jndi/PropertiesFileInitialContextFactory.java
>  (original)
> +++ 
> qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/jndi/PropertiesFileInitialContextFactory.java
>  Wed Oct 21 14:04:00 2009
> @@ -108,7 +108,8 @@
>         }
>         catch (IOException ioe)
>         {
> -            _logger.warn("Unable to load property file specified in 
> Provider_URL:" + environment.get(Context.PROVIDER_URL));
> +            _logger.warn("Unable to load property file specified in 
> Provider_URL:" + environment.get(Context.PROVIDER_URL) +"\n" +
> +                         "Due to:"+ioe.getMessage());
>         }
>
>         createConnectionFactories(data, environment);
>
>
>
> ---------------------------------------------------------------------
> Apache Qpid - AMQP Messaging Implementation
> Project:      http://qpid.apache.org
> Use/Interact: mailto:[email protected]
>
>



-- 
Martin Ritchie

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:[email protected]

Reply via email to