Done.
Andrus Adamchik wrote:
> Jeff, IIRC this list used to strip attachments. So cc'ing it to your
> address as well.
>
> Andrus
>
>
> ------------------------------------------------------------------------
>
> Index: geronimo-spec-jpa/src/main/java/javax/persistence/Persistence.java
> ===================================================================
> --- geronimo-spec-jpa/src/main/java/javax/persistence/Persistence.java
> (revision 398324)
> +++ geronimo-spec-jpa/src/main/java/javax/persistence/Persistence.java
> (working copy)
> @@ -144,18 +144,29 @@
> static EntityManagerFactory createFactory(
> String providerName,
> String persistenceUnitName,
> - Map properties) {
> + Map properties)
> + throws PersistenceException {
>
> - try {
> - Class providerClass = Class.forName(providerName, true, Thread
> - .currentThread()
> - .getContextClassLoader());
> - PersistenceProvider provider = (PersistenceProvider)
> providerClass
> - .newInstance();
> - return provider.createEntityManagerFactory(persistenceUnitName,
> properties);
> - }
> - catch (Exception e) {
> - return null;
> - }
> - }
> + Class providerClass;
> + try {
> + providerClass = Class.forName(providerName, true, Thread
> +
> .currentThread().getContextClassLoader());
> + }
> + catch (Exception e) {
> + throw new PersistenceException(
> + "Invalid or inaccessible provider
> class: " + providerName,
> + e);
> + }
> +
> + try {
> + PersistenceProvider provider = (PersistenceProvider)
> providerClass
> + .newInstance();
> + return
> provider.createEntityManagerFactory(persistenceUnitName,
> + properties);
> + }
> + catch (Exception e) {
> + throw new PersistenceException("Provider error.
> Provider: "
> + + providerName, e);
> + }
> + }
> }
>
>
> ------------------------------------------------------------------------
>
>
>
>
> On Apr 30, 2006, at 12:03 AM, Jeff Genender wrote:
>
>> Can you please attach the patch. If I copy/paste from this email it
>> chokes.
>>
>> Thanks,
>>
>> Jeff
>>
