スキーマ調査用のSQLログの出し分け議論

MOROHASHI Kyosuke @moro

Rails.env.development? の時もスキーマ調査用のSQLをログに出さなくするオプションが欲しいな。実行されてるSQL見るときにさすがにうざったい。

Akira Matsuda @a_matsuda

@moro あー、なるほどその需要はありそうですね。しかしどう実装したものか難しいかも。log_levelじゃない何か別の概念を持ち込むしかないかなぁ。

MOROHASHI Kyosuke @moro

@a_matsuda ざっと見たけど、いまの作りだとAdapterごとに別れてるのね。むーんてなった。作るならconfig.activerecord.schema_query_loglevel ||= :debug でどうか。

MOROHASHI Kyosuke @moro

nilか'none'で出力しない

Akira Matsuda @a_matsuda

@moro んでもってそのクエリがschema_queryかどうかのフラグかなんかをlogしてるとこまで引き回さなきゃいけないんですよねぇ。なんかちょっとuglyだなぁ。

MOROHASHI Kyosuke @moro

@a_matsuda abstract_connectionでなんとかならんかしら。とゆうかクエリの元がユーザの都合のものなのか、フレームワークの事情なのかを判別する手立てがあること自体は悪くないと思うのだけど。という抗弁でどうか。

Akira Matsuda @a_matsuda

@moro んーと、なんとかというと、connection_adapters/abstract/schema_statements.rb のほうの各メソッドになんかフラグを返させてアダプタはいちいちsuper呼ぶようにする感じですかねい？どうなんだろ。

MOROHASHI Kyosuke @moro

@a_matsuda あー、実装的にはAbstraceだけでいければ楽でいいけど無理そうねー、というのがさっきと、いま改めてみての感想です。各アダプタに書いてまわるのがださいというのは認める。

Akira Matsuda @a_matsuda

@moro いやぁ、ってかぜんぜんadapterごとに実装バラバラですね。こりゃめんどいわ。って、かぶりましたね。

Akira Matsuda @a_matsuda

@moro とりあえずJavaとかのInterfaceっていいもんだったんだな、とこれ見てちょっと思いました。

MOROHASHI Kyosuke @moro

@a_matsuda ちょww モンキーでいいならAbstractAdapter.subclasses.each{|k| k.alias_method_chain (以下略) }でいいんだろうけど、ちゃんと書こうとするとしんどい感じねぇ。

Akira Matsuda @a_matsuda

@moro そうすねぇ。やるんだったらその機能を入れる前に今のコードをリファクタリングして、ASのhookかなんかAOP的なものでlogするようにしておくべきな感じですかね。