Tarantool development patches archive
 help / color / mirror / Atom feed
From: Mergen Imeev via Tarantool-patches <tarantool-patches@dev.tarantool.org>
To: Vladislav Shpilevoy <v.shpilevoy@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org
Subject: Re: [Tarantool-patches] [PATCH v1 7/7] sql: fix comparison with SCALAR value
Date: Fri, 13 Aug 2021 01:23:52 +0300	[thread overview]
Message-ID: <20210812222352.GA206429@tarantool.org> (raw)
In-Reply-To: <bfdedada-9a41-310f-c3cc-31c752c24df8@tarantool.org>

On Thu, Aug 12, 2021 at 09:50:35PM +0300, Vladislav Shpilevoy wrote:
> Thanks for the patch!
> 
> On 11.08.2021 19:01, Mergen Imeev via Tarantool-patches wrote:
> > After this patch, SCALAR values will be able to be compared with values
> > of any other scalar type. The comparison will be done according to the
> > SCALAR rules, which means boolean values < numeric values < string
> > values < binary values < uuid values.
> > 
> > Closes #6221
> 
> Could you please add a docbot request?

Fixed:


commit 6179b16e422a1349ebe5b0ed340c81d8f5a4e6a2
Author: Mergen Imeev <imeevma@gmail.com>
Date:   Wed Aug 11 14:58:23 2021 +0300

    sql: fix comparison with SCALAR value
    
    After this patch, SCALAR values will be able to be compared with values
    of any other scalar type. The comparison will be done according to the
    SCALAR rules, which means boolean values < numeric values < string
    values < binary values < uuid values.
    
    Closes #6221
    
    @TarantoolBot document
    Title: SCALAR and NUMBER values in SQL
    
    SCALAR values can now be compared with values of any other scalar type,
    but cannot be implicitly cast to any other scalar type. This means that
    SCALAR values cannot participate in arithmetic, bitwise operations,
    concatenation, or functions that, by definition, do not accept SCALAR
    values.
    
    NUMBER values now also cannot be implicitly cast to any other numeric
    type, which means that NUMBER values cannot participate in arithmetic
    and bitwise operations, or in functions that, by definition, do not
    accept NUMBER values.

  reply	other threads:[~2021-08-12 22:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-11 16:01 [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 1/7] sql: remove enum field_type from struct Mem Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 2/7] sql: re-introduce NUMBER and SCALAR meta-types Mergen Imeev via Tarantool-patches
2021-08-12 18:51   ` Vladislav Shpilevoy via Tarantool-patches
2021-08-12 22:22     ` Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 3/7] sql: disallow implicit cast from NUMBER and SCALAR Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 4/7] sql: disallow arithmetic for " Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 5/7] sql: disallow bitwise " Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 6/7] sql: disallow concatination for SCALAR Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 7/7] sql: fix comparison with SCALAR value Mergen Imeev via Tarantool-patches
2021-08-12 18:50   ` Vladislav Shpilevoy via Tarantool-patches
2021-08-12 22:23     ` Mergen Imeev via Tarantool-patches [this message]
2021-08-13  3:12 [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL Mergen Imeev via Tarantool-patches
2021-08-13  3:13 ` [Tarantool-patches] [PATCH v1 7/7] sql: fix comparison with SCALAR value Mergen Imeev via Tarantool-patches

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=20210812222352.GA206429@tarantool.org \
    --to=tarantool-patches@dev.tarantool.org \
    --cc=imeevma@tarantool.org \
    --cc=v.shpilevoy@tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH v1 7/7] sql: fix comparison with SCALAR value' \
    /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