diff options
author | Svilen Mihaylov <svilen.mihaylov@mongodb.com> | 2022-02-14 16:27:22 +0000 |
---|---|---|
committer | Evergreen Agent <no-reply@evergreen.mongodb.com> | 2022-02-14 19:13:13 +0000 |
commit | 81b430cc20641e21037ee58e5124897974fdb2ae (patch) | |
tree | 195c4c91132dd4a12f3ab6d36aad49ca254f3173 | |
parent | 7a1ecab0a42ce278d592fc274d59cbe09af2caaf (diff) | |
download | mongo-81b430cc20641e21037ee58e5124897974fdb2ae.tar.gz |
SERVER-63583 Address coverity issues in the new optimizer.
-rw-r--r-- | src/mongo/db/query/optimizer/cascades/logical_props_derivation.cpp | 1 | ||||
-rw-r--r-- | src/mongo/db/query/optimizer/cascades/memo.cpp | 1 | ||||
-rw-r--r-- | src/mongo/db/query/optimizer/explain.cpp | 1 |
3 files changed, 3 insertions, 0 deletions
diff --git a/src/mongo/db/query/optimizer/cascades/logical_props_derivation.cpp b/src/mongo/db/query/optimizer/cascades/logical_props_derivation.cpp index 4f2a655a363..510bad791e3 100644 --- a/src/mongo/db/query/optimizer/cascades/logical_props_derivation.cpp +++ b/src/mongo/db/query/optimizer/cascades/logical_props_derivation.cpp @@ -330,6 +330,7 @@ public: } else { distributions.erase(it++); } + break; default: it++; diff --git a/src/mongo/db/query/optimizer/cascades/memo.cpp b/src/mongo/db/query/optimizer/cascades/memo.cpp index d39dc7ccbc5..111247e3138 100644 --- a/src/mongo/db/query/optimizer/cascades/memo.cpp +++ b/src/mongo/db/query/optimizer/cascades/memo.cpp @@ -104,6 +104,7 @@ PhysOptimizationResult::PhysOptimizationResult(size_t index, _physProps(std::move(physProps)), _costLimit(std::move(costLimit)), _nodeInfo(), + _rejectedNodeInfo(), _lastImplementedNodePos(0), _queue() {} diff --git a/src/mongo/db/query/optimizer/explain.cpp b/src/mongo/db/query/optimizer/explain.cpp index 064ffc1c0aa..b813a84ac99 100644 --- a/src/mongo/db/query/optimizer/explain.cpp +++ b/src/mongo/db/query/optimizer/explain.cpp @@ -317,6 +317,7 @@ public: ExplainPrinterImpl(ExplainPrinterImpl&& other) noexcept { _nextFieldName = std::move(other._nextFieldName); _initialized = other._initialized; + _canAppend = other._canAppend; _tag = other._tag; _val = other._val; _fieldNameSet = std::move(other._fieldNameSet); |