Skip to content

Conversation

@zhenlineo
Copy link
Contributor

@zhenlineo zhenlineo commented Apr 16, 2019

Added a stub test to verify that after a leader in a cluster is removed, the driver should be able to retry the tx on

  • writer no longer available error (connection with a writer breaks in the middle of a tx)
  • server no longer reachable error (a broken writer is still included in routing table)
  • no writer available error (no writer found in routing table)

Changed the error SEVERE: ServiceUnavailableException: Connection to the database terminated. which happens on the second error condition, to WARN: SessionExpired to avoid noise in logs.

…ed, the driver should be able to retry the tx on 1) writer no longer available error and 2) no writer available error.
@zhenlineo zhenlineo requested a review from ali-ince April 17, 2019 10:00
@zhenlineo zhenlineo force-pushed the 1.7-retry-no-writer branch from ac502de to bb038cf Compare April 23, 2019 14:58
Copy link
Contributor

@ali-ince ali-ince left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@ali-ince ali-ince merged commit 71ff948 into neo4j:1.7 Apr 24, 2019
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