[teiid-designer-dev] [Teiid-Designer/teiid-designer] ef7c74: Improves performance of testing the jboss server c...

[Paul Richardson]

  Branch: refs/heads/master
  Home:   https://github.com/Teiid-Designer/teiid-designer
  Commit: ef7c74772b7c5abedbfc861a1ca9906e2387df32
      https://github.com/Teiid-Designer/teiid-designer/commit/ef7c74772b7c5abedbfc861a1ca9906e2387df32
  Author: Paul Richardson <p.g.richardson at 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.