Fix full text index fix

rschu1ze · rschu1ze · commit 8c7a2566327b · 2023-09-13T07:57:48.000Z
diff --git a/src/Storages/MergeTree/MergeTreeIndexFullText.cpp b/src/Storages/MergeTree/MergeTreeIndexFullText.cpp
@@ -346,13 +346,13 @@ bool MergeTreeConditionFullText::extractAtomFromTree(const RPNBuilderTreeNode &
 
             if (const_value.getType() == Field::Types::Int64)
             {
-                out.function = const_value.get<UInt64>() ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;
+                out.function = const_value.get<Int64>() ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;
                 return true;
             }
 
             if (const_value.getType() == Field::Types::Float64)
             {
-                out.function = const_value.get<UInt64>() ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;
+                out.function = const_value.get<Float64>() != 0.0 ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;
                 return true;
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -346,13 +346,13 @@ bool MergeTreeConditionFullText::extractAtomFromTree(const RPNBuilderTreeNode &`
`346`	`346`
`347`	`347`	`if (const_value.getType() == Field::Types::Int64)`
`348`	`348`	`{`
`349`		`- out.function = const_value.get<UInt64>() ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;`
	`349`	`+ out.function = const_value.get<Int64>() ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;`
`350`	`350`	`return true;`
`351`	`351`	`}`
`352`	`352`
`353`	`353`	`if (const_value.getType() == Field::Types::Float64)`
`354`	`354`	`{`
`355`		`- out.function = const_value.get<UInt64>() ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;`
	`355`	`+ out.function = const_value.get<Float64>() != 0.0 ? RPNElement::ALWAYS_TRUE : RPNElement::ALWAYS_FALSE;`
`356`	`356`	`return true;`
`357`	`357`	`}`
`358`	`358`	`}`