Tarantool development patches archive
 help / color / mirror / Atom feed
From: "v.shpilevoy@tarantool.org" <v.shpilevoy@tarantool.org>
To: tarantool-patches@freelists.org
Cc: Vladimir Davydov <vdavydov.dev@gmail.com>
Subject: Re: [tarantool-patches] Re: [PATCH v2 1/1] netbox: show is_nullable and collation fields
Date: Mon, 26 Mar 2018 20:44:27 +0300	[thread overview]
Message-ID: <8B0870B9-34B3-4525-AF9C-702FB2410739@tarantool.org> (raw)
In-Reply-To: <02f53c50-c3eb-9e8c-6d15-9adfb7d23c0d@tarantool.org>

Looks good to me. Vova, please, take a look.

> 26 марта 2018 г., в 20:41, Kirill Shcherbatov <kshcherbatov@tarantool.org> написал(а):
> 
> diff --git a/test/box/net.box.test.lua b/test/box/net.box.test.lua
> index fe45721..3b4ce05 100644
> --- a/test/box/net.box.test.lua
> +++ b/test/box/net.box.test.lua
> @@ -799,11 +799,9 @@ box.schema.user.revoke('guest','read,write,execute','universe')
> 
>  space:drop()
> 
> -
>  --
>  -- gh-3256 net.box is_nullable and collation options output
>  --
> -
>  space = box.schema.create_space('test')
>  box.schema.user.grant('guest','read,write,execute','space', 'test')
>  pk = space:create_index('pk')
> @@ -821,4 +819,5 @@ c = net:connect(box.cfg.listen)
>  c.space.test.index.sk.parts
>  c:close()
>  box.internal.collation.drop('test')
> -space:drop()
> \ No newline at end of file
> +space:drop()
> +
> 
> 
> On 26.03.2018 20:24, v.shpilevoy@tarantool.org wrote:
>> See below 2 minor comments.
>> 
>>> 26 марта 2018 г., в 20:20, Kirill Shcherbatov <kshcherbatov@tarantool.org> написал(а):
>>> 
>>> Fixes #3256
>>> ---
>>> 
>>> Issue: https://github.com/tarantool/tarantool/issues/3256
>>> Branch: https://github.com/tarantool/tarantool/compare/gh-3256-net-box-nullable-collation-options-output
>>> 
>>> src/box/lua/net_box.lua   |  6 ++++-
>>> test/box/net.box.result   | 65 +++++++++++++++++++++++++++++++++++++++++++++++
>>> test/box/net.box.test.lua | 24 +++++++++++++++++
>>> 3 files changed, 94 insertions(+), 1 deletion(-)
>>> 
>>> diff --git a/test/box/net.box.test.lua b/test/box/net.box.test.lua
>>> index c7541ed..fe45721 100644
>>> --- a/test/box/net.box.test.lua
>>> +++ b/test/box/net.box.test.lua
>>> @@ -798,3 +798,27 @@ c.space.test.index.test_index ~= nil
>>> box.schema.user.revoke('guest','read,write,execute','universe')
>>> 
>>> space:drop()
>>> +
>>> +
>> 1. Please, do not put multiple empty lines, when it is not necessary.
>> 
>>> +--
>>> +-- gh-3256 net.box is_nullable and collation options output
>>> +--
>>> +
>>> +space = box.schema.create_space('test')
>>> +box.schema.user.grant('guest','read,write,execute','space', 'test')
>>> +pk = space:create_index('pk')
>>> +sk = space:create_index('sk', {parts = {{2, 'unsigned', is_nullable = true}}})
>>> +c = net:connect(box.cfg.listen)
>>> +c.space.test.index.sk.parts
>>> +c:close()
>>> +space:drop()
>>> +
>>> +space = box.schema.create_space('test')
>>> +box.schema.user.grant('guest','read,write,execute','space', 'test')
>>> +box.internal.collation.create('test', 'ICU', 'ru-RU')
>>> +sk = space:create_index('sk', { type = 'tree', parts = {{1, 'str', collation = 'test'}}, unique = true })
>>> +c = net:connect(box.cfg.listen)
>>> +c.space.test.index.sk.parts
>>> +c:close()
>>> +box.internal.collation.drop('test')
>>> +space:drop()
>>> \ No newline at end of file
>> 2. Please, add an empty line at the end of the file.
>> 
>>> -- 
>>> 2.7.4
>>> 
>> 
> 
> 

  reply	other threads:[~2018-03-26 17:44 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <cf039f93de4ddc5ae7d7d1578ec4c436d8a37d80.1522083375.git.kshcherbatov@tarantool.org>
2018-03-26 17:24 ` v.shpilevoy
2018-03-26 17:41   ` Kirill Shcherbatov
2018-03-26 17:44     ` v.shpilevoy [this message]
2018-03-27 16:26     ` Vladimir Davydov
2018-03-28  7:40       ` Kirill Shcherbatov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8B0870B9-34B3-4525-AF9C-702FB2410739@tarantool.org \
    --to=v.shpilevoy@tarantool.org \
    --cc=tarantool-patches@freelists.org \
    --cc=vdavydov.dev@gmail.com \
    --subject='Re: [tarantool-patches] Re: [PATCH v2 1/1] netbox: show is_nullable and collation fields' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox