Merge apache de6d963 #60
Closed
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
* Improve birth example dashboard * Fix tests * pylint
* Adding support for filter box in annotations * Fixing tox lint errors * fixing lint errors
* Fix: Chain var declaration and webpack invocation * enh: Add cross-env to make package.json work cross-platform * Fix: Update lock-file for new package cross-env * Chores: Update lock-file for new package cross-env
* bump a set of test-related JS libs Mostly upgrading jest as I was trying to tackle issues related to wrong source-maps reporting in jest. This addresses it * Bumping more things * lint * using core-js/stable * pointing to the right place
* Move more stuff behind ff * Fix unit tests * Fix more tests * Fix lint
…e found (apache#8037) * fix: handle case where result exists but query cannot be found * changed error code to 404 for query not found
* dashboard filter indicators * add/fix unit tests
* Add typing to db_engine_specs * Add more type annotations and docstrings * Add docstrings and typing to sql_parse and db_engine_specs * Refine select_star * Fix execute and add more docstrings * Revert kwargs change from execute * Remove redundant or * Align view and table getter schema types * Fix return type of latest_partition * Remove some typing from presto * Improve docstring for __extract_from_token
* add new chart plugins * fix: pin aphrodite version * fix: package * fix: unit test dependencies * fix: memory
Turns out since we use `git archive`, there's no need for `git clean` which was pretty disruptive.
…pache#7569) * Separate vis-specific controls from centralized controls * Remove comment * Update state's vizType when switching to a new visualization. This fixes the renderTrigger bug. * Rename functions for better readability * Fix lint issues * Fix nits * Fix vscode rename mistake
* Fix HistogramViz bug. * Fix ci black error.
* wip * Finish PR * Remove annoying snap * Fix lint * Fix js tests
* [logging] add query id to logs * add query_id to hive and presto logging
* removed database deps from reqs-dev.txt and typo. * Revert requirements-dev.txt to limit scope
* Add LogConfigurator, and add logging for CSV export * tweak for PR feedback * fix linter issue
* Add check for calls to cache_key_wrapper to avoid unavoidable compilation of query * Add fetch_values_predicate to check * Only check relevant attributes * Address nit
…ache#8115) * feat: bump superset-ui dependencies to v0.12 and plugins to v0.11 * fix: update transformProps * fix: update hooks * fix: pin lunar version * fix: lint * fix: revert table
…row (apache#8069) * Add support for msgpack results_backend serialization * Serialize DataFrame with PyArrow rather than JSON * Adjust dependencies, de-lint * Add tests for (de)serialization methods * Add MessagePack config info to Installation docs * Enable msgpack/arrow serialization by default * [Fix] Prevent msgpack serialization on synchronous queries * Add type annotations
* Moving towards a whitelist approach for MANIFEST.in when it comes to static resources * Tuning static exclude * Fix for fetching version string from package.json, which no longer exists * Adding package.json fallback for unit tests
Codecov Report
@@ Coverage Diff @@
## lyft-master #60 +/- ##
===============================================
+ Coverage 64.63% 73.14% +8.5%
===============================================
Files 478 116 -362
Lines 22993 12224 -10769
Branches 2503 0 -2503
===============================================
- Hits 14862 8941 -5921
+ Misses 7997 3283 -4714
+ Partials 134 0 -134
Continue to review full report at Codecov.
|
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.
CATEGORY
Choose one
SUMMARY
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
ADDITIONAL INFORMATION
REVIEWERS