You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by jo...@apache.org on 2019/03/28 17:08:13 UTC

[incubator-superset] branch master updated: [migration] Fixing issue with fb13d49b72f9 downgrade (#7145)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 36a6fad  [migration] Fixing issue with fb13d49b72f9 downgrade (#7145)
36a6fad is described below

commit 36a6fade908510429f41d2c6831c44faed565e88
Author: John Bodley <45...@users.noreply.github.com>
AuthorDate: Thu Mar 28 10:07:55 2019 -0700

    [migration] Fixing issue with fb13d49b72f9 downgrade (#7145)
    
    * [migration] Fixing issue with fb13d49b72f9 downgrade
    
    This PR fixes an issue with the downgrade step of migration fb13d49b72f9 which wrongfully labeled the field `metrics` rather than `metric`.
    
    to: @graceguo-supercat @michellethomas @mistercrunch
    
    * Update fb13d49b72f9_better_filters.py
---
 superset/migrations/versions/fb13d49b72f9_better_filters.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset/migrations/versions/fb13d49b72f9_better_filters.py b/superset/migrations/versions/fb13d49b72f9_better_filters.py
index cac42c3..9ea7880 100644
--- a/superset/migrations/versions/fb13d49b72f9_better_filters.py
+++ b/superset/migrations/versions/fb13d49b72f9_better_filters.py
@@ -94,7 +94,7 @@ def downgrade():
             flts = params.get('filter_configs')
             if not flts:
                 continue
-            params['metrics'] = [flts[0].get('metric')]
+            params['metric'] = flts[0].get('metric')
             params['groupby'] = [o.get('column') for o in flts]
             slc.params = json.dumps(params, sort_keys=True)
         except Exception as e: