Bugfix on date sorting (Thanks to @trinity-1686a) (#3619)

fulmicoton · web-flow · commit 8a27ddb19122 · 2023-07-10T04:53:48.000Z
Not my finding. All credit to @trinity. Just extracting the bugfix to release 0.6.2.
diff --git a/quickwit/quickwit-search/src/collector.rs b/quickwit/quickwit-search/src/collector.rs
@@ -100,7 +100,7 @@ impl SortingFieldComputer {
                     SortFieldType::U64 => SortValue::U64(sort_value),
                     SortFieldType::I64 => SortValue::I64(i64::from_u64(sort_value)),
                     SortFieldType::F64 => SortValue::F64(f64::from_u64(sort_value)),
-                    SortFieldType::DateTime => SortValue::U64(sort_value),
+                    SortFieldType::DateTime => SortValue::I64(i64::from_u64(sort_value)),
                     SortFieldType::Bool => SortValue::Boolean(sort_value == 1u64),
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ impl SortingFieldComputer {`
`100`	`100`	`SortFieldType::U64 => SortValue::U64(sort_value),`
`101`	`101`	`SortFieldType::I64 => SortValue::I64(i64::from_u64(sort_value)),`
`102`	`102`	`SortFieldType::F64 => SortValue::F64(f64::from_u64(sort_value)),`
`103`		`- SortFieldType::DateTime => SortValue::U64(sort_value),`
	`103`	`+ SortFieldType::DateTime => SortValue::I64(i64::from_u64(sort_value)),`
`104`	`104`	`SortFieldType::Bool => SortValue::Boolean(sort_value == 1u64),`
`105`	`105`	`}`
`106`	`106`	`}`