Bug 231527 - Multiple Persistence Unit Warning Message
Summary: Multiple Persistence Unit Warning Message
Status: VERIFIED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: General (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows XP
: P3 normal with 1 vote (vote)
Target Milestone: 2.1 M3   Edit
Assignee: Neil Hauge CLA
QA Contact:
URL:
Whiteboard:
Keywords: PII
Depends on:
Blocks:
 
Reported: 2008-05-12 08:16 EDT by Shaun Smith CLA
Modified: 2008-11-20 10:02 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Shaun Smith CLA 2008-05-12 08:16:25 EDT
The "no support for multiple persistence units" warning message is currently "Multiple persistence units defined - tooling only supports 1 persistence unit per project".  "tooling only supports" has to go :-)

I propose something closer to "Multiple persistence units defined - only first persistence unit will be validated".  This better explains the limitation and provides the developer with notice that they'll have to manually check persistence units after the first one.  Even the notion of "first" isn't great but I'm not sure how else to say it succinctly.
Comment 1 Neil Hauge CLA 2008-06-10 15:44:17 EDT
Need to figure out an appropriate message for the next release, since we weren't able to address this issue in the 2.0 RC's.

We need to convey to the user that only one PU is going to be recognized for Dali functionality.  Any Entities defined in additional PU's will now be treated like they are in the 1 recognized PU for the purposes of defaults, validation, etc.  These unrecognized Entities will be marked with an Error, informing the user that they are not listed in the recognized PU.

So, perhaps the message could simply say, "Multiple persistence units defined - only the first persistence unit will be recognized".

The first PU in the persistence.xml is what we will recognize.  Perhaps there is a better word than "recognized", but that is my suggestion for the moment.

Comment 2 Neil Hauge CLA 2008-11-04 10:22:17 EST
Changed to "Multiple persistence units defined - only the first persistence unit will be recognized".

Comment 3 Karen Butzke CLA 2008-11-20 10:02:07 EST
verified in current code