grn_pat_get_key2() returns 1 for id == 0 (GRN_ID_NIL)

[s-yata]

Description

grn_pat_get_key2() is a function that returns the length of a key, but grn_pat_get_key2() returns 1 for id == 0 (GRN_ID_NIL).

The original reports are as follows:

Groongaの参照型ベクターカラムでブランクのタグが入っていると
出力に適当なタグの１文字目だけが出力されます。

https://gist.github.com/naoa/6bd6c19244b37ced2ce9

これを出力されないようにするか、ブランクが出力できるようにできませんか？

• https://gist.github.com/naoa/6bd6c19244b37ced2ce9

こちら少しデバッグしたのですが、思ったより深いようでした。
grn_pat_get_key2にid=0を渡した時、なんかnodeが見つかるようで
長さが0でなく1で返ります。
これのせいで１つ前にかかれたバッファから１の長さでとってしまうようです。

https://github.com/groonga/groonga/blob/master/lib/pat.c#L1470-L1472

他方、grn_hash_get_key2やgrn_dat_get_key2では長さ0で返ります。

• https://github.com/groonga/groonga/blob/master/lib/pat.c#L1470-L1472

Thanks to Naoya Murakami!

[s-yata]

単純に， grn_pat_get_key() および grn_pat_get_key2() に id == 0 のときに対する例外処理を追加すれば解決できそうです．

grn_pat のノードに保存される length が 1 以上 4,096 以下なので，例外処理で回避しないと id == 0 に対応するダミーノードでも length == 1 と出てしまうようです．
なお，短いキーしか存在しない状況では immediate == 1 なノードばかりでキー保存用の領域が確保されず， pat_node_get_key() が NULL を返すために再現しませんでした．

[s-yata]

上述の修正を加えると，結果は以下のようになりました．

$ rm -rf /tmp/groonga/* && src/groonga -n /tmp/gonga/db
> table_create Tags TABLE_PAT_KEY ShortText
[[0,1447980288.16017,0.00117969512939453],true]
> table_create Memos TABLE_HASH_KEY ShortText
[[0,1447980290.73807,0.00681185722351074],true]
> column_create Memos tags COLUMN_VECTOR Tags
[[0,1447980292.89716,0.00232911109924316],true]
> load --table Memos
> [
> {"_key": "Rroonga", "tags": ["Groonga", "Ruby"]},
> {"_key": "Groonga", "tags": ["Groonga"]},
> {"_key": "Blank", "tags": [""]},
> {"_key": "Nothing"}
> ]
[[0,1447980294.74791,16.7109251022339],4]
> select Memos --output_columns 'tags'
[[0,1447980313.97276,0.00151491165161133],
 [[[4],[["tags","Tags"]],[["Groonga","Ruby"]],[["Groonga"]],[[""]],[[]]]]]

[s-yata]

ユニットテスト，コマンドテストともに問題ないことを確認しました．