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

MXS-2486: Add missing schemarouter capabilities

Browse Source 
The capabilities that the schemarouter declared were missing the
RCAP_TYPE_PACKET_OUTPUT which caused partial packets to be returned to it.
This commit is contained in:
Markus Mäkelä
2019-07-23 09:46:05 +03:00
parent fbeb5d9c84
commit 12bd26398f
2 changed files with 18 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
server/modules/protocol/MySQL/rwbackend.cc
View File

@ -294,6 +294,7 @@ void RWBackend::process_packets(GWBUF* result)
auto it = buffer.begin(); auto it = buffer.begin();
MXB_AT_DEBUG(size_t total_len = buffer.length()); MXB_AT_DEBUG(size_t total_len = buffer.length());
MXB_AT_DEBUG(size_t used_len = 0); MXB_AT_DEBUG(size_t used_len = 0);
mxb_assert(dcb()->session->service->capabilities & (RCAP_TYPE_PACKET_OUTPUT | RCAP_TYPE_STMT_OUTPUT));
while (it != buffer.end()) while (it != buffer.end())
{ {

15
server/modules/routing/schemarouter/schemarouterinstance.cc
View File

@ -265,9 +265,12 @@ json_t* SchemaRouter::diagnostics_json() const
return rval; return rval;
} }
static const uint64_t CAPABILITIES = RCAP_TYPE_CONTIGUOUS_INPUT | RCAP_TYPE_PACKET_OUTPUT
| RCAP_TYPE_RUNTIME_CONFIG;
uint64_t SchemaRouter::getCapabilities() uint64_t SchemaRouter::getCapabilities()
{ {
return RCAP_TYPE_CONTIGUOUS_INPUT | RCAP_TYPE_RUNTIME_CONFIG; return schemarouter::CAPABILITIES;
} }
} }
@ -288,12 +291,12 @@ extern "C" MXS_MODULE* MXS_CREATE_MODULE()
MXS_ROUTER_VERSION, MXS_ROUTER_VERSION,
"A database sharding router for simple sharding", "A database sharding router for simple sharding",
"V1.0.0", "V1.0.0",
RCAP_TYPE_CONTIGUOUS_INPUT | RCAP_TYPE_RUNTIME_CONFIG, schemarouter::CAPABILITIES,
&schemarouter::SchemaRouter::s_object, &schemarouter::SchemaRouter::s_object,
NULL, /* Process init. */ NULL,
NULL, /* Process finish. */ NULL,
NULL, /* Thread init. */ NULL,
NULL, /* Thread finish. */ NULL,
{ {
{"ignore_databases", MXS_MODULE_PARAM_STRING }, {"ignore_databases", MXS_MODULE_PARAM_STRING },
{"ignore_databases_regex", MXS_MODULE_PARAM_STRING }, {"ignore_databases_regex", MXS_MODULE_PARAM_STRING },