Release 20190531b #39
Merged
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
) (apache#7518) * [WIP] Live query validation, where supported This builds on apache#7422 to build check-as-you-type sql query validation in Sql Lab. This closes apache#6707 too. It adds a (debounced) call to the validate_sql_json API endpoint with the querytext, and on Lyft infra is able to return feedback to the user (end to end) in $TBD seconds. At present feedback is provided only through the "annotations" mechanism build in to ACE, although I'd be open to adding full text elsewhere on the page if there's interest. * fix: Unbreak lints and tests
…#7517) (apache#7519) This change makes the query progress bar only show whole number percentage changes, instead of numbers like 12.13168276%.
* Making Talisman configurable * Fixing double quotes * Fixing flake8 * Removing default
* Validate start/end when scheduling queries * Use chrono instead of Sugar
* Show scheduled queries * Remove column * Secure views * Add import * Fix unit tests * Reuse existing db connection from view * Remove unnecessary import
* feat: add header tooltip (apache#7531)
flask-talisman was enabled recently and while it may be virtuous in some cases, it seems to break things out of the box. Locally and in dev mode, upon my first redirect it sends to HTTPS and things it crashes. I think it should be opt-in, maybe we can recommend turning this on in production in the docs?
* resolved conflict * Docs updated re: Anaconda/certifi issue re apache#7373 * Removed --console-log "not working" note * A note about Anaconda virtualenvs * Make anaconda comment fit on page * Added README to docker directory * Added install doc reference to master copy of contrib/docker/README.md * merged master, removed mysqlclient * Removed mysql dependency, Anaconda and --console-log references * Add cypress install command to cypress test instructions * Fixed cypress instructions re: port 8081 * Removed anaconda reference, runserver references * Remove anaconda reference * Added back a self-contained version of mysqlclient to dev requirements * Added ASF license to docker README.md
* [security] New, deprecate merge_perm, FAB method is fixed * [style] Fix, flakes * [tests] Fix, change merge_perm to add_permission_view_menu * [security] Fix, maintain merge_perm for compatibility * [security] New, deprecation warning on merge_perm method * [style] Fix, flake8 C812
* Creating withVerification HOC * Updating to use componentDidMount and componentDidUpdate and adding propTypes * Adding tests to withVerification * Adding documentation to withVerification
* Better message for scheduling queries * Only allow scheduling after success * Ask for query name and description * Use CSS instead of <br />
* Fix for polling queries * Revert changes due to apache#7575
* Add link to scheduled pipeline * Split utils into separate file * Fix unit test * Fix separator recursion
* Move schema name handling in table names from frontend to backend * Rename all_schema_names to get_all_schema_names * Fix js errors * Fix additional js linting errors * Refactor datasource getters and fix linting errors * Update js unit tests * Add python unit test for get_table_names method * Add python unit test for get_table_names method * Fix js linting error
When the same metric is used in a World Map panel for both bubble size and an axis (either X or Y), it currently breaks the rendering. The change in behavior was introduced in: apache@71e0c07#diff-f451672348fc6071e8d627778bdc4e96L1730 The use of .iloc() is not needed anymore since the code that used to duplicate the metric is not there anymore. Should fix apache#7006
* Add Auto option to point radius to make it work as expected since the example using this didn't work because of this (and also as the tooltip says the auto option should be available) * Remove trailing space
* Bump python libs, address insecure releases Using https://requires.io/github/mistercrunch/superset/requirements/?branch=apache_release_improv to identify insecure, old releases we're using and bumping. * redis <3.0 * fix new flakes
* Add support for Apache Drill * Updated Docs * Removed Extraneous Functions * Removed Extraneous Functions * Final Mods * Fixed Unit Test Error * Fixed Epoch Conversion Functions
* Add native sqla component for time expressions * Add unit tests and remove old tests * Remove redundant _grains_dict method * Clarify time_grain logic * Add docstrings and typing * Fix flake8 errors * Add missing typings * Rename to TimestampExpression * Remove redundant tests * Fix broken reference to db.database_name due to refactor
* Add relative start param for time filters * Fix typo and add types to parse_human_datetime * Add relative start/end to query_object * Fix linting error
* Show expanded columns in gray * Remove payload mocking * Remove empty line * Safety fallback if expanded_columns is not in the payload * Fix typo
* feat: rough check in for Presto rows and arrays * fix: presto arrays * fix: return selected and expanded columns * fix: add helper methods and unit tests * fix: only allow exploration of selected columns * fix: address Beto's comments and add more unit tests
khtruong
approved these changes
May 31, 2019
|
💨 |
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.
Merge
apache/masterintolyft/lyft-master.