

<HTML><BODY><div>Cause osx_10_15_lto was passed this is my QA LGTM on the last patch</div><div>Hooray!</div><div> </div><div> </div><div> </div><div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Vitaliia Ioffe</div></div></div><div> </div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Четверг, 19 августа 2021, 15:26 +03:00 от Kirill Yukhin <kyukhin@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16293760031368342448_BODY">Hello,<br><br>On 19 авг 15:02, <a href="/compose?To=imeevma@tarantool.org">imeevma@tarantool.org</a> wrote:<div class="mail-quote-collapse">> This patch introduces static and dynamic type checks for SQL built-in functions.<br>><br>> <a href="https://github.com/tarantool/tarantool/issues/6105" target="_blank">https://github.com/tarantool/tarantool/issues/6105</a><br>> <a href="https://github.com/tarantool/tarantool/tree/imeevma/gh-6105-properly-check-funcs-args-types" target="_blank">https://github.com/tarantool/tarantool/tree/imeevma/gh-6105-properly-check-funcs-args-types</a><br>><br>> Mergen Imeev (9):<br>> sql: modify signature of TRIM()<br>> sql: rework SQL built-in functions hash table<br>> sql: check number of arguments during parsing<br>> sql: static type check for SQL built-in functions<br>> sql: runtime type check for SQL built-in functions<br>> sql: enable types checking for some functions<br>> sql: fix result type of min() and max() functions<br>> sql: check argument types of sum(), avg(), total()<br>> sql: arguments check for string value functions</div><br>LGTM.<br><br>--<br>Regards, Kirill Yukhin</div></div></div></div></blockquote><div> </div></BODY></HTML>