Skip to content

Conversation

rachmatowicz
Copy link
Contributor

This PR does the following:

  • reverts three commits for EJBCLIENT-356 which introduced discovery-additional-timeout and eagerNodes
  • add in commit for introducing the connection attempt timeout DISCOVERY_CONNECTION_TIMEOUT
  • add in EJBCLIENT-392 which results in all test cases using AbstractEJBClientTestCase

I will also write a test case to better test the DiscoveryAttempt part of the timeout mechanism before merging - thus this is a draft PR for the time being.

@chengfang chengfang added the Hold label Jan 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants