Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[revert] Revert #5445 #7095

Closed

Conversation

john-bodley
Copy link
Member

This PR reverts #5445 due to the fallout related to potentially ill-defined metadata. As discussed here the collective decision was to revert the logic (which will be cherry-picked into v0.31) and re-remedy the entire problem prior to the v0.32 branch cut.

Note this PR "reverts" in the sense of it creates a new database migration to rollback the state of the database to something representative prior to #5445.

to: @graceguo-supercat @michellethomas @mistercrunch @xtinec

@john-bodley john-bodley changed the title John bodley revert 5445 [revert] Revert #5445 Mar 22, 2019
@john-bodley john-bodley deleted the john-bodley--revert-5445 branch March 22, 2019 22:03
@codecov-io
Copy link

codecov-io commented Mar 22, 2019

Codecov Report

Merging #7095 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7095      +/-   ##
==========================================
- Coverage   64.57%   64.56%   -0.01%     
==========================================
  Files         422      422              
  Lines       20576    20574       -2     
  Branches     2251     2251              
==========================================
- Hits        13286    13284       -2     
  Misses       7168     7168              
  Partials      122      122
Impacted Files Coverage Δ
superset/__init__.py 74.04% <ø> (-0.4%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33d67da...66eee14. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants