Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[query] Improve GVCF/VCF import performance #8535

Merged
merged 9 commits into from
Apr 23, 2020
Merged

[query] Improve GVCF/VCF import performance #8535

merged 9 commits into from
Apr 23, 2020

Conversation

tpoterba
Copy link
Contributor 

                                    Benchmark Name      Ratio     Time 1     Time 2
                                    --------------      -----     ------     ------
                         import_and_transform_gvcf      69.9%    128.759     90.060
                           import_gvcf_force_count      66.5%    109.791     73.018```

chrisvittal
chrisvittal previously requested changes Apr 13, 2020
View reviewed changes
hail/src/main/scala/is/hail/io/vcf/LoadVCF.scala Show resolved Hide resolved
hail/src/main/scala/is/hail/io/tabix/TabixReader.scala Outdated Show resolved Hide resolved
hail/src/main/scala/is/hail/variant/ReferenceGenome.scala Outdated Show resolved Hide resolved
hail/src/main/scala/is/hail/io/tabix/TabixReader.scala Outdated Show resolved Hide resolved
hail/src/main/scala/is/hail/io/tabix/TabixReader.scala Outdated Show resolved Hide resolved
hail/src/main/scala/is/hail/io/tabix/TabixReader.scala Outdated Show resolved Hide resolved
@tpoterba tpoterba mentioned this pull request Apr 13, 2020
Merged
@tpoterba tpoterba dismissed chrisvittal’s stale review April 21, 2020 19:00

addressed, ready for another review

@danking danking merged commit 3302850 into hail-is:master Apr 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chrisvittal chrisvittal chrisvittal approved these changes

Assignees

@chrisvittal chrisvittal

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tpoterba @chrisvittal @danking