diff options
author | Caspar De Groot | 2010-10-14 02:42:55 +0000 |
---|---|---|
committer | Caspar De Groot | 2010-10-14 02:42:55 +0000 |
commit | db01d92da832fa7f5b1cddc6b0e0831fcc466d84 (patch) | |
tree | 19fbea0599eb3d41f0b7ad5920d8a7a2f7dffbd1 | |
parent | 5048ec0ffc5b19002ab771ffd5fbb9b5a7cb5ce9 (diff) | |
download | cdo-db01d92da832fa7f5b1cddc6b0e0831fcc466d84.tar.gz cdo-db01d92da832fa7f5b1cddc6b0e0831fcc466d84.tar.xz cdo-db01d92da832fa7f5b1cddc6b0e0831fcc466d84.zip |
[327428] Failed-over session broken
https://bugs.eclipse.org/bugs/show_bug.cgi?id=327428
2 files changed, 8 insertions, 3 deletions
diff --git a/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionConfigurationImpl.java b/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionConfigurationImpl.java index ee92d40d4c..b604ceaeb9 100644 --- a/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionConfigurationImpl.java +++ b/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionConfigurationImpl.java @@ -82,6 +82,5 @@ public class FailoverCDOSessionConfigurationImpl extends CDONet4jSessionConfigur sessionImpl.setMonitorConnectionDescription(monitorConnectorDescription); sessionImpl.setRepositoryGroup(repositoryGroup); sessionImpl.setContainer(container); - sessionImpl.updateConnectorAndRepositoryName(); // TODO (CD) Can't we leave it to the session to call this? } } diff --git a/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionImpl.java b/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionImpl.java index 28d56429d1..a364980590 100644 --- a/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionImpl.java +++ b/plugins/org.eclipse.emf.cdo.net4j/src/org/eclipse/emf/cdo/internal/net4j/FailoverCDOSessionImpl.java @@ -145,8 +145,14 @@ public class FailoverCDOSessionImpl extends CDONet4jSessionImpl } } - // TODO (CD) Default access allows config object to call this once. Does this make sense? - void updateConnectorAndRepositoryName() + @Override + protected void activateSession() throws Exception + { + updateConnectorAndRepositoryName(); + super.activateSession(); + } + + private void updateConnectorAndRepositoryName() { // System.out.println("Querying fail-over monitor..."); queryRepositoryInfoFromMonitor(); |