Details

    • Type: Bug
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.2
    • Component/s: .querying
    • Labels:
      None

      Description

      In line 670, the class org.terrier.applications.batchquerying.TRECQuerying has this:

      ((Request)rs).setResultSet(rs);

      That is an error. The correct code should be:

      ((Request)srq).setResultSet(rs);

      Otherwise, the application will crash when we try to use a cache strategy.

      (Sorry for my English)

       

        Attachments

          Activity

          Hide
          craigm Craig Macdonald added a comment -

          Thanks, committed for the next release.

          Show
          craigm Craig Macdonald added a comment - Thanks, committed for the next release.
          Hide
          ahMarrone Agustin Marrone added a comment -

          Yes Craig, it's Universidad Nacional de Luján, a public university from Argentina.

          Than you!

          Show
          ahMarrone Agustin Marrone added a comment - Yes Craig, it's Universidad Nacional de Luján, a public university from Argentina. Than you!
          Hide
          craigm Craig Macdonald added a comment -

          I agree, this is a (trivial) bug. As results cache are not used by default, I have dropped the priority. We will fix for the next release. Is this your affiliation: Universidad Nacional de Luján ? As we will credit you for finding the bug in the next release.

          Show
          craigm Craig Macdonald added a comment - I agree, this is a (trivial) bug. As results cache are not used by default, I have dropped the priority. We will fix for the next release. Is this your affiliation: Universidad Nacional de Luján ? As we will credit you for finding the bug in the next release.
          Hide
          ahMarrone Agustin Marrone added a comment -

          Line number is 666, not 670 .

          Show
          ahMarrone Agustin Marrone added a comment - Line number is 666, not 670 .

            People

            • Assignee:
              craigm Craig Macdonald
              Reporter:
              ahMarrone Agustin Marrone
            • Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: