expose results #17
Closed
expose results #17
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… in druid (apache#5162) * added disabled message for custom sql tab on druid datasources * Update AdhocFilterEditPopover.jsx Fixing indentation. * Update AdhocMetricEditPopover.jsx Fixing indent. * Update AdhocFilterEditPopover.jsx Fixing indentation. * Update AdhocMetricEditPopover.jsx Fixing indentation.
This commit will try to dockerize superset in local development
environment.
The basic design is:
- Enable superset, redis and postgres service instead of using sqlite,
just want to simulate production environment settings
- Use environment variables to config various app settings. It's easy to
run and config superset to any environment if we use environment than
traditional config files
- For local development environment, we just expose postgres and redis
to local host machine thus you can connect local port via `psql` or
`redis-cli`
- Wrap start up command in a standard `docker-entrypoint.sh`, and use
`tail -f /dev/null` combined with manually `superset runserver -d` to
make sure that code error didn't cause the container to fail.
- Use volumes to share code between host and container, thus you can use
your favourite tools to modify code and your code will run in
containerized environment
- Use volumes to persistent postgres and redis data, and also
`node_modules` data.
- If we don't cache `node_modules` in docker volume, then every time
run docker build, the `node_modules` directory, will is about 500 MB
large, will be sent to docker daemon, and make the build quite slow.
- Wrap initialization commands to a single script `docker-init.sh`
After this dockerize setup, any developers who want to contribute to
superset, just follow three easy steps:
```
git clone https://github.com/apache/incubator-superset/
cd incubator-superset
cp contrib/docker/{docker-build.sh,docker-compose.yml,docker-entrypoint.sh,docker-init.sh,Dockerfile} .
cp contrib/docker/superset_config.py superset/
bash -x docker-build.sh
docker-compose up -d
docker-compose exec superset bash
bash docker-init.sh
```
* Fix bullet chart rendering * Fix code style errors
…pache#5183) * fetch datasources from broker endpoint when refresh new datasources * remove get_base_coordinator_url as out of use * add broker_endpoint in get_test_cluster_obj
[bugfix] form_data - fix empty metrics
Never did this when I renamed the folder. `npm run cover` currently doesn't generate a proper coverage report, this fixes it
…l_sort_by Adding the MetricsControl to the timeseries_limit_metric field
…ately pass error message separately
* [webpack] setup lazy loading for all visualizations * [lazy-load] push renderVis function to <Chart /> state * no mapbox token * [lazy loading] use native webpack import func to fix chunk names, add babel-plugin-syntax-dynamic-import, fix rebase bug. * fix geojson import, undefined t, and fix async css bug * [lazy load] actually add babel-plugin-syntax-dynamic-import * [webpack] working dev version of webpack v4 * [webpack 4] fix url issues, use mini-css-extract-plugin and webpack-assets-manifest plugins * [webpack 4] use splitchunks for all files, update templates to multi-file entrypoints * [webpack 4] multiple theme entry files for markup vis css, don't uglify mapbox * [webpack 4] lint python manifest changes, update yarn lock. * [webpack 4] fix tests with babel-plugin-dynamic-import-node * [webpack 4] only use 'dynamic-import-node' plugin in tests, update <Chart /> vis promise when vis type changes * [webpack 4] clean up package.json and yarn.lock after rebase * [webpack 4] lint? * [webpack 4] lint for real
* [explore] fix autocomplete on verbose names Currently when searching for metrics or groupbys, the autocomplete search functionality only matches based on the metric_name, though in some cases the verbose_name is displayed and disregarded for search purposes. Also another issue is that not all pre-defined metrics show up in the drop down which is confusing. Users may have simple metrics for which they setup a nice verbose name and/or description and expect to see those in the dropdown. This PR addresses it for metric and column-related dropdowns. * Add unit test
…5219) * Revert "[explore] fix autocomplete on verbose names (apache#5204)" This reverts commit d5ebc43. * Revert "[webpack] setup lazy loading for all visualizations (apache#4727)" This reverts commit de0aaf4.
* Downgrade celery and kombu * Remove kombu from requirements.txt * Pin kombu at 4.1.0 * Bump celery to 4.2.0
The check_ownership was failing when Alpha user was saving a datasource from the datasource editor.
I noticed that `secondary_metric` was required in the chart that users it, namely sunburst, parallel coordinates, and world_map. I set the control as allowing null, and found related bugs in the process: * parallel coordinates did not have support for the new MetricControl * added color scheme support for parallel coordinates * added option to set row_limit on these viz types * sunburst to support numeric columns (number would show as blank)
…apache#5641) * Field names in big query can contain only alphanumeric and underscore * bad quote * better place for mutating labels * lint * bug fix thanks to mistercrunch * lint * lint again
* Fix unicode error message response * Update
* Refactor and add props to Force Directed Graph * update label and enable renderTrigger
* create reactize function * update function signature * adjust word cloud controls * Fix broken word cloud logic * Repair and refactor word cloud * remove file * Add PropTypes * change module.exports to export default * change order * update proptypes
* Fixes to the play slider * Address comments * Make CSS rules more strict * Fix CSS and improve code
* Fix legend position * Add categories and play slider to arc viz * New functionality to arc viz
* [viz flow] detect TIMESTAMP, transition to line chart * Refactor is_date
* Make brush send events * Make sure object is date * Send event only on brushend
* [bugfix] geohash lat/long is reversed This allows support for reversing geohashes. Note that the default option was the wrong way. * addressing comments * make reverse_geohash_decode a staticmethod
* Add geojson for Myanmar * Adds 'myanmar.geojson' file * Adds "iso-3166" codes
* allow user to choose color * add color control * remove rgbToHex reference * change to color picker control * add d3-color * remove module * use d3-colors * add semicolon
Fix menu on 'import dashboards' page
|
meow @hughhhh @betodealmeida |
|
Let me try to deploy this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
resultsapi endpoint for service-to-service requests.Notes