Uploaded image for project: 'JBRULES'
  1. JBRULES
  2. JBRULES-3363

Add support to inline constraints on multi-function accumulate

This issue belongs to an archived project. You can view it, but you can't modify it. Learn more

    XMLWordPrintable

Details

    Description

      FROM MARK:
      ===========
      Edson,

      While I don't want to change to acc/for keyword quite yet, until we are sure what drastic changes we want to make to syntax. I think we can evolve accumulate()
      1) make the first ", " optionall [,;]. Where the documented new separate is ";"
      2) functions are still "," separated
      3) allow an optional second ";" after this boolean expression, an eval without the eval wrapper.

      3. can be done now with a separate eval(....) after the accumulate. But I think good to encapsulate the intent of the guard within the accumulate itself, it's also another opportunity to remove the "eval" keyword for a common use case. For now we'll just rewrite the accumulate to place an eval() CE after the acc. It's a small win, which I think will make the DRL look nicer, especially for CEP use cases.

      ============
      FROM EDSON:
      ============
      Mark,

      (1) and (2) are ok. Regarding (3), we can embed the expression in the accumulate node itself. No need for a separate node. And if we use ; as the separator, we don't need the eval() keyword.

      Attachments

        Activity

          People

            etirelli@redhat.com Edson Tirelli
            etirelli@redhat.com Edson Tirelli
            Archiver:
            rhn-support-ceverson Clark Everson

            Dates

              Created:
              Updated:
              Resolved:
              Archived:

              PagerDuty