fix(table): change required for RxJS v6.0

[bowenni]

this._sort.sortChange and this._paginator.page are of type EventEmitter<Sort> and EventEmitter<PageEvent>.

In RxJS 6.0 the empty observable returns Observable<never> instead of the currently Observable<Sort> and Observable<PageEvent>. Therefore with RxJS v6.0 sortChange is of type EventEmitter<Sort>|Observable<never> and pageChange is of type EventEmitter<PageEvent>|Observable<never>. The union Observable types throw TS2554: Expected 0 arguments, but got 1. error. Details in ReactiveX/rxjs#3388

[crisbeto]

LGTM

[bowenni]

I think the failed tests are unrelated to my changes?
master was already failing on those tests at https://travis-ci.org/angular/material2/jobs/353088323

[crisbeto]

I’ve submitted #10409 to fix the e2e failure.

[admosity]

This issue occurs in rxjs 5.5.x+ (with pipe). I believe this is a typescript 2.7.x issue with stricter type checking.

[andrewseguin]

Needs rebase

[bowenni]

#10482 looks like a super set of this change.
I'm closing this one in favor of #10482

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}