From cfb08cc9f1c68fb0864b0045941bb3471c42048d Mon Sep 17 00:00:00 2001 From: Andrew Lamb Date: Sat, 16 Dec 2023 08:07:00 -0500 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Ruihang Xia --- datafusion/physical-expr/src/utils/guarantee.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/datafusion/physical-expr/src/utils/guarantee.rs b/datafusion/physical-expr/src/utils/guarantee.rs index 3969346f2b55..afe61fd63d58 100644 --- a/datafusion/physical-expr/src/utils/guarantee.rs +++ b/datafusion/physical-expr/src/utils/guarantee.rs @@ -49,7 +49,7 @@ use std::sync::Arc; /// **Important**: If a `LiteralGuarantee` is not satisfied, the relevant /// expression is *guaranteed* to evaluate to `false` or `null`. **However**, /// the opposite does not hold. Even if all `LiteralGurantee`s are satisfied, -/// that does does not guarantee that the predicate will actually evaluate to +/// that does not guarantee that the predicate will actually evaluate to /// `true`: it may still evaluate to `true`, `false` or `null`. /// /// # Creating `LiteralGuarantee`s @@ -60,7 +60,7 @@ use std::sync::Arc; /// # Details /// A guarantee can be one of two forms: /// -/// 1. The column must be one a particular set of values for the predicate +/// 1. The column must be one of a particular set of values for the predicate /// to be `true`. If the column takes on any other value, the predicate can not /// evaluate to `true`. For example, /// `(a = 1)`, `(a = 1 OR a = 2) or `a IN (1, 2, 3)`