Remove groupBy with selector.

[benjchristensen]

I think we can use groupBy(keySelector).map(elementSelector) instead. Is there any reason to keep this signature?

Related to 02ccc4d#commitcomment-5430646

[daveray]

Well, groupBy returns Observable<Observable> so you need one more level of map to apply elementSelector. I'm ambivalent.

[benjchristensen]

Merging for now to remove the broken functionality ... can be re-added before 0.17.0 release if we decide to.

[benjchristensen]

Well, groupBy returns Observable so you need one more level of map to apply elementSelector. I'm ambivalent.

Yes sorry, it's within the map or flatMap, but we already have to do that anyways when using groupBy as that's what it does, emit Observable<GroupedObservable<T>>>. I don't see the need for a special operator that emits Observable<GroupedObservable<R>>>.

Thanks @daveray for the feedback. Anyone else have a good reason for the existence of this overload? The preference is allow composition of existing operators.

[cloudbees-pull-request-builder]

RxJava-pull-requests #845 SUCCESS
This pull request looks good