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

Timestamp column regression in cat/health and cat/count #4782

Closed
drewr opened this issue Jan 17, 2014 · 1 comment
Closed

Timestamp column regression in cat/health and cat/count #4782

drewr opened this issue Jan 17, 2014 · 1 comment

Comments

@drewr
Copy link
Contributor

drewr commented Jan 17, 2014

In #4696 we made the epoch time more Unix-compatible without realizing it was used in the DateTimeFormatter which outputs the HMS timestamp string. Now that column shows the wrong time.

Also take this opportunity to use TimeUnit instead of literal math.

@spinscale
Copy link
Contributor

LGTM

@drewr drewr closed this as completed in 3f14725 Jan 17, 2014
brusic pushed a commit to brusic/elasticsearch that referenced this issue Jan 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants