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
>

Reply via email to