matthiasmullie/scrapbook

Bottleneck in Couchbase Adapter

pesha opened this issue · 1 comments

pesha commented

We have uncovered issue with logic that is assessing cluster health. This check is performed using regular http call which can lead to significant performance issues (higher response time). We didn't identified the root cause exactly but it can be caused by reaching of some limit (TCP connections).

scrapbook/src/Adapters/Couchbase.php

Line 38 in 044ff95

$this->assertServerHealhy();

The graph below shows the impact on response time before and after the method assertServerHealhy was commented out.
response time

pesha commented

It might not be needed to solve that anymore as we had slightly older version. The current version in m aster has an option using which we can disable this check. I would just consider whether the check shouldn't be disabled by default.