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.

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

Merge apache 1a02d615 #66

Merged
merged 20 commits into from
Sep 10, 2019
Merged

Merge apache 1a02d615 #66

merged 20 commits into from
Sep 10, 2019

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

Erik Ritter and others added 18 commits September 3, 2019 14:13
* [Codemod] Rename react unsafe lifecycles

* Fix lint errors

* [Codemod] Rename react unsafe lifecycles

* Fix lint errors
* [sqllab] add retries for stop_query

* use backoff for retries

* address PR comments

* import statement order
…#8117)

* Fix to werkzeug proxy; expose additional configuration items

* Forced to all x-forwarded configurations ON; black done

* added comments related to x_port after testing

* Updated UPDATING.md

* Removed accidental notebook; added *.ipynb to gitignore

* Delete Untitled-checkpoint.ipynb
"Pop"这里应该翻译成"弹出",而不是"流行"。
* Fix select_star table quoting bug

* Add unit test for fully qualified names in select_star

* Rename main_db to db

* Rename test function
* Fix regression in time grain addons

* Revert privatization of time_grain_functions

* Fix test

* Rename variable

* Fix test

* Fix typing error

* Refactor and add tests

* Add TODO
* Bump flask-appbuilder to 2.1.12

To pick up this fix dpgaspar/Flask-AppBuilder#1095 which addresses apache#7739

* Bump FAB again to 2.1.13
* prevent 'main' database connection creation

* fix tests

* removing get_main_database

* Kill get_main_database

* Point to examples tables
* Bump FAB to 2.1.13

* [SQLLab] dummy commit

* [SQLLab] remove dummy commit

* [SQLLab] test

* [SQLLab] bump back to 13
…#8204)

* change log from csv_export to data_export; fix db name in log

* add exported_format field to data_export event log
@DiggidyDave DiggidyDave merged commit d0ef9c0 into lyft-master Sep 10, 2019
@DiggidyDave DiggidyDave deleted the merge-apache-1a02d615 branch November 1, 2019 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants