> 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 > >