Re: Very Dangerous bug in XA connection pooling and SL EJBs with jboss-4.2.3 & latest postgresql-8.4-701.jdbc3.jar

Στις Wednesday 03 March 2010 10:54:53 ο/η Heikki Linnakangas έγραψε:
> Achilleas Mantzios wrote:
> > i'll try to make a test case, but i think now there is enuf material for anyone to make his own case,
> > please ask whatever i may have missed, and thank you for your attention.
>
> If you can set logLevel to DEBUG (ie. loglevel=2 in connection string),
> run the code, and send over the logs, that would help a lot. From that
> we should see what exactly is the sequence of XA commands the JDBC
> driver is receiving from JBoss.
>

As loglevel configuration property to org.postgresql.ds.common.BaseDataSource  is not supported in the lastest driver,
i made this patch (i attach it) against postgresql-jdbc-8.4-701.src, also i set in jboss

<datasources>
   <xa-datasource>
     <jndi-name>pgsql</jndi-name>
     <use-java-context>true</use-java-context>
     <xa-datasource-class>org.postgresql.xa.PGXADataSource</xa-datasource-class>
     <xa-datasource-property name="ServerName">localhost</xa-datasource-property>
     <xa-datasource-property name="PortNumber">5432</xa-datasource-property>
     <xa-datasource-property name="DatabaseName">dynacom</xa-datasource-property>
     <xa-datasource-property name="User">postgres</xa-datasource-property>
     <xa-datasource-property name="Password">xxxx</xa-datasource-property>
     <xa-datasource-property name="Compatible">8.2</xa-datasource-property>
     <xa-datasource-property name="Loglevel">2</xa-datasource-property>
     <track-connection-by-tx/>
     <min-pool-size>1</min-pool-size>
     <max-pool-size>2</max-pool-size>
     <metadata>
         <type-mapping>PostgreSQL 8.0</type-mapping>
     </metadata>
   </xa-datasource>
</datasources>

But i dont see anything spectacular comming out of PostgreSQL logs.
in postgresql.conf I have set
client_min_messages = debug5
log_min_messages = debug5

however playing with <xa-datasource-property name="Loglevel"></xa-datasource-property> does not seem to change a thing.
Am i missing something?

> --
>   Heikki Linnakangas
>   EnterpriseDB   http://www.enterprisedb.com
>



--
Achilleas Mantzios