[Tarantool-patches] [PATCH v1 2/4] test: fix xlog-py/big_lsn.test.py
Sergey Bronnikov
sergeyb at tarantool.org
Thu Jan 14 12:51:50 MSK 2021
Thanks for review!
On 14.01.2021 12:02, Leonid Vasiliev wrote:
> Hi! Thank you for the patch.
> LGTM.
>
> On 13.01.2021 11:48, sergeyb at tarantool.org wrote:
>> From: Sergey Bronnikov <estetus at gmail.com>
>>
>> Set correct encoding on opening file.
>>
>> Part of #5538
>
> AFAIU, if the task has been closed, new patches corresponding with the
> task are marked as "Follows up".
>
Fixed!
>> ---
>> test/xlog-py/big_lsn.test.py | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/test/xlog-py/big_lsn.test.py b/test/xlog-py/big_lsn.test.py
>> index edc1e5620..73dcf7a48 100644
>> --- a/test/xlog-py/big_lsn.test.py
>> +++ b/test/xlog-py/big_lsn.test.py
>> @@ -1,4 +1,5 @@
>> import os
>> +import codecs
>> #
>> # Check that Tarantool handles huge LSNs well (gh-4033).
>> @@ -17,7 +18,7 @@ new_lsn = 123456789123
>> wal_dir = os.path.join(server.vardir, server.name)
>> old_wal = os.path.join(wal_dir, "%020d.xlog" % old_lsn)
>> new_wal = os.path.join(wal_dir, "%020d.xlog" % new_lsn)
>> -with open(old_wal, "r+") as f:
>> +with codecs.open(old_wal, "r+", encoding = "ISO-8859-1") as f:
>> s = f.read()
>> s = s.replace("VClock: {{1: {}}}".format(old_lsn),
>> "VClock: {{1: {}}}".format(new_lsn))
>>
More information about the Tarantool-patches
mailing list