Menu

#12 TransactionImpl.canUseOnePhaseCommit() error

open-fixed
None
5
2004-09-08
2004-08-18
No

TransactionImpl.canUseOnePhaseCommit() line 918:

// if there is only one delisted resource then yes
if ( ( null != _delisted ) &&
( _delisted._nextHolder != null ) &&
( null == _enlisted ) )
return true;

shouldn't be
_delisted._nextHolder == null instead of
_delisted._nextHolder != null ?

In the first case it means that there are more than one
delisted resources.

The same error appears on the "enlisted resources
check" (feedback of Rost Vashevnik), line 912:

// if there is only one enlisted resource then yes
if ( ( null != _enlisted ) &&
( _enlisted._nextHolder != null ) &&
( null == _delisted ) )
return true;

Regards
Massimo

Discussion

  • Massimo Ferrari

    Massimo Ferrari - 2004-08-18

    Logged In: YES
    user_id=173555

    PS: this bug is a duplicate / extension of bug 634975

     
  • Rost Vashevnik

    Rost Vashevnik - 2004-09-08
    • status: open --> open-fixed
     
  • Rost Vashevnik

    Rost Vashevnik - 2004-09-08
    • assigned_to: nobody --> metaboss
     
  • Rost Vashevnik

    Rost Vashevnik - 2004-09-08

    Logged In: YES
    user_id=883678

    Hi Massimo !

    I have fixed it. It is available in the latest revision of the file
    in CVS. Will include in the next buld.

    Thank you

     

Log in to post a comment.