Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SwitchFirst operator #645

Closed
jinroh opened this issue Nov 4, 2015 · 4 comments
Closed

SwitchFirst operator #645

jinroh opened this issue Nov 4, 2015 · 4 comments

Comments

@jinroh
Copy link
Contributor

jinroh commented Nov 4, 2015

Is anyone working on the implementation of SwitchFirst right now ? If not I'd be happy to ! (I really need it in one of my project)

Some bikeshed I'd like to discuss before starting implementation:

@benlesh
Copy link
Member

benlesh commented Nov 4, 2015

I don't think anyone is working on it. I think you're right, though, the name needs work. switchFirst doesn't really describe what it's doing. I'm not sure what would though.

It subscribes to the first Observable it gets until it completes, dropping all others, then subscribes to the next one. It's almost a weird specialization of sample and merge. I've honestly never used it.

For now it could be named switchFirst and switchFirstMap... there just seems like there could be some more descriptive name or term for it.

Also, it can land in KitchenSink for now until we bikeshed what to do with it.

@benlesh
Copy link
Member

benlesh commented Nov 4, 2015

... also for now, let's skip the concurrency parameter until that can be discussed further. I'm not against it, I just haven't thought about it.

@kwonoj
Copy link
Member

kwonoj commented Nov 13, 2015

I think this can be closed now - @jinroh , may I possibly ask if there's additional PR might be required for this issue?

@benlesh
Copy link
Member

benlesh commented Nov 13, 2015

I don't think there is any more work to do here. If there is we can open another issue. Closing for now.

@benlesh benlesh closed this as completed Nov 13, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jun 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants