Validate key_type as a required parameter for some cases

lib/droonga/plugins/groonga/table_create.rb

@@ -35,6 +35,8 @@ def process_request(request)
                                      :result => false)
             end
 
+            validate_key_type
+
             options = parse_command
             ::Groonga::Schema.define(:context => @context) do |schema|
               schema.create_table(name, options)
@@ -88,6 +90,29 @@ def parse_normalizer(options)
             return unless @command["normalizer"]
             options[:normalizer] = @command["normalizer"]
           end
+
+          def validate_key_type
+            if @command.table_hash_key? and @command["key_type"].nil?
+              message = "key_type is required for TABLE_HASH_KEY table"
+              raise CommandError.new(:status => Status::INVALID_ARGUMENT,
+                                     :message => message,
+                                     :result => false)
+            end
+
+            if @command.table_pat_key? and @command["key_type"].nil?
+              message = "key_type is required for TABLE_PAT_KEY table"
+              raise CommandError.new(:status => Status::INVALID_ARGUMENT,
+                                     :message => message,
+                                     :result => false)
+            end
+
+            if @command.table_dat_key? and @command["key_type"].nil?
+              message = "key_type is required for TABLE_DAT_KEY table"
+              raise CommandError.new(:status => Status::INVALID_ARGUMENT,
+                                     :message => message,
+                                     :result => false)
+            end
+          end
         end
 
         class Handler < Droonga::Handler