-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
[sql lab] preserve schema through visualize flow #4742
Conversation
apache#4696 got tangled into refactoring views out of views/core.py and onto views/sql_lab.py This is the same PR without the refactoring.
Codecov Report
@@ Coverage Diff @@
## master #4742 +/- ##
=========================================
+ Coverage 72.38% 72.58% +0.2%
=========================================
Files 205 205
Lines 15374 15376 +2
Branches 1182 1183 +1
=========================================
+ Hits 11128 11161 +33
+ Misses 4243 4212 -31
Partials 3 3
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(y)
* [sql lab] preserve schema through visualize flow apache#4696 got tangled into refactoring views out of views/core.py and onto views/sql_lab.py This is the same PR without the refactoring. * Fix lint
* [sql lab] preserve schema through visualize flow apache#4696 got tangled into refactoring views out of views/core.py and onto views/sql_lab.py This is the same PR without the refactoring. * Fix lint
* [sql lab] preserve schema through visualize flow apache#4696 got tangled into refactoring views out of views/core.py and onto views/sql_lab.py This is the same PR without the refactoring. * Fix lint
* [sql lab] preserve schema through visualize flow apache#4696 got tangled into refactoring views out of views/core.py and onto views/sql_lab.py This is the same PR without the refactoring. * Fix lint
#4696 got tangled
into refactoring views out of views/core.py and onto views/sql_lab.py
This is the same PR without the refactoring.