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

Merge Bug: Missing Emissions #1463

Merged
merged 1 commit into from
Jul 18, 2014
Merged

Merge Bug: Missing Emissions #1463

merged 1 commit into from
Jul 18, 2014

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

@benjchristensen
Merge Bug: Missing Emissions
a720f58 
Notes and temporary fix
@cloudbees-pull-request-builder
Copy link

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

benjchristensen added a commit that referenced this pull request Jul 18, 2014
@benjchristensen
Merge pull request #1463 from benjchristensen/merge-1420
8f1f6f2 
Merge Bug: Missing Emissions
@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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@benjchristensen @cloudbees-pull-request-builder