[Groonga-mysql-commit] mroonga/mroonga at be59bd6 [master] Add a forgotten check of error status

Back to archive index

Kouhei Sutou kou****@clear*****
Tue Feb 17 11:11:14 JST 2015


おぉ。。。
これ、4.09に私がエンバグさせちゃったやつですね。。。

修正ありがとうございます!

MySQL 5.5で開発していて気づいていませんでした。。。
ちゃんとTravis CIの結果を確認しておかないとダメですね。。。

少なくともリリース前にはTravis CIの結果を確認するべきですね。。。

In <be59bd65a71b96e0d2e1847e543ce8a7785195dd �� jenkins.clear-code.com>
  "[Groonga-mysql-commit] mroonga/mroonga �� be59bd6 [master] Add a forgotten check of error status" on Tue, 17 Feb 2015 10:59:40 +0900,
  shibakentoku <null+mroonga �� clear-code.com> wrote:

> shibakentoku	2015-02-17 10:59:40 +0900 (Tue, 17 Feb 2015)
> 
>   New Revision: be59bd65a71b96e0d2e1847e543ce8a7785195dd
>   https://github.com/mroonga/mroonga/commit/be59bd65a71b96e0d2e1847e543ce8a7785195dd
> 
>   Message:
>     Add a forgotten check of error status
> 
>   Modified files:
>     ha_mroonga.cpp
> 
>   Modified: ha_mroonga.cpp (+1 -1)
> ===================================================================
> --- ha_mroonga.cpp    2015-02-17 00:02:08 +0900 (2912644)
> +++ ha_mroonga.cpp    2015-02-17 10:59:40 +0900 (3ff89d0)
> @@ -13696,7 +13696,7 @@ bool ha_mroonga::storage_inplace_alter_table_index(
>      if (error == HA_ERR_FOUND_DUPP_UNIQUE) {
>        my_printf_error(ER_DUP_UNIQUE, ER(ER_DUP_UNIQUE), MYF(0),
>                        table_share->table_name);
> -    } else {
> +    } else if (error) {
>        my_message(error, "failed to create multiple column index", MYF(0));
>      }
>      for (i = 0; i < n_columns; ++i) {




More information about the Groonga-mysql-commit mailing list
Back to archive index