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

Correct usage of $count

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: V4.0_CS02
    • Fix Version/s: V4.0_CSD04
    • Component/s: Data Aggregation
    • Labels:
      None
    • Environment:

      Applied

    • Proposal:
      Hide

      Update request in example 57:

      GET ~/Products?$apply=groupby((Name),aggregate(Sales/$count with sum as SalesCount))

      Rewrite section on aggregate, last item in list on "virtual property $count": replace it with "$count, which is a shortcut for 1 with sum"

      Show
      Update request in example 57: GET ~/Products?$apply=groupby((Name),aggregate(Sales/$count with sum as SalesCount)) Rewrite section on aggregate, last item in list on "virtual property $count": replace it with "$count, which is a shortcut for 1 with sum"
    • Resolution:
      Hide

      https://github.com/oasis-tcs/odata-abnf/pull/12

      Show
      https://github.com/oasis-tcs/odata-abnf/pull/12

      Description

      Section 3.1 introduces $count as one of several options for aggregate expressions.

      Example 57 applies $count relative to a collection-valued navigation property, which is a different use case that is backed by the commonExpr ABNF rule to address the count of a collection (see URL spec, section 4.8). Since section 3.1 requires commonExpr to be augmented with an aggregation method, such a method must be added to the request in the example.

      The current aggregation ABNF contains a rule for $count with a path prefix, which needs to be cleaned up accordingly.

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: