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

Make BasicPullResponseHandler support emitting signals in and out lock #1233

Merged
merged 1 commit into from
May 31, 2022

Conversation

injectives
Copy link
Contributor

@injectives injectives commented May 26, 2022

This update adds configuration param to BasicPullResponseHandler to emit signals to recordConsumer and summaryConsumer either in or out of lock.

@injectives injectives requested a review from gjmwoods May 26, 2022 11:34
@injectives injectives force-pushed the feature/basic-pull branch 3 times, most recently from cf6c4c3 to d0b1ee0 Compare May 27, 2022 13:21
@injectives injectives changed the title Update BasicPullResponseHandler to emit signals outside lock Add support for emitting signals outside sync on BasicPullResponseHandler May 27, 2022
@injectives injectives force-pushed the feature/basic-pull branch from d0b1ee0 to 6a03958 Compare May 27, 2022 13:23
@injectives injectives changed the title Add support for emitting signals outside sync on BasicPullResponseHandler Make BasicPullResponseHandler support emitting signals inside and outside synchronization lock May 27, 2022
@injectives injectives changed the title Make BasicPullResponseHandler support emitting signals inside and outside synchronization lock Make BasicPullResponseHandler support emitting signals in and out lock May 27, 2022
@injectives injectives force-pushed the feature/basic-pull branch from 6a03958 to 3495ef0 Compare May 27, 2022 13:25
@injectives injectives force-pushed the feature/basic-pull branch from 3495ef0 to cee99fa Compare May 27, 2022 14:23
This update adds configuration param to `BasicPullResponseHandler` to emit signals to `recordConsumer` and `summaryConsumer` either in or out of lock.
@injectives injectives force-pushed the feature/basic-pull branch from cee99fa to 21e31f2 Compare May 27, 2022 14:26
@injectives injectives merged commit 505f214 into neo4j:5.0 May 31, 2022
@injectives injectives deleted the feature/basic-pull branch May 31, 2022 10:07
injectives added a commit to injectives/neo4j-java-driver that referenced this pull request May 31, 2022
neo4j#1233)

This update adds configuration param to `BasicPullResponseHandler` to emit signals to `recordConsumer` and `summaryConsumer` either in or out of lock.
injectives added a commit that referenced this pull request May 31, 2022
#1233) (#1235)

This update adds configuration param to `BasicPullResponseHandler` to emit signals to `recordConsumer` and `summaryConsumer` either in or out of lock.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants