Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

IntervalScan doesn't work with arrays #279

Open
severb opened this issue Mar 11, 2016 · 1 comment
Open

IntervalScan doesn't work with arrays #279

severb opened this issue Mar 11, 2016 · 1 comment

Comments

@severb
Copy link

severb commented Mar 11, 2016

I'm using this code to smear a tight loop but it doesn't work on arrays. I think the problem is here: https://github.com/uber/hyperbahn/blob/master/lib/interval-scan.js#L141.

@kriskowal
Copy link
Contributor

Correct, this should be an array of indexes, which is easiest to obtain with Object.keys(array), proper to generate an array of indexes, better to iterate differently for arrays and objects.

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

No branches or pull requests

2 participants