[groonga-dev,02349] Re: 管理画面を改良したい!

アーカイブの一覧に戻る

Kouhei Sutou kou****@clear*****
2014年 5月 23日 (金) 23:50:18 JST


須藤です。

In <CANM+HhdT_X6XQ4Si-VmF80ey=+YOWenKLoXAyd4+=M4swi****@mail*****>
  "[groonga-dev,02348] Re: 管理画面を改良したい!" on Fri, 23 May 2014 22:05:56 +0900,
  Naoya Murakami <visio****@gmail*****> wrote:

> わざわざ詳細な解説ありがとうございます!すごく勉強になります。

よかったです!
こちらも助かっています!

> grn_dbの配下には、grn_typeとgrn_tableとgrn_procとかが
> いて、全部grn_table_cursor_openで動かせて、
> obj->header.typeをみるとgrn_objの種類が判定できるんですね!

そうです!

>> 戻り値は、今は["NormalizerAuto", "NormalizerNFKC51"]のように
>> 名前の配列になっていますが、
>>
>>   [
>>     {"name": "NormalizerAuto"},
>>     {"name": "NormalizerNFKC51"}
>>   ]
>>
>> のようにオブジェクトにしておいた方がいいかなぁと思います。理
>> 由は、ノーマライザー毎に新しい情報を追加したいときにAPIを変
>> えずに対応できるからです。
>>
>> 今は、名前だけで十分だとは思うんですが、もしかしたら、どのプ
>> ラグインが追加したのか、みたいな情報もあると助かることがある
>> かも?とか思いました。
>>
> 
> 上記を考慮しなおして書き直して、(といっても、ほとんど書いて
> いただいている通りですが)Pull Requestをしました。
> 
> https://github.com/groonga/groonga/pull/171
> 
> よければ、ご検討ください。

ありがとうございます!
取り込みました!

で、table_listとかcolumn_listとか見なおしてみたんですが、

  [
    ["id", "UInt32"],
    ["name", "ShortText"]
  ],
  [
    1,
    "TokenMecab"
  ],
  [
    ...
  ]

みたいな感じで、tokenizer_listとかとフォーマットが違いました
ね。。。

個人的にはオブジェクトの配列の方が使いやすいと思うんですけど、
一貫性がなくなるのがアレなんですよね。。。command_version>=2
以降で使えるとかにするのはどうかしら、とか検討します!


あと、"name"の他に"id"も入っていてもいいかも、と思いました。
table_listとかには入っていたので。必要になってからでもよさげ
な気はするので、後ででもいいかしら。

> (ドキュメントの修正が一番大変でした。。すいません、翻訳ファイル
> がうまく効かせられず、まだ追加できていません。。今後、うまくでき
> そうなら追加します。)

わかりました!
そうなんですよ。ドキュメント、大変なんですよねぇ。
なので、手伝ってもらえるとすごく助かるんですよ。

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

Groongaサポート:
  http://groonga.org/ja/support/
パッチ採用:
  http://www.clear-code.com/recruitment/
コードリーダー育成支援:
  http://www.clear-code.com/services/code-reader/




groonga-dev メーリングリストの案内
アーカイブの一覧に戻る