-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
[load_examples] download data at runtime #7314
Conversation
When running `superset load_examples` to load example data sets, Superset used to load from the local package. This created a few issues notably around licensing (what are these datasets licensed as?) and around package size. For now, I moved the data sets here: https://github.com/apache-superset/examples-data Altered the logic to download the data from where it is stored.
7db504d
to
2a50665
Compare
Codecov Report
@@ Coverage Diff @@
## master #7314 +/- ##
==========================================
- Coverage 64.93% 64.91% -0.02%
==========================================
Files 424 424
Lines 20589 20580 -9
Branches 2281 2281
==========================================
- Hits 13369 13360 -9
Misses 7097 7097
Partials 123 123
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.
👍
* [load_examples] download data at runtime When running `superset load_examples` to load example data sets, Superset used to load from the local package. This created a few issues notably around licensing (what are these datasets licensed as?) and around package size. For now, I moved the data sets here: https://github.com/apache-superset/examples-data Altered the logic to download the data from where it is stored. * flakes (cherry picked from commit 3d08266)
* [load_examples] download data at runtime When running `superset load_examples` to load example data sets, Superset used to load from the local package. This created a few issues notably around licensing (what are these datasets licensed as?) and around package size. For now, I moved the data sets here: https://github.com/apache-superset/examples-data Altered the logic to download the data from where it is stored. * flakes (cherry picked from commit 0088895)
CATEGORY
Choose one
SUMMARY
This PR will help us push our first Apache release.
When running
superset load_examples
to load example data sets,Superset used to load from the local package. This created a few issues
notably around licensing (what are these datasets licensed as?) and
around package size.
For now, I moved the data sets here https://github.com/apache-superset/examples-data
and altered the logic to download the data from where it is stored (another Github repo).
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A
TEST PLAN
Ran
superset load_examples
and validated examples look okREVIEWERS
@kristw