rroonga 1.0.1 → 1.0.2

data/ext/groonga/mkmf.log

@@ -1,6 +1,6 @@
 checking for GCC... -------------------- yes
 
-"gcc -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -I/tmp/local/include/groonga   -c conftest.c"
+"cc -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC     -c conftest.c"
 checked program was:
 /* begin */
 1: /*top*/
@@ -18,7 +18,7 @@ checking for Win32 OS... -------------------- no
 
 have_macro: checking for HAVE_RUBY_ST_H in ruby.h... -------------------- no
 
-"gcc -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -I/tmp/local/include/groonga -Wall   -c conftest.c"
+"cc -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -Wall -I/tmp/local/include/groonga     -c conftest.c"
 conftest.c:4:3: error: #error 
 conftest.c:5: error: expected identifier or ‘(’ before ‘>>’ token
 checked program was:
@@ -35,7 +35,7 @@ checked program was:
 
 have_header: checking for ruby/st.h... -------------------- no
 
-"gcc -E -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -I/tmp/local/include/groonga -Wall  conftest.c -o conftest.i"
+"cc -E -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -Wall -I/tmp/local/include/groonga    conftest.c -o conftest.i"
 conftest.c:1:21: error: ruby/st.h: そのようなファイルやディレクトリはありません
 checked program was:
 /* begin */
@@ -46,7 +46,7 @@ checked program was:
 
 have_func: checking for rb_errinfo() in ruby.h... -------------------- no
 
-"gcc -o conftest -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -I/tmp/local/include/groonga -Wall conftest.c  -L. -L/usr/lib -L.  -rdynamic -Wl,-export-dynamic -L/tmp/local/lib     -lgroonga -lruby1.8-static  -lgroonga -lpthread -lrt -ldl -lcrypt -lm   -lc"
+"cc -o conftest -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -Wall -I/tmp/local/include/groonga   conftest.c  -L. -L/usr/lib -L.  -rdynamic -Wl,-export-dynamic -L/tmp/local/lib     -lgroonga   -lruby1.8-static  -lgroonga   -lpthread -lrt -ldl -lcrypt -lm   -lc"
 conftest.c: In function ‘t’:
 conftest.c:5: error: ‘rb_errinfo’ undeclared (first use in this function)
 conftest.c:5: error: (Each undeclared identifier is reported only once
@@ -60,10 +60,10 @@ checked program was:
 5: int t() { void ((*volatile p)()); p = (void ((*)()))rb_errinfo; return 0; }
 /* end */
 
-"gcc -o conftest -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -I/tmp/local/include/groonga -Wall conftest.c  -L. -L/usr/lib -L.  -rdynamic -Wl,-export-dynamic -L/tmp/local/lib     -lgroonga -lruby1.8-static  -lgroonga -lpthread -lrt -ldl -lcrypt -lm   -lc"
+"cc -o conftest -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -Wall -I/tmp/local/include/groonga   conftest.c  -L. -L/usr/lib -L.  -rdynamic -Wl,-export-dynamic -L/tmp/local/lib     -lgroonga   -lruby1.8-static  -lgroonga   -lpthread -lrt -ldl -lcrypt -lm   -lc"
 conftest.c: In function ‘t’:
 conftest.c:5: warning: implicit declaration of function ‘rb_errinfo’
-/tmp/cc9zPpDk.o: In function `t':
+/tmp/ccRn7Jyd.o: In function `t':
 /home/kou/work/ruby/rroonga.18/ext/groonga/conftest.c:5: undefined reference to `rb_errinfo'
 collect2: ld returned 1 exit status
 checked program was:
@@ -79,7 +79,7 @@ checked program was:
 
 have_type: checking for enum ruby_value_type in ruby.h... -------------------- no
 
-"gcc -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -I/tmp/local/include/groonga -Wall   -c conftest.c"
+"cc -I. -I/usr/lib/ruby/1.8/x86_64-linux -I/home/kou/work/ruby/rroonga.18/ext/groonga    -fno-strict-aliasing -g -g -O2  -fPIC   -Wall -I/tmp/local/include/groonga     -c conftest.c"
 conftest.c:5: error: invalid application of ‘sizeof’ to incomplete type ‘conftest_type’ 
 conftest.c:5: error: size of array ‘conftestval’ is negative
 checked program was:

data/ext/groonga/rb-grn-column.c

@@ -552,6 +552,68 @@ rb_grn_column_is_locked (int argc, VALUE *argv, VALUE self)
     return CBOOL2RVAL(grn_obj_is_locked(context, column));
 }
 
+/*
+ * Document-method: reference_column?
+ *
+ * call-seq:
+ *   column.reference_column? -> true/false
+ *
+ * _column_の値がテーブルのレコードとなる場合は+true+を返し、
+ * そうでない場合は+false+を返す。
+ */
+static VALUE
+rb_grn_column_reference_column_p (VALUE self)
+{
+    grn_ctx *context;
+    grn_obj *column;
+    grn_id range_id;
+    grn_obj *range;
+    unsigned short int type;
+
+    rb_grn_column_deconstruct(SELF(self), &column, &context,
+			     NULL, NULL,
+			     NULL, NULL, NULL);
+
+    range_id = grn_obj_get_range(context, column);
+    range = grn_ctx_at(context, range_id);
+    type = range->header.type;
+    grn_obj_unlink(context, range);
+    switch (type) {
+      case GRN_TABLE_HASH_KEY:
+      case GRN_TABLE_PAT_KEY:
+      case GRN_TABLE_NO_KEY:
+	return Qtrue;
+      default:
+	return Qfalse;
+    }
+}
+
+/*
+ * Document-method: index_column?
+ *
+ * call-seq:
+ *   column.index_column? -> true/false
+ *
+ * _column_がGroonga::IndexColumnの場合は+true+を返し、
+ * そうでない場合は+false+を返す。
+ */
+static VALUE
+rb_grn_column_index_column_p (VALUE self)
+{
+    grn_ctx *context;
+    grn_obj *column;
+
+    rb_grn_column_deconstruct(SELF(self), &column, &context,
+			     NULL, NULL,
+			     NULL, NULL, NULL);
+
+    if (column->header.type == GRN_COLUMN_INDEX) {
+	return Qtrue;
+    } else {
+	return Qfalse;
+    }
+}
+
 void
 rb_grn_init_column (VALUE mGrn)
 {
@@ -566,6 +628,10 @@ rb_grn_init_column (VALUE mGrn)
     rb_define_method(rb_cGrnColumn, "unlock", rb_grn_column_unlock, -1);
     rb_define_method(rb_cGrnColumn, "clear_lock", rb_grn_column_clear_lock, -1);
     rb_define_method(rb_cGrnColumn, "locked?", rb_grn_column_is_locked, -1);
+    rb_define_method(rb_cGrnColumn, "reference_column?",
+		     rb_grn_column_reference_column_p, 0);
+    rb_define_method(rb_cGrnColumn, "index_column?",
+		     rb_grn_column_index_column_p, 0);
 
     rb_grn_init_fix_size_column(mGrn);
     rb_grn_init_variable_size_column(mGrn);

data/ext/groonga/rb-grn-context.c

@@ -56,19 +56,27 @@ rb_grn_context_from_ruby_object (VALUE object)
     return rb_grn_context->context;
 }
 
-void
-rb_grn_context_fin (grn_ctx *context)
+static void
+rb_grn_context_unlink_database (grn_ctx *context)
 {
     grn_obj *database;
 
-    if (context->stat == GRN_CTX_FIN)
-	return;
-
     database = grn_ctx_db(context);
     debug("context:database: %p:%p\n", context, database);
     if (database && database->header.type == GRN_DB) {
+	debug("context:database: %p:%p: unlink\n", context, database);
 	grn_obj_unlink(context, database);
     }
+    debug("context:database: %p:%p: done\n", context, database);
+}
+
+void
+rb_grn_context_fin (grn_ctx *context)
+{
+    if (context->stat == GRN_CTX_FIN)
+	return;
+
+    rb_grn_context_unlink_database(context);
     grn_ctx_fin(context);
 }
 
@@ -92,6 +100,33 @@ rb_grn_context_alloc (VALUE klass)
     return Data_Wrap_Struct(klass, NULL, rb_grn_context_free, NULL);
 }
 
+static grn_obj *
+rb_grn_context_finalizer (grn_ctx *context, int n_args, grn_obj **grn_objects,
+			  grn_user_data *user_data)
+{
+    RbGrnContext *rb_grn_context;
+
+    if (rb_grn_exited)
+	return NULL;
+
+    rb_grn_context = user_data->ptr;
+
+    GRN_CTX_USER_DATA(context)->ptr = NULL;
+    grn_ctx_set_finalizer(context, NULL);
+
+    debug("context-finalize: %p:%p:%p\n",
+	  context, rb_grn_context, rb_grn_context->context);
+
+    rb_grn_context->context = NULL;
+
+    rb_grn_context_unlink_database(context);
+
+    debug("context-finalize: %p:%p:%p: done\n",
+	  context, rb_grn_context, rb_grn_context->context);
+
+    return NULL;
+}
+
 VALUE
 rb_grn_context_to_exception (grn_ctx *context, VALUE related_object)
 {
@@ -301,6 +336,7 @@ rb_grn_context_initialize (int argc, VALUE *argv, VALUE self)
     rb_grn_context_check(context, self);
 
     GRN_CTX_USER_DATA(context)->ptr = rb_grn_context;
+    grn_ctx_set_finalizer(context, rb_grn_context_finalizer);
 
     if (!NIL_P(rb_encoding)) {
 	grn_encoding encoding;
@@ -380,6 +416,52 @@ rb_grn_context_set_encoding (VALUE self, VALUE rb_encoding)
     return rb_encoding;
 }
 
+/*
+ * call-seq:
+ *   context.support_zlib?
+ *
+ * groongaがZlibサポート付きでビルドされていれば+true+、そう
+ * でなければ+false+を返す。
+ */
+static VALUE
+rb_grn_context_support_zlib_p (VALUE self)
+{
+    VALUE rb_support_p;
+    grn_ctx *context;
+    grn_obj support_p;
+
+    context = SELF(self);
+    GRN_BOOL_INIT(&support_p, 0);
+    grn_obj_get_info(context, NULL, GRN_INFO_SUPPORT_ZLIB, &support_p);
+    rb_support_p = CBOOL2RVAL(GRN_BOOL_VALUE(&support_p));
+    GRN_OBJ_FIN(context, &support_p);
+
+    return rb_support_p;
+}
+
+/*
+ * call-seq:
+ *   context.support_lzo?
+ *
+ * groongaがLZOサポート付きでビルドされていれば+true+、そう
+ * でなければ+false+を返す。
+ */
+static VALUE
+rb_grn_context_support_lzo_p (VALUE self)
+{
+    VALUE rb_support_p;
+    grn_ctx *context;
+    grn_obj support_p;
+
+    context = SELF(self);
+    GRN_BOOL_INIT(&support_p, 0);
+    grn_obj_get_info(context, NULL, GRN_INFO_SUPPORT_LZO, &support_p);
+    rb_support_p = CBOOL2RVAL(GRN_BOOL_VALUE(&support_p));
+    GRN_OBJ_FIN(context, &support_p);
+
+    return rb_support_p;
+}
+
 /*
  * call-seq:
  *   context.database -> Groonga::Database
@@ -639,6 +721,11 @@ rb_grn_init_context (VALUE mGrn)
     rb_define_method(cGrnContext, "encoding", rb_grn_context_get_encoding, 0);
     rb_define_method(cGrnContext, "encoding=", rb_grn_context_set_encoding, 1);
 
+    rb_define_method(cGrnContext, "support_zlib?",
+		     rb_grn_context_support_zlib_p, 0);
+    rb_define_method(cGrnContext, "support_lzo?",
+		     rb_grn_context_support_lzo_p, 0);
+
     rb_define_method(cGrnContext, "database", rb_grn_context_get_database, 0);
 
     rb_define_method(cGrnContext, "[]", rb_grn_context_array_reference, 1);

data/ext/groonga/rb-grn-database.c

@@ -1,6 +1,6 @@
 /* -*- c-file-style: "ruby" -*- */
 /*
-  Copyright (C) 2009  Kouhei Sutou <kou@clear-code.com>
+  Copyright (C) 2009-2010  Kouhei Sutou <kou@clear-code.com>
 
   This library is free software; you can redistribute it and/or
   modify it under the terms of the GNU Lesser General Public
@@ -441,6 +441,27 @@ rb_grn_database_is_locked (VALUE self)
     return CBOOL2RVAL(grn_obj_is_locked(context, database));
 }
 
+/*
+ * Document-method: touch
+ *
+ * call-seq:
+ *   database.touch
+ *
+ * _database_の最終更新時刻を現在時刻にする。
+ */
+static VALUE
+rb_grn_database_touch (VALUE self)
+{
+    grn_ctx *context;
+    grn_obj *database;
+
+    rb_grn_database_deconstruct(SELF(self), &database, &context,
+				NULL, NULL, NULL, NULL);
+
+    grn_db_touch(context, database);
+    return Qnil;
+}
+
 void
 rb_grn_init_database (VALUE mGrn)
 {
@@ -469,4 +490,6 @@ rb_grn_init_database (VALUE mGrn)
     rb_define_method(rb_cGrnDatabase, "clear_lock",
 		     rb_grn_database_clear_lock, 0);
     rb_define_method(rb_cGrnDatabase, "locked?", rb_grn_database_is_locked, 0);
+
+    rb_define_method(rb_cGrnDatabase, "touch", rb_grn_database_touch, 0);
 }

data/ext/groonga/rb-grn-exception.c

@@ -93,6 +93,7 @@ static VALUE eGrnTooSmallOffset;
 static VALUE eGrnTooLargeOffset;
 static VALUE eGrnTooSmallLimit;
 static VALUE eGrnCASError;
+static VALUE eGrnUnsupportedCommandVersion;
 
 VALUE
 rb_grn_rc_to_exception (grn_rc rc)
@@ -316,6 +317,9 @@ rb_grn_rc_to_exception (grn_rc rc)
       case GRN_CAS_ERROR:
         exception = eGrnCASError;
         break;
+      case GRN_UNSUPPORTED_COMMAND_VERSION:
+        exception = eGrnUnsupportedCommandVersion;
+        break;
     }
 
     if (NIL_P(exception))
@@ -546,6 +550,9 @@ rb_grn_rc_to_message (grn_rc rc)
       case GRN_CAS_ERROR:
         message = "CAS error";
         break;
+      case GRN_UNSUPPORTED_COMMAND_VERSION:
+        message = "unsupported command version";
+        break;
     }
 
     if (!message)
@@ -1166,4 +1173,12 @@ rb_grn_init_exception (VALUE mGrn)
      * CAS（Compare and Swap）が失敗したときに発生する。
      */
     eGrnCASError = rb_define_class_under(mGrn, "CASError", rb_eGrnError);
+
+    /*
+     * Document-class: Groonga::UnsupportedCommandVersion
+     *
+     * 未サポートのコマンドバージョンを指定したときに発生する。
+     */
+    eGrnUnsupportedCommandVersion =
+	rb_define_class_under(mGrn, "UnsupportedCommandVersion", rb_eGrnError);
 }

data/ext/groonga/rb-grn-hash.c

@@ -81,7 +81,7 @@ VALUE rb_cGrnHash;
  *   ともでき、その場合は自動的にGroonga::Recordからレコード
  *   IDを取得する。
  *
- *   省略した場合は文字列をキーとして使用する。この場合、
+ *   省略した場合はShortText型をキーとして使用する。この場合、
  *   4096バイトまで使用可能である。
  *
  * [+:value_type+]
@@ -192,7 +192,7 @@ rb_grn_hash_s_create (int argc, VALUE *argv, VALUE klass)
 	flags |= GRN_OBJ_KEY_NORMALIZE;
 
     if (NIL_P(rb_key_type)) {
-	flags |= GRN_OBJ_KEY_VAR_SIZE;
+	key_type = grn_ctx_at(context, GRN_DB_SHORT_TEXT);
     } else {
 	key_type = RVAL2GRNOBJECT(rb_key_type, &context);
     }

data/ext/groonga/rb-grn-index-column.c

@@ -267,7 +267,8 @@ rb_grn_index_column_get_sources (VALUE self)
 }
 
 static grn_id
-resolve_source_id (grn_ctx *context, grn_obj *column, VALUE rb_source)
+resolve_source_id (grn_ctx *context, grn_obj *column, grn_id range_id,
+		   VALUE rb_source)
 {
     grn_id source_id;
 
@@ -316,7 +317,7 @@ resolve_source_id (grn_ctx *context, grn_obj *column, VALUE rb_source)
 		rb_raise(rb_eArgError,
 			 "source accessor must be '_key': <%s>", name);
 	    }
-	    source_id = grn_obj_id(context, column);
+	    source_id = range_id;
 	} else {
 	    source_id = grn_obj_id(context, source);
 	}
@@ -339,19 +340,22 @@ rb_grn_index_column_set_sources (VALUE self, VALUE rb_sources)
     grn_obj *column;
     int i, n;
     VALUE *rb_source_values;
+    grn_id range_id;
     grn_id *sources;
     grn_rc rc;
 
     rb_grn_index_column_deconstruct(SELF(self), &column, &context,
 				    NULL, NULL,
-				    NULL, NULL, NULL, NULL,
+				    NULL, NULL,
+				    &range_id, NULL,
 				    NULL, NULL);
 
     n = RARRAY_LEN(rb_sources);
     rb_source_values = RARRAY_PTR(rb_sources);
     sources = ALLOCA_N(grn_id, n);
     for (i = 0; i < n; i++) {
-	sources[i] = resolve_source_id(context, column, rb_source_values[i]);
+	sources[i] = resolve_source_id(context, column, range_id,
+				       rb_source_values[i]);
     }
 
     {

data/ext/groonga/rb-grn-object.c

@@ -19,7 +19,7 @@
 /*
  * Document-class: Groonga::Object
  *
- * Ruby/groongaが提供するクラスのベースとなるクラス。
+ * rroongaが提供するクラスのベースとなるクラス。
  * Groonga::ContextとGroonga::Logger以外はGroonga::Objectを継
  * 承している。
  */
@@ -829,7 +829,9 @@ rb_grn_object_inspect_content (VALUE self, VALUE inspected)
     object = rb_grn_object->object;
 
     rb_str_cat2(inspected, " ");
-    if (object) {
+    if (rb_grn_exited) {
+	rb_str_cat2(inspected, "(finished)");
+    } else if (object) {
 	rb_grn_object_inspect_object_content(inspected, context, object);
     } else {
 	rb_str_cat2(inspected, "(closed)");

data/ext/groonga/rb-grn-patricia-trie.c

@@ -90,7 +90,7 @@ VALUE rb_cGrnPatriciaTrie;
  *   ともでき、その場合は自動的にGroonga::Recordからレコード
  *   IDを取得する。
  *
- *   省略した場合は文字列をキーとして使用する。この場合、
+ *   省略した場合はShortText型をキーとして使用する。この場合、
  *   4096バイトまで使用可能である。
  *
  * [+:value_type+]
@@ -206,7 +206,7 @@ rb_grn_patricia_trie_s_create (int argc, VALUE *argv, VALUE klass)
 	flags |= GRN_OBJ_KEY_WITH_SIS;
 
     if (NIL_P(rb_key_type)) {
-	flags |= GRN_OBJ_KEY_VAR_SIZE;
+	key_type = grn_ctx_at(context, GRN_DB_SHORT_TEXT);
     } else {
 	key_type = RVAL2GRNOBJECT(rb_key_type, &context);
     }

data/ext/groonga/rb-grn-snippet.c

@@ -302,6 +302,7 @@ rb_grn_snippet_execute (VALUE self, VALUE rb_string)
 
     rc = grn_snip_exec(context, snippet, string, string_length,
                        &n_results, &max_tagged_length);
+    rb_grn_context_check(context, self);
     rb_grn_rc_check(rc, self);
 
     rb_results = rb_ary_new2(n_results);

data/ext/groonga/rb-grn-table-key-support.c

@@ -457,23 +457,6 @@ rb_grn_table_key_support_get_column_value (int argc, VALUE *argv, VALUE self)
     return rb_grn_table_get_column_value_raw(self, id, rb_name);
 }
 
-/*
- * call-seq:
- *   table.find(key) -> Groonga::Record
- *
- * テーブルの_key_に対応するレコードを返す。
- *
- * 0.9.0から非推奨。代わりにtable[key]を使うこと。
- */
-static VALUE
-rb_grn_table_key_support_find (VALUE self, VALUE rb_key)
-{
-    rb_warn("#find is deprecated. Use #[] instead: %s",
-	    rb_grn_inspect(self));
-
-    return rb_grn_table_key_support_array_reference(self, rb_key);
-}
-
 static VALUE
 rb_grn_table_key_support_get_value_by_key (VALUE self, VALUE rb_key)
 {
@@ -691,6 +674,25 @@ rb_grn_table_key_support_normalize_key_p (VALUE self)
     return CBOOL2RVAL(table->header.flags & GRN_OBJ_KEY_NORMALIZE);
 }
 
+/*
+ * call-seq:
+ *   table.support_key? -> true/false
+ *
+ * テーブルでキーを使えるなら+true+、使えないなら+false+を返
+ * す。キーを使えないテーブルはGroonga::Arrayだけ。
+ */
+static VALUE
+rb_grn_table_key_support_support_key_p (VALUE self)
+{
+    grn_obj *table;
+
+    rb_grn_table_key_support_deconstruct(SELF(self), &table, NULL,
+					 NULL, NULL, NULL,
+					 NULL, NULL, NULL,
+					 NULL);
+    return table->header.domain == GRN_ID_NIL ? Qfalse : Qtrue;
+}
+
 void
 rb_grn_init_table_key_support (VALUE mGrn)
 {
@@ -709,8 +711,6 @@ rb_grn_init_table_key_support (VALUE mGrn)
     rb_define_method(rb_mGrnTableKeySupport, "delete",
 		     rb_grn_table_key_support_delete, 1);
 
-    rb_define_method(rb_mGrnTableKeySupport, "find",
-		     rb_grn_table_key_support_find, 1);
     rb_define_method(rb_mGrnTableKeySupport, "[]",
 		     rb_grn_table_key_support_array_reference, 1);
     rb_define_method(rb_mGrnTableKeySupport, "[]=",
@@ -733,4 +733,7 @@ rb_grn_init_table_key_support (VALUE mGrn)
 
     rb_define_method(rb_mGrnTableKeySupport, "normalize_key?",
 		     rb_grn_table_key_support_normalize_key_p, 0);
+
+    rb_define_method(rb_mGrnTableKeySupport, "support_key?",
+		     rb_grn_table_key_support_support_key_p, 0);
 }

data/ext/groonga/rb-grn-table.c

@@ -75,6 +75,7 @@ rb_grn_table_bind (RbGrnTable *rb_grn_table,
     rb_grn_object = RB_GRN_OBJECT(rb_grn_table);
     rb_grn_table->value = grn_obj_open(context, GRN_BULK, 0,
 				       rb_grn_object->range_id);
+    rb_grn_table->columns = Qnil;
     rb_grn_table->columns = rb_ary_new();
 }
 
@@ -202,7 +203,7 @@ rb_grn_table_inspect_content (VALUE self, VALUE inspected)
 
 /*
  * call-seq:
- *   _table_.inspect -> String
+ *   table.inspect -> String
  *
  * テーブルの中身を人に見やすい文字列で返す。
  */
@@ -428,8 +429,14 @@ rb_grn_table_define_index_column (int argc, VALUE *argv, VALUE self)
 	flags |= GRN_OBJ_PERSISTENT;
     }
 
-    if (RVAL2CBOOL(rb_with_section))
+    if (NIL_P(rb_with_section)) {
+	if (TYPE(rb_sources) == T_ARRAY && RARRAY_LEN(rb_sources) > 1) {
+	    flags |= GRN_OBJ_WITH_SECTION;
+	}
+    } else if (RVAL2CBOOL(rb_with_section)) {
 	flags |= GRN_OBJ_WITH_SECTION;
+    }
+
 
     if (RVAL2CBOOL(rb_with_weight))
 	flags |= GRN_OBJ_WITH_WEIGHT;
@@ -734,6 +741,11 @@ rb_grn_table_open_grn_cursor (int argc, VALUE *argv, VALUE self,
 		 rb_grn_inspect(rb_order));
     }
     if (NIL_P(rb_order_by)) {
+	if (table->header.type == GRN_TABLE_PAT_KEY) {
+	    flags |= GRN_CURSOR_BY_KEY;
+	} else {
+	    flags |= GRN_CURSOR_BY_ID;
+	}
     } else if (rb_grn_equal_option(rb_order_by, "id")) {
 	flags |= GRN_CURSOR_BY_ID;
     } else if (rb_grn_equal_option(rb_order_by, "key")) {
@@ -798,10 +810,12 @@ rb_grn_table_open_grn_cursor (int argc, VALUE *argv, VALUE self,
  *   取り出す。
  *
  * [+:order_by+]
- *   +:id+を指定するとID順にレコードを取り出す。（デフォルト）
+ *   +:id+を指定するとID順にレコードを取り出す。（Arrayと
+ *   Hashのデフォルト）
  *
  *   +:key+指定するとキー順にレコードを取り出す。ただし、
- *   Groonga::PatriciaTrieにしか使えない。
+ *   Groonga::PatriciaTrieにしか使えない。（PatriciaTrieのデ
+ *   フォルト）
  *
  * [+:greater_than+]
  *   +true+を指定すると+:min+で指定した値に一致した[+key+]を
@@ -1234,7 +1248,11 @@ rb_grn_table_group (int argc, VALUE *argv, VALUE self)
 VALUE
 rb_grn_table_array_reference (VALUE self, VALUE rb_id)
 {
-    return rb_grn_record_new_raw(self, rb_id, Qnil);
+    if (FIXNUM_P(rb_id)) {
+	return rb_grn_record_new_raw(self, rb_id, Qnil);
+    } else {
+	return Qnil;
+    }
 }
 
 VALUE
@@ -1935,6 +1953,19 @@ rb_grn_table_merge_bang (VALUE self, VALUE rb_other)
     return rb_grn_table_set_operation_bang(self, rb_other, GRN_OP_ADJUST);
 }
 
+/*
+ * call-seq:
+ *   table.support_key? -> true/false
+ *
+ * _table_に主キーが設定されていれば+true+、されていなければ
+ * +false+を返す。
+ */
+static VALUE
+rb_grn_table_support_key_p (VALUE self)
+{
+    return Qfalse;
+}
+
 /*
  * call-seq:
  *   table.support_sub_records? -> true/false
@@ -1955,6 +1986,29 @@ rb_grn_table_support_sub_records_p (VALUE self)
     return CBOOL2RVAL(table->header.flags & GRN_OBJ_WITH_SUBREC);
 }
 
+/*
+ * call-seq:
+ *   table.exist?(id) -> true/false
+ *
+ * _table_に_id_で指定したIDのレコードが存在する場合は+true+、
+ * 存在しない場合は+false+を返す。
+ *
+ * 注意: 実行には相応のコストがかかるのであまり頻繁に呼ばな
+ * いようにして下さい。
+ */
+static VALUE
+rb_grn_table_exist_p (VALUE self, VALUE id)
+{
+    grn_ctx *context;
+    grn_obj *table;
+
+    rb_grn_table_deconstruct(SELF(self), &table, &context,
+			     NULL, NULL,
+			     NULL, NULL, NULL,
+			     NULL);
+    return CBOOL2RVAL(grn_table_at(context, table, NUM2UINT(id)));
+}
+
 void
 rb_grn_init_table (VALUE mGrn)
 {
@@ -2022,9 +2076,13 @@ rb_grn_init_table (VALUE mGrn)
     rb_define_method(rb_cGrnTable, "merge!",
 		     rb_grn_table_merge_bang, 1);
 
+    rb_define_method(rb_cGrnTable, "support_key?",
+		     rb_grn_table_support_key_p, 0);
     rb_define_method(rb_cGrnTable, "support_sub_records?",
 		     rb_grn_table_support_sub_records_p, 0);
 
+    rb_define_method(rb_cGrnTable, "exist?", rb_grn_table_exist_p, 1);
+
     rb_grn_init_table_key_support(mGrn);
     rb_grn_init_array(mGrn);
     rb_grn_init_hash(mGrn);

data/ext/groonga/rb-grn.h

@@ -69,7 +69,7 @@ RB_GRN_BEGIN_DECLS
 
 #define RB_GRN_MAJOR_VERSION 1
 #define RB_GRN_MINOR_VERSION 0
-#define RB_GRN_MICRO_VERSION 1
+#define RB_GRN_MICRO_VERSION 2
 
 typedef int rb_grn_boolean;
 #define RB_GRN_FALSE (0)

data/ext/groonga/rb-groonga.c

@@ -26,6 +26,7 @@ finish_groonga (VALUE data)
     debug("finish\n");
     grn_fin();
     rb_grn_exited = RB_GRN_TRUE;
+    debug("finish: done\n");
 }
 
 static void