Yasuhiro Horimoto
null+****@clear*****
Mon Jan 15 17:04:42 JST 2018
Yasuhiro Horimoto 2018-01-15 17:04:42 +0900 (Mon, 15 Jan 2018) New Revision: b4fa223ce220c61ef9f962a2d464e12b76d875e2 https://github.com/groonga/fluent-plugin-groonga/commit/b4fa223ce220c61ef9f962a2d464e12b76d875e2 Merged 2b71481: Merge pull request #25 from komainu8/fluentd-v0.12 Message: Modify type of value of protocol to "enum" Because become same code with master and simply. Modified files: lib/fluent/plugin/in_groonga.rb lib/fluent/plugin/out_groonga.rb Modified: lib/fluent/plugin/in_groonga.rb (+1 -8) =================================================================== --- lib/fluent/plugin/in_groonga.rb 2018-01-15 17:03:57 +0900 (0082151) +++ lib/fluent/plugin/in_groonga.rb 2018-01-15 17:04:42 +0900 (716c384) @@ -33,14 +33,7 @@ module Fluent super end - config_param :protocol, :default => :http do |value| - case value - when "http", "gqtp" - value.to_sym - else - raise ConfigError, "must be http or gqtp: <#{value}>" - end - end + config_param :protocol, :enum, :list => [:http, :gqtp], :default => :http def configure(conf) super Modified: lib/fluent/plugin/out_groonga.rb (+1 -9) =================================================================== --- lib/fluent/plugin/out_groonga.rb 2018-01-15 17:03:57 +0900 (8c842eb) +++ lib/fluent/plugin/out_groonga.rb 2018-01-15 17:04:42 +0900 (ab7c608) @@ -29,15 +29,7 @@ module Fluent super end - config_param :protocol, :default => :http do |value| - case value - when "http", "gqtp", "command" - value.to_sym - else - raise ConfigError, "must be http, gqtp or command: <#{value}>" - end - end - + config_param :protocol, :enum, :list => [:http, :gqtp, :command], :default => :http # alias is just for backward compatibility config_param :store_table, :string, :default => nil, :alias => :table -------------- next part -------------- HTML����������������������������... URL: https://lists.osdn.me/mailman/archives/groonga-commit/attachments/20180115/a3dd86a1/attachment-0002.htm
