hcq/MaxScale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MXS-3330: Book only expected responses

Browse Source 
Responses generated by replayed session commands must not be treated as
actual responses to retained statements. In 2.5 this is not a problem as
it is done implicitly with the pre-assignment of the server that delivers
the session command response.
This commit is contained in:
Markus Mäkelä
2021-02-10 09:42:12 +02:00
parent 47bcb6ad02
commit 2657cc100e
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
server/modules/routing/readwritesplit/rwsplitsession.cc
View File

@ -734,7 +734,10 @@ void RWSplitSession::clientReply(GWBUF* writebuf, DCB* backend_dcb)
/** Got a complete reply, decrement expected response count */ /** Got a complete reply, decrement expected response count */
m_expected_responses--; m_expected_responses--;
if (!backend->is_replaying_history())
{
session_book_server_response(m_pSession, backend->backend()->server, m_expected_responses == 0); session_book_server_response(m_pSession, backend->backend()->server, m_expected_responses == 0);
}
mxb_assert(m_expected_responses >= 0); mxb_assert(m_expected_responses >= 0);
mxb_assert(backend->get_reply_state() == REPLY_STATE_DONE); mxb_assert(backend->get_reply_state() == REPLY_STATE_DONE);