[groonga-dev,02034] Re: grn_io_lockのタイムアウト値がハードコードされている件について

Back to archive index

yoku ts. yoku0****@gmail*****
2014年 1月 6日 (月) 12:30:43 JST


あけましておめでとうございます、yokuです。

> よいと思うので、定数で一気に変更できるようにしました。
> 定数はconfigureなら--with-lock-timeout、CMakeな
> ら-DGRN_LOCK_TIMEOUTで変更できるようにしてあります。

やった! ありがとうございます! :)
これでオレオレパッチの呪縛から逃れられます :)


2014年1月6日 11:51 Kouhei Sutou <kou****@clear*****>:
> 須藤です。
>
> In <20131****@clear*****>
>   "[groonga-dev,02020] Re: grn_io_lockのタイムアウト値がハードコードされている件について" on Fri, 20 Dec 2013 13:31:13 +0900 (JST),
>   Kouhei Sutou <kou****@clear*****> wrote:
>
>>> grn_io_lockのタイムアウト値を短くしてやることで最悪の事態
>>> (参照系まで影響が出る && mysqldのクラッシュ)は
>>> 回避できるかと思っているので短くしたいのですが、
>>> Redmineにも上がっている通りこの値はハードコードです。
>>>
>>> http://redmine.groonga.org/issues/109
>>>
>>>
>>> このままの状態だとバージョンアップのたびに手を入れる箇所が増えるので、
>>> 定数でまとめてしまいたいのですがどうでしょうか?
>>> (APIまで作る力量はありませんでした。。)
>>>
>>> https://github.com/yoku0825/groonga/compare/grn_io_lock_timeout
>>>
>>> 良さそうであればPull-requestを投げるか、大したパッチではないので、
>>> 開発チーム側でご対応いただければ幸いです(インデントや定数の名前とかの問題もありますし
>>>
>>> ご一考くださいm(_"_)m
>>
>> 一考するので少々お待ちください!
>
> 一考しました!
>
> よいと思うので、定数で一気に変更できるようにしました。
> 定数はconfigureなら--with-lock-timeout、CMakeな
> ら-DGRN_LOCK_TIMEOUTで変更できるようにしてあります。
>
> APIは用意していないのでビルド時にしか変更できませんが。。。
>
> --
> 須藤 功平 <kou****@clear*****>
> 株式会社クリアコード <http://www.clear-code.com/> (03-6231-7270)
>
> Groongaサポート:
>   http://groonga.org/ja/support/
> パッチ採用はじめました:
>   http://www.clear-code.com/recruitment/
> コミットへのコメントサービスはじめました:
>   http://www.clear-code.com/services/commit-comment.html
>
> _______________________________________________
> groonga-dev mailing list
> groon****@lists*****
> http://lists.sourceforge.jp/mailman/listinfo/groonga-dev




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