[Tarantool-patches] [PATCH v1] test: flaky replication/gh-5195-qsync-*

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Thu Sep 10 22:21:56 MSK 2020


>>> diff --git a/test/replication/gh-5195-qsync-replica-write.result b/test/replication/gh-5195-qsync-replica-write.result
>>> index 3999e8f5e..b47359fde 100644
>>> --- a/test/replication/gh-5195-qsync-replica-write.result
>>> +++ b/test/replication/gh-5195-qsync-replica-write.result
>>> @@ -96,10 +96,12 @@ test_run:wait_downstream(replica_id, {status='follow'})
>>>   | - true
>>>   | ...
>>>  test_run:wait_cond(function()                                                   \
>>> -        local info = box.info.replication[replica_id]                           \
>>> -        local lsn = info.downstream.vclock[replica_id]                          \
>>> -        return lsn and lsn >= replica_lsn                                       \
>>> -end)                                                                            \
>>> +        local lsn = box.info.replication[replica_id].downstream.vclock          \
>>
>> 3. It is not lsn, it is vclock now. Also I don't think you need to inline 'info'
>> variable, it was perfectly fine.
> 
> Right, the variable name was not correct - fixed. 'info' variable wasn't
> need to be set due to used only once.

That is not about number of usages. It is about code being easier to read
by avoiding too big one-liners.


More information about the Tarantool-patches mailing list