Skip to content

2.25.1.0-b97

@vpatibandla-yb vpatibandla-yb tagged this 27 Dec 11:49
Summary:
The change here is to fix the unit test failures noticed with connection manager enabled:
  # Increase timeout - Some of the ASan, TSan tests with connection manger enabled are timing out. So, increased the time out duration for these tests. The increase in timeout duration has been taken based on the historical time taken noticed with the help of detective.
  # In the case of `TestPgTimeout`, during the cleanUp operation, the test has been timing out.  To avoid this timeout scenario, I have disabled the STATEMENT_TIMEOUT.
  # The change in router.c is a bug fix. The lock has to be not released before the code block which deals with server warmup state typically used in our tests. I have moved the unlock code from before the server warmup to post the server warmup.
  # In sequence.c, the change is to only log the increment the log stickiness count and never pass it as a message from the server to the client. This is also the right behaviour as we want connection manager to be transparent to the user.

Test Plan: Jenkins: enable connection manager, all tests

Reviewers: rbarigidad, skumar, mkumar

Reviewed By: rbarigidad

Subscribers: stiwary, yql

Differential Revision: https://phorge.dev.yugabyte.com/D40906
Assets 2
Loading