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.

Sign up for GitHub

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

Jump to bottom

Remove Pivot Operator in 0.20? #1402

Closed
benjchristensen opened this issue Jul 2, 2014 · 4 comments
Closed

Remove Pivot Operator in 0.20? #1402

benjchristensen opened this issue Jul 2, 2014 · 4 comments

Comments

@benjchristensen
Copy link
Member

The pivot operator continues to have concurrency issues. I'm considering removing it in 0.20 (it probably should never have been added).

Does anyone use it? Does anyone have issues with me removing it?

i think it is better suited to being a custom operator, rather than part of rxjava-core.
@headinthebox
Copy link
Contributor

I'd be totally fine with that; it is a monster that is hard to tame.

@benjchristensen
Copy link
Member Author

Great ... I'll remove it in 0.20. I'd rather wait until it matures and possibly bring it back in the 1.x series than leave it in its current state.

@headinthebox
Copy link
Contributor

+1

@benjchristensen benjchristensen mentioned this issue Jul 16, 2014
Merged
benjchristensen added a commit to benjchristensen/RxJava that referenced this issue Jul 17, 2014
@benjchristensen
Remove Pivot Operator
a4eb42e 
as per ReactiveX#1402
@benjchristensen benjchristensen mentioned this issue Jul 17, 2014
Merged
benjchristensen added a commit to benjchristensen/RxJava that referenced this issue Jul 17, 2014
@benjchristensen
Remove Pivot Operator
68c7332 
as per ReactiveX#1402
@benjchristensen
Copy link
Member Author

It has been removed.

zsxwing pushed a commit to zsxwing/RxScala that referenced this issue Aug 19, 2014
@benjchristensen
Remove Pivot Operator
9e3117e 
as per ReactiveX/RxJava#1402
benjchristensen added a commit to ReactiveX/RxScala that referenced this issue Aug 19, 2014
@benjchristensen
Remove Pivot Operator
bebca3d 
as per ReactiveX/RxJava#1402
benjchristensen added a commit to ReactiveX/RxScala that referenced this issue Aug 19, 2014
@benjchristensen
Remove Pivot Operator
f3d77ea 
as per ReactiveX/RxJava#1402
benjchristensen added a commit to ReactiveX/RxScala that referenced this issue Aug 19, 2014
@benjchristensen
Remove Pivot Operator
91265ad 
as per ReactiveX/RxJava#1402
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@benjchristensen @headinthebox