Kouhei Sutou
kou****@clear*****
2014年 1月 6日 (月) 11:51:47 JST
須藤です。 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
