[GitHub] [incubator-superset] DiggidyDave commented on a change in pull request #8085: Add LogConfigurator, and add logging for CSV export

2019-08-27 Thread GitBox
DiggidyDave commented on a change in pull request #8085: Add LogConfigurator, and add logging for CSV export URL: https://github.com/apache/incubator-superset/pull/8085#discussion_r318273900 ## File path: superset/views/core.py ## @@ -2685,7 +2685,17 @@ def csv(self,

[GitHub] [incubator-superset] DiggidyDave commented on a change in pull request #8085: Add LogConfigurator, and add logging for CSV export

2019-08-21 Thread GitBox
DiggidyDave commented on a change in pull request #8085: Add LogConfigurator, and add logging for CSV export URL: https://github.com/apache/incubator-superset/pull/8085#discussion_r316467927 ## File path: superset/utils/logging_configurator.py ## @@ -0,0 +1,72 @@ +#

[GitHub] [incubator-superset] DiggidyDave commented on a change in pull request #8085: Add LogConfigurator, and add logging for CSV export

2019-08-21 Thread GitBox
DiggidyDave commented on a change in pull request #8085: Add LogConfigurator, and add logging for CSV export URL: https://github.com/apache/incubator-superset/pull/8085#discussion_r316467018 ## File path: tests/logging_configurator_tests.py ## @@ -0,0 +1,55 @@ +#