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

UI: Adds the ability to shift the metric graph timescales #6145

Merged
merged 1 commit into from
Apr 19, 2016

Conversation

maxlang
Copy link
Contributor

@maxlang maxlang commented Apr 19, 2016

Currently the UI chokes on values larger than 1 day, so I've
restricted it to a day until we can fix the performance.

Partially addresses #4986

screen shot 2016-04-19 at 12 32 17 am


This change is Reviewable

@maxlang maxlang force-pushed the maxlang/ui-graph-timescales2 branch 3 times, most recently from f8947de to 1ba8e38 Compare April 19, 2016 19:26
Currently the UI chokes on values larger than 1 day, so I've
restricted it to a day until we can fix the performance.

Partially addresses cockroachdb#4986
@maxlang maxlang force-pushed the maxlang/ui-graph-timescales2 branch from 1ba8e38 to 3362ebe Compare April 19, 2016 19:31
@mrtracy
Copy link
Contributor

mrtracy commented Apr 19, 2016

LGTM

@maxlang maxlang merged commit a550ae9 into cockroachdb:master Apr 19, 2016
@maxlang maxlang deleted the maxlang/ui-graph-timescales2 branch April 19, 2016 20:33
@tbg
Copy link
Member

tbg commented Apr 19, 2016

Just came here to say "sweet"!

On Tue, Apr 19, 2016 at 4:33 PM Max Lang notifications@github.com wrote:

Merged #6145 #6145.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#6145 (comment)

-- Tobias

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants