Branch: refs/heads/master
Home:
https://github.com/Teiid-Designer/teiid-designer
Commit: ef7c74772b7c5abedbfc861a1ca9906e2387df32
https://github.com/Teiid-Designer/teiid-designer/commit/ef7c74772b7c5abed...
Author: Paul Richardson <p.g.richardson(a)redhat.com>
Date: 2013-07-29 (Mon, 29 Jul 2013)
Changed paths:
M
plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/adapter/JBoss7ServerUtil.java
M
plugins/org.teiid.designer.dqp/src/org/teiid/designer/runtime/adapter/JBossServerUtil.java
Log Message:
-----------
Improves performance of testing the jboss server connection
* Testing the connection of both teiid and its parent jboss server is a
very frequent occurrence and as such needs to do as little as possible.
The current implementation generates at least 2 thread runnables to
return the jboss server name, which is never used. It is also slow,
taking on average 1.5 - 3 seconds, with the possibility of using much
needed memory / threads and other resources. Given the frequency of the
operation, this is incredibly wasteful.
* Scales back the connection test to determine the availability of the
jboss management port on the jboss host. Each connection has a timeout of
a second so a failure is returned quickly. Should this become as issue
in the future then this timeout could be included as a preference. The
socket is always closed and nulled to ensure it is garbage collected.