Amp client warning for pipeline method

[kelunik]

Hey @MacFJA,

It's great to see support for amphp/redis built-in! Could you tell me what the notice in

php-redisearch/src/Redis/Client/AmpRedisClient.php

Line 75 in 6d3a675

&& trigger_error('Warning, \\Amp\\Redis\\Redis don\'t use a real Redis Pipeline', E_USER_NOTICE);

is supposed to mean?

[MacFJA]

It's just that, AFAIK, there is not pipeline in Amp\Redis, so each command need to be send one by one, and are also executed one by one.

Where with a Redis pipeline:

• all commands are sent to the server in one go
• the server execute all commands
• the server send one message containing all commands response.

With pipeline only one network request is made instead of one per command.

[kelunik]

While amphp/redis uses separate write calls, pipelining is more about avoiding the RTT cost and it's totally supported by amphp/redis. The client won't wait for a server response before sending the next command. It uses pipelining by default.