> On May 4, 2018, 9:11 p.m., Varun Thacker wrote:
> > solr/core/src/test/org/apache/solr/update/MockStreamingSolrClients.java
> > Line 28 (original)
> > <https://reviews.apache.org/r/66967/diff/1/?file=2016949#file2016949line32>
> >
> >     We can use @LogLevel("org.apache.solr.update.SolrCmdDistributor=DEBUG) 
> > for this test to make it easier to check manually if the retires are 
> > happening or not . This would avoid changing the log4j2.xml file

Thanks! I didn't know that existed.


- Tomás


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66967/#review202485
-----------------------------------------------------------


On May 4, 2018, 7:49 p.m., Tomás Fernández Löbbe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66967/
> -----------------------------------------------------------
> 
> (Updated May 4, 2018, 7:49 p.m.)
> 
> 
> Review request for lucene.
> 
> 
> Repository: lucene-solr
> 
> 
> Description
> -------
> 
> Still need to do some more testing. Won't be committing the log4j changes
> 
> 
> Diffs
> -----
> 
>   solr/core/src/java/org/apache/solr/update/SolrCmdDistributor.java 
> 08b397fbb2 
>   
> solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java
>  d5e4194b15 
>   
> solr/core/src/java/org/apache/solr/update/processor/TimeRoutedAliasUpdateProcessor.java
>  224bd4b6ba 
>   solr/core/src/test-files/log4j2.xml 7d0ebf7a49 
>   solr/core/src/test/org/apache/solr/update/MockStreamingSolrClients.java 
> 72d39ff89b 
>   solr/core/src/test/org/apache/solr/update/SolrCmdDistributorTest.java 
> 1699b0d70d 
> 
> 
> Diff: https://reviews.apache.org/r/66967/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Tomás Fernández Löbbe
> 
>

Reply via email to