Uploaded image for project: 'Torque'
  1. Torque
  2. TORQUE-124

Thread safety problem in JndiDataSourceFactory

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.3
    • 4.0-beta1
    • Runtime
    • None

    Description

      javax.naming.InitialContext is not thread safe.
      http://java.sun.com/j2se/1.5.0/docs/api/javax/naming/InitialContext.html
      Threads that need to access a single InitialContext instance concurrently should synchronize
      amongst themselves and provide the necessary locking.

      In JndiDataSourceFactory#getDataSource, InitialContext #lookup is called without synchronization.
      This can be potential thread safe problems under the high load systems.

      public DataSource getDataSource() throws TorqueException
      {
      long time = System.currentTimeMillis();

      if (ds == null || time - lastLookup > ttl)
      {
      try

      { ds = ((DataSource) ctx.lookup(path)); lastLookup = time; }

      catch (Exception e)

      { throw new TorqueException(e); }

      }

      return ds;
      }

      Attachments

        Activity

          People

            tfischer Thomas Fox
            knambo Kazu Nambo
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: