Uploaded image for project: 'OASIS Open Data Protocol (OData) TC'
  1. OASIS Open Data Protocol (OData) TC
  2. ODATA-1201

Improve the syntax of aggregation exceptions with "from"

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: V4.0_CSD03
    • Fix Version/s: V4.0_CSD04
    • Component/s: Data Aggregation
    • Labels:
      None
    • Environment:

      Applied

    • Proposal:
      Hide

      Change the current syntax in section 3.1.4 to

      aggregateExpression
        from groupableProperty_1 with aggregationMethod_1 [ with aggregationMethod_1 ]
        ...
        from groupableProperty_n with aggregationMethod_n [ with aggregationMethod_n ]
        as alias
      
      Show
      Change the current syntax in section 3.1.4 to aggregateExpression from groupableProperty_1 with aggregationMethod_1 [ with aggregationMethod_1 ] ... from groupableProperty_n with aggregationMethod_n [ with aggregationMethod_n ] as alias
    • Resolution:
      Show
      https://github.com/oasis-tcs/odata-abnf/pull/10 , already merged

      Description

      Current syntax:

       

      aggregateExpression    (1)
        as alias             (3)
        from groupableProperty_1 with aggregationMethod_1 [ with aggregationMethod_1 ] (2) 
        ... 
        from groupableProperty_n with aggregationMethod_n [ with aggregationMethod_n ] 
      

      It first takes (1) the standard aggregate expression followed by (3) the alias for the final aggregated value followed by (2) "from" clauses for exceptions. Since there is a certain processing order defined, which is 1, 2, 3, the syntax would express the intention more clearly, if it would be (1) (2) (3)

       

        Attachments

          Activity

          Hide
          mikep Michael Pizzo (Inactive) added a comment -

          Resolved as proposed 2018-7-26

          Show
          mikep Michael Pizzo (Inactive) added a comment - Resolved as proposed 2018-7-26

            People

            • Assignee:
              Unassigned
              Reporter:
              gerald.krause1 Gerald Krause
            • Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: