Skip to content

Commit 417cce3

Browse files
alambTed-Jiang
authored andcommitted
Minor: Improve TableProviderFilterPushDown docs (apache#7685)
1 parent d804793 commit 417cce3

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

datafusion/expr/src/table_source.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,14 @@ use std::any::Any;
3030
pub enum TableProviderFilterPushDown {
3131
/// The expression cannot be used by the provider.
3232
Unsupported,
33-
/// The expression can be used to help minimise the data retrieved,
34-
/// but the provider cannot guarantee that all returned tuples
35-
/// satisfy the filter. The Filter plan node containing this expression
36-
/// will be preserved.
33+
/// The expression can be used to reduce the data retrieved,
34+
/// but the provider cannot guarantee it will omit all tuples that
35+
/// may be filtered. In this case, DataFusion will apply an additional
36+
/// `Filter` operation after the scan to ensure all rows are filtered correctly.
3737
Inexact,
38-
/// The provider guarantees that all returned data satisfies this
39-
/// filter expression. The Filter plan node containing this expression
40-
/// will be removed.
38+
/// The provider **guarantees** that it will omit **all** tuples that are
39+
/// filtered by the filter expression. This is the fastest option, if available
40+
/// as DataFusion will not apply additional filtering.
4141
Exact,
4242
}
4343

0 commit comments

Comments
 (0)