[seasar-dotnet:990] Re: SqlLogRegistryについて

[kubo]

久保(jflute)です。

おおお、ありがとうございます！素敵です。
今こっちでもDBFluteで対応してしまおうかと
思った次第でした。。。参考にさせて頂きます。
とりあえずDBFluteでは個別に対応してしまおうかと思います。
古いバージョンのS2Dao.NETで動かなくなってしまうので、
DBFluteはDBFluteで対応した方が良いという判断です。
(S2Dao.NETに組み込むかどうかは他のコミッタの判断に任せます)

> ・BasicHandler でログ出力をしていない
> ログを出力する必要があるのか？
ちょっと構造がJavaとは微妙に異なるので、
厳密に対応しようとすると大変かもですね。
（把握するのが大変って意味で）

> ・SqlLogRegistryImplのget(int)が未実装
まず使わないメソッドだと思うので大丈夫です。

> ・Java版と同様にデフォルトで機能がONになっている
> パフォーマンス等の問題でデフォルトはOFFがよいか？
個人的にはOFFが良いかと思います。
(もしSeasar側でONで組み込まれたらDBFluteで
 必死にOFFにします（＾＾)

>> 要望がSeasar.NETの方でなかったからだと思います。
> なるほど、そうだったんですか。
> Java版では例外発生時のログに含めたりとかいろいろ役に立ったのですが…
ちなみに、最新のDBFluteではSQL実行時の例外には、
デバッグログで出力されるSQLと同じものをメッセージとして
乗せています。例外発生してから文字列を生成するため、
パフォーマンスも劣化しません。
なので、目的が「例外発生時のログ」だけであればそんなに
この機能が必要になることはないはずです。
(無論、SQL文字列をDBに書き込むとかそういう時に必要ですが)

2008/10/17 k. moriwaki <[E-MAIL ADDRESS DELETED]>:
> 久保さん
> 森脇です
>
>> 要望がSeasar.NETの方でなかったからだと思います。
> なるほど、そうだったんですか。
> Java版では例外発生時のログに含めたりとかいろいろ役に立ったのですが…
>
> で、Java版をC#版に書き換えてみました。
> （C#自体の理解がまだ浅いのでちょっと汚いかもしれません）
> ご迷惑かもしれませんが取り込んでいただけるとありがたく思います。
>
> また、移植するにあたりちょっと困った点が以下になります。
> ・BasicHandler でログ出力をしていない
> ログを出力する必要があるのか？
> ・SqlLogRegistryImplのget(int)が未実装
> Java版と同様にLinkedListで実装したが、ランダムアクセスをサポートしていなかった。
> おそらくListを使うよりは早いと思われたのでLinkedListのままにしてある。
> ・Java版と同様にデフォルトで機能がONになっている
> パフォーマンス等の問題でデフォルトはOFFがよいか？
>
> 以上、よろしくお願いいたします。
>
> 2008/10/17 15:10 kubo <[E-MAIL ADDRESS DELETED]>:
>> 久保(jflute)です。
>>
>> 森脇さん、こんにちは
>>
>> 無いです。残念ですが。。。
>> なぜ無いかというと、単に今までそういった
>> 要望がSeasar.NETの方でなかったからだと思います。
>>
>> 2008/10/17 k. moriwaki <[E-MAIL ADDRESS DELETED]>:
>>> 森脇です
>>>
>>> 表題のクラスについてですが、Seasar2.NETに実装されていますでしょうか？
>>>
>>> Java版のDBFluteでは
>>> org.seasar.extension.jdbc.SqlLogRegistry
>>> がデフォルトで無効にされていたので、有効にしようと思ったのですが
>>> Seasar2.NETのソースを見たところ同様の機能をもったクラスが
>>> 無いみたいだったので質問させていただきました。
>>> （もしあったのであればすいません）
>>> _______________________________________________
>>> seasar-dotnet mailing list
>>> [E-MAIL ADDRESS DELETED]
>>> https://ml.seasar.org/mailman/listinfo/seasar-dotnet
>>>
>> _______________________________________________
>> seasar-dotnet mailing list
>> [E-MAIL ADDRESS DELETED]
>> https://ml.seasar.org/mailman/listinfo/seasar-dotnet
>>
>
> _______________________________________________
> seasar-dotnet mailing list
> [E-MAIL ADDRESS DELETED]
> https://ml.seasar.org/mailman/listinfo/seasar-dotnet
>
>