try to get times from cyanite for Reader

[sgran]

This makes an attempt to pull the information about metric interval validity from the cyanite backend, falling back to the current behavior if this request fails.

Signed-off-by: Stephen Gran [email protected]

[sgran]

I see that this is going to be harder with the way you're mocking things in the test. Not sure how to work around that just yet.

[brutasse]

The implementation makes sequential calls, which is probably going to introduce a pretty significant performance hit… We'd need to discuss this with @pyr but ideally these calls to cyanite should be batched.

[sgran]

Yeah, I had another thought about how to solve this in a cleaner way. Is there a good place where you and pyr and I can have a chat?

[sgran]

should we just discuss here? I was hoping for something a little more synchronous so we could bounce ideas back and forth, but this will do if nowhere else works.