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

Merge Bug: Missing Emissions#1463

Merged
benjchristensen merged 1 commit intoReactiveX:masterfrom
benjchristensen:merge-1420
Jul 18, 2014
Merged

Merge Bug: Missing Emissions#1463
benjchristensen merged 1 commit intoReactiveX:masterfrom
benjchristensen:merge-1420

Conversation

@benjchristensen
Copy link
Member

Notes and temporary fix using synchronized to achieve correctness until a proper solution can be found.

This code does not exhibit the issue when I do production testing. Without the two synchronized methods I get errors in prod that I've temporarily added to BlockingOperatorToIterator until this is fixed so it's visible.

/apps/tomcat/logs$ tail -f catalina.out |grep "Timed out waiting for value"
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava
Timed out waiting for value. File a bug at github.com/Netflix/RxJava

Notes and temporary fix
@cloudbees-pull-request-builder

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

benjchristensen added a commit that referenced this pull request Jul 18, 2014
@benjchristensen benjchristensen merged commit 8f1f6f2 into ReactiveX:master Jul 18, 2014
@benjchristensen benjchristensen deleted the merge-1420 branch July 18, 2014 17:02
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