quickfix-developers

[Quickfix-developers] MySQLStore: SQL statements exchanged

[Pasquale d'A. <pas...@ga...>]

I think the "UPDATE" statements in functions =
MySQLStore::setNextSenderMsgSeqNum() and =
MySQLStore::setNextTargetMsgSeqNum() have been exchanged.
setNextSenderMsgSeqNum() must set the field "outgoing_seqnum" while =
setNextTargetMsgSeqNum() must set the field "incoming_seqnum", I =
suppose.

Here is the wrong code:

void MySQLStore::setNextSenderMsgSeqNum( int value ) throw ( =
IOException& )
{=20
.....
    query << "UPDATE sessions SET incoming_seqnum=3D" << value << " =
WHERE "
...
}

void MySQLStore::setNextTargetMsgSeqNum( int value ) throw ( =
IOException& )
{=20
....
    query << "UPDATE sessions SET outgoing_seqnum=3D" << value << " =
WHERE "
...
}

Regards.
Pasquale d'Aloise

Re: [Quickfix-developers] MySQLStore: SQL statements exchanged

[Oren M. <or...@qu...>]

Yes, thank you.  I've checked this fix it in to CVS.

http://cvs.sourceforge.net/viewcvs.py/quickfix/quickfix/src/C%2B%2B/=20
MySQLStore.cpp?r1=3D1.6&r2=3D1.7

On Jun 4, 2004, at 5:37 PM, Pasquale d'Aloise wrote:

> I think the "UPDATE" statements in functions =20
> MySQLStore::setNextSenderMsgSeqNum() and =20
> MySQLStore::setNextTargetMsgSeqNum() have been exchanged.
> setNextSenderMsgSeqNum() must set the=A0field "outgoing_seqnum" while =20=

> setNextTargetMsgSeqNum() must set the=A0field "incoming_seqnum", I =20
> suppose.
>
> Here is the wrong code:
>
> void MySQLStore::setNextSenderMsgSeqNum( int value ) throw ( =20
> IOException& )
> {
>  .....
> =A0=A0=A0 query << "UPDATE sessions SET incoming_seqnum=3D" << value =
<< " =20
> WHERE "
> ...
> }
>
> void MySQLStore::setNextTargetMsgSeqNum( int value ) throw ( =20
> IOException& )
> {
>  ....
> =A0=A0=A0 query << "UPDATE sessions SET outgoing_seqnum=3D" << value =
<< " =20
> WHERE "
> ...
> }
> =A0
> Regards.
> Pasquale d'Aloise