Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

GroupBy & Time Gap Fixes#975

Merged
benjchristensen merged 1 commit intoReactiveX:masterfrom
benjchristensen:groupBy-fixes
Mar 20, 2014
Merged

GroupBy & Time Gap Fixes#975
benjchristensen merged 1 commit intoReactiveX:masterfrom
benjchristensen:groupBy-fixes

Conversation

@benjchristensen
Copy link
Member

I ran head-on into the "time gap" (#844) issue while working on a stream processing use case (and new 'pivot' operator I'm writing).
This is a solution. It's still not ideal as the Javadocs of BufferUntilSubscriber mention, but this is working better than nothing and does not require blocking threads.
A better solution will come as part of the back pressure work where BufferUntilSubscriber will evolve to have a bounded buffer.

I ran head-on into the "time gap" (ReactiveX#844) issue while working on a stream processing use case (and new 'pivot' operator I'm writing).
This is a solution. It's still not ideal as the Javadocs of BufferUntilSubscriber mention, but this is working better than nothing and does not require blocking threads.
A better solution will come as part of the back pressure work where BufferUntilSubscriber will evolve to have a bounded buffer.
benjchristensen added a commit that referenced this pull request Mar 20, 2014
@benjchristensen benjchristensen merged commit 17652fc into ReactiveX:master Mar 20, 2014
@benjchristensen benjchristensen deleted the groupBy-fixes branch March 20, 2014 17:47
@cloudbees-pull-request-builder

RxJava-pull-requests #914 FAILURE
Looks like there's a problem with this pull request

@benjchristensen
Copy link
Member Author

I should probably fix that random compilation error sitting in master branch ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants