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

Port propagating query execution errors from Celery tasks properly #458

Closed
jezdez opened this issue Jul 13, 2018 · 4 comments
Closed

Port propagating query execution errors from Celery tasks properly #458

jezdez opened this issue Jul 13, 2018 · 4 comments
Assignees
@alison985
Labels
merge upstream
Milestone
17

Comments

@jezdez
Copy link

jezdez commented Jul 13, 2018

Refs #290 (80d9ab6)
@rafrombrc rafrombrc added this to the 17 milestone Jul 16, 2018
@alison985 alison985 self-assigned this Jul 30, 2018
alison985 added a commit to StantonVentures/redash that referenced this issue Aug 1, 2018
@alison985
ports propagating query execution errors from Celery tasks properly
0c074d2 
re: mozilla#458
@alison985 alison985 mentioned this issue Aug 1, 2018
Merged
@alison985
Copy link

PR upstream getredash#2713

@alison985 alison985 added the in review upstream PR in review in getredash/redash label Aug 1, 2018
@jezdez
Copy link
Author

jezdez commented Aug 8, 2018

Yays!

@jezdez jezdez closed this as completed Aug 8, 2018
@jezdez jezdez added merge upstream and removed in review upstream PR in review in getredash/redash labels Aug 8, 2018
@jezdez jezdez reopened this Aug 8, 2018
@jezdez
Copy link
Author

jezdez commented Aug 8, 2018

Sorry, wrong PR.

@jezdez jezdez added in review upstream PR in review in getredash/redash and removed merge upstream labels Aug 8, 2018
jezdez pushed a commit to getredash/redash that referenced this issue Aug 23, 2018
@alison985 @jezdez
Propagate query execution errors from Celery tasks properly (#2713)
341a68c 
Refs mozilla#458
@jezdez jezdez added merge upstream and removed in review in review upstream PR in review in getredash/redash labels Aug 23, 2018
@jezdez
Copy link
Author

jezdez commented Aug 23, 2018

Merged upstream!

@jezdez jezdez closed this as completed Aug 23, 2018
@MaxMood96 MaxMood96 mentioned this issue Nov 5, 2022
Open
@MaxMood96 MaxMood96 mentioned this issue Dec 26, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alison985 alison985

Labels
merge upstream
Projects
None yet
Milestone
17
Development

No branches or pull requests
3 participants
@jezdez @rafrombrc @alison985