I filed https://issues.apache.org/jira/browse/SOLR-5380 and just committed a 
fix.

- Mark

On Oct 23, 2013, at 11:15 AM, Shawn Heisey <s...@elyograg.org> wrote:

> On 10/23/2013 3:59 AM, Thomas Egense wrote:
>> Using cloudSolrServer.setDefaultCollection(collectionId) does not work as
>> intended for an alias spanning more than 1 collection.
>> The virtual collection-alias collectionID is recoqnized as a existing
>> collection, but it does only query one of the collections it is mapped to.
>> 
>> You can confirm this easy in AliasIntegrationTest.
>> 
>> The test-class AliasIntegrationTest creates to cores with 2 and 3 different
>> documents. And then creates an alias pointing to both of them.
>> 
>> Line 153:
>>    // search with new cloud client
>>    CloudSolrServer cloudSolrServer = new
>> CloudSolrServer(zkServer.getZkAddress(), random().nextBoolean());
>>    cloudSolrServer.setParallelUpdates(random().nextBoolean());
>>    query = new SolrQuery("*:*");
>>    query.set("collection", "testalias");
>>    res = cloudSolrServer.query(query);
>>    cloudSolrServer.shutdown();
>>    assertEquals(5, res.getResults().getNumFound());
>> 
>> No unit-test bug here, however if you change it from setting the
>> collectionid on the query but on CloudSolrServer instead,it will produce
>> the bug:
>> 
>>    // search with new cloud client
>>    CloudSolrServer cloudSolrServer = new
>>    CloudSolrServer(zkServer.getZkAddress(), random().nextBoolean());
>>    cloudSolrServer.setDefaultCollection("testalias");
>>    cloudSolrServer.setParallelUpdates(random().nextBoolean());
>>    query = new SolrQuery("*:*");
>>    //query.set("collection", "testalias");
>>    res = cloudSolrServer.query(query);
>>    cloudSolrServer.shutdown();
>>    assertEquals(5, res.getResults().getNumFound());  <-- Assertion failure
>> 
>> Should I create a Jira issue for this?
> 
> Thomas,
> 
> I have confirmed this with the following test patch, which adds to the
> test rather than changing what's already there:
> 
> http://apaste.info/9ke5
> 
> I'm about to head off to the train station to start my commute, so I
> will be unavailable for a little while.  If you haven't gotten the jira
> filed by the time I get to another computer, I will create it.
> 
> Thanks,
> Shawn
> 

Reply via email to