You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by cc...@apache.org on 2018/11/30 04:58:01 UTC

[incubator-superset] branch master updated (5261d8a -> 5f7817a)

This is an automated email from the ASF dual-hosted git repository.

ccwilliams pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git.


    from 5261d8a  Fixing error when clearing big number (#6465)
     add c11e9c8  [SIP-5] Build metrics in query_object in the client - Unify the metric interface (absorb the current plain string metric for built-in metric keys into the format used by adhoc metric) - Port the logic in adhocMetric on the client and process_metrics in the backend to the new typed Metrics class - Omit hasCustomLabel and formFromData properties from the new metric interface as their value can be inferred from label and optionName - Expose from the Metrics class both metri [...]
     add a4597af  addressing PR comments
     add e06f873  Adding a comment for metrictype values
     new 5f7817a  [SIP-5] Build metrics in query_object in the client (#6423)

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 superset/assets/package.json                       |   4 +-
 .../spec/javascripts/superset-ui/Metric.test.ts    |  95 ++++++++++++++++++++
 .../superset-ui/buildQueryObject.test.ts           |  19 +++-
 superset/assets/src/query/Column.ts                |  24 +++++
 superset/assets/src/query/FormData.ts              |  21 +++--
 superset/assets/src/query/Metric.ts                | 100 +++++++++++++++++++++
 superset/assets/src/query/buildQueryObject.ts      |   3 +
 superset/common/query_object.py                    |  16 ++--
 8 files changed, 262 insertions(+), 20 deletions(-)
 create mode 100644 superset/assets/spec/javascripts/superset-ui/Metric.test.ts
 create mode 100644 superset/assets/src/query/Column.ts
 create mode 100644 superset/assets/src/query/Metric.ts


[incubator-superset] 01/01: [SIP-5] Build metrics in query_object in the client (#6423)

Posted by cc...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

ccwilliams pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git

commit 5f7817a6a3a532f4f9b9a72aabc1d9755715137d
Merge: 5261d8a e06f873
Author: Chris Williams <wi...@users.noreply.github.com>
AuthorDate: Thu Nov 29 20:57:55 2018 -0800

    [SIP-5] Build metrics in query_object in the client (#6423)
    
    * [SIP-5] Build metrics in query_object in the client
    - Unify the metric interface (absorb the current plain string metric for built-in metric keys into the format used by adhoc metric)
    - Port the logic in adhocMetric on the client and process_metrics in the backend to the new typed Metrics class
    - Omit hasCustomLabel and formFromData properties from the new metric interface as their value can be inferred from label and optionName
    - Expose from the Metrics class both metrics and their labels as public methods to match the all_metrics and metric_labels fields in the backend code
    - Provide defaut values for filters, metrics and groupby in the backend
    
    * addressing PR comments
    
    * Adding a comment for metrictype values

 superset/assets/package.json                       |   4 +-
 .../spec/javascripts/superset-ui/Metric.test.ts    |  95 ++++++++++++++++++++
 .../superset-ui/buildQueryObject.test.ts           |  19 +++-
 superset/assets/src/query/Column.ts                |  24 +++++
 superset/assets/src/query/FormData.ts              |  21 +++--
 superset/assets/src/query/Metric.ts                | 100 +++++++++++++++++++++
 superset/assets/src/query/buildQueryObject.ts      |   3 +
 superset/common/query_object.py                    |  16 ++--
 8 files changed, 262 insertions(+), 20 deletions(-)