[GitHub] IanWE commented on issue #4575: Remove Permission from View Error

2018-03-13 Thread GitBox
IanWE commented on issue #4575: Remove Permission from View Error URL: https://github.com/apache/incubator-superset/issues/4575#issuecomment-372922177 @leehwmj-junta Thank you This is an automated message from the Apache Git

[GitHub] IanWE closed issue #4575: Remove Permission from View Error

2018-03-13 Thread GitBox
IanWE closed issue #4575: Remove Permission from View Error URL: https://github.com/apache/incubator-superset/issues/4575 This is an automated message from the Apache Git Service. To respond to the message, please log on GitH

[GitHub] yzfranklin opened a new issue #4612: In the “SQL Lab”, the “Results” section shows “Pending” constantly

2018-03-13 Thread GitBox
yzfranklin opened a new issue #4612: In the “SQL Lab”, the “Results” section shows “Pending” constantly URL: https://github.com/apache/incubator-superset/issues/4612 Following the instructions of "superset document", I deployed the "superset" to my Linux server. Everything seems ok. Ho

[GitHub] leehwmj-junta commented on issue #4575: Remove Permission from View Error

2018-03-13 Thread GitBox
leehwmj-junta commented on issue #4575: Remove Permission from View Error URL: https://github.com/apache/incubator-superset/issues/4575#issuecomment-372920872 I had got same logs on terminal. My superset version is 0.22 So, to solve this problem, I had deleted ~/.superset/superset

[GitHub] sunyongmofang closed issue #4502: How to use two filter_box to control two slice, and not to be interfered on the same dashboard?(divide and rule)

2018-03-13 Thread GitBox
sunyongmofang closed issue #4502: How to use two filter_box to control two slice, and not to be interfered on the same dashboard?(divide and rule) URL: https://github.com/apache/incubator-superset/issues/4502 This is an auto

[GitHub] NaukhaizAhmad opened a new issue #4611: [Slice] 'Visualization Type' should display the drop down (listing all the visualization types) rather than displaying an editable string field.

2018-03-13 Thread GitBox
NaukhaizAhmad opened a new issue #4611: [Slice] 'Visualization Type' should display the drop down (listing all the visualization types) rather than displaying an editable string field. URL: https://github.com/apache/incubator-superset/issues/4611 ### Superset version 0.22.1

[GitHub] mistercrunch commented on issue #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
mistercrunch commented on issue #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609#issuecomment-372895890 Wouldn't be against them having their own tab, though that might be a bit much --

[GitHub] mistercrunch commented on issue #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
mistercrunch commented on issue #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609#issuecomment-372895777 LGTM This is an automated message from the Ap

[GitHub] mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts

2018-03-13 Thread GitBox
mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts URL: https://github.com/apache/incubator-superset/pull/4610#issuecomment-372895371 Also means the overlay won't show up at all even when changing the data, @betodealmeida any thoughts? -

[GitHub] mistercrunch commented on a change in pull request #4604: Show "Range Filter" by default

2018-03-13 Thread GitBox
mistercrunch commented on a change in pull request #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604#discussion_r174345221 ## File path: superset/assets/visualizations/nvd3_vis.js ## @@ -158,9 +159,10 @@ function nvd3Vis(sli

[GitHub] codecov-io commented on issue #4604: Show "Range Filter" by default

2018-03-13 Thread GitBox
codecov-io commented on issue #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604#issuecomment-372526668 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4604?src=pr&el=h1) Report > Merging [#4604](https://codecov.io/gh/apa

[GitHub] codecov-io commented on issue #4604: Show "Range Filter" by default

2018-03-13 Thread GitBox
codecov-io commented on issue #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604#issuecomment-372526668 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4604?src=pr&el=h1) Report > Merging [#4604](https://codecov.io/gh/apa

[GitHub] betodealmeida commented on issue #4604: Show "Range Filter" by default

2018-03-13 Thread GitBox
betodealmeida commented on issue #4604: Show "Range Filter" by default URL: https://github.com/apache/incubator-superset/pull/4604#issuecomment-372876554 ![brush](https://user-images.githubusercontent.com/1534870/37378923-6ec9dd40-26ee-11e8-8504-a8d8d10eb411.gif) -

[GitHub] GabeLoins commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts

2018-03-13 Thread GitBox
GabeLoins commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts URL: https://github.com/apache/incubator-superset/pull/4610#issuecomment-372874331 That was part of the confusion. We should still merge this PR, or the overlay will appear incorrectly when deck

[GitHub] codecov-io commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts

2018-03-13 Thread GitBox
codecov-io commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts URL: https://github.com/apache/incubator-superset/pull/4610#issuecomment-372872875 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4610?src=pr&el=h1) Report > Merging [#46

[GitHub] mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts

2018-03-13 Thread GitBox
mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts URL: https://github.com/apache/incubator-superset/pull/4610#issuecomment-372869409 After discussion, there was confusion around the fact that `autozoom` was overwriting the `viewport` setting

[GitHub] mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts

2018-03-13 Thread GitBox
mistercrunch commented on issue #4610: [Explore] removing refresh chart overlay on deck_gl charts URL: https://github.com/apache/incubator-superset/pull/4610#issuecomment-372869409 After discussion, there was confusion around the fact that `autozoom` was overwriting the `viewport` setting

[GitHub] GabeLoins opened a new pull request #4610: [Explore] removing refresh chart overlay on deck_gl charts

2018-03-13 Thread GitBox
GabeLoins opened a new pull request #4610: [Explore] removing refresh chart overlay on deck_gl charts URL: https://github.com/apache/incubator-superset/pull/4610 Deck gl charts have this weird thing where scrolling the canvas isn't considered `renderTrigger: true` but also immediately upda

[GitHub] betodealmeida commented on a change in pull request #4572: Legend for deck.gl scatterplot

2018-03-13 Thread GitBox
betodealmeida commented on a change in pull request #4572: Legend for deck.gl scatterplot URL: https://github.com/apache/incubator-superset/pull/4572#discussion_r174322258 ## File path: superset/assets/visualizations/deckgl/layers/scatter.jsx ## @@ -68,8 +89,10 @@ functio

[GitHub] betodealmeida commented on a change in pull request #4572: Legend for deck.gl scatterplot

2018-03-13 Thread GitBox
betodealmeida commented on a change in pull request #4572: Legend for deck.gl scatterplot URL: https://github.com/apache/incubator-superset/pull/4572#discussion_r174321354 ## File path: superset/assets/visualizations/Legend.jsx ## @@ -0,0 +1,52 @@ +import React from 'reac

[GitHub] codecov-io commented on issue #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
codecov-io commented on issue #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609#issuecomment-372839264 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4609?src=pr&el=h1) Report > Merging [#4609

[GitHub] codecov-io commented on issue #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
codecov-io commented on issue #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609#issuecomment-372839264 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4609?src=pr&el=h1) Report > Merging [#4609

[GitHub] williaster commented on issue #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
williaster commented on issue #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609#issuecomment-372836901 thanks for updating this 🙌 LGTM, only possible suggestion would be making `data` a variable constant instead for

[GitHub] williaster commented on issue #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
williaster commented on issue #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609#issuecomment-372836901 thanks for updating this 🙌 LGTM, only possible suggestion would be making `data` a variable constant instead for

[GitHub] williaster commented on issue #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
williaster commented on issue #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609#issuecomment-372836901 thanks for updating this 🙌 LGTM, only possible suggestion would be making `data` a variable constant instead for

[GitHub] williaster closed pull request #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
williaster closed pull request #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] graceguo-supercat commented on issue #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
graceguo-supercat commented on issue #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607#issuecomment-372830359 LGTM! This is an automated message from the Apache

[GitHub] graceguo-supercat commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
graceguo-supercat commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607#discussion_r174284347 ## File path: superset/views/core.py ## @@ -1122,10 +1123,10 @@ def generate_json

[GitHub] GabeLoins opened a new pull request #4609: [Explore] overriding Annotations and Layers' control tab

2018-03-13 Thread GitBox
GabeLoins opened a new pull request #4609: [Explore] overriding Annotations and Layers' control tab URL: https://github.com/apache/incubator-superset/pull/4609 as reported by @williaster- the annotations and layers control should live in the Data tab and not the Style tab, despite its rend

[GitHub] graceguo-supercat commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
graceguo-supercat commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607#discussion_r174284347 ## File path: superset/views/core.py ## @@ -1122,10 +1123,10 @@ def generate_json

[GitHub] jeffreythewang commented on issue #4005: [BUGFIX]: JavaScripts max int is 2^53 - 1, longs are bigger

2018-03-13 Thread GitBox
jeffreythewang commented on issue #4005: [BUGFIX]: JavaScripts max int is 2^53 - 1, longs are bigger URL: https://github.com/apache/incubator-superset/pull/4005#issuecomment-372789596 Any updates on this besides a rebase (and a possible test fix)? -

[GitHub] jeffreythewang commented on issue #4005: [BUGFIX]: JavaScripts max int is 2^53 - 1, longs are bigger

2018-03-13 Thread GitBox
jeffreythewang commented on issue #4005: [BUGFIX]: JavaScripts max int is 2^53 - 1, longs are bigger URL: https://github.com/apache/incubator-superset/pull/4005#issuecomment-372789596 Any updates on this besides a rebase? Th

[GitHub] fabianmenges commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
fabianmenges commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607#discussion_r174256060 ## File path: superset/views/core.py ## @@ -1122,10 +1123,10 @@ def generate_json(self

[GitHub] john-bodley closed pull request #4590: Removing escape_sql so we dont double escape

2018-03-13 Thread GitBox
john-bodley closed pull request #4590: Removing escape_sql so we dont double escape URL: https://github.com/apache/incubator-superset/pull/4590 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As t

[GitHub] williaster commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
williaster commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607#discussion_r174253852 ## File path: superset/views/core.py ## @@ -1122,10 +1123,10 @@ def generate_json(self,

[GitHub] codecov-io commented on issue #4608: Set filter_select_enabled default to True for Druid

2018-03-13 Thread GitBox
codecov-io commented on issue #4608: Set filter_select_enabled default to True for Druid URL: https://github.com/apache/incubator-superset/pull/4608#issuecomment-372785763 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4608?src=pr&el=h1) Report > Merging [#4608](htt

[GitHub] codecov-io commented on issue #4608: Set filter_select_enabled default to True for Druid

2018-03-13 Thread GitBox
codecov-io commented on issue #4608: Set filter_select_enabled default to True for Druid URL: https://github.com/apache/incubator-superset/pull/4608#issuecomment-372785763 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4608?src=pr&el=h1) Report > Merging [#4608](htt

[GitHub] fabianmenges commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
fabianmenges commented on a change in pull request #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607#discussion_r174252687 ## File path: superset/views/core.py ## @@ -1122,10 +1123,10 @@ def generate_json(self

[GitHub] codecov-io commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text

2018-03-13 Thread GitBox
codecov-io commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text URL: https://github.com/apache/incubator-superset/pull/4194#issuecomment-372785385 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4194?src=pr&el=h1) Report > Merging [#4194](htt

[GitHub] codecov-io commented on issue #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
codecov-io commented on issue #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607#issuecomment-372784539 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4607?src=pr&el=h1) Report > Merging [#4607](https://c

[GitHub] john-bodley closed pull request #4579: Removing [dashboard] and [slice] titles to show name

2018-03-13 Thread GitBox
john-bodley closed pull request #4579: Removing [dashboard] and [slice] titles to show name URL: https://github.com/apache/incubator-superset/pull/4579 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenanc

[GitHub] mistercrunch opened a new pull request #4608: Set filter_select_enabled default to True for Druid

2018-03-13 Thread GitBox
mistercrunch opened a new pull request #4608: Set filter_select_enabled default to True for Druid URL: https://github.com/apache/incubator-superset/pull/4608 This is an automated message from the Apache Git Service. To respo

[GitHub] williaster commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text

2018-03-13 Thread GitBox
williaster commented on issue #4194: [bug fixes] annotations <> x domains, zeros in text URL: https://github.com/apache/incubator-superset/pull/4194#issuecomment-372780297 fixed annotations separately in #4607, updated this PR / should be good to go so PTAL ? --

[GitHub] williaster opened a new pull request #4607: [slice_json] pass slice id to get_form_data()

2018-03-13 Thread GitBox
williaster opened a new pull request #4607: [slice_json] pass slice id to get_form_data() URL: https://github.com/apache/incubator-superset/pull/4607 @mistercrunch @graceguo-supercat @fabianmenges [This line](https://github.com/apache/incubator-superset/commit/83524f97d7fcf3c8c86ef

[GitHub] codecov-io commented on issue #4606: Use 'count' as the default metric when available

2018-03-13 Thread GitBox
codecov-io commented on issue #4606: Use 'count' as the default metric when available URL: https://github.com/apache/incubator-superset/pull/4606#issuecomment-372776199 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4606?src=pr&el=h1) Report > Merging [#4606](https:

[GitHub] john-bodley commented on issue #4590: Removing escape_sql so we dont double escape

2018-03-13 Thread GitBox
john-bodley commented on issue #4590: Removing escape_sql so we dont double escape URL: https://github.com/apache/incubator-superset/pull/4590#issuecomment-372774659 @michellethomas upon further investigation this regression doesn't seem to be related to the auto-escape feature but rather

[GitHub] john-bodley commented on issue #4590: Removing escape_sql so we dont double escape

2018-03-13 Thread GitBox
john-bodley commented on issue #4590: Removing escape_sql so we dont double escape URL: https://github.com/apache/incubator-superset/pull/4590#issuecomment-372774659 For context this SQLAlchemy [commit](https://github.com/zzzeek/sqlalchemy/commit/2392ae1900f112c44ed966783d1dedfb88f13353)

[GitHub] codecov-io commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
codecov-io commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1) Report > Merging [#4605](https://codecov.io/gh/apache/incu

[GitHub] john-bodley closed pull request #4573: [cache] Ensuring that the datasource UID is defined

2018-03-13 Thread GitBox
john-bodley closed pull request #4573: [cache] Ensuring that the datasource UID is defined URL: https://github.com/apache/incubator-superset/pull/4573 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance

[GitHub] michellethomas commented on issue #4573: [cache] Ensuring that the datasource UID is defined

2018-03-13 Thread GitBox
michellethomas commented on issue #4573: [cache] Ensuring that the datasource UID is defined URL: https://github.com/apache/incubator-superset/pull/4573#issuecomment-372759162 lgtm This is an automated message from the Apach

[GitHub] john-bodley closed pull request #4602: [druid] Adding cluster filter for refresh

2018-03-13 Thread GitBox
john-bodley closed pull request #4602: [druid] Adding cluster filter for refresh URL: https://github.com/apache/incubator-superset/pull/4602 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] michellethomas commented on issue #4602: [druid] Adding cluster filter for refresh

2018-03-13 Thread GitBox
michellethomas commented on issue #4602: [druid] Adding cluster filter for refresh URL: https://github.com/apache/incubator-superset/pull/4602#issuecomment-372758640 lgtm This is an automated message from the Apache Git Serv

[GitHub] mistercrunch opened a new pull request #4606: Use 'count' as the default metric when available

2018-03-13 Thread GitBox
mistercrunch opened a new pull request #4606: Use 'count' as the default metric when available URL: https://github.com/apache/incubator-superset/pull/4606 Count is a much better default than the current behavior which is to use whatever the first metric in the list happens to be.

[GitHub] john-bodley commented on issue #4169: Dashboard Filtering Issue across different data sources

2018-03-13 Thread GitBox
john-bodley commented on issue #4169: Dashboard Filtering Issue across different data sources URL: https://github.com/apache/incubator-superset/issues/4169#issuecomment-372751566 @kkalyan I believe this change is a side effect of the caching restructure (cc @mistercrunch) however one coul

[GitHub] john-bodley commented on issue #4169: Dashboard Filtering Issue across different data sources

2018-03-13 Thread GitBox
john-bodley commented on issue #4169: Dashboard Filtering Issue across different data sources URL: https://github.com/apache/incubator-superset/issues/4169#issuecomment-372751566 @kkalyan I believe this change is a side effect of the caching restructure (cc @mistercrunch) however one coul

[GitHub] john-bodley commented on issue #4169: Dashboard Filtering Issue across different data sources

2018-03-13 Thread GitBox
john-bodley commented on issue #4169: Dashboard Filtering Issue across different data sources URL: https://github.com/apache/incubator-superset/issues/4169#issuecomment-372751566 @kkalayn I believe this change is a side effect of the caching restructure (cc @mistercrunch) however one coul

[GitHub] rumbin commented on issue #2814: [explore] Filter Select not working for int and bool

2018-03-13 Thread GitBox
rumbin commented on issue #2814: [explore] Filter Select not working for int and bool URL: https://github.com/apache/incubator-superset/issues/2814#issuecomment-372742376 @ZamoraAndres check out #3007 please and whether the Boolean representation in Impala is false/true (and not 0/1 like

[GitHub] rumbin commented on issue #2814: [explore] Filter Select not working for int and bool

2018-03-13 Thread GitBox
rumbin commented on issue #2814: [explore] Filter Select not working for int and bool URL: https://github.com/apache/incubator-superset/issues/2814#issuecomment-372739842 I guess, we need to file a new issue for the bool-specific part, as this one is closed already. @ZamoraAndres could

[GitHub] ZamoraAndres commented on issue #2814: [explore] Filter Select not working for int and bool

2018-03-13 Thread GitBox
ZamoraAndres commented on issue #2814: [explore] Filter Select not working for int and bool URL: https://github.com/apache/incubator-superset/issues/2814#issuecomment-372731509 Hello, I experience the same behavior connecting the tool to an Impala table. The only workaround that I found

[GitHub] ZamoraAndres commented on issue #2814: [explore] Filter Select not working for int and bool

2018-03-13 Thread GitBox
ZamoraAndres commented on issue #2814: [explore] Filter Select not working for int and bool URL: https://github.com/apache/incubator-superset/issues/2814#issuecomment-372731509 Hello, I experience the same behavior connecting the tool to an Impala table. The only workaround that I Found

[GitHub] codecov-io commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
codecov-io commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1) Report > Merging [#4605](https://codecov.io/gh/apache/incu

[GitHub] codecov-io commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
codecov-io commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1) Report > Merging [#4605](https://codecov.io/gh/apache/incu

[GitHub] mistercrunch closed pull request #4581: Making timeseries_limit not required for phase 2

2018-03-13 Thread GitBox
mistercrunch closed pull request #4581: Making timeseries_limit not required for phase 2 URL: https://github.com/apache/incubator-superset/pull/4581 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] mistercrunch closed issue #4208: Druid run_query executes phase 1 query with no limit

2018-03-13 Thread GitBox
mistercrunch closed issue #4208: Druid run_query executes phase 1 query with no limit URL: https://github.com/apache/incubator-superset/issues/4208 This is an automated message from the Apache Git Service. To respond to the

[GitHub] codecov-io commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
codecov-io commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372724014 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4605?src=pr&el=h1) Report > Merging [#4605](https://codecov.io/gh/apache/incu

[GitHub] xrmx commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
xrmx commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372719111 You added the messages.json to pt_br and please rename the ptpt dir to pt with git mv T

[GitHub] maver1ck commented on issue #4543: Setting up compression using flask-compress

2018-03-13 Thread GitBox
maver1ck commented on issue #4543: Setting up compression using flask-compress URL: https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921 @mistercrunch How to disable this behaviour? I have nginx in front of Superset for ssl and compression. For sure it

[GitHub] maver1ck commented on issue #4543: Setting up compression using flask-compress

2018-03-13 Thread GitBox
maver1ck commented on issue #4543: Setting up compression using flask-compress URL: https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921 @mistercrunch How to disable this option ? I have nginx in front of Superset for ssl and compression. For sure it s

[GitHub] maver1ck commented on issue #4543: Setting up compression using flask-compress

2018-03-13 Thread GitBox
maver1ck commented on issue #4543: Setting up compression using flask-compress URL: https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921 How to disable this option ? I have nginx in front of Superset for ssl and compression. For sure it should be configura

[GitHub] maver1ck commented on issue #4543: Setting up compression using flask-compress

2018-03-13 Thread GitBox
maver1ck commented on issue #4543: Setting up compression using flask-compress URL: https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921 How to disable this option ? I have nginx in front of Superset for ssl and compression. For sure it should be configura

[GitHub] maver1ck commented on issue #4543: Setting up compression using flask-compress

2018-03-13 Thread GitBox
maver1ck commented on issue #4543: Setting up compression using flask-compress URL: https://github.com/apache/incubator-superset/pull/4543#issuecomment-372709921 How to disable this option ? I have nginx in front of Superset for ssl and compression. -

[GitHub] nunohelibeires commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
nunohelibeires commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372708237 @xrmx honestly it should be just pt but as I noticed there was a ptbr version, I changed to ptpt. added the missing json file -

[GitHub] xrmx commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
xrmx commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372654679 Are you sure portuguese is ptpt and not just pt or pt_PT? This is an automated message f

[GitHub] xrmx commented on issue #4605: Portuguese Translation

2018-03-13 Thread GitBox
xrmx commented on issue #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605#issuecomment-372654822 Also you are missing the messages.json for the frontend This is an automated message fro

[GitHub] maver1ck commented on issue #4428: Proper error handling in Hive Queries

2018-03-13 Thread GitBox
maver1ck commented on issue #4428: Proper error handling in Hive Queries URL: https://github.com/apache/incubator-superset/pull/4428#issuecomment-372642657 ping @mistercrunch This is an automated message from the Apache Git

[GitHub] nunohelibeires opened a new pull request #4605: Portuguese Translation

2018-03-13 Thread GitBox
nunohelibeires opened a new pull request #4605: Portuguese Translation URL: https://github.com/apache/incubator-superset/pull/4605 complete. my contribution to superset, with the help of @jcoelho-pt This is an autom

[GitHub] scott-retair commented on issue #3613: Time Series Table: group-by keys must be strings

2018-03-13 Thread GitBox
scott-retair commented on issue #3613: Time Series Table: group-by keys must be strings URL: https://github.com/apache/incubator-superset/issues/3613#issuecomment-372617052 I met the same issue This is an automated message f