Tarantool development patches archive
 help / color / mirror / Atom feed
From: Vladimir Davydov <vdavydov.dev@gmail.com>
To: imeevma@tarantool.org
Cc: tarantool-patches@freelists.org
Subject: Re: [PATCH v2 3/3] box: remove _sql_stat1 and _sql_stat4 system tables
Date: Thu, 4 Apr 2019 19:11:20 +0300	[thread overview]
Message-ID: <20190404161120.lycknoar6elfh3ct@esperanza> (raw)
In-Reply-To: <8907dd9a025b0cafe9858289e4e3e2d64618f12b.1554310018.git.imeevma@gmail.com>

On Wed, Apr 03, 2019 at 07:58:38PM +0300, imeevma@tarantool.org wrote:
> New version:
> 
> commit 8907dd9a025b0cafe9858289e4e3e2d64618f12b
> Author: Mergen Imeev <imeevma@gmail.com>
> Date:   Wed Apr 3 19:05:09 2019 +0300

When you insert a patch into an email like this, please use
`git show --format=email` so that one can apply it.

> 
>     box: remove _sql_stat1 and _sql_stat4 system tables
>     
>     These tables won't be used anymore and should be deleted.
>     
>     This patch breaks backward compatibility.

This statement deserves an explanation:

	Note, this patch breaks backward compatibility between 2.1.1 and 2.1.2,
	but that's okay as 2.1.1 was beta and we didn't recommend anyone to use
	it.

Or something like this.

The patch itself is okay although I'd squash it with patch 2.

  parent reply	other threads:[~2019-04-04 16:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-03 16:58 [PATCH v2 0/3] box: drop _sql_stat1 and _sql_stat4 tables imeevma
2019-04-03 16:58 ` [PATCH v2 1/3] sql: allocate memory for index_id in VDBE imeevma
2019-04-04 16:00   ` Vladimir Davydov
2019-04-03 16:58 ` [PATCH v2 2/3] sql: remove space_by_id() from analyze.c imeevma
2019-04-04 16:03   ` Vladimir Davydov
2019-04-04 17:41     ` Vladimir Davydov
2019-04-03 16:58 ` [PATCH v2 3/3] box: remove _sql_stat1 and _sql_stat4 system tables imeevma
2019-04-03 17:19   ` Vladimir Davydov
2019-04-03 17:38     ` Re[2]: " Мерген Имеев
2019-04-03 17:58       ` Vladimir Davydov
2019-04-03 18:04         ` [tarantool-patches] Re: [tarantool-patches] " Мерген Имеев
2019-04-04 16:11   ` Vladimir Davydov [this message]
2019-04-04 18:18     ` Mergen Imeev
2019-04-05 11:36 ` [tarantool-patches] [PATCH v2 0/3] box: drop _sql_stat1 and _sql_stat4 tables Kirill Yukhin

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=20190404161120.lycknoar6elfh3ct@esperanza \
    --to=vdavydov.dev@gmail.com \
    --cc=imeevma@tarantool.org \
    --cc=tarantool-patches@freelists.org \
    --subject='Re: [PATCH v2 3/3] box: remove _sql_stat1 and _sql_stat4 system tables' \
    /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