-
Notifications
You must be signed in to change notification settings - Fork 458
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
Tweak parquet BEIR yaml configs #2609
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2609 +/- ##
============================================
+ Coverage 65.56% 65.57% +0.01%
Complexity 1479 1479
============================================
Files 225 225
Lines 13012 13009 -3
Branches 1551 1551
============================================
Hits 8531 8531
+ Misses 3953 3950 -3
Partials 528 528
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@valamuri2020 please take a look? |
@@ -49,24 +49,19 @@ public class ParquetDenseVectorDocumentGenerator<T extends SourceDocument> imple | |||
public Document createDocument(T src) throws InvalidDocumentException { | |||
|
|||
try { | |||
LOG.info("Processing document ID: " + src.id() + " with thread: " + Thread.currentThread().getName()); |
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.
I removed logging to make less noisy.
Entire slate of flat, cached regression: