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.

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

Pulses not showing figures - metabase.pulse.render.NumericWrapper #7210

Closed
JBrame opened this issue Mar 27, 2018 · 12 comments
Closed

Pulses not showing figures - metabase.pulse.render.NumericWrapper #7210

JBrame opened this issue Mar 27, 2018 · 12 comments
Assignees
Labels
Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness Reporting/Pulses Now called Subscriptions Type:Bug Product defects
Milestone

Comments

@JBrame
Copy link

JBrame commented Mar 27, 2018

I've just updated to version 0.28.3 and Pulses seem to be broken:
image

Viewed in Outlook and Outlook web (Office 365)
Your operating system: Win 10
Your databases: MySQL 5.7.19
Metabase version: 0.28.3
Metabase hosting environment: Win10 Powershell jar
Metabase internal database: H2 (default)

@JoeSham
Copy link

JoeSham commented Mar 30, 2018

We also have some problems with pulses since the upgrade. They often don't get sent as scheduled. But when manually sending them, they work.

But we have been having multiple other issues since the upgrade, most notably the java process running out of memory quite often (with -Xms and -Xmx set to more than 6 GB and even more GB available to the container), causing Metabase docker container restart, so it might be connected.

@thedavidprice
Copy link

Same issue for me with pulses. Started with 28.2, continued on 28.3 and now 28.4.

Will continue an attempt to diagnose but no idea as of yet.

@senior senior self-assigned this Mar 30, 2018
@senior senior added Type:Bug Product defects Reporting/Pulses Now called Subscriptions labels Mar 30, 2018
@senior
Copy link
Contributor

senior commented Mar 30, 2018

I fixed this bug in a few other places, must have missed this one. Thanks for reporting this!

@thedavidprice
Copy link

@senior greatly appreciate it!

@salsakran salsakran added this to the 0.29 milestone Mar 31, 2018
@salsakran salsakran added the Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness label Mar 31, 2018
@brnt
Copy link

brnt commented Apr 2, 2018

Sounds like you've got a handle on it @senior, so I won't pile on to the report.

Can we make sure this gets pushed into the heroku binary branch once the issue is resolved?

@valtlfelipe
Copy link
Contributor

This is still present in 0.28.5.

captura de tela 2018-04-05 as 10 05 33

@senior
Copy link
Contributor

senior commented Apr 5, 2018

Sorry for misleading everyone on this issue, but it's already been fixed, I just didn't realize that the fix was in master, not the 0.28.x release branches. The fix was merged into master quite a while ago (early March) #7074. It will be included with 0.29.0 which should be soon.

I spent all morning trying to reproduce this issue on master and couldn't, which is good news as it validated the fix, just waiting for a release now.

@camsaul
Copy link
Member

camsaul commented Apr 9, 2018

Fixed by #7074

@salsakran
Copy link
Contributor

Heads up @TheCondorman @thedavidprice @JoeSham and @brnt , this is embarrassing enough that we kicked out an 0.28.6 to fix it while we do final testing + tweaking on 0.29. Sorry for the mess!

@JBrame
Copy link
Author

JBrame commented Apr 13, 2018

@salsakran Thanks for the update. Installed and verified 👍

@bishwo633
Copy link

This is occurring again on v0.32.9. I pulled a docker image of metabase "docker pull metabase/metabase:v0.32.9", configured postgresql database data source, then setup pulse to send an email to exchange mail.

Output (outlook email):
image

Output (web browser):
image

@flamber
Copy link
Contributor

flamber commented Jul 8, 2019

@bishwo633 I think you're seeing issue #6874

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness Reporting/Pulses Now called Subscriptions Type:Bug Product defects
Projects
None yet
Development

No branches or pull requests

10 participants