[GitHub] williaster commented on issue #4905: [wip][missing values] Removing replacing missing values

2018-04-30 Thread GitBox
williaster commented on issue #4905: [wip][missing values] Removing replacing missing values URL: https://github.com/apache/incubator-superset/pull/4905#issuecomment-385473973 I think that option 1 above is the best^ it's super misleading to have nulls replaced with 0s, and if you

[GitHub] williaster commented on issue #4905: [wip][missing values] Removing replacing missing values

2018-04-29 Thread GitBox
williaster commented on issue #4905: [wip][missing values] Removing replacing missing values URL: https://github.com/apache/incubator-superset/pull/4905#issuecomment-385288293  why was filling `null`s with `0`s ever introduced?