[GitHub] mistercrunch commented on issue #4366: Add filters to dashboard in single slice without filter box view - interactive filters
mistercrunch commented on issue #4366: Add filters to dashboard in single slice without filter box view - interactive filters URL: https://github.com/apache/incubator-superset/issues/4366#issuecomment-385869786 It's planned for the next version of the dashboard app to have "scoped filters" that apply to sections / tabs / overall and perhaps individual charts. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #945: Google BigQuery Support
mistercrunch commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385869242 I wouldn't recommend running on Windows since Superset isn't tested against it. You will hit problems like this one. Even if you'd fix this issue there could be regressions in the future since no one uses Windows to run Superset in production. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] swamy16 commented on issue #945: Google BigQuery Support
swamy16 commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385862029 @darylerwin @mistercrunch sorry it is a typo from my end. The actual error is SIGALRM as mentioned by @mxmzdlv Module 'signal has no attribute 'SIGALRM' Basically I am able to add tables to superset and am able to visualize them in dashboards (Be it tables from bigquery, or the default main db "world bank dataset" ) but I am not able to query them (even the main db tables from superset sql lab)  As you can see the highlighted part in the above image "tsukuyomi" is a dataset in my bigquery and "2017_agency_mapping" is a table. I am able to create a slice view using tables from my bigquery.  As you can see in the above image I am querying from the main db of the default world bank. But it is not able to query the same in sql labs sql editor. @mxmzdlv mentioned that this is happening due to SIGALRM not working in windows. Please let em know if there is a way to solve this error in windows. I tried replicate what ASPePeX mentioned in the link you have given Unity-Technologies/ml-agents#7 But I am not able to locate the file in my computer. /python/unityagents/environment.py I am also not sure if this is the right approach This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] swamy16 commented on issue #945: Google BigQuery Support
swamy16 commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385862029 @darylerwin @mistercrunch sorry it is a typo from my end. The actual error is SIGALRM as mentioned by @mxmzdlv Module 'signal has no attribute 'SIGALRM' Basically I am able to add tables to superset and am able to visualize them in dashboards (Be it tables from bigquery, or the default main db "world bank dataset" ) but I am not able to query them (even the main db tables from superset sql lab)  As you can see the highlighted part in the above image "tsukuyomi" is a dataset in my bigquery and "2017_agency_mapping" is a table. I am able to create a slice view using tables from my bigquery.  As you can see in the above image I am querying from the main db of the default world bank. But it is not able to query the same in sql labs sql editor. @mxmzdlv mentioned that this is happening due to SIGALRM not working in windows. Please let em know if there is a way to solve this error in windows. I tried replicate what ASPePeX mentioned in the link you have given Unity-Technologies/ml-agents#7 But I am not able to locate the file in my computer. /python/unityagents/environment.py This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch closed pull request #4917: Add doc entry for BigQuery support
mistercrunch closed pull request #4917: Add doc entry for BigQuery support URL: https://github.com/apache/incubator-superset/pull/4917 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 is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/docs/installation.rst b/docs/installation.rst index ca9cf3f595..06afbc637a 100644 --- a/docs/installation.rst +++ b/docs/installation.rst @@ -320,6 +320,8 @@ Here's a list of some of the recommended packages. +---+-+-+ | Kylin| ``pip install kylinpy`` | ``kylin://`` | +---+-+-+ +| BigQuery | ``pip install pybigquery`` | ``bigquery://`` | ++---+-+-+ Note that many other database are supported, the main criteria being the existence of a functional SqlAlchemy dialect and Python driver. Googling This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch closed issue #945: Google BigQuery Support
mistercrunch closed issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations
williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385803221 > I'm guessing the load time improvements have to do with the fact that some of the work can be started upfront while the bundles are downloading Yes, and this is great from a user perspective because it doesn't block page render like it currently does. I think this will have the effect of making users feel like the page is loading _much_ faster. > I'm thinking for visualizations that use React, they shouldn't have to import it and should somehow pick it up from global, same goes for jquery and perhaps a set of other things. React [is not that large](https://bundlephobia.com/result?p=react@16.3.2) (even smaller if we could actually upgrade to `16.0.0`) and I can't tell if you mean literal globals (`window.jquery`) but we should move away from that pattern (and `jquery` in general, it's just syntactic sugar for JS and ideally we'd be using fetch not ajax) rather than embrace it. It seems like the proper way to pull out "globally shared" code is using the webpack `CommonChunksPlugin`, if chunks share code it will pull them into a new chunk. I used this to create a separate `chunk` for all shared deck.gl code, and I also added it in a way that it _should_ pull out all shared code into a separate chunk if it is imported by >=2 modules ... BUT this didn't appear to have any effect so the vis chunks must not really share that much code 路♀️ This is the new final webpack output (note the deckgl layer size decreases, and the new larger `deckgl.fb9e56a3bf69cd1ce139.chunk.js`): ``` Version: webpack 3.11.0 Time: 46756ms Asset Size ChunksChunk Names deckgl/layers/grid.a8beccbf9f5b13e97f98.chunk.js8.74 kB 32 [emitted] deckgl/layers/grid f4769f9bdb7466be65088239c12046d1.eot20.1 kB [emitted] fa2772327f55d8198301fdb8bcfc8158.woff23.4 kB [emitted] e18bbf611f2a2e43afc071aa2f4e1512.ttf45.4 kB [emitted] 89889688147bd7575d6327160d64e760.svg 109 kB [emitted] nvd3_vis.7b1466ac696cd8d30467.chunk.js 6.8 MB 0 [emitted] [big] nvd3_vis sankey.0da04cf564e3607b0313.chunk.js 268 kB 1 [emitted] [big] sankey paired_ttest.d9654f93c94179c43de4.chunk.js 304 kB 2 [emitted] [big] paired_ttest pivot_table.752772a0c72696357184.chunk.js2.08 MB 3 [emitted] [big] pivot_table parallel_coordinates.0d0923b8adf4d520192a.chunk.js 204 kB 4 [emitted] parallel_coordinates EventFlow.04abc2ccfab65027f920.chunk.js2.57 MB 5 [emitted] [big] EventFlow big_number.39d33798fefe5e02632e.chunk.js41.8 kB 6 [emitted] big_number markup.0ee56d948bd6cc1366fb.chunk.js14.7 kB 7 [emitted] markup filter_box.f76d5a45eff1c943db92.chunk.js 12 MB 8 [emitted] [big] filter_box time_table.49446835d78c8f19c439.chunk.js1.51 MB 9 [emitted] [big] time_table mapbox.b927c5a82390d9078f99.chunk.js4.43 MB 10 [emitted] [big] mapbox partition.408cf680da8bcce8ad58.chunk.js 159 kB 11 [emitted] partition deckgl/multi.bced534d9159ab18934d.chunk.js 512 kB 12, 13, 14, 29, 30, 31, 32, 33, 34 [emitted] [big] deckgl/multi deckgl/layers/scatter.ce1c712e5113e386fc6a.chunk.js 429 kB 13 [emitted] [big] deckgl/layers/scatter deckgl/layers/screengrid.3619502e1991df96e5d5.chunk.js 400 kB 14 [emitted] [big] deckgl/layers/screengrid heatmap.a587cf5f653617b1305a.chunk.js 108 kB 15 [emitted] heatmap table.48aed4719820c3e79b32.chunk.js2.09 MB 16 [emitted] [big] table cal_heatmap.ed95b9fdabe40cb02ef1.chunk.js 348 kB 17 [emitted] [big] cal_heatmap
[GitHub] graceguo-supercat commented on issue #4919: [Profile]Gamma, Alpha user can't see their profile page
graceguo-supercat commented on issue #4919: [Profile]Gamma, Alpha user can't see their profile page URL: https://github.com/apache/incubator-superset/issues/4919#issuecomment-385798104 @mistercrunch Alpha and Gamma user can't access their own profile page. is this logic in FAB code base? Thank you! This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] graceguo-supercat commented on issue #4919: [Profile]Gamma, Alpha user can't see their profile page
graceguo-supercat commented on issue #4919: [Profile]Gamma, Alpha user can't see their profile page URL: https://github.com/apache/incubator-superset/issues/4919#issuecomment-385798104 @mistercrunch Alpha and Gamma user can't see their profile page. is this logic in FAB code base? Thank you! This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] graceguo-supercat opened a new issue #4919: [Profile]Gamma, Alpha user can't see their profile page
graceguo-supercat opened a new issue #4919: [Profile]Gamma, Alpha user can't see their profile page URL: https://github.com/apache/incubator-superset/issues/4919 Make sure these boxes are checked before submitting your issue - thank you! - [x ] I have checked the superset logs for python stacktraces and included it here as text if any - [x ] I have reproduced the issue with at least the latest released version of superset - [x ] I have checked the issue tracker for the same issue and I haven't found one similar ### Superset version latest ### Expected results visit user's profile content ### Actual results page got 302 redirect. ### Steps to reproduce clicks on Profile icon from Superset header https://user-images.githubusercontent.com/27990562/39494701-46b5db18-4d4c-11e8-923a-8cc526406d79.png;> This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #1413: Added average metric AVG() to default metrics
mistercrunch commented on issue #1413: Added average metric AVG() to default metrics URL: https://github.com/apache/incubator-superset/pull/1413#issuecomment-385794518 You have to create post-aggregations for averages on Druid. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] graceguo-supercat commented on a change in pull request #4760: URL shortner for dashboards
graceguo-supercat commented on a change in pull request #4760: URL shortner for
dashboards
URL:
https://github.com/apache/incubator-superset/pull/4760#discussion_r185330381
##
File path:
superset/assets/javascripts/explore/components/ExploreViewContainer.jsx
##
@@ -157,27 +151,6 @@ class ExploreViewContainer extends React.Component {
}
}
- addHistory({ isReplace = false, title }) {
Review comment:
i feel it's safe to keep query state in browser history.
- we still need to keep url scheme backward compatible right? Even we
display url in browser like /r/123, when user request with old scheme, we
should handle it correctly.
- after query state added into browser history, there is no real long url
request happened unless user refreshed browser or someone accessed the shared
url. So i guess the rate of old url scheme is pretty low.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
[GitHub] graceguo-supercat commented on issue #4833: [sqllab] Help sqllab forget query history
graceguo-supercat commented on issue #4833: [sqllab] Help sqllab forget query history URL: https://github.com/apache/incubator-superset/pull/4833#issuecomment-385783066 current auto query refresh component work like this: - in client-side, from all queries in local storage, it find out - status in `'running', 'started', 'pending', 'fetching'`, and - started in < 6 hours - if has any queries, polling with latest updated timestamp - if there is no such queries, stop polling. - in server-side, for every poll request, we get all queries - by user_id, and - changed after updated timestamp - if some query went wrong, and query status kept as `running` but never got to updated after 6 hours, when client-side polling with a newer timestamp (< 6 hours), this query will not have a chance to be set as `timeout` state. But it won't cause client-side keep polling because now client-side added timeout threshold. this PR LGTM. but will not merge until @mistercrunch feel comfortable. Thank you! This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch closed issue #4915: Exploring BQ table throws query error as date is incorrectly formatted
mistercrunch closed issue #4915: Exploring BQ table throws query error as date is incorrectly formatted URL: https://github.com/apache/incubator-superset/issues/4915 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch closed pull request #4918: Fix typos from linting
mistercrunch closed pull request #4918: Fix typos from linting
URL: https://github.com/apache/incubator-superset/pull/4918
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 is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/superset/db_engine_specs.py b/superset/db_engine_specs.py
index e95b4d3928..ee8f632da4 100644
--- a/superset/db_engine_specs.py
+++ b/superset/db_engine_specs.py
@@ -1260,7 +1260,7 @@ class BQEngineSpec(BaseEngineSpec):
def convert_dttm(cls, target_type, dttm):
tt = target_type.upper()
if tt == 'DATE':
-return "{}'".format(dttm.strftime('%Y-%m-%d'))
+return "'{}'".format(dttm.strftime('%Y-%m-%d'))
return "'{}'".format(dttm.strftime('%Y-%m-%d %H:%M:%S'))
@classmethod
@@ -1291,7 +1291,7 @@ class ImpalaEngineSpec(BaseEngineSpec):
def convert_dttm(cls, target_type, dttm):
tt = target_type.upper()
if tt == 'DATE':
-return "{}'".format(dttm.strftime('%Y-%m-%d'))
+return "'{}'".format(dttm.strftime('%Y-%m-%d'))
return "'{}'".format(dttm.strftime('%Y-%m-%d %H:%M:%S'))
@classmethod
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
[GitHub] codecov-io commented on issue #4760: URL shortner for dashboards
codecov-io commented on issue #4760: URL shortner for dashboards URL: https://github.com/apache/incubator-superset/pull/4760#issuecomment-378721411 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4760?src=pr=h1) Report > Merging [#4760](https://codecov.io/gh/apache/incubator-superset/pull/4760?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/510ae84b3b3c3880393b84ba1db6d9979ecb63e0?src=pr=desc) will **decrease** coverage by `0.01%`. > The diff coverage is `100%`. [](https://codecov.io/gh/apache/incubator-superset/pull/4760?src=pr=tree) ```diff @@Coverage Diff @@ ## master#4760 +/- ## == - Coverage 77.13% 77.12% -0.02% == Files 44 44 Lines8542 8538 -4 == - Hits 6589 6585 -4 Misses 1953 1953 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/4760?src=pr=tree) | Coverage Δ | | |---|---|---| | [superset/views/core.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvdmlld3MvY29yZS5weQ==) | `74.66% <100%> (-0.02%)` | :arrow_down: | | [superset/utils.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvdXRpbHMucHk=) | `87.85% <0%> (-0.22%)` | :arrow_down: | | [superset/views/sql\_lab.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvdmlld3Mvc3FsX2xhYi5weQ==) | `93.33% <0%> (ø)` | :arrow_up: | | [superset/cache\_util.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvY2FjaGVfdXRpbC5weQ==) | `54.16% <0%> (ø)` | :arrow_up: | | [superset/connectors/druid/models.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvY29ubmVjdG9ycy9kcnVpZC9tb2RlbHMucHk=) | `81.16% <0%> (ø)` | :arrow_up: | | [superset/models/annotations.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvbW9kZWxzL2Fubm90YXRpb25zLnB5) | `92.85% <0%> (ø)` | :arrow_up: | | [superset/jinja\_context.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvamluamFfY29udGV4dC5weQ==) | `73.33% <0%> (ø)` | :arrow_up: | | [superset/data/\_\_init\_\_.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvZGF0YS9fX2luaXRfXy5weQ==) | `100% <0%> (ø)` | :arrow_up: | | [superset/models/sql\_lab.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvbW9kZWxzL3NxbF9sYWIucHk=) | `98.59% <0%> (ø)` | :arrow_up: | | [superset/connectors/sqla/views.py](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree#diff-c3VwZXJzZXQvY29ubmVjdG9ycy9zcWxhL3ZpZXdzLnB5) | `70.47% <0%> (ø)` | :arrow_up: | | ... and [29 more](https://codecov.io/gh/apache/incubator-superset/pull/4760/diff?src=pr=tree-more) | | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4760?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4760?src=pr=footer). Last update [510ae84...d3e1e76](https://codecov.io/gh/apache/incubator-superset/pull/4760?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] LanDeQuHuXi commented on issue #4366: Add filters to dashboard in single slice without filter box view - interactive filters
LanDeQuHuXi commented on issue #4366: Add filters to dashboard in single slice without filter box view - interactive filters URL: https://github.com/apache/incubator-superset/issues/4366#issuecomment-385780560 +1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch closed pull request #4911: Fix for week_start_sunday and week_ending_saturday
mistercrunch closed pull request #4911: Fix for week_start_sunday and
week_ending_saturday
URL: https://github.com/apache/incubator-superset/pull/4911
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 is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/superset/assets/spec/javascripts/modules/time_spec.js
b/superset/assets/spec/javascripts/modules/time_spec.js
new file mode 100644
index 00..59dab8effe
--- /dev/null
+++ b/superset/assets/spec/javascripts/modules/time_spec.js
@@ -0,0 +1,58 @@
+import { it, describe } from 'mocha';
+import { expect } from 'chai';
+import { getPlaySliderParams } from '../../../src/modules/time';
+
+describe('getPlaySliderParams', () => {
+ it('is a function', () => {
+assert.isFunction(getPlaySliderParams);
+ });
+
+ it('handles durations', () => {
+const timestamps = [
+ new Date('2018-01-01'),
+ new Date('2018-01-02'),
+ new Date('2018-01-03'),
+ new Date('2018-01-04'),
+ new Date('2018-01-05'),
+ new Date('2018-01-06'),
+ new Date('2018-01-07'),
+ new Date('2018-01-08'),
+ new Date('2018-01-09'),
+ new Date('2018-01-10'),
+].map(d => d.getTime());
+const { start, end, step, values, disabled } =
getPlaySliderParams(timestamps, 'P2D');
+expect(new Date(start)).to.eql(new Date('2018-01-01'));
+expect(new Date(end)).to.eql(new Date('2018-01-11'));
+expect(step).to.equal(2 * 24 * 60 * 60 * 1000);
+expect(values.map(v => new Date(v))).to.eql([
+ new Date('2018-01-01'),
+ new Date('2018-01-03'),
+]);
+expect(disabled).to.equal(false);
+ });
+
+ it('handles intervals', () => {
+const timestamps = [
+ new Date('2018-01-01'),
+ new Date('2018-01-02'),
+ new Date('2018-01-03'),
+ new Date('2018-01-04'),
+ new Date('2018-01-05'),
+ new Date('2018-01-06'),
+ new Date('2018-01-07'),
+ new Date('2018-01-08'),
+ new Date('2018-01-09'),
+ new Date('2018-01-10'),
+].map(d => d.getTime());
+// 1970-01-03 was a Saturday
+const { start, end, step, values, disabled } =
getPlaySliderParams(timestamps, 'P1W/1970-01-03T00:00:00Z');
+expect(new Date(start)).to.eql(new Date('2017-12-30')); // Saturday
+expect(new Date(end)).to.eql(new Date('2018-01-13')); // Saturday
+expect(step).to.equal(7 * 24 * 60 * 60 * 1000);
+expect(values.map(v => new Date(v))).to.eql([
+ new Date('2017-12-30'),
+ new Date('2018-01-06'),
+]);
+expect(disabled).to.equal(false);
+ });
+});
diff --git a/superset/assets/src/explore/controls.jsx
b/superset/assets/src/explore/controls.jsx
index ffe264b547..b3248df454 100644
--- a/superset/assets/src/explore/controls.jsx
+++ b/superset/assets/src/explore/controls.jsx
@@ -776,8 +776,8 @@ export const controls = {
['P1D', '1 day'],
['P7D', '7 days'],
['P1W', 'week'],
- ['P1W', 'week_starting_sunday'],
- ['P1W', 'week_ending_saturday'],
+ ['week_starting_sunday', 'week starting Sunday'],
+ ['week_ending_saturday', 'week ending Saturday'],
['P1M', 'month'],
],
description: t('The time granularity for the visualization. Note that you
' +
diff --git a/superset/assets/src/modules/time.js
b/superset/assets/src/modules/time.js
index 0c13dae859..7ebc4d7766 100644
--- a/superset/assets/src/modules/time.js
+++ b/superset/assets/src/modules/time.js
@@ -4,11 +4,31 @@ import parseIsoDuration from 'parse-iso-duration';
export const getPlaySliderParams = function (timestamps, timeGrain) {
let start = Math.min(...timestamps);
let end = Math.max(...timestamps);
+ let step;
- // lock start and end to the closest steps
- const step = parseIsoDuration(timeGrain);
- start -= start % step;
- end += step - end % step;
+ if (timeGrain.indexOf('/') > 0) {
+// Here, time grain is a time interval instead of a simple duration, either
+// `reference/duration` or `duration/reference`. We need to parse the
+// duration and make sure that start and end are in the right places. For
+// example, if `reference` is a Saturday and `duration` is 1 week (P1W)
+// then both start and end should be Saturdays.
+const parts = timeGrain.split('/', 2);
+let reference;
+if (parts[0].endsWith('Z')) { // ISO string
+ reference = new Date(parts[0]).getTime();
+ step = parseIsoDuration(parts[1]);
+} else {
+ reference = new Date(parts[1]).getTime();
+ step = parseIsoDuration(parts[0]);
+}
+start = reference + step * Math.floor((start - reference) / step);
+end = reference + step * (Math.floor((end - reference) / step) + 1);
+ } else {
+// lock start and end to the closest steps
+step = parseIsoDuration(timeGrain);
+start -= start % step;
+end += step - end % step;
+ }
[GitHub] timifasubaa commented on issue #4833: [sqllab] Help sqllab forget query history
timifasubaa commented on issue #4833: [sqllab] Help sqllab forget query history URL: https://github.com/apache/incubator-superset/pull/4833#issuecomment-385761026 PING Quick update on the state of the logic. What is really happening is that the frontend ceases to poll for queries older than 6 hours old. This is because on polls, it checks for all queries by that user whose state has changed after the last updated time. Meaning if a query is on pending for a very long time (> 6hours) and another query that's running at the same time changes state. We will no longer poll and also no longer receive any more updates. Is this backend code worth having? Especially since it may trick people into believing we actually timeout the queries when the query will most likely not be updated to timeout. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mohadelrezk commented on issue #1413: Added average metric AVG() to default metrics
mohadelrezk commented on issue #1413: Added average metric AVG() to default metrics URL: https://github.com/apache/incubator-superset/pull/1413#issuecomment-385751870 Hi, I am using latest version of superset (0.24) but average is not activated as a metric when I create a chart/slice from druid datasource note: it is available when I load data from csv files directly This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mohadelrezk commented on issue #1413: Added average metric AVG() to default metrics
mohadelrezk commented on issue #1413: Added average metric AVG() to default metrics URL: https://github.com/apache/incubator-superset/pull/1413#issuecomment-385751870 Hi, I am using latest version of superset (0.24) but average is not activated as a metric This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch closed pull request #4908: Replace NaN/Infinity with null
mistercrunch closed pull request #4908: Replace NaN/Infinity with null
URL: https://github.com/apache/incubator-superset/pull/4908
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 is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/superset/views/core.py b/superset/views/core.py
index 2e01e99b3a..a79ff2b4da 100755
--- a/superset/views/core.py
+++ b/superset/views/core.py
@@ -6,7 +6,6 @@
from __future__ import unicode_literals
from datetime import datetime, timedelta
-import json
import logging
import os
import re
@@ -24,6 +23,7 @@
from flask_babel import gettext as __
from flask_babel import lazy_gettext as _
import pandas as pd
+import simplejson as json
from six import text_type
import sqlalchemy as sqla
from sqlalchemy import create_engine
@@ -2327,7 +2327,8 @@ def results(self, key):
payload_json = json.loads(payload)
payload_json['data'] = payload_json['data'][:display_limit]
return json_success(
-json.dumps(payload_json, default=utils.json_iso_dttm_ser))
+json.dumps(
+payload_json, default=utils.json_iso_dttm_ser,
ignore_nan=True))
@has_access_api
@expose('/stop_query/', methods=['POST'])
@@ -2435,7 +2436,7 @@ def sql_json(self):
resp = json_success(json.dumps(
{'query': query.to_dict()}, default=utils.json_int_dttm_ser,
-allow_nan=False), status=202)
+ignore_nan=True), status=202)
session.commit()
return resp
@@ -2453,7 +2454,7 @@ def sql_json(self):
rendered_query,
return_results=True)
payload = json.dumps(
-data, default=utils.pessimistic_json_iso_dttm_ser)
+data, default=utils.pessimistic_json_iso_dttm_ser,
ignore_nan=True)
except Exception as e:
logging.exception(e)
return json_error_response('{}'.format(e))
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
[GitHub] betodealmeida commented on a change in pull request #4908: Replace NaN/Infinity with null
betodealmeida commented on a change in pull request #4908: Replace NaN/Infinity
with null
URL:
https://github.com/apache/incubator-superset/pull/4908#discussion_r185292674
##
File path: superset/views/core.py
##
@@ -2435,7 +2436,7 @@ def sql_json(self):
resp = json_success(json.dumps(
{'query': query.to_dict()}, default=utils.json_int_dttm_ser,
-allow_nan=False), status=202)
+ignore_nan=True), status=202)
Review comment:
Digging in a bit more, I found the first commit where this was introduced by
some guy called @mistercrunch:
https://github.com/apache/incubator-superset/commit/38b8db805 :-P
Initially both sync and async responses used `allow_nan=False`. Eventually
it got dropped from the sync response in
https://github.com/apache/incubator-superset/commit/269f55c29 when the
pessimistic encoder was introduced. The pessimistic encoder was not added to
the async response because the commit is trying to fix gigantic HTML error
messages, but IMHO it should've been added as well.
The problem is that the pessimistic encoder doesn't handle `NaN` and
`±Infinity` because they're floats, so the JSON encoder never calls the
`default` method on them.
I think the change above is safe — it will return _some_ result instead of
failing with a `ValueError` exception. But I think the best approach would be
using a more robust serialization to send data to the browser, like BSON.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
[GitHub] codecov-io commented on issue #4819: Visualization for multiple line charts
codecov-io commented on issue #4819: Visualization for multiple line charts URL: https://github.com/apache/incubator-superset/pull/4819#issuecomment-383682187 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4819?src=pr=h1) Report > Merging [#4819](https://codecov.io/gh/apache/incubator-superset/pull/4819?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/8c94e1f710dcdea03c3e1f04c42439f1b4127fa0?src=pr=desc) will **increase** coverage by `0.06%`. > The diff coverage is `100%`. [](https://codecov.io/gh/apache/incubator-superset/pull/4819?src=pr=tree) ```diff @@Coverage Diff@@ ## master #4819 +/- ## = + Coverage 77.14% 77.2% +0.06% = Files 44 44 Lines85398563 +24 = + Hits 65876611 +24 Misses 19521952 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/4819?src=pr=tree) | Coverage Δ | | |---|---|---| | [superset/viz.py](https://codecov.io/gh/apache/incubator-superset/pull/4819/diff?src=pr=tree#diff-c3VwZXJzZXQvdml6LnB5) | `80.1% <100%> (+0.2%)` | :arrow_up: | | [superset/cli.py](https://codecov.io/gh/apache/incubator-superset/pull/4819/diff?src=pr=tree#diff-c3VwZXJzZXQvY2xpLnB5) | `44.69% <100%> (+0.62%)` | :arrow_up: | | [superset/data/\_\_init\_\_.py](https://codecov.io/gh/apache/incubator-superset/pull/4819/diff?src=pr=tree#diff-c3VwZXJzZXQvZGF0YS9fX2luaXRfXy5weQ==) | `100% <100%> (ø)` | :arrow_up: | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4819?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4819?src=pr=footer). Last update [8c94e1f...1e58489](https://codecov.io/gh/apache/incubator-superset/pull/4819?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] codecov-io commented on issue #4911: Fix for week_start_sunday and week_ending_saturday
codecov-io commented on issue #4911: Fix for week_start_sunday and week_ending_saturday URL: https://github.com/apache/incubator-superset/pull/4911#issuecomment-385575078 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=h1) Report > Merging [#4911](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/e1d2150391ba827a64cebd0c0573aab0a82a3634?src=pr=desc) will **not change** coverage. > The diff coverage is `n/a`. [](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#4911 +/- ## === Coverage 77.14% 77.14% === Files 44 44 Lines8539 8539 === Hits 6587 6587 Misses 1952 1952 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=tree) | Coverage Δ | | |---|---|---| | [superset/db\_engine\_specs.py](https://codecov.io/gh/apache/incubator-superset/pull/4911/diff?src=pr=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzLnB5) | `52.69% <ø> (ø)` | :arrow_up: | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=footer). Last update [e1d2150...158e02e](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] darylerwin commented on issue #945: Google BigQuery Support
darylerwin commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385737552 @swamy16 is that a typo in the alarm and perhaps related to this windows signal python issue? https://github.com/Unity-Technologies/ml-agents/issues/7 Basically it is timing out? Is the dataset too big? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] codecov-io commented on issue #4911: Fix for week_start_sunday and week_ending_saturday
codecov-io commented on issue #4911: Fix for week_start_sunday and week_ending_saturday URL: https://github.com/apache/incubator-superset/pull/4911#issuecomment-385575078 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=h1) Report > Merging [#4911](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/e1d2150391ba827a64cebd0c0573aab0a82a3634?src=pr=desc) will **not change** coverage. > The diff coverage is `n/a`. [](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#4911 +/- ## === Coverage 77.14% 77.14% === Files 44 44 Lines8539 8539 === Hits 6587 6587 Misses 1952 1952 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=tree) | Coverage Δ | | |---|---|---| | [superset/db\_engine\_specs.py](https://codecov.io/gh/apache/incubator-superset/pull/4911/diff?src=pr=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzLnB5) | `52.69% <ø> (ø)` | :arrow_up: | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=footer). Last update [e1d2150...158e02e](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #3724: SECRET_KEY encrypts database secrets without warning, possibly breaking web interface if changed
mistercrunch commented on issue #3724: SECRET_KEY encrypts database secrets without warning, possibly breaking web interface if changed URL: https://github.com/apache/incubator-superset/issues/3724#issuecomment-385736855 FYI the easiest way is to use `DB_CONNECTION_MUTATOR` that you can define in your `superset_config.py`. More info here: https://github.com/apache/incubator-superset/blob/master/superset/config.py#L386 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] codecov-io commented on issue #4911: Fix for week_start_sunday and week_ending_saturday
codecov-io commented on issue #4911: Fix for week_start_sunday and week_ending_saturday URL: https://github.com/apache/incubator-superset/pull/4911#issuecomment-385575078 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=h1) Report > Merging [#4911](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/e1d2150391ba827a64cebd0c0573aab0a82a3634?src=pr=desc) will **not change** coverage. > The diff coverage is `n/a`. [](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#4911 +/- ## === Coverage 77.14% 77.14% === Files 44 44 Lines8539 8539 === Hits 6587 6587 Misses 1952 1952 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=tree) | Coverage Δ | | |---|---|---| | [superset/db\_engine\_specs.py](https://codecov.io/gh/apache/incubator-superset/pull/4911/diff?src=pr=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzLnB5) | `52.69% <ø> (ø)` | :arrow_up: | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=footer). Last update [e1d2150...158e02e](https://codecov.io/gh/apache/incubator-superset/pull/4911?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations
mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385718217 I'm worried about the way things look here without bundling visualizations together. The sum of all visualization is much more than the original `explore` bundle. A simple dashboard with a `filter_box` , nvd3, `table` and `pivot_table` will load like 24mb which is significantly more than the old explore bundle. I'm guessing the load time improvements have to do with the fact that some of the work can be started upfront while the bundles are downloading. Results may look different on a network-bound scenario. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations
mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385719915 We need to fix the fact that `filter_box` is 12mb. I'm thinking for visualizations that use `React`, they shouldn't have to import it and should somehow pick it up from `global`, same goes for `jquery` and perhaps a set of other things. We probably need some sort of `global_imports` package that all apps (explore, dashboard, profile, ...) would import and build, and that visualization would pick from without importing/bundling. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] johnsks commented on issue #4037: table view pagination
johnsks commented on issue #4037: table view pagination URL: https://github.com/apache/incubator-superset/issues/4037#issuecomment-385718766 Hello all, We are currently using 0.23.0dev, and have quite a few table views in our implementation. The lack of pagination is impacting the performance of the dashboards, which I am sure are all aware of. I have funding to get the table view pagination changes done, and a resource available. We will gladly contribute all the changes back to product. However, it will be helpful if any of you who have looked into the issue can share your thought son the changes needed. For example to table.js, viz.py etc. Your insights into how we can go about making the changes is much appreciated. Thank you in advance. John This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations
mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385718217 I'm worried about the way things look here without bundling visualizations together. The sum of all visualization is much more than the original `explore` bundle. A simple dashboard with a `filter_box` , nvd3, `table` and `pivot_table` will load like 24mb which is significantly more than the old explore bundle. I'm guessing the load time improvements have to do with the fact that some of the work can be started upfront while the bundles are downloading. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations
williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385716957 I can give deck.gl a shot if you want tho, I think the d3 bundle is almost over optimizing. That'll be harder to untangle potentially as we refactor vis libraries. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations
williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385716957 I can give deck.gl a shot if you want tho, I think the d3 bundle is almost over optimizing. That'll be harder to untangle potential totally as we refactor vis libraries. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations
williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385715788 It's not as trivial as using the same chunk name. The final name consists of chunk name and hash. I'd rather do that optimization in another PR because I think you'd have to create an index file that exports all of deckgl and async import that file. I'm not as familiar with the deckgl code organization so you all might be best suited to do that? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] codecov-io commented on issue #4918: Fix typos from linting
codecov-io commented on issue #4918: Fix typos from linting URL: https://github.com/apache/incubator-superset/pull/4918#issuecomment-385713818 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4918?src=pr=h1) Report > Merging [#4918](https://codecov.io/gh/apache/incubator-superset/pull/4918?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/8c94e1f710dcdea03c3e1f04c42439f1b4127fa0?src=pr=desc) will **not change** coverage. > The diff coverage is `0%`. [](https://codecov.io/gh/apache/incubator-superset/pull/4918?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#4918 +/- ## === Coverage 77.14% 77.14% === Files 44 44 Lines8539 8539 === Hits 6587 6587 Misses 1952 1952 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-superset/pull/4918?src=pr=tree) | Coverage Δ | | |---|---|---| | [superset/db\_engine\_specs.py](https://codecov.io/gh/apache/incubator-superset/pull/4918/diff?src=pr=tree#diff-c3VwZXJzZXQvZGJfZW5naW5lX3NwZWNzLnB5) | `52.69% <0%> (ø)` | :arrow_up: | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4918?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4918?src=pr=footer). Last update [8c94e1f...36ebd6e](https://codecov.io/gh/apache/incubator-superset/pull/4918?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] codecov-io commented on issue #4917: Add doc entry for BigQuery support
codecov-io commented on issue #4917: Add doc entry for BigQuery support URL: https://github.com/apache/incubator-superset/pull/4917#issuecomment-385710264 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4917?src=pr=h1) Report > Merging [#4917](https://codecov.io/gh/apache/incubator-superset/pull/4917?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/e1d2150391ba827a64cebd0c0573aab0a82a3634?src=pr=desc) will **not change** coverage. > The diff coverage is `n/a`. [](https://codecov.io/gh/apache/incubator-superset/pull/4917?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#4917 +/- ## === Coverage 77.14% 77.14% === Files 44 44 Lines8539 8539 === Hits 6587 6587 Misses 1952 1952 ``` -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4917?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4917?src=pr=footer). Last update [e1d2150...ad3ecd4](https://codecov.io/gh/apache/incubator-superset/pull/4917?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4819: Visualization for multiple line charts
mistercrunch commented on issue #4819: Visualization for multiple line charts URL: https://github.com/apache/incubator-superset/pull/4819#issuecomment-385708415 Cool, please fix merge conflicts, as this looks like it's ready to be merged. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4915: Exploring BQ table throws query error as date is incorrectly formatted
mistercrunch commented on issue #4915: Exploring BQ table throws query error as date is incorrectly formatted URL: https://github.com/apache/incubator-superset/issues/4915#issuecomment-385708066 Thanks for reporting this, https://github.com/apache/incubator-superset/pull/4918 addresses the issue. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch opened a new pull request #4918: Fix typos from linting
mistercrunch opened a new pull request #4918: Fix typos from linting URL: https://github.com/apache/incubator-superset/pull/4918 Caused by https://github.com/apache/incubator-superset/pull/3847 Fixes https://github.com/apache/incubator-superset/issues/4915 Greped for more of this pattern but didn't find other errors. @john-bodley please check if you can think of what went wrong and whether other similar patterns may be affected. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on a change in pull request #3847: [flake8] Resolving Q??? errors
mistercrunch commented on a change in pull request #3847: [flake8] Resolving
Q??? errors
URL:
https://github.com/apache/incubator-superset/pull/3847#discussion_r185256095
##
File path: superset/db_engine_specs.py
##
@@ -1043,22 +1043,22 @@ class BQEngineSpec(BaseEngineSpec):
engine = 'bigquery'
time_grains = (
-Grain("Time Column", _('Time Column'), "{col}"),
-Grain("second", _('second'), "TIMESTAMP_TRUNC({col}, SECOND)"),
-Grain("minute", _('minute'), "TIMESTAMP_TRUNC({col}, MINUTE)"),
-Grain("hour", _('hour'), "TIMESTAMP_TRUNC({col}, HOUR)"),
-Grain("day", _('day'), "TIMESTAMP_TRUNC({col}, DAY)"),
-Grain("week", _('week'), "TIMESTAMP_TRUNC({col}, WEEK)"),
-Grain("month", _('month'), "TIMESTAMP_TRUNC({col}, MONTH)"),
-Grain("quarter", _('quarter'), "TIMESTAMP_TRUNC({col}, QUARTER)"),
-Grain("year", _('year'), "TIMESTAMP_TRUNC({col}, YEAR)"),
+Grain('Time Column', _('Time Column'), '{col}'),
+Grain('second', _('second'), 'TIMESTAMP_TRUNC({col}, SECOND)'),
+Grain('minute', _('minute'), 'TIMESTAMP_TRUNC({col}, MINUTE)'),
+Grain('hour', _('hour'), 'TIMESTAMP_TRUNC({col}, HOUR)'),
+Grain('day', _('day'), 'TIMESTAMP_TRUNC({col}, DAY)'),
+Grain('week', _('week'), 'TIMESTAMP_TRUNC({col}, WEEK)'),
+Grain('month', _('month'), 'TIMESTAMP_TRUNC({col}, MONTH)'),
+Grain('quarter', _('quarter'), 'TIMESTAMP_TRUNC({col}, QUARTER)'),
+Grain('year', _('year'), 'TIMESTAMP_TRUNC({col}, YEAR)'),
)
@classmethod
def convert_dttm(cls, target_type, dttm):
tt = target_type.upper()
if tt == 'DATE':
-return "'{}'".format(dttm.strftime('%Y-%m-%d'))
+return "{}'".format(dttm.strftime('%Y-%m-%d'))
Review comment:
This caused: https://github.com/apache/incubator-superset/issues/4915
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
[GitHub] mistercrunch closed pull request #4913: Fix country_map visualization URL
mistercrunch closed pull request #4913: Fix country_map visualization URL
URL: https://github.com/apache/incubator-superset/pull/4913
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 is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/superset/assets/src/visualizations/country_map.js
b/superset/assets/src/visualizations/country_map.js
index a5ab3c431d..09d325de67 100644
--- a/superset/assets/src/visualizations/country_map.js
+++ b/superset/assets/src/visualizations/country_map.js
@@ -132,7 +132,7 @@ function countryMapChart(slice, payload) {
.attr('x', 20)
.attr('y', 60);
- const url =
`/static/assets/visualizations/countries/${fd.select_country.toLowerCase()}.geojson`;
+ const url =
`/static/assets/src/visualizations/countries/${fd.select_country.toLowerCase()}.geojson`;
d3.json(url, function (error, mapData) {
const features = mapData.features;
const center = d3.geo.centroid(mapData);
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
[GitHub] mistercrunch opened a new pull request #4917: Add doc entry for BigQuery support
mistercrunch opened a new pull request #4917: Add doc entry for BigQuery support URL: https://github.com/apache/incubator-superset/pull/4917 closes https://github.com/apache/incubator-superset/issues/945 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations
mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385704283 We should use the same webchunk name for all `deckgl`-related charts, same for `nvd3` and same all the ones that only require `d3`. I'm thinking we could have 4-5 bundles with all the visualizations in them. The "DeckGL Demo" dashboard should be much fast as it will load one 8mb chunk instead of 6 of them. Is it just a matter of sharing/reusing webchunknames? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations
mistercrunch commented on issue #4727: [webpack] setup lazy loading for all visualizations URL: https://github.com/apache/incubator-superset/pull/4727#issuecomment-385704283 We should use the same webchunk name for all `deckgl`-related charts, same for `nvd3` and same all the ones that only require `d3`. I'm thinking we could have 4-5 bundles with all the visualizations in them. Is it just a matter of sharing/reusing webchunknames? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch commented on issue #4916: Readonly Databases not Supported (eg Dremio or lower permissioned connections)
mistercrunch commented on issue #4916: Readonly Databases not Supported (eg Dremio or lower permissioned connections) URL: https://github.com/apache/incubator-superset/issues/4916#issuecomment-385699143 I think you're misinterpreting the logs. Superset does not require writing to target databases. `SELECT EXISTS` is against your Superset metadata database as it should as it tries to add a new entry on the Superset side to store the table's metadata from Dremio. My guess it that the call that tries to extract the metadata from Dremio here https://github.com/apache/incubator-superset/blob/master/superset/models/core.py#L809 isn't working properly in the dialect. Some database don't have a direct notion or schema, or sometimes have 2 levels of namespacing (Postgres has database and schema/owner) and that doesn't always fits the SQLAlchemy model perfectly. Dialects implementation have to be careful to line things up properly. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] xrmx commented on issue #4916: Readonly Databases not Supported (eg Dremio or lower permissioned connections)
xrmx commented on issue #4916: Readonly Databases not Supported (eg Dremio or lower permissioned connections) URL: https://github.com/apache/incubator-superset/issues/4916#issuecomment-385671694 You may want to raise debugging level to see more interesting things. Superset works perfectly fine with multiple databases. It looks like it's just matter of ironing out the compatibility with dremio. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] malsmith opened a new issue #4916: Readonly Databases not Supported (eg Dremio or lower permissioned connections)
malsmith opened a new issue #4916: Readonly Databases not Supported (eg Dremio or lower permissioned connections) URL: https://github.com/apache/incubator-superset/issues/4916 Make sure these boxes are checked before submitting your issue - thank you! - [X] I have checked the superset logs for python stacktraces and included it here as text if any - [X] I have reproduced the issue with at least the latest released version of superset - [X] I have checked the issue tracker for the same issue and I haven't found one similar ### Superset version 0.24 ### Expected results I should be able to add a table from any database connection. Note that when I add more database connections I can use them just fine in the SQL Lab UI - just can't add tables from them in the Tables function - so I can't actually use them in charts or dashboards. It seems like the Superset system is only designed to use a single database at a time and requires read/write access to that database, which would block my planned use of the system where I planned to have a small postgresql database for metadata and management and a large readonly database (Dremio in my case) as the analytical datastore. ### Actual results When adding a new Table item in the UI the system gets confused and queries the target database to check if the "tables" table already contains the new entry - rather than checking the main database connection (sqlite in my case). The system returns an Unknown error from database Flash message and the query itself fails in the Superset system logs (table "Tables" does not exist) ### Steps to reproduce Install the system as is with the Sqlite datasource/database as the primary database (load samples etc). Then Add a new Database in addition to the SQLLite one. In my case add a database where new tables cannot be created (although this seems to be beside the point). Attempt to add a new Table that is known to exist in the secondary database. Watch the system fail to add the new table (generating the unknown database flash message) Appears to be a problem in (or leading up to) ``` superset/connectors/sqla/views.py line 236 pre_add method ``` This seems to grab the wrong database sqla engine value and implications are that the system needs to know which is the database engine that holds the Superset metadata and which is the system where the new table is to be added from. SQL Trace ``` 2018-04-30 20:14:20,274 INFO sqlalchemy.engine.base.Engine SELECT EXISTS (SELECT 1 FROM tables WHERE tables.table_name = ? AND tables.schema = ? AND tables.database_id = ?) AS anon_1 2018-04-30 20:14:20,274:INFO:sqlalchemy.engine.base.Engine:SELECT EXISTS (SELECT 1 FROM tables WHERE tables.table_name = ? AND tables.schema = ? AND tables.database_id = ?) AS anon_1 2018-04-30 20:14:20,275 INFO sqlalchemy.engine.base.Engine (u'sf_incidents_clean', u'', 3) 2018-04-30 20:14:20,275:INFO:sqlalchemy.engine.base.Engine:(u'sf_incidents_clean', u'', 3) 2018-04-30 20:14:20,275:INFO:root:Database.get_sqla_engine(). Masked URL: dremio+pyodbc://malsmith:XX@mydremio 2018-04-30 20:14:20,562 INFO sqlalchemy.engine.base.Engine ROLLBACK 2018-04-30 20:14:20,562:INFO:sqlalchemy.engine.base.Engine:ROLLBACK 2018-04-30 20:14:20,564:INFO:werkzeug:127.0.0.1 - - [30/Apr/2018 20:14:20] "POST /tablemodelview/add HTTP/1.1" 302 - ``` This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] swamy16 commented on issue #945: Google BigQuery Support
swamy16 commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385653782 @mxmzdlv thanks for the help. I shall try installing superset in virtualbox. also is it possible to share the dashboards I create on superset with others in my organisation who don't have superset or python installed? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mxmzdlv commented on issue #945: Google BigQuery Support
mxmzdlv commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385631372 @swamy16 seems to be an issue with calling signal(signal.SIGALRM) — SIGALRM is not available on Windows. https://github.com/apache/incubator-superset/blob/d533ce09676921ec2f72e1032bbca0d2a37f65b6/superset/utils.py#L508 I think the easiest way to handle this would be to run Superset in a Docker / Vagrant or just install Virtualbox with Linux on it and run Superset from there? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mxmzdlv commented on issue #945: Google BigQuery Support
mxmzdlv commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385631372 @swamy16 seems to be an issue with calling signal(signal.SIGALRM) — SIGALRM is not available on Windows. https://github.com/apache/incubator-superset/blob/d533ce09676921ec2f72e1032bbca0d2a37f65b6/superset/utils.py#L508 I think the easiest way to handle this would be to run Superset in a Docker / Vagrant. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] swamy16 commented on issue #945: Google BigQuery Support
swamy16 commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385619874 @tswast @darylerwin @mistercrunch @mxmzdlv @Bamieh Thanks to you guys (your guidance) I was finally able to connect my superset to bigquery in windows. For some reason I was not able to set env variable through command promt or windows power shell. I did it manually through mycomputer/properties/settings/environment variables. I had to restart superset to make it work though. @Bamieh Idid not try using docker for running this. I have only heard about docker. Have never practically used it or worked with it. It would be great if you could point me to some resources on it so that I can learn the same. Now I can see tables and get their count and other things. Please refer below image for the same.  But when I am trying to run the same query from sql lab I am not able to generate any result. It just keeps on running and timesout. The same is happening If I try to query from the default 'main' DB that gets installed for visualising World Bank Data and other dashboards. I get the following error `Module 'signal has no attribute 'SILGRAM' ` and the query keeps on running for ever.  Currently I am creating dashboards using a manual process (Upload csv). Please help me resolve this error. Also is there a way to share the dashboard made on superset to others who dont have superset (even python) installed on them. Can I share it privately within the organisation I work in? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] swamy16 commented on issue #945: Google BigQuery Support
swamy16 commented on issue #945: Google BigQuery Support URL: https://github.com/apache/incubator-superset/issues/945#issuecomment-385619874 @tswast @darylerwin @mistercrunch @mxmzdlv Thanks to you guys (your guidance) I was finally able to connect my superset to bigquery in windows. For some reason I was not able to set env variable through command promt or windows power shell. I did it manually through mycomputer/properties/settings/environment variables. I had to restart superset to make it work though. Now I can see tables and get their count and other things. Please refer below image for the same.  But when I am trying to run the same query from sql lab I am not able to generate any result. It just keeps on running and timesout. The same is happening If I try to query from the default 'main' DB that gets installed for visualising World Bank Data and other dashboards. I get the following error `Module 'signal has no attribute 'SILGRAM' ` and the query keeps on running for ever.  Currently I am creating dashboards using a manual process (Upload csv). Please help me resolve this error. Also is there a way to share the dashboard made on superset to others who dont have superset (even python) installed on them. Can I share it privately within the organisation I work in? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4727: [webpack] setup lazy loading for all visualizations
williaster commented on issue #4727: [webpack] setup lazy loading for all
visualizations
URL:
https://github.com/apache/incubator-superset/pull/4727#issuecomment-385615006
I got chunk names to update (see exact webpack input / output below) and
wanted to post some numbers from local dashboard testing which all look pretty
good:
- average total payload **decreased by 5 MB**
- DOM content load time **decreased 3.4s**
- Page load time **decreased 1.1s**
- the average # requests **increased by 6**
| Total payload (MB) | |
-- | -- | -- | --
Dashboard | Before | After |
Births example | 12.4 | 6.9 | -5.5
World data example | 12.4 | 8.7 | -3.7
Misc charts example | 14.9 | 8.5 | -6.4
Misc charts example w Deck gl | 15.4 | 11.1 | -4.3
| DOM content loaded (seconds) | |
-- | -- | -- | --
Dashboard | Before | After | Delta
Births example | 7.69 | 2.93 | -4.76
World data example | 6.58 | 2.63 | -3.95
Misc charts example | 6.38 | 3.88 | -2.50
Misc charts example w Deck gl | 6.20 | 2.71 | -3.49
| Total time (seconds) | |
-- | -- | -- | --
Dashboard | Before | After | Delta
Births example | 9.25 | 6.87 | -2.38
World data example | 9.22 | 6.78 | -2.44
Misc charts example | 7.69 | 5.53 | -2.16
Misc charts example w Deck gl | 7.73 | 10.19 | +2.45
| Requests (count) | |
-- | -- | -- | --
Dashboard | Before | After | Delta
Births example | 32 | 36 | +4
World data example | 29 | 34 | +5
Misc charts example | 45 | 53 | +8
Misc charts example w Deck gl | 59 | 67 | +8
webpack output before
```
Hash: 09e816a63d54c8a794f4
Version: webpack 3.10.0
Time: 44176ms (44s)
Asset Size Chunks
Chunk Names
welcome.f1949d0f24772fdd90be.entry.js8.49 MB5, 6 [emitted] [big]
welcome
f4769f9bdb7466be65088239c12046d1.eot20.1 kB [emitted]
fa2772327f55d8198301fdb8bcfc8158.woff23.4 kB [emitted]
e18bbf611f2a2e43afc071aa2f4e1512.ttf45.4 kB [emitted]
89889688147bd7575d6327160d64e760.svg 109 kB [emitted]
dashboard.7aae3890d2f0d6521e6e.entry.js48.1 MB0, 6 [emitted] [big]
dashboard
explore.7876f80fe4be57a17e6c.entry.js46.3 MB1, 6 [emitted] [big]
explore
sqllab.80cb1149088d4df08c50.entry.js17.8 MB2, 6 [emitted] [big]
sqllab
profile.87ba695abcd8682ad921.entry.js8.57 MB3, 6 [emitted] [big]
profile
addSlice.f27f6a8368fce90f27ca.entry.js9.18 MB4, 6 [emitted] [big]
addSlice
448c34a56d699c29117adc64c43affeb.woff2 18 kB [emitted]
common.540032da3aeaf1cdc78e.entry.js3.85 MB 6 [emitted] [big]
common
theme.912f2edddcf1890cf5f2.entry.js6.12 kB 7 [emitted]
theme
theme.912f2edddcf1890cf5f2.css 321 kB 7 [emitted] [big]
theme
explore.7876f80fe4be57a17e6c.css85.5 kB1, 6 [emitted]
explore
dashboard.7aae3890d2f0d6521e6e.css92.9 kB0, 6 [emitted]
dashboard
sqllab.80cb1149088d4df08c50.css7.01 kB2, 6 [emitted]
sqllab
welcome.f1949d0f24772fdd90be.css 846 bytes5, 6 [emitted]
welcome
profile.87ba695abcd8682ad921.css1.04 kB3, 6 [emitted]
profile
manifest.json1.17 kB [emitted]
[31] ./src/locales.jsx 5.63 kB {0} {1} {2} {3} {4} {5} {6} [built]
[544] ./src/common.js 1.52 kB {0} {1} {2} {3} {4} {5} {6} [built]
[2877] ./src/theme.js 152 bytes {7} [built]
[2879] ./stylesheets/react-select/select.less 41 bytes {7} [built]
[2881] multi babel-polyfill ./src/addSlice/index.jsx 40 bytes {4} [built]
[2882] ./src/addSlice/index.jsx 793 bytes {4} [built]
[2884] multi babel-polyfill ./src/explore/index.jsx 40 bytes {1} [built]
[2885] ./src/explore/index.jsx 3.53 kB {1} [built]
[2910] multi babel-polyfill ./src/dashboard/index.jsx 40 bytes {0} [built]
[2911] ./src/dashboard/index.jsx 1.47 kB {0} [built]
[2976] multi babel-polyfill ./src/SqlLab/index.jsx 40 bytes {2} [built]
[2977] ./src/SqlLab/index.jsx 1.55 kB {2} [built]
[3062] multi babel-polyfill ./src/welcome/index.jsx 40 bytes {5} [built]
[3066] multi babel-polyfill ./src/profile/index.jsx 40 bytes {3} [built]
[3067] ./src/profile/index.jsx 775 bytes {3} [built]
+ 3119 hidden modules
```
webpack output after
```
Hash: 460ae189f7f00025375d
Version: webpack 3.11.0
Time: 45052ms
Asset Size
ChunksChunk Names
parallel_coordinates.e19f30010f7bbf13546b.chunk.js 204 kB
22 [emitted] parallel_coordinates
f4769f9bdb7466be65088239c12046d1.eot
[GitHub] williaster commented on issue #4913: Fix country_map visualization URL
williaster commented on issue #4913: Fix country_map visualization URL URL: https://github.com/apache/incubator-superset/pull/4913#issuecomment-385612991 lgtm This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] kuriancheeramelil opened a new issue #4915: Exploring BQ table throws query error as date is incorrectly formatted
kuriancheeramelil opened a new issue #4915: Exploring BQ table throws query
error as date is incorrectly formatted
URL: https://github.com/apache/incubator-superset/issues/4915
Make sure these boxes are checked before submitting your issue - thank you!
- [X] I have checked the superset logs for python stacktraces and included
it here as text if any
- [X] I have reproduced the issue with at least the latest released version
of superset
- [X] I have checked the issue tracker for the same issue and I haven't
found one similar
### Superset version
0.25
### Expected results
Executing BQ table query via table explore page executes query with date
field correctly.
### Actual results
the query generated throws syntax error
### Steps to reproduce
Add BQ table with date in superset and explore table. Any metric throws
syntax error.
The issue seems to be with this line
https://github.com/apache/incubator-superset/blob/e1d2150391ba827a64cebd0c0573aab0a82a3634/superset/db_engine_specs.py#L1263
Instead of
return "'{}'".format(dttm.strftime('%Y-%m-%d'))
it is missing the starting single quote
return "{}'".format(dttm.strftime('%Y-%m-%d'))
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
[GitHub] mistercrunch commented on issue #4343: fixing Issue #4335 - Not quoting column name when doing a time series
mistercrunch commented on issue #4343: fixing Issue #4335 - Not quoting column name when doing a time series URL: https://github.com/apache/incubator-superset/pull/4343#issuecomment-385610298 @kstan79 I think this is fixed on master, can you confirm? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] codecov-io commented on issue #4913: Fix country_map visualization URL
codecov-io commented on issue #4913: Fix country_map visualization URL URL: https://github.com/apache/incubator-superset/pull/4913#issuecomment-385609581 # [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4913?src=pr=h1) Report > Merging [#4913](https://codecov.io/gh/apache/incubator-superset/pull/4913?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-superset/commit/e1d2150391ba827a64cebd0c0573aab0a82a3634?src=pr=desc) will **not change** coverage. > The diff coverage is `n/a`. [](https://codecov.io/gh/apache/incubator-superset/pull/4913?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#4913 +/- ## === Coverage 77.14% 77.14% === Files 44 44 Lines8539 8539 === Hits 6587 6587 Misses 1952 1952 ``` -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4913?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-superset/pull/4913?src=pr=footer). Last update [e1d2150...9b91c9b](https://codecov.io/gh/apache/incubator-superset/pull/4913?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4910: Use React.PureComponent everywhere
williaster commented on issue #4910: Use React.PureComponent everywhere URL: https://github.com/apache/incubator-superset/pull/4910#issuecomment-385608926 I would not pursue this, [from Dan Abranov ]( https://twitter.com/dan_abramov/status/75938353012011008) (on React team, creator of redux): > Don‘t use PureComponent everywhere. Measure. On that Twitter thread he notes that equality checks for every `prop` for every component (which is what happens when you use `PureComponent`) is not necessarily cheap, and it can actually slow down your app in some cases. This is what I've been told as well, you should only start doing this when you measure something (with dev tools) and know something expensive is happening. Or do some explicit measuring in this PR and only update significant improvements you see. https://user-images.githubusercontent.com/4496521/39463701-2db7b4f2-4cce-11e8-80e6-f6b784dddac0.png; width="400" /> This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4910: Use React.PureComponent everywhere
williaster commented on issue #4910: Use React.PureComponent everywhere URL: https://github.com/apache/incubator-superset/pull/4910#issuecomment-385608926 I would not pursue this, [from Dan Abranov ]( https://twitter.com/dan_abramov/status/75938353012011008) (on React team, creator of redux): > Don‘t use PureComponent everywhere. Measure. On that Twitter thread he notes that equality checks for every `prop` for every component (which is what happens when you use `PureComponent`) is not necessarily cheap, and it can actually slow down your app in some cases. This is what I've been told as well, you should only start doing this when you measure something (with dev tools) and know something expensive is happening. Or do some explicit measuring in this PR and only update significant improvements you see. https://user-images.githubusercontent.com/4496521/39463701-2db7b4f2-4cce-11e8-80e6-f6b784dddac0.png This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4910: Use React.PureComponent everywhere
williaster commented on issue #4910: Use React.PureComponent everywhere URL: https://github.com/apache/incubator-superset/pull/4910#issuecomment-385608926 I would not pursue this, [from Dan Abranov ]( https://twitter.com/dan_abramov/status/75938353012011008) (on React team, creator of redux): > Don‘t use PureComponent everywhere. Measure. On that Twitter thread he notes that equality checks for every `prop` for every component (which is what happens when you use `PureComponent`) is not necessarily cheap, and it can actually slow down your app in some cases. This is what I've been told as well, you should only start doing this when you measure something (with dev tools) and know something expensive is happening. Or do some explicit measuring in this PR and see what's improving significantly. https://user-images.githubusercontent.com/4496521/39463701-2db7b4f2-4cce-11e8-80e6-f6b784dddac0.png This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] williaster commented on issue #4910: Use React.PureComponent everywhere
williaster commented on issue #4910: Use React.PureComponent everywhere URL: https://github.com/apache/incubator-superset/pull/4910#issuecomment-385608926 I would not pursue this, [from Dan Abranov ]( https://twitter.com/dan_abramov/status/75938353012011008) (on React team, creator of redux): > Don‘t use PureComponent everywhere. Measure. https://user-images.githubusercontent.com/4496521/39463701-2db7b4f2-4cce-11e8-80e6-f6b784dddac0.png; width="350" /> On that Twitter thread he notes that equality checks for every `prop` for every component (which is what happens when you use `PureComponent`) is not necessarily cheap, and it can actually slow down your app in some cases. This is what I've been told as well, you should only start doing this when you measure something (with dev tools) and know something expensive is happening. Or do some explicit measuring in this PR and see what's improving significantly. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] mistercrunch opened a new pull request #4913: Fix country_map visualization URL
mistercrunch opened a new pull request #4913: Fix country_map visualization URL URL: https://github.com/apache/incubator-superset/pull/4913 When moving the visualizations/ folder to src/ I missed this reference. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
