Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-7127

remove epsilon-based testing from lucene/spatial

Details

    • Test
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 6.1, 7.0
    • None
    • None
    • New

    Description

      Currently, the random tests here allow a TOLERANCE and will fail if the error exceeds. But this is not fun to debug! It also keeps the door wide open for bugs to creep in.

      Alternatively, we can rework the tests like we did for sandbox/ points. This means the test is aware of the index-time quantization and so it can demand exact answers.

      Its more difficult at first, because even floating point error can cause a failure. It requires us to maybe work through corner cases/rework optimizations. If any epsilons must be added, they can be added to the optimizations themselves (e.g. bounding box) instead of the user's result.

      Attachments

        1. LUCENE-7127.patch
          65 kB
          Robert Muir
        2. LUCENE-7127.patch
          43 kB
          Robert Muir
        3. LUCENE-7127.patch
          43 kB
          Robert Muir
        4. LUCENE-7127.patch
          23 kB
          Robert Muir

        Activity

          People

            Unassigned Unassigned
            rcmuir Robert Muir
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment