You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openjpa.apache.org by Albert Lee <al...@gmail.com> on 2009/12/14 20:21:09 UTC
Re: [jira] Commented: (OPENJPA-1258) make warning message more
friendly when other persistence provider configured by application
I saw quite a few of revisions committed into 1.1.x branch but was not in
other forward releases, i.e. 1.3.0 and trunk. It is a general practice to
commit the same changes to the current release to avoid breakage and
regression when users move up to the current release. Please consider
migrating these relevant commits to (at least) trunk.
Thanks,
Albert Lee.
On Mon, Dec 14, 2009 at 12:01 PM, David Ezzio (JIRA) <ji...@apache.org>wrote:
>
> [
> https://issues.apache.org/jira/browse/OPENJPA-1258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790264#action_12790264]
>
> David Ezzio commented on OPENJPA-1258:
> --------------------------------------
>
> Applied Amy's patch to 1.1.x branch at 890415.
>
> > make warning message more friendly when other persistence provider
> configured by application
> >
> --------------------------------------------------------------------------------------------
> >
> > Key: OPENJPA-1258
> > URL: https://issues.apache.org/jira/browse/OPENJPA-1258
> > Project: OpenJPA
> > Issue Type: Improvement
> > Components: logging
> > Affects Versions: 1.1.0
> > Environment: Windows XP
> > java version "1.5.0_14"
> > Reporter: Amy Yang
> > Priority: Minor
> > Attachments: OPENJPA1258.patch, OPENJPA1258_trunk.patch
> >
> >
> > When application configures other persistence provider rather than
> openjpa, there will be a warning message printed out:
> > WARNING: Found unrecognized persistence provider
> "org.eclipse.persistence.jpa.PersistenceProvider" in place of OpenJPA
> provider. This provider's properties will not be used.
> > It is legal that other persistence provider be configured. The message
> doesn't look friendly and might be confusing.
> > I change the message like:
> > INFO: Found persistence provider
> "org.eclipse.persistence.jpa.PersistenceProvider". OpenJPA will not be used.
>
> --
> This message is automatically generated by JIRA.
> -
> You can reply to this email to add a comment to the issue online.
>
>
--
Albert Lee.
Re: [jira] Commented: (OPENJPA-1258) make warning message more
friendly when other persistence provider configured by application
Posted by Albert Lee <al...@gmail.com>.
David,
Thanks for the follow up and clarifications.
Albert Lee.
On Tue, Dec 15, 2009 at 11:45 AM, David Ezzio <de...@apache.org> wrote:
> Hi Albert,
>
> Yes, I have been making commits for Amy Yang and Ravi Palacherla in the
> past few days. Patches for OpenJPA-386, 681, 817, 1200, and 1260 were made
> to the 1.1.x branch. It is my understanding from the JIRA issues that the
> same patches were made to trunk some time ago. In several cases, these
> issues were marked as closed or resolved.
>
> For OpenJPA-1258, I have not yet applied the patch to trunk. Amy and I are
> discussing the suitability of the patch for trunk, and I will either apply
> the patch to trunk or one of us will update the JIRA issue with our
> thinking.
>
> Hope this clarifies the situation.
>
> Thanks,
>
> David
>
>
> Albert Lee wrote:
>
>> I saw quite a few of revisions committed into 1.1.x branch but was not in
>> other forward releases, i.e. 1.3.0 and trunk. It is a general practice to
>> commit the same changes to the current release to avoid breakage and
>> regression when users move up to the current release. Please consider
>> migrating these relevant commits to (at least) trunk.
>>
>> Thanks,
>> Albert Lee.
>>
>> On Mon, Dec 14, 2009 at 12:01 PM, David Ezzio (JIRA) <jira@apache.org
>> >wrote:
>>
>> [
>>>
>>> https://issues.apache.org/jira/browse/OPENJPA-1258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790264#action_12790264
>>> ]
>>>
>>> David Ezzio commented on OPENJPA-1258:
>>> --------------------------------------
>>>
>>> Applied Amy's patch to 1.1.x branch at 890415.
>>>
>>> make warning message more friendly when other persistence provider
>>>>
>>> configured by application
>>>
>>> --------------------------------------------------------------------------------------------
>>>
>>>> Key: OPENJPA-1258
>>>> URL: https://issues.apache.org/jira/browse/OPENJPA-1258
>>>> Project: OpenJPA
>>>> Issue Type: Improvement
>>>> Components: logging
>>>> Affects Versions: 1.1.0
>>>> Environment: Windows XP
>>>> java version "1.5.0_14"
>>>> Reporter: Amy Yang
>>>> Priority: Minor
>>>> Attachments: OPENJPA1258.patch, OPENJPA1258_trunk.patch
>>>>
>>>>
>>>> When application configures other persistence provider rather than
>>>>
>>> openjpa, there will be a warning message printed out:
>>>
>>>> WARNING: Found unrecognized persistence provider
>>>>
>>> "org.eclipse.persistence.jpa.PersistenceProvider" in place of OpenJPA
>>> provider. This provider's properties will not be used.
>>>
>>>> It is legal that other persistence provider be configured. The message
>>>>
>>> doesn't look friendly and might be confusing.
>>>
>>>> I change the message like:
>>>> INFO: Found persistence provider
>>>>
>>> "org.eclipse.persistence.jpa.PersistenceProvider". OpenJPA will not be
>>> used.
>>>
>>> --
>>> This message is automatically generated by JIRA.
>>> -
>>> You can reply to this email to add a comment to the issue online.
>>>
>>>
>>>
>>
>>
--
Albert Lee.
Re: [jira] Commented: (OPENJPA-1258) make warning message more friendly
when other persistence provider configured by application
Posted by David Ezzio <de...@apache.org>.
Hi Albert,
Yes, I have been making commits for Amy Yang and Ravi Palacherla in the
past few days. Patches for OpenJPA-386, 681, 817, 1200, and 1260 were
made to the 1.1.x branch. It is my understanding from the JIRA issues
that the same patches were made to trunk some time ago. In several
cases, these issues were marked as closed or resolved.
For OpenJPA-1258, I have not yet applied the patch to trunk. Amy and I
are discussing the suitability of the patch for trunk, and I will either
apply the patch to trunk or one of us will update the JIRA issue with
our thinking.
Hope this clarifies the situation.
Thanks,
David
Albert Lee wrote:
> I saw quite a few of revisions committed into 1.1.x branch but was not in
> other forward releases, i.e. 1.3.0 and trunk. It is a general practice to
> commit the same changes to the current release to avoid breakage and
> regression when users move up to the current release. Please consider
> migrating these relevant commits to (at least) trunk.
>
> Thanks,
> Albert Lee.
>
> On Mon, Dec 14, 2009 at 12:01 PM, David Ezzio (JIRA) <ji...@apache.org>wrote:
>
>> [
>> https://issues.apache.org/jira/browse/OPENJPA-1258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12790264#action_12790264]
>>
>> David Ezzio commented on OPENJPA-1258:
>> --------------------------------------
>>
>> Applied Amy's patch to 1.1.x branch at 890415.
>>
>>> make warning message more friendly when other persistence provider
>> configured by application
>> --------------------------------------------------------------------------------------------
>>> Key: OPENJPA-1258
>>> URL: https://issues.apache.org/jira/browse/OPENJPA-1258
>>> Project: OpenJPA
>>> Issue Type: Improvement
>>> Components: logging
>>> Affects Versions: 1.1.0
>>> Environment: Windows XP
>>> java version "1.5.0_14"
>>> Reporter: Amy Yang
>>> Priority: Minor
>>> Attachments: OPENJPA1258.patch, OPENJPA1258_trunk.patch
>>>
>>>
>>> When application configures other persistence provider rather than
>> openjpa, there will be a warning message printed out:
>>> WARNING: Found unrecognized persistence provider
>> "org.eclipse.persistence.jpa.PersistenceProvider" in place of OpenJPA
>> provider. This provider's properties will not be used.
>>> It is legal that other persistence provider be configured. The message
>> doesn't look friendly and might be confusing.
>>> I change the message like:
>>> INFO: Found persistence provider
>> "org.eclipse.persistence.jpa.PersistenceProvider". OpenJPA will not be used.
>>
>> --
>> This message is automatically generated by JIRA.
>> -
>> You can reply to this email to add a comment to the issue online.
>>
>>
>
>