docs(MIGRATION): remove the non-exist usecase for throttle()

[tetsuharuohzeki]

RxJS v4's throttle() does not accept Observable<T> as a durationSelector. There are no need the migration path.

• https://github.com/Reactive-Extensions/RxJS/blob/8f12f812d497acf639588e90f74d504a9fc801ec/src/core/linq/observable/throttle.js#L9
• https://github.com/Reactive-Extensions/RxJS/blob/8f12f812d497acf639588e90f74d504a9fc801ec/ts/core/linq/observable/throttle.ts

[kwonoj]

Doesn't RxJS5 column need to maintain throttle(durationSelector) variant?

[tetsuharuohzeki]

@kwonoj

I thought this as a migration doc to move from RxJS v4. So I removed throttle(durationSelector) which is just a new feature. If the comparison doc introduces a new feature, I feel we should keep it.

[kwonoj]

Those table's for operator splits, introduces new operator variant from RxJS4's polymorphic behavior as well such as window, windowWhen, windowToggle comes from RxJS4. in case of throttle(selector) is somewhat different though since original throttle didn't have those behavior though, thought it belongs to operator split allows easier lookup. Maybe I'm incorrect still.

[kwonoj]

Merged with a3cc4e7, think it's ok as-is by not displaying RxJS 5 variant is trivial compare to showing non exist one. Thanks @saneyuki .

[tetsuharuohzeki]

@kwonoj Thank you for accepting my proposal!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.