

<HTML><BODY><div><div><br>Hi,<br> </div><div>Agreed with everything except one place:</div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">> diff --git a/test/box/gh-3290-ddl_collation_deleted.test.lua b/test/box/gh-3290-ddl_collation_deleted.test.lua<br>> new file mode 100644<br>> index 000000000..f36838e7a<br>> --- /dev/null<br>> +++ b/test/box/gh-3290-ddl_collation_deleted.test.lua<br>> @@ -0,0 +1,10 @@<br>> +env = require('test_run')<br>> +test_run = env.new()<br><br>unused variables<br><br>> +--<br>> +-- gh-3290: expose ICU into Lua. It uses built-in collations, that<br>> +-- must work even if a collation is deleted from _collation.<br>> +--<br>> +t = box.space._collation:delete{1}<br>> +utf8.cmp('abc', 'def')<br>> +box.space._collation:replace(t)<br><br>missed cleanup<br> </blockquote></div><div>In this place cleanup not needed.</div><div>The last line restores the original state or «cleanup»</div><div> </div><div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">--<br>sergeyb@</blockquote> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Oleg Piskunov</div></div></div><div> </div></div></BODY></HTML>