Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Markup chart raise Exception(_('Empty query?')) #4222

Closed
3 tasks done
zhizunbao514 opened this issue Jan 17, 2018 · 0 comments · Fixed by #4225
Closed
3 tasks done

Markup chart raise Exception(_('Empty query?')) #4222

zhizunbao514 opened this issue Jan 17, 2018 · 0 comments · Fixed by #4225

Comments

@zhizunbao514
Copy link

zhizunbao514 commented Jan 17, 2018 •
edited

  • I have checked the superset logs for python stacktraces and included it here as text if any
  • I have reproduced the issue with at least the latest released version of superset
  • I have checked the issue tracker for the same issue and I haven't found one similar

Superset version

0.23

Expected results

When I was in use markup chart, he reported to the error.
I am a novice, help me, please.Thank you!

Actual results

Traceback (most recent call last):
File "/home/superset/incubator-superset-master/superset/views/core.py", line 1053, in generate_json
force=force)
File "/home/superset/incubator-superset-master/superset/viz.py", line 246, in get_payload
cache_key = self.cache_key(query_obj)
File "/home/superset/incubator-superset-master/superset/viz.py", line 239, in cache_key
self.datasource.get_query_str(query_obj),
File "/home/superset/incubator-superset-master/superset/connectors/sqla/models.py", line 404, in get_query_str
qry = self.get_sqla_query(**query_obj)
File "/home/superset/incubator-superset-master/superset/connectors/sqla/models.py", line 483, in get_sqla_query
raise Exception(_('Empty query?'))

Steps to reproduce
mistercrunch added a commit to mistercrunch/superset that referenced this issue Jan 17, 2018
@mistercrunch
[bugfix] markup and iframe viz raise 'Empty query'
de2b615 
closes apache#4222

Related to: apache#4016
@mistercrunch mistercrunch mentioned this issue Jan 17, 2018
Merged
mistercrunch added a commit that referenced this issue Jan 17, 2018
@mistercrunch
[bugfix] markup and iframe viz raise 'Empty query' (#4225)
5897d85 
closes #4222

Related to: #4016
mistercrunch pushed a commit that referenced this issue Jan 18, 2018
@hughhhh @mistercrunch
Added DeckGL.Polygon Layer w/ JS controls (#4227)
5079b2a 
* Working polygon layer for deckGL

* add js controls

* add thumbnail

* better description

* refactor to leverage line_column controls

* templates: open code and documentation on a new tab (#4217)

As they are external resources.

* Fix tutorial doesn't match the current interface #4138 (#4215)

* [bugfix] markup and iframe viz raise 'Empty query' (#4225)

closes #4222

Related to: #4016

* [bugfix] time_pivot entry got missing in merge conflict (#4221)

PR here #3518 missed a
line of code while merging conflicts with time_pivot viz

* Improve deck.gl GeoJSON visualization (#4220)

* Improve geoJSON

* Addressing comments

* lint

* refactor to leverage line_column controls

* refactor to use DeckPathViz

* oops
hughhhh pushed a commit to lyft/incubator-superset that referenced this issue Jan 18, 2018
@mistercrunch @hughhhh
[bugfix] markup and iframe viz raise 'Empty query' (apache#4225)
46411bc 
closes apache#4222

Related to: apache#4016
hughhhh added a commit to lyft/incubator-superset that referenced this issue Jan 18, 2018
@hughhhh
Added DeckGL.Polygon Layer w/ JS controls (apache#4227)
1c76d58 
* Working polygon layer for deckGL

* add js controls

* add thumbnail

* better description

* refactor to leverage line_column controls

* templates: open code and documentation on a new tab (apache#4217)

As they are external resources.

* Fix tutorial doesn't match the current interface apache#4138 (apache#4215)

* [bugfix] markup and iframe viz raise 'Empty query' (apache#4225)

closes apache#4222

Related to: apache#4016

* [bugfix] time_pivot entry got missing in merge conflict (apache#4221)

PR here apache#3518 missed a
line of code while merging conflicts with time_pivot viz

* Improve deck.gl GeoJSON visualization (apache#4220)

* Improve geoJSON

* Addressing comments

* lint

* refactor to leverage line_column controls

* refactor to use DeckPathViz

* oops
michellethomas pushed a commit to michellethomas/panoramix that referenced this issue May 24, 2018
@mistercrunch @michellethomas
[bugfix] markup and iframe viz raise 'Empty query' (apache#4225)
53fffdc 
closes apache#4222

Related to: apache#4016
michellethomas pushed a commit to michellethomas/panoramix that referenced this issue May 24, 2018
@hughhhh @michellethomas
Added DeckGL.Polygon Layer w/ JS controls (apache#4227)
1356943 
* Working polygon layer for deckGL

* add js controls

* add thumbnail

* better description

* refactor to leverage line_column controls

* templates: open code and documentation on a new tab (apache#4217)

As they are external resources.

* Fix tutorial doesn't match the current interface apache#4138 (apache#4215)

* [bugfix] markup and iframe viz raise 'Empty query' (apache#4225)

closes apache#4222

Related to: apache#4016

* [bugfix] time_pivot entry got missing in merge conflict (apache#4221)

PR here apache#3518 missed a
line of code while merging conflicts with time_pivot viz

* Improve deck.gl GeoJSON visualization (apache#4220)

* Improve geoJSON

* Addressing comments

* lint

* refactor to leverage line_column controls

* refactor to use DeckPathViz

* oops
wenchma pushed a commit to wenchma/incubator-superset that referenced this issue Nov 16, 2018
@mistercrunch
[bugfix] markup and iframe viz raise 'Empty query' (apache#4225)
ba9ed18 
closes apache#4222

Related to: apache#4016
wenchma pushed a commit to wenchma/incubator-superset that referenced this issue Nov 16, 2018
@hughhhh @mistercrunch
Added DeckGL.Polygon Layer w/ JS controls (apache#4227)
3482bce 
* Working polygon layer for deckGL

* add js controls

* add thumbnail

* better description

* refactor to leverage line_column controls

* templates: open code and documentation on a new tab (apache#4217)

As they are external resources.

* Fix tutorial doesn't match the current interface apache#4138 (apache#4215)

* [bugfix] markup and iframe viz raise 'Empty query' (apache#4225)

closes apache#4222

Related to: apache#4016

* [bugfix] time_pivot entry got missing in merge conflict (apache#4221)

PR here apache#3518 missed a
line of code while merging conflicts with time_pivot viz

* Improve deck.gl GeoJSON visualization (apache#4220)

* Improve geoJSON

* Addressing comments

* lint

* refactor to leverage line_column controls

* refactor to use DeckPathViz

* oops
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[bugfix] markup and iframe viz raise 'Empty query' mistercrunch/superset
1 participant
@zhizunbao514