-
Type: Task
-
Resolution: Works as Designed
-
Priority: Major - P3
-
Affects Version/s: None
-
Labels:None
Description
We have marked one of the methods related to $or operator as dead code. We are certain that no user query can lead to the execution of this method, so this change should not impact anything.
Still, we would like other teams to be informed just in case.
Description of Linked Ticket
During the discussion of SERVER-52914, we have noticed that OrMatchExpression::matchesSingleElement has a different handling of MatchDetails object compared to OrMatchExpression::matches. During the investigation we have started to wonder if this method ever gets called. It looks like query parser simply does not allow a query where this method would be called.
Our coverage report for July 2019 says that this method is not covered by tests. Recent Evergreen patch where this method was replaced with MONGO_UNREACHABLE passes without any failures.
It is highly unlikely that our tests do not cover some case where OrMatchExpression::matchesSingleElement is used, so we concluded that this is a dead code. We should replace the body of this method with MONGO_UNREACHABLE_TASSERT to avoid further confusion.
Scope of changes
Impact to Other Docs
MVP (Work and Date)
Resources (Scope or Design Docs, Invision, etc.)
- documents
-
SERVER-54299 Mark OrMatchExpression::matchesSingleElement as dead code
- Closed