Hi There, In order to resolve junit test failure with Solr 9.x Output connector, I modified MockSolrService to support HTTP2C. I prepared a PR for this: https://github.com/apache/manifoldcf/pull/157 I confirmed that all test cases of ant test passed. ``` test: BUILD SUCCESSFUL Total time: 51 minutes 37 seconds ```
Regards, Mingchun 2023年10月28日(土) 1:50 Guylaine BASSETTE <guylaine.basse...@francelabs.com>: > Just a precision about the Solr NavigationHSQLDBUI test was talking > about, my problem is about the test environment I failed to configure > properly in the time I got to work on it. So I don't know if it could > succeed or not. > > Guylaine > Le 27/10/2023 à 17:34, Guylaine BASSETTE a écrit : > > Hi Karl, > > The errors shown by Mingchun were puzzling me, so I did some more testing: > actually, if we set up a real solr 9 server, and modify the Solr test to > point to it rather than to the mockupsolr, then everything runs fine, the > tests ends successfully and the data are actually uploaded into solr 9. My > conclusion is that the error is therefore not at all related to conflicting > libraries, but probably to the mockupsolrservice that is now somehow > outdated in the way that it interacts with the jetty client. > > > Joint to this mail, the logs in Solr server and the test results (I have > added extra logs on job status when I tested) > > Here are the modification I have made: > Remove Mock server launch in SolrCrawlHSQLDBIT class: > > public class SolrCrawlHSQLDBIT extends BaseITHSQLDB > { > > protected SolrTester tester; > protected MockSolrService solrService = null; > > public SolrCrawlHSQLDBIT() > { > tester = new SolrTester(mcfInstance); > } > > @Test > public void simpleCrawl() > throws Exception > { > tester.executeTest(); > } > } > > > In SolrTester: > > configParams.setParameter(SolrConfig.PARAM_SERVER,"IP_SOLR_SERVER"); > // To be replaced with the Solr server IP > configParams.setParameter(SolrConfig.PARAM_PORT,"8983"); > > configParams.setParameter(SolrConfig.PARAM_WEBAPPNAME,"solr/techproducts"); > configParams.setParameter(SolrConfig.PARAM_COLLECTION, "solr"); > > I was not able to pass the NavigationHSQLDBUI test. > > Sorry, I won't be available next week to go further. I hope this will help > ! > > Guylaine > Le 27/10/2023 à 13:40, Karl Wright a écrit : > > That is indeed reassuring. It probably means that the test needs some > changes, is all. But we cannot be sure the newer Zookeeper wouldn't mess > things up without having the test be successful at least with the older > zookeeper. > > Karl > > > On Fri, Oct 27, 2023 at 7:32 AM Guylaine BASSETTE > <guylaine.basse...@francelabs.com> wrote: > > > Sorry for not executing all of the tests: I'm quite new to the MCF > project, and I focused only on the core-framework tests as it was the > key aspect highlighted in the related jira issue. I didn't know I had to > run other tests such as the IT ones. It is to be noted that we tested > the MCF with Solr 9 embedded in our Datafari, and after quite some > indexing, we have seen no problems at all, which sounds quite reassuring > > Le 27/10/2023 à 12:49, Karl Wright a écrit : > > Okay, well I wouldnt have approved the upgrade had I known that the tests > didn't pass! So we need to understand the problem as soon as possible. > > Karl > > > On Fri, Oct 27, 2023 at 2:50 AM Guylaine BASSETTE > <guylaine.basse...@francelabs.com> wrote: > > > Hello all, > > Sadly, the error remains the same even with the previous Zookeeper > version (3.8.0). Actually, I'm not able to pass the test since we have > passed to Solr 9. > > Le 27/10/2023 à 01:51, Karl Wright a écrit : > > It is possible that Solr needs the older version of Zookeeper. If you > swap out the current one and replace it with the one the version of > SolrJ we use references, does the test pass then? If it does, we're > going to have to figure out how to address the fact that we have two > connectors that each depend on a different version of zookeeper. But > first please let me know if it works. I'll suggest a way of > reconciling these once I know. > > -- > Cordialement, > Guylaine > > France Labs – Your knowledge, now > Datafari Enterprise Search – Découvrez la version 5 / Discover our > > version > > 5www.datafari.com <http://www.datafari.com> <http://www.datafari.com> > > -- > Cordialement, > Guylaine > > France Labs – Your knowledge, now > Datafari Enterprise Search – Découvrez la version 5 / Discover our version > 5www.datafari.com <http://www.datafari.com> <http://www.datafari.com> > > -- > Cordialement, > Guylaine > > France Labs – Your knowledge, now > Datafari Enterprise Search – Découvrez la version 5 / Discover our version > 5 > www.datafari.com > > -- > Cordialement, > Guylaine > > France Labs – Your knowledge, now > Datafari Enterprise Search – Découvrez la version 5 / Discover our version > 5 > www.datafari.com >