add support for f64 for concat fields

quickwit-oss · Aug 30, 2024 · 213ecbf · 213ecbf
1 parent 67091c5
commit 213ecbf
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/quickwit/quickwit-doc-mapper/src/default_doc_mapper/mapping_tree.rs b/quickwit/quickwit-doc-mapper/src/default_doc_mapper/mapping_tree.rs
@@ -314,14 +314,13 @@ impl LeafType {
 
     fn supported_for_concat(&self) -> bool {
         use LeafType::*;
-        matches!(self, Text(_) | U64(_) | I64(_) | Bool(_) | Json(_))
+        matches!(self, Text(_) | U64(_) | I64(_) | F64(_) | Bool(_) | Json(_))
         /*
-            // will be supported if possible
-            DateTime(_),
+            // Since concat is a JSON field, anything that JSON supports can be supported
+            DateTime(_), // Could be supported if the date is converted to Rfc3339
             IpAddr(_),
             // won't be supported
             Bytes(_),
-            F64(_),
         */
     }
 }