

<HTML><BODY><div>Hi team, </div><div> </div><div>QA LGTM</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;">Среда, 4 августа 2021, 17:08 +03:00 от Kirill Yukhin via Tarantool-patches <tarantool-patches@dev.tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16280861350616991891_BODY">Hello,<br><br>On 04 авг 11:30, <a href="/compose?To=imeevma@tarantool.org">imeevma@tarantool.org</a> wrote:<div class="mail-quote-collapse">> This opcode was used to convert INTEGER values to REAL. It is not<br>> necessary in Tarantool and causes errors.<br>><br>> Due to OP_Realify two type of errors appeared:<br>> 1) In some cases in trigger INTEGER may be converted to DOUBLE.<br>> For example:<br>> box.execute("CREATE TABLE t (i NUMBER PRIMARY KEY, n NUMBER);")<br>> box.execute("CREATE TRIGGER t AFTER INSERT ON t FOR EACH ROW BEGIN UPDATE t SET n = new.n; END;")<br>> box.execute("INSERT INTO t VALUES (1, 1);")<br>> box.execute("SELECT i / 2, n / 2 FROM t;")<br>><br>> Result:<br>> tarantool> box.execute("SELECT i / 2, n / 2 FROM t;")<br>> ---<br>> - metadata:<br>> - name: COLUMN_1<br>> type: number<br>> - name: COLUMN_2<br>> type: number<br>> rows:<br>> - [0, 0.5]<br>> ...<br>><br>> 2) If SELECT uses GROUP BY then it may return DOUBLE instead of INTEGER.<br>> For example:<br>> box.execute("CREATE TABLE t (i NUMBER PRIMARY KEY, n NUMBER);")<br>> box.execute("INSERT INTO t VALUES (1,1);")<br>> box.execute("SELECT i / 2, n / 2 FROM t GROUP BY n;")<br>><br>> Result:<br>> tarantool> box.execute("SELECT i / 2, n / 2 FROM t GROUP BY n;")<br>> ---<br>> - metadata:<br>> - name: COLUMN_1<br>> type: number<br>> - name: COLUMN_2<br>> type: number<br>> rows:<br>> - [0.5, 0.5]<br>> ...<br>><br>> This patch removes OP_Realify, after which these errors disappear.<br>><br>> Closes #5335<br>> ---<br>> <a href="https://github.com/tarantool/tarantool/issues/5335" target="_blank">https://github.com/tarantool/tarantool/issues/5335</a><br>> <a href="https://github.com/tarantool/tarantool/tree/imeevma/gh-5335-remove-op-realify" target="_blank">https://github.com/tarantool/tarantool/tree/imeevma/gh-5335-remove-op-realify</a></div><br>The patch LGTM.<br><br>--<br>Regards, Kirill Yukhin</div></div></div></div></blockquote><div> </div></BODY></HTML>