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: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: V4.0_CSD03
    • Fix Version/s: V4.0_CSD04
    • Component/s: Data Aggregation
    • Labels:
      None
    • Environment:

      Proposed

    • 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

      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

            People

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

              Dates

              • Created:
                Updated:
                Resolved: