This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | persistence.xml version 2.1 without jta-data-source should default the datasource to java:comp/DefaultDataSource | ||
---|---|---|---|
Product: | javaee | Reporter: | Petr Jiricka <pjiricka> |
Component: | Persistence | Assignee: | Sergey Petrov <sj-nb> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | P2 | ||
Version: | 7.4 | ||
Hardware: | PC | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | 235027 | ||
Bug Blocks: |
Description
Petr Jiricka
2013-08-21 14:36:34 UTC
Shou;d it really be prepopulated? As it can be either specified or omitted with the same result, it ay be two separate options. Also selection of this default may produce 2 different persistence.xml. More general approach should involve server code to populate existed datasources list with this default. 3rd one is a real problem in my opinion. It may have no sense to specify jta-data-source with empty content and general fix may be to remove it if field is empty. > Shou;d it really be prepopulated? Maybe not, you are right. Leaving this empty is a possibility. > 3rd one is a real problem in my opinion. I agree. #2 should be fixed too. > Shou;d it really be prepopulated? Though the default datasource should be taken into account in some other situations - see e.g. bug 235026 for the Entity from Database wizard. it's a question if DefaultDataSource is available for applications with ee6 and lower level running on gf4 and why it's not listed in asadmin output. http://hg.netbeans.org/web-main/rev/62d45444f9b8 fix for 2,3. it's still a question if default datasource can be obtained common way as all others from server plugin from gf4 or it's required to hardcode it somewhere because of some very special handling in gf4 environment/ee7 environment. it may have sense to split remaining part of default datasource support into separate issue. 2,3 is fixed, 1 may be filed as separate issue/enhancment if consider some custom behaviour like replace empty ds with default etc. |