Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-3430

DIH: More-realistic testing & subsequent bug fix

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    Description

      A couple of people have mentioned on the users' list that 3.6 changes break CachedSqlEntityProcessor. Part of the problem is the current DIH tests are not realistic: they mostly test 1 class at a time and not the full end-to-end interactions between components.

      This is a test of (Cached) SqlEntityProcessor using an in-memory Derby database. This hopefully is a first step in creating a series of new, more-rigorous DIH tests, which is needed in order to make changes without worry.

      This also revealed a bug in SortedMapBackedCache, which is fixed also. I'm not sure this is the same bug causing problems for the 2 users but once committed I will post a reply on the users' list to see if they can try it.

      Attachments

        1. derby-10.8.2.2.jar
          2.55 MB
          James Dyer
        2. derby-10.8.2.2.jar.sha1
          0.0 kB
          James Dyer
        3. derby-LICENSE-ASL.txt
          11 kB
          James Dyer
        4. derby-NOTICE.txt
          7 kB
          James Dyer
        5. SOLR-3430_build.patch
          3 kB
          Robert Muir
        6. SOLR-3430.patch
          36 kB
          James Dyer
        7. SOLR-3430.patch
          35 kB
          James Dyer
        8. SOLR-3430.patch
          34 kB
          James Dyer
        9. SOLR-3430.patch
          33 kB
          James Dyer

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            jdyer James Dyer
            jdyer James Dyer
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment