Commits on Mar 14, 2012
  1. @bbrowning

    Verify injection works from Padrino controllers (TORQUE-734)

    The key is to make sure to include TorqueBox::Injectors in the Padrino
    controller. It's a by-product of how our injection analysis works that
    each file needs the include statement even if a class inherits or
    extends another class that has already included TorqueBox::Injectors.
    bbrowning committed Mar 14, 2012
  2. @dobozysaurus
  3. @dobozysaurus
Commits on Mar 13, 2012
  1. @bbrowning

    Don't fail a deployment if we can't parse database.yml (TORQUE-728)

    We just log a warning that XA won't be enabled and let the deployment
    continue. The only known case where this should happen is when a
    database.yml contains ERB code.
    bbrowning committed Mar 13, 2012
  2. @bbrowning
  3. @lance
  4. @bbrowning

    Only enable XA for current environment's database (TORQUE-732)

    If an application uses multiple databases, 'xa: true' will need to be
    set in the database.yml for the database entries that don't match the
    current environment (typically one of development/test/production).
    bbrowning committed Mar 12, 2012
Commits on Mar 12, 2012
  1. @lance

    TORQUE-724 Fix task concurrency issues with newrelic being included m…

    …ultiple times. Thanks msavy
    lance committed Mar 12, 2012
  2. @lance
Commits on Mar 9, 2012
  1. @lance

    [TORQUE-731] Allows services and jobs to receive configuration

    options when specified in a config block.
    lance committed Mar 9, 2012
  2. @dobozysaurus

    fixes for TORQUE-729

    dobozysaurus committed Mar 9, 2012
Commits on Mar 7, 2012
  1. @bobmcwhirter
Commits on Mar 2, 2012
  1. @projectodd-ci
  2. @projectodd-ci
  3. @projectodd-ci
  4. @projectodd-ci
  5. @bobmcwhirter
Commits on Mar 1, 2012
  1. @bbrowning
  2. @bbrowning
  3. @bbrowning

    Add "app/", "models/", and "." to the default injection paths (TORQUE…

    …-721)
    
    This lets any runtime use objects under these paths with injections.
    
    This also required adding some additional logic to ensure services
    can't inject each other and end up in a circular dependency.
    bbrowning committed Feb 29, 2012
  4. @bbrowning

    Look for injections inside FCallNodes that aren't inject calls (TORQU…

    …E-721)
    
    This fixes an issue with our analyzer where we weren't picking up on
    injections inside a Padrino or Sinatra app like below:
    
    get '/' do
      inject('/queue/foo')
    end
    bbrowning committed Feb 29, 2012
  5. @bbrowning
Commits on Feb 29, 2012
  1. @jcrossley3
Commits on Feb 28, 2012
  1. @bbrowning

    Check PostgreSQL's max_prepared_transactions before attempting XA (TO…

    …RQUE-717)
    
    The XAResource warnings we were generated were caused by attempting an
    XA transaction against PostgreSQL during our probe to determine if the
    target database supports XA. When it didn't (max_prepared_transactions
    equal to 0) the recovery manager would periodically try to fix these
    transactions it believe were partially committed.
    
    The more times you booted TorqueBox, the more transactions we'd
    collect inside $JBOSS_HOME/standalone/data/tx-object-store and
    multiple warnings are logged for each one as the recovery manager
    attempts recovery.
    
    This commit prevents that behavior by not attempting an XA transaction
    against a PostgreSQL database unless it has max_prepared_transactions
    set to a value greater than 0.
    bbrowning committed Feb 28, 2012
  2. @lance
Commits on Feb 27, 2012
  1. @lance
Commits on Feb 24, 2012
  1. @bbrowning

    Don't remove the XaDataSourceService if the datasource doesn't suppor…

    …t XA (TORQUE-685)
    
    Removing the XaDataSourceService triggers an entire chain of Services
    to restart and ultimately leads to the null classloader seen in
    TORQUE-685.
    
    There was a flawed assumption that when an optional dependency is
    removed that it doesn't trigger the dependent to stop. Not only does
    it trigger the dependent to stop but the dependent also immediately
    restarts because all its dependencies are satisfied even with the
    optional dependency removed.
    bbrowning committed Feb 24, 2012
  2. @bobmcwhirter

    Version for EAP builds.

    bobmcwhirter committed Feb 24, 2012
  3. @bobmcwhirter
  4. @bobmcwhirter

    Build against an EAP repo.

    Assuming you have a settings.xml pointing to an EAP repo:
    
    	mvn install -Pjboss-eap-repository
    bobmcwhirter committed Feb 13, 2012
Commits on Feb 23, 2012
  1. @bbrowning
  2. @bbrowning

    remove trailing whitespace

    bbrowning committed Feb 23, 2012
  3. @bbrowning
Commits on Feb 22, 2012
  1. @bbrowning
  2. @bbrowning