windowTime race condition

Bug Report

Current Behavior windowTime for some parameter values throws

Error in /turbo_modules/rxjs@6.5.2/internal/operators/windowTime.js (105:27)
Cannot read property 'closed' of undefined

Couldn’t figure out the exact relation between parameters, but basically it reliably broke for me with windowTimeSpan > windowCreationInterval && maxWindowSize > 2,

Reproduction https://stackblitz.com/edit/rxjs-gajdpw?devtoolsheight=60

import { interval } from 'rxjs';
import { windowTime, mergeAll } from 'rxjs/operators';

const timer = interval(200)
const result = timer.pipe(
  windowTime(1000, 600, 4),
  mergeAll(),
);
result.subscribe(x => console.log(x));

Expected behavior Not breaking. Not sure as to exact console output, though. Same as with bufferTime? I am pretty new to RxJS, just experimenting with operators.

Environment

• Runtime: Chrome 76.0.3806.1 (Official Build) dev (64-bit)
• RxJS version: 6.5.2

Additional context/Screenshots I’m assuming windowTime tries to close a window stream before it creates one, so a race condition?

[kschaefe]

As the original reporter suggested, I can confirm that bufferTime is dropping events when you are using positive values for all three parameters. Do you need a different issue report for that?

[cartant]

@kschaefe

Do you need a different issue report for that?

I’ve not been able to reproduce this easily with bufferTime, so if you could open another issue - complete with a minimal reproduction - that would be appreciated.