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: | Add method to StartServer SPI for handling the need to restart server | ||
|---|---|---|---|
| Product: | serverplugins | Reporter: | Nam Nguyen <nnguyen> |
| Component: | Infrastructure | Assignee: | Nam Nguyen <nnguyen> |
| Status: | RESOLVED FIXED | ||
| Severity: | blocker | CC: | apireviews, ludo, marcow, pbuzek |
| Priority: | P1 | Keywords: | API_REVIEW_FAST |
| Version: | 4.x | ||
| Hardware: | PC | ||
| OS: | Linux | ||
| Issue Type: | ENHANCEMENT | Exception Reporter: | |
| Bug Depends on: | |||
| Bug Blocks: | 52346 | ||
| Attachments: |
diff of apichanges.xml, StartServer.java, ServerInstance.java included...
update with improved implementation code, no changes in signature over the previous attachement |
||
Will attach the diff tomorrow. Created attachment 19806 [details]
diff of apichanges.xml, StartServer.java, ServerInstance.java included...
Created attachment 19836 [details]
update with improved implementation code, no changes in signature over the previous attachement
Patch checked in: /cvs/j2eeserver/apichanges.xml,v <-- apichanges.xml new revision: 1.9; previous revision: 1.8 /cvs/j2eeserver/src/org/netbeans/modules/j2ee/deployment/impl/ServerInstance.java,v <-- ServerInstance.java /cvs/j2eeserver/src/org/netbeans/modules/j2ee/deployment/plugins/api/StartServer.java,v <-- StartServer.java new revision: 1.6; previous revision: 1.5 Close now. Planned changes already checked in. |
When server instance is configured and the changes require a restart to take effect in next deployment j2eeserver state manager needs to be aware of this condition. Hence this method addition: StartServer { /** * Returns true if target server needs a * restart for last configuration changes to * take effect. * @param target target server; null implies * the case where target is also admin server. */ boolean needsRestart(Target target) { return false; } }