[groonga-dev,00824] Re: 連続INSERT時に全文検索でエラー

Back to archive index

Kouhei Sutou kou****@clear*****
2012年 5月 8日 (火) 14:58:53 JST


須藤です。

In <CABnM****@mail*****>
  "[groonga-dev,00823] Re: 連続INSERT時に全文検索でエラー" on Tue, 8 May 2012 13:22:12 +0900,
  "IWAI, Masaharu" <iwaim****@gmail*****> wrote:

> 2012年5月8日11:41 Kouhei Sutou <kou****@clear*****>:
>> あ、upgradeするときは一度uninstallしてからinstallし直したいん
>> です。そうしないと、新しいha_mroonga.soを読みなおしてくれない
>> ので。これだとupgrade時はuninstallもinstallもしないですよね。
> 
> あー、なるほど。だから昨日Twitterであんな感じにおっしゃってたんですね。
> 昨日須藤さんが仰ってた通りでOKですね。
> 
> https://github.com/iwaim/mroonga/commit/08718960b841ea8afe912648d0dc559029f8db86#diff-0
> # Fedoraの方の「fi」消し忘れは無視してください。

おぉ!

>>> こちらですが、上記修正を今やっても「次のリリース」のときは
>>> mysql-mroonga-2.01-0のpostunスクリプトが走るので消されちゃいますね……。
>>
>> そうですね。。。
>> 今の%postを%posttransにすればうまくいきそうな気がするんです
>> が、そうでもないですかねぇ。。。
> 
> それでも行けそうですけど、トランザクション内の方がよさそうなのと、
> そのあたりはあまり使われないので、全部そっちにもってっちゃうとspec
> 読む人がはまる懸念があるので、triggerpostunでバージョン指定いれちゃった
> 方がいいんじゃないですかねぇ。

まぁ、何回もアンインストールされてもエラーメッセージがでるだ
けで実害はないので、古いバージョンからのアップグレードのとき
は気にしなくてもよい気がしてきました!


> あと、MySQLのプラグインは詳しくないんですが、postunのやつは、
> ファイルがあるうちに実行するpreunの方が美しい気もしています。

たしかにそうですね!

  1. ifのやつと
  2. それに合わせてインデントを直したやつと
  3. postunをpreunにしたやつを

pull requestしてもらえるとうれしいです!
(できれば、それぞれを別のコミットにして計3コミットくらいに
なっているとうれしい感じがします!が、無理そうなら無理をしな
くても大丈夫です!)

-- 
須藤 功平 <kou****@clear*****>
株式会社クリアコード <http://www.clear-code.com/> (03-6231-7270)

groongaサポート:
  http://groonga.org/ja/support/
プログラミングが好きなソフトウェア開発者を募集中:
  http://www.clear-code.com/recruitment/




groonga-dev メーリングリストの案内
Back to archive index