Why does zipping with periodic() hangs?

Summary

I want to print a stream of increasing numbers every 1s, but the code below hangs Node.js forever:

// test.js
const { iterate, periodic } = require('most')

iterate(x => x + 1, 0)
  .take(10)
  .zip(x => x, periodic(1000))
  .forEach(x => console.log(x))

while this

iterate(x => x + 1, 0)
  .take(10)
  .forEach(x => console.log(x))

terminates as expected.

Expected result

Expected output would be:

$ node test.js
0
1 # after 1s
2 # after 2s
...

Actual Result

$ node test.js
0 # <- hangs here and CPU usage goes up

Versions

• most.js: 1.5.0

[briancavalier]

Hi @iwinux, thanks for reporting this. The sample code to reproduce was very helpful! It appears to be a regression in take introduced in #428 (technically, a regression in slice, on which take is based). The regression is causing iterate to run forever, and continuously preempt the periodic task.

The reason it manifests with zip is that zip has to buffer the very fast iterate events, while waiting for corresponding periodic events, which never occur due to the preemption mentioned above.

The fix is relatively simple, and I should have time tonight to create a branch so you can try it.

If you’re indeed blocked by needing a periodic counter, then this will work and hopefully unblock you until the fix is ready:

periodic(1000)
  .slice(1, 10)
  .scan((x, _) => x + 1, 0)
  .forEach(x => console.log(x))

I’m guessing your example is a simplified version of a real use case, so this may not help, but figured it was worth a shot. Cheers!

[briancavalier]

@iwinux There’s a candidate fix in #467 (fix-466 branch). Could you give it a try and let us know if it works for you?

UPDATE: There’s a slightly cleaner fix in #468 (fix-466-2 branch). Please give that one a try and let us know. Thanks!