Re: [eclipselink-dev] bug 326777 - Some Core LRG tests hang on MaxDB

["Goerler, Adrian" <adrian.goerler@xxxxxxx>]

Hi,

 

https://bugs.eclipse.org/bugs/show_bug.cgi?id=326777

 

reviewed by Andrei,

tested on MaxDB (core LRG),

checked in at #8286.

 

-Adrian

 

SAP AG
Dietmar-Hopp-Allee 16, 69190 Walldorf, Germany

Sitz der Gesellschaft/Registered Office: Walldorf, Germany
Vorstand/SAP Executive Board: Henning Kagermann (Sprecher/CEO), Léo Apotheker (stellvertretender Sprecher/Deputy CEO), Werner Brandt, Claus Heinrich, Gerhard Oswald, John Schwarz, Peter Zencke

Vorsitzender des Aufsichtsrats/Chairperson of the SAP Supervisory Board: Hasso Plattner
Registergericht/Commercial Register Mannheim No HRB 350269

 

---

Von: Andrei Ilitchev [mailto:andrei.ilitchev@xxxxxxxxxx]
Gesendet: Freitag, 1. Oktober 2010 18:12
An: Goerler, Adrian
Cc: Dev mailing list for Eclipse Persistence Services
Betreff: Re: AW: bug 326777 - Some Core LRG tests hang on MaxDB

Looks good.

On 10/1/2010 12:06 PM, Goerler, Adrian wrote:

Hi Andrei,

 

good catch. I have updated the patch. Please double-check again.

 

Thanks,

 

Adrian

 

 

---

Von: Andrei Ilitchev [mailto:andrei.ilitchev@xxxxxxxxxx]
Gesendet: Freitag, 1. Oktober 2010 17:35
An: Goerler, Adrian
Cc: Dev mailing list for Eclipse Persistence Services
Betreff: Re: bug 326777 - Some Core LRG tests hang on MaxDB

Hi Adrian,

The listener added by checkTransactionIsolation method on Symfoware (and now on MaxDB, too) to the ServerSession is never removed.
That seems to be easy to fix - alter checkTransactionIsolation method to return the listener and cache it in the calling tests' listener attribute.
This attribute is already defined by all the tests using checkTransactionIsolation method, and all of these tests already remove the listener in reset method.

Thanks,

Andrei

On 10/1/2010 10:56 AM, Goerler, Adrian wrote:

Hi Andrei,

 

https://bugs.eclipse.org/bugs/show_bug.cgi?id=326777

 

on MaxDB, we are facing the same issue with hanging LRG tests as on Sybase (https://bugs.eclipse.org/bugs/show_bug.cgi?id=240210) and Symfoware.

 

I am basically proposing the same solution (temporaily reduce the iso level of the hanging transaction).

 

Could you please have a look?

 

Tested sucessfully on MaxDB.

 

Thanks,

 

Adrian

 

 

SAP AG

Dietmar-Hopp-Allee 16, 69190 Walldorf, Germany

 

Sitz der Gesellschaft/Registered Office: Walldorf, Germany

Vorstand/SAP Executive Board: Henning Kagermann (Sprecher/CEO), Léo Apotheker (stellvertretender Sprecher/Deputy CEO), Werner Brandt, Claus Heinrich, Gerhard Oswald, John Schwarz, Peter Zencke

Vorsitzender des Aufsichtsrats/Chairperson of the SAP Supervisory Board: Hasso Plattner

Registergericht/Commercial Register Mannheim No HRB 350269