diff options
| author | David Storch <david.storch@10gen.com> | 2018-01-05 17:09:45 -0500 |
|---|---|---|
| committer | David Storch <david.storch@10gen.com> | 2018-01-09 17:58:11 -0500 |
| commit | c3c908788d6630969055b7fad674403a476cf481 (patch) | |
| tree | 112240b0d66d8327e44e62b52b408eba0fae04dc /src/mongo/db | |
| parent | a9d076d8642c13859588c3bc111b3c9af08cea2f (diff) | |
| download | mongo-c3c908788d6630969055b7fad674403a476cf481.tar.gz | |
SERVER-32570 Clean up stale TODO comments in the query system.
Diffstat (limited to 'src/mongo/db')
| -rw-r--r-- | src/mongo/db/clientcursor.h | 3 | ||||
| -rw-r--r-- | src/mongo/db/exec/count_scan.cpp | 1 | ||||
| -rw-r--r-- | src/mongo/db/exec/index_scan.cpp | 1 | ||||
| -rw-r--r-- | src/mongo/db/pipeline/lite_parsed_pipeline.h | 2 |
4 files changed, 0 insertions, 7 deletions
diff --git a/src/mongo/db/clientcursor.h b/src/mongo/db/clientcursor.h index f75d41e3089..2e8d7894ad3 100644 --- a/src/mongo/db/clientcursor.h +++ b/src/mongo/db/clientcursor.h @@ -333,9 +333,6 @@ private: // CursorManager, at which point it has sole ownership of the ClientCursor. // - // TODO SERVER-28309 Remove this field and instead use _exec->markedAsKilled(). - bool _killed = false; - // While a cursor is being used by a client, it is marked as "pinned". See ClientCursorPin // below. // diff --git a/src/mongo/db/exec/count_scan.cpp b/src/mongo/db/exec/count_scan.cpp index 876a5a89d56..8fa0433fe7d 100644 --- a/src/mongo/db/exec/count_scan.cpp +++ b/src/mongo/db/exec/count_scan.cpp @@ -157,7 +157,6 @@ void CountScan::doRestoreState() { _cursor->restore(); // This can change during yielding. - // TODO this isn't sufficient. See SERVER-17678. _shouldDedup = _descriptor->isMultikey(getOpCtx()); } diff --git a/src/mongo/db/exec/index_scan.cpp b/src/mongo/db/exec/index_scan.cpp index d453150c1ec..3459739ed0d 100644 --- a/src/mongo/db/exec/index_scan.cpp +++ b/src/mongo/db/exec/index_scan.cpp @@ -94,7 +94,6 @@ boost::optional<IndexKeyEntry> IndexScan::initIndexScan() { if (_params.doNotDedup) { _shouldDedup = false; } else { - // TODO it is incorrect to rely on this not changing. SERVER-17678 _shouldDedup = _params.descriptor->isMultikey(getOpCtx()); } diff --git a/src/mongo/db/pipeline/lite_parsed_pipeline.h b/src/mongo/db/pipeline/lite_parsed_pipeline.h index 33b2f695d75..e6918ee0d1c 100644 --- a/src/mongo/db/pipeline/lite_parsed_pipeline.h +++ b/src/mongo/db/pipeline/lite_parsed_pipeline.h @@ -93,8 +93,6 @@ public: /** * Returns true if the pipeline has a $changeStream stage. - * - * TODO SERVER-29506 Require $changeStream to be the first stage. */ bool hasChangeStream() const { return std::any_of(_stageSpecs.begin(), _stageSpecs.end(), [](auto&& spec) { |