-
Notifications
You must be signed in to change notification settings - Fork 24
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
EVA-486 PopulationStatistics: add chr, start, ref, alt fields #44
Conversation
@@ -101,9 +101,10 @@ public void shouldCreateIndexesInCollection() throws Exception { | |||
|
|||
// check vid has an index |
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.
This comments need to be updated to reflect the new structure.
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.
forgot about this PR, fixed in next commit
dda81aa
to
159d29f
Compare
This was necessary if we want to do better queries like going to a specific region. this would we harder if we had only the merged vid. {vid: {$lte: 20_7000}} would not only query the positions before 7000, but all the previous chromosomes as well.
159d29f
to
611514b
Compare
".populationStatistics\"}, { \"v\" : 1 , \"unique\" : true , \"key\" : " + | ||
"{ \"chr\" : 1 , \"start\" : 1 , \"ref\" : 1 , \"alt\" : 1 , \"sid\" : 1 , \"cid\" : 1} , " + | ||
"\"name\" : \"vscid\" , \"ns\" : \"" + databaseName + ".populationStatistics\"}]", | ||
statsCollection.getIndexInfo().toString()); |
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.
This test could compare the DBObjects extracted from getIndexInfo()
using the equals()
method.
EVA-486 PopulationStatistics: add chr, start, ref, alt fields
This was necessary if we want to do better queries like going
to a specific region. this would we harder if we had only the
merged vid. {vid: {$lte: 20_7000}} would not only query the
positions before 7000, but all the previous chromosomes as well.