summaryrefslogtreecommitdiff
path: root/src/mongo/db/exec
diff options
context:
space:
mode:
authorProjjal Chanda <projjal.chanda@mongodb.com>2023-02-09 11:57:09 +0000
committerEvergreen Agent <no-reply@evergreen.mongodb.com>2023-02-09 18:55:31 +0000
commita579f1d19e87c0fbf5f86e3aacb39d710d8fb757 (patch)
treec31b9fc5e3176239e54638a063dc47c37df120e5 /src/mongo/db/exec
parentc136914675b484500b10ab2350483e8bafa08862 (diff)
downloadmongo-a579f1d19e87c0fbf5f86e3aacb39d710d8fb757.tar.gz
SERVER-71525: Removed failOnPoisonedFieldLookup fail point and associated tests relying on it.
Performs an invalid operation (divide-by-zero) instead of a "POISON" field lookup for short-circuit testing
Diffstat (limited to 'src/mongo/db/exec')
-rw-r--r--src/mongo/db/exec/sbe/vm/vm.cpp7
1 files changed, 0 insertions, 7 deletions
diff --git a/src/mongo/db/exec/sbe/vm/vm.cpp b/src/mongo/db/exec/sbe/vm/vm.cpp
index 35dab757964..99464310d2f 100644
--- a/src/mongo/db/exec/sbe/vm/vm.cpp
+++ b/src/mongo/db/exec/sbe/vm/vm.cpp
@@ -64,9 +64,6 @@
#define MONGO_LOGV2_DEFAULT_COMPONENT ::mongo::logv2::LogComponent::kQuery
-
-MONGO_FAIL_POINT_DEFINE(failOnPoisonedFieldLookup);
-
namespace mongo {
namespace sbe {
namespace vm {
@@ -962,10 +959,6 @@ FastTuple<bool, value::TypeTags, value::Value> ByteCode::getField(value::TypeTag
FastTuple<bool, value::TypeTags, value::Value> ByteCode::getField(value::TypeTags objTag,
value::Value objValue,
StringData fieldStr) {
- if (MONGO_unlikely(failOnPoisonedFieldLookup.shouldFail())) {
- uassert(4623399, "Lookup of $POISON", fieldStr != "POISON");
- }
-
if (objTag == value::TypeTags::Object) {
auto [tag, val] = value::getObjectView(objValue)->getField(fieldStr);
return {false, tag, val};