Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-207

Rename Afterburner plugins to CatalogCalculation plugins

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      A new plugin system type was introduced to handle plugins which only operated on a catalog, after all "measurement" plugins had been successfully run. At the time the term afterburner was used as a way to describe a system that was optionally run after other measurements. However in the processing pipeline there may be many other steps after a catalog has been created and measurements have been made. As such I propose we rename the catalog level afterburner plugin system to CatalogCalculation plugin system to better clarify its purpose and to leave the term 'afterburner' free to be used in the context of something that may be run outside the context of normal processing flow.

        Attachments

          Issue Links

            Activity

            Hide
            jsick Jonathan Sick added a comment -

            +1

            Show
            jsick Jonathan Sick added a comment - +1
            Hide
            rowen Russell Owen added a comment -

            +1. I find the current name opaque.

            Show
            rowen Russell Owen added a comment - +1. I find the current name opaque.
            Hide
            ktl Kian-Tat Lim added a comment -

            OK by me. Naming is important, but this is one I don't want to get in the middle of

            Show
            ktl Kian-Tat Lim added a comment - OK by me. Naming is important, but this is one I don't want to get in the middle of
            Hide
            Parejkoj John Parejko added a comment -

            +1, and as a bonus I now know what Afterburner was supposed to mean.

            Show
            Parejkoj John Parejko added a comment - +1, and as a bonus I now know what Afterburner was supposed to mean.
            Hide
            krughoff Simon Krughoff added a comment -

            +1

            Show
            krughoff Simon Krughoff added a comment - +1

              People

              Assignee:
              nlust Nate Lust
              Reporter:
              nlust Nate Lust
              Watchers:
              Jim Bosch, John Parejko, John Swinbank, Jonathan Sick, Kian-Tat Lim, Nate Lust, Paul Price, Robert Lupton, Russell Owen, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.