Hoss Man created SOLR-13210: ------------------------------- Summary: TriLevelCompositeIdRoutingTest makes no sense -- can never fail Key: SOLR-13210 URL: https://issues.apache.org/jira/browse/SOLR-13210 Project: Solr Issue Type: Sub-task Security Level: Public (Default Security Level. Issues are Public) Reporter: Hoss Man
i recently fixed tweaked TriLevelCompositeIdRoutingTest to lower the node/shard count on TEST_NIGHTLY because it was constantly causing an OOM. While skimming this test i realized that (other then the OOM, or other catastrophic failure in solr) it was garunteed to never fail, rgardless of what bugs might exist in solr when routing an update/query: * it doesn't sanity check that any docs are returned from any query -- so if commit does nothing and it gets no results from each of the shard queries, it will still pass * the {{getKey()}} method -- which throws away anything after the last "!" in a String -- is called redundently on it's own output to populate an {{idMap}} ... but not before the first result is used do to acontainsKey assertion on that same {{idMap}} ** ie: if {{app42/7!user33!doc1234}} is a uniqueKey value, then {{app42/7!user33}} is what the assert !containsKey checks the Map for, but {{app42/7}} is what gets put in the Map -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org