Probably wrong assert in executeClientQuery

Issue #55 new
Artyom Smirnov created an issue

File phoenixdb/src/network/ClientMessageHandleJob.cpp

void ClientMessageHandleJob::executeClientQuery()
...
assert(Query::getQueryByID(queryResult.queryID)->queryString == queryString);

Query string required only in prepare message, in execute message it can be omitted.

Comments (1)

  1. Roman Simakov Account Deactivated

    I guess preparing is never used after implementation:) But if such assert is I'm afraid some place hopes on it. I agree I should be removed. Regressions we can fix later.

  2. Log in to comment