-
-
Notifications
You must be signed in to change notification settings - Fork 625
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixes order_by_u64_field and order_by_fast_field should allow sorting in ascending order #1676 #2111
fixes order_by_u64_field and order_by_fast_field should allow sorting in ascending order #1676 #2111
Conversation
)) | ||
.unwrap(); | ||
index_writer | ||
.add_document(doc!( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add an empty doc to the test
index_writer.add_document(doc!()).unwrap();
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry I meant a doc without a value
index_writer.add_document(doc!(title => "empty beer"))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ahh ok. without the size field.
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## main #2111 +/- ##
========================================
Coverage 94.39% 94.39%
========================================
Files 321 321
Lines 60650 60797 +147
========================================
+ Hits 57248 57391 +143
- Misses 3402 3406 +4
|
src/collector/top_score_collector.rs
Outdated
@@ -157,8 +171,13 @@ impl CustomScorer<u64> for ScorerByField { | |||
sort_column_opt.ok_or_else(|| FastFieldNotAvailableError { | |||
field_name: self.field.clone(), | |||
})?; | |||
let mut value = 0u64; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let mut value = 0u64; | |
let mut default_value = 0u64; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks!
No description provided.