Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Merge apache 6dc760a to lyft-master #59

Closed
wants to merge 28 commits into from
Closed

Merge apache 6dc760a to lyft-master #59

wants to merge 28 commits into from

Conversation

DiggidyDave
Copy link

CATEGORY

Choose one

  • Bug Fix
  • Enhancement (new features, refinement)
  • Refactor
  • Add tests
  • Build / Development Environment
  • Documentation

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

REVIEWERS

mistercrunch and others added 28 commits August 18, 2019 22:36
@mistercrunch
Improve birth example dashboard (apache#8042)
4e7ea3f 
* Improve birth example dashboard

* Fix tests

* pylint
@KKcorps @kristw
Adding support for filter box in annotations (apache#8057)
ae16e69 
* Adding support for filter box in annotations

* Fixing tox lint errors

* fixing lint errors
@fzzylogic @kristw
Make package.json cross platform (apache#8035)
c97a71b 
* 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
[Explore view] Fix extra data fetch when user clicks Run Query (apach…
319c1df 
…e#8070)
@mistercrunch
bump a set of test-related JS libs (apache#8051)
d872511 
* 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
@betodealmeida
Move more Presto nested types behind feature flag (apache#8075)
aae9b56 
* Move more stuff behind ff

* Fix unit tests

* Fix more tests

* Fix lint
@DiggidyDave @mistercrunch
remove idna req from setup.py (apache#8067)
995b96f
@serenajiang @mistercrunch
fix: handle case where result exists but corresponding query cannot b…
638fbb1 
…e found (apache#8037)

* fix: handle case where result exists but query cannot be found

* changed error code to 404 for query not found
@mistercrunch
Remove gevent installation as a separate layer (apache#8078)
0fd7364
[Feature] Dashboard filter indicators (apache#7908)
40776bd 
* dashboard filter indicators

* add/fix unit tests
@villebro
Add docstrings and typing to db_engine_specs and sql_parse (apache#8058)
fb51632 
* 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
@kristw
feat: replace table and box plot vis with new plugins (apache#8083)
0d76fc7 
* add new chart plugins

* fix: pin aphrodite version

* fix: package

* fix: unit test dependencies

* fix: memory
@mistercrunch
CHANGELOG for 0.30...0.34 (apache#8089)
763af85
@mistercrunch
fix: minor release/license related issues (apache#8087)
279307c
@serenajiang @mistercrunch
[fix] checks for stopped queries (apache#8097)
e6956f8
@friendzis @mistercrunch
Move Darwin-specific JS libs to optional (apache#8093)
fd119c1
@mistercrunch
docs: remove unused release instruction (apache#8100)
610b35a 
Turns out since we use `git archive`, there's no need for `git clean`
which was pretty disruptive.
@FelixCodes @kristw
refactor: separate vis-specific controls from centralized controls (a…
6ca3e34 
…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 sqla query cache keys function (apache#8105)
01aa17c
Fix cache set kwargs (apache#8106)
ee75d03
@kuckjwi0928 @villebro
Fixed Histogram visualization bug. (apache#8077)
7ac1a29 
* Fix HistogramViz bug.

* Fix ci black error.
@betodealmeida
Allow resizing width of SQL Lab left bar / editor (apache#8099)
ed3360b 
* wip

* Finish PR

* Remove annoying snap

* Fix lint

* Fix js tests
@michellethomas
[Hive Engine Spec] Fix latest partition logic (apache#8098)
5d8da6a
@serenajiang @michellethomas
[logging] add query id to SQL Lab logs (apache#8104)
6244498 
* [logging] add query id to logs

* add query_id to hive and presto logging
@MarcusSorealheis @mistercrunch
removed database deps from reqs-dev.txt and typo. (apache#8111)
b1269c3 
* removed database deps from reqs-dev.txt and typo.

* Revert requirements-dev.txt to limit scope
@rusackas @mistercrunch
adding spaces missing from string concatenations (apache#8126)
2085e89
@DiggidyDave @betodealmeida
Add LogConfigurator, and add logging for CSV export (apache#8085)
6dc760a 
* Add LogConfigurator, and add logging for CSV export

* tweak for PR feedback

* fix linter issue
Merge apache/master@6dc760a
927f453
@hongbosong
Copy link

👍

@DiggidyDave DiggidyDave deleted the merge-apache-6dc760a branch August 28, 2019 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@khtruong khtruong khtruong approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.