[Groonga-commit] droonga/drntest at b50e31e [master] Move config_dir to Engine

Back to archive index

Kouhei Sutou kou****@clear*****
Thu Jan 16 17:17:45 JST 2014


In <52D78F19.9060406 �� clear-code.com>
  "Re: [Groonga-commit] droonga/drntest �� b50e31e [master] Move config_dir to Engine" on Thu, 16 Jan 2014 16:49:45 +0900,
  YUKI Hiroshi <yuki �� clear-code.com> wrote:

>> で、この後、またAPI変えちゃいました。
>> Hashでいろいろパラメーターを渡すんじゃなくて、新しく作った
>> Configurationオブジェクトをドンと渡すようにしました。
> 
> EngineをDrntestから余計に切り離しにくくなってしまいませんか?
> これは僕は賛成しづらいです。。。

そんなことないと思いますよ。

ConfigurationオブジェクトはEngineが必要なattr_accessorを持っ
たオブジェクトならなんでもよいので、必要ならdrnbench用
Cnofigurationオブジェクトを作ってそれを渡してもよいです。
(必要にならない気はしています。)

module Drnbench
  class Configuration
    attr_accessor :port, :host, :tag
    attr_accessor :base_path, :engine_config
    attr_accessor :fluentd, :fluentd_options
  end
end

みたいに。

Configurationオブジェクトは何か操作するメソッドは持たずに、単
にデータを管理するオブジェクトのままで居続けるはずなので、な
にか困ることがでることはないんじゃないかと思っています。




More information about the Groonga-commit mailing list
Back to archive index