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

Refactor all Stream Evaluators in solrj.io.eval to simplify them

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 7.1, master (8.0)
    • Component/s: None
    • Security Level: Public (Default Security Level. Issues are Public)
    • Labels:
      None

      Description

      As Stream Evaluators have been evolving we are seeing a need to better handle differing types of data within evaluators. For example, allowing some to evaluate over individual values or arrays of values, like

      sin(a)
      sin(a,b,c,d)
      sin([a,b,c,d])
      

      The current structure of Evaluators makes this difficult and repetitive work.

      Also, the hierarchy of classes behind evaluators can be confusing for developers creating new evaluators. For example, when to use a ComplexEvaluator vs a BooleanEvaluator.

      A full refactoring of these classes will greatly enhance the usability and future evolution of evaluators.

        Attachments

        1. SOLR-11283.patch
          455 kB
          Dennis Gove
        2. SOLR-11283.patch
          433 kB
          Dennis Gove

          Activity

            People

            • Assignee:
              dpgove Dennis Gove
              Reporter:
              dpgove Dennis Gove
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: