[Tarantool-patches] [PATCH v2 2/2] uuid: support uuid comparison with strings
Oleg Babin
olegrok at tarantool.org
Tue Nov 24 22:23:11 MSK 2020
Thanks for review!
On 24/11/2020 18:20, Igor Munkin wrote:
> Oleg,
>
> Thanks for the patch! Considering the version on the remote branch, LGTM
> except a single nit below.
>
> On 18.11.20,olegrok at tarantool.org wrote:
>> From: Oleg Babin<babinoleg at mail.ru>
>>
>> Before this patch it was impossible to compare uuid values with
>> string representations of uuid. However we have cases when such
>> comparisons is possible (e.g. "decimal" where we can compare
>> decimal values with strings and numbers).
>>
>> This patch extends uuid comparators (eq, lt, le) and every string
> Minor: Strictly saying, you're right, but AFAICS we usually use the key
> name (i.e. __eq, __le and __lt) for metamethods in description.
I've updated and force-pushed branch. I cite only changed part:
This patch extends uuid comparators (__eq, __lt and __le) and
every string argument is tried to be converted to uuid value to
compare then.
More information about the Tarantool-patches
mailing list