[jboss-jira] [JBoss JIRA] (REMJMX-86) RemotingConnector & VersionedConectionFactory need configurable connection, channel & versioned connection timeouts

[Brad Maxwell (JIRA)]

     [ https://issues.jboss.org/browse/REMJMX-86?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brad Maxwell updated REMJMX-86:
-------------------------------

    Summary: RemotingConnector & VersionedConectionFactory need configurable connection, channel & versioned connection timeouts   (was: RemotingConnector needs system property for default value when opening a channel)


> RemotingConnector & VersionedConectionFactory need configurable connection, channel & versioned connection timeouts 
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: REMJMX-86
>                 URL: https://issues.jboss.org/browse/REMJMX-86
>             Project: Remoting JMX
>          Issue Type: Bug
>          Components: Connection
>    Affects Versions: 2.0.0.Final
>            Reporter: Mustafa Musaji
>            Assignee: Brad Maxwell
>
> In [1] we have the following snippet of code which has a hardcoded value of 5 seconds.
> {code:title=RemotingConnector.java}
> // Now open the channel
>         final IoFuture<Channel> futureChannel = connection.openChannel(serviceName, OptionMap.EMPTY);
>         IoFuture.Status result = futureChannel.await(5, TimeUnit.SECONDS);
>         if (result == IoFuture.Status.DONE) {
>             channel = futureChannel.get();
>         } else if (result == IoFuture.Status.FAILED) {
>             throw new IOException(futureChannel.getException());
>         } else {
>             throw new RuntimeException("Operation failed with status " + result);
>         }
> {code}
> This should be a system property or the default value should be raised.
> [1] https://github.com/jbossas/remoting-jmx/blob/master/src/main/java/org/jboss/remotingjmx/RemotingConnector.java



--
This message was sent by Atlassian JIRA
(v6.2.6#6264)