// 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.
Regards
Massimo
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
This one appears to be a bug (in fact it appears to be similar for the enlisted resources @ line 912), please file this one in as a bug report and I will fix it.
Kind regards
Rost
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
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.
Regards
Massimo
Hi Massimo !
This one appears to be a bug (in fact it appears to be similar for the enlisted resources @ line 912), please file this one in as a bug report and I will fix it.
Kind regards
Rost