[Seasar-user:7037] Re: [Teeda]Gridの幅調整スクリプトが出力されない

Shinpei Ohtani [E-MAIL ADDRESS DELETED]
2007年 4月 3日 (火) 16:46:29 JST 

大谷です。

> 回避して、1.0.7のSNAPSHOTで問題なく動作することを確認しました。
> お役に立ちましたでしょうか。

今回だけでなく、色々とフィードバックいただき、ありがとうございます。
これからもぜひ宜しくお願いします。

> なんかパフォーマンスがよくなっているようですね^^;
> こちらはすでに1.0.4でフリーズしたので残念ですが。

はい、ここ1ヶ月くらいパフォーマンスチューニングを
主にやっていたので、コンポーネントの種類によりけりですが
数倍のスピードにはなっていると思います。

宜しくお願いします。

07/04/03 に Asarima<[E-MAIL ADDRESS DELETED]> さんは書きました:
> Asarimaです。
>
> 回避して、1.0.7のSNAPSHOTで問題なく動作することを確認しました。
> お役に立ちましたでしょうか。
>
> なんかパフォーマンスがよくなっているようですね^^;
> こちらはすでに1.0.4でフリーズしたので残念ですが。
>
> >Asarimaです。
> >
> >　すいません、genorimo-jtaを1.1に上げたら起動時エラーはなくなりました。
> >
> >　あと ExtensionConstants.TRANSITION_BY_TEEDA_PREPARED_METHOD が無くなっ
> >てコンパイルエラーになっているところを回避すればいけそうです。
> >
> >>Asarimaさん、
> >>
> >>大谷です。
> >>
> >>確認ありがとうございます。
> >>起動時のエラーとはどのようなものでしょうか。
> >>スタックトレース等見せていただけますか？
> >>
> >>宜しくお願いします。
> >>
> >>07/04/03 に Asarima<[E-MAIL ADDRESS DELETED]> さんは書きました:
> >>> Asarimaです。
> >>>
> >>> ありがとうございます。助かります。
> >>>
> >>> ただ、S2.4.9+Teeda 1.0.4からS2.4.11+SNAPSHOTに変更するとTeedaの
> >>> 起動時にエラーが発生するようになってしまい、確認することができません
> >>> でした。1.0.5以降の仕様面で何か対応する必要があるようですね。
> >>>
> >>> SNAPSHOTのソースを拝見しましたところ、ComponentAttributesMapを変更
> >>> されているようでしたので、現プロジェクト（1.0.4環境）において優先的に
> >>> 参照されるように新しいComponentAttributesMap＋InternalConstants（必要
> >>> だったので）の２クラスを追加して確認したところ、問題なく動作しました。
> >>>
> >>> そういうわけで、1.0.4ではＯＫですがSNAPSHOTは確認できないです。
> >>> 申し訳ありません。
> >>>
> >>> >大谷です。
> >>> >
> >>> >調べた結果、Asarimaさんのフィードバックでおそらく問題ないです。
> >>> >ありがとうございます。
> >>> >
> >>> >この対応を盛り込んだSNAPSHOTを作成してみました。
> >>> >お手数ですが、下記URLのjarで一旦動かしてみていただけないでしょうか。
> >>> >（Seasar2は2.4.11が必要です。）
> >>> >宜しくお願いします。
> >>> >
> >>> >
> >>> >http://maven.seasar.org/maven2-snapshot/org/seasar/teeda/teeda-ajax/1.0.7
> >>> >-
> >>> >SNAPSHOT/teeda-ajax-1.0.7-20070403.051906-1.jar
> >>> >
> >>> >http://maven.seasar.org/maven2-snapshot/org/seasar/teeda/teeda-core/1.0.7
> >>> >-
> >>> >SNAPSHOT/teeda-core-1.0.7-20070403.051906-1.jar
> >>> >
> >>> >http://maven.seasar.org/maven2-snapshot/org/seasar/teeda/teeda-extension/
> >>> >1.0.
> >>> >7-SNAPSHOT/teeda-extension-1.0.7-20070403.051906-1.jar
> >>> >
> >>> >
> >>> >07/04/03 に Shinpei Ohtani<[E-MAIL ADDRESS DELETED]> さんは書きました:
> >>> >> 大谷です。
> >>> >> 返答おそくなり、すいません。
> >>> >>
> >>> >> こちらでも確認してみます。
> >>> >> 宜しくお願いします。
> >>> >>
> >>> >> 07/04/03 に Asarima<[E-MAIL ADDRESS DELETED]> さんは書きました:
> >>> >> > Asarimaです。
> >>> >> >
> >>> >> > 下記の件、どなたかお分かりになる方はいらっしゃいませんか。
> >>> >> >
> >>> >> > 同一ページのSerializedViewからUIBodyに復元されるAttributesMapが同一
> >>> >> > インスタンスではないか、それがUIBodyのAttributesMapの分離性を欠いてい
> >>> >> > ないでしょうか。
> >>> >> >
> >>> >> > 複数のGridをもつ同一ページに対し、複数クライアントから同時にリクエス
> >>> >> > トを処理したとき、UIBodyのencodeBegin〜encodeEndまでの処理が重なる
> >>> >> > と、
> >>> >> > RenderListenerが保持するListenerの数がおかしくなることはデバッグで確
> >>> >> > 認
> >>> >> > しています。
> >>> >> >
> >>> >> > 例：３つのGridを持つページへの２つの同時リクエストを処理するスレッド
> >>> >> > Ａ，
> >>> >> > Ｂが、UIBodyのメソッドを以下の順番で呼び出したとすると、先のリクエ
> >>> >> > ストに対してスクリプトが５つ、後に対して１つ出力される。
> >>> >> > （本来は３つずつ出力されなければならない）
> >>> >> >
> >>> >> > （Ａ）encodeBegin
> >>> >> > （Ａ）addRenderListener → RenderListenerの数: 1
> >>> >> > （Ｂ）encodeBegin
> >>> >> > （Ａ）addRenderListener → RenderListenerの数: 2
> >>> >> > （Ｂ）addRenderListener → RenderListenerの数: 3
> >>> >> > （Ａ）addRenderListener → RenderListenerの数: 4
> >>> >> > （Ｂ）addRenderListener → RenderListenerの数: 5
> >>> >> > （Ａ）encodeEnd         → Ａ出力 → RenderListenerの数: 0
> >>> >> > （Ｂ）addRenderListener → RenderListenerの数: 1
> >>> >> > （Ｂ）encodeEnd         → Ｂ出力
> >>> >> >
> >>> >> > それで、AttributeMapのシャローコピーを復元するようにしたら問題が回避
> >>> >> > されることを確認したのですが、この変更で他の問題が発生しないでしょう
> >>> >> > か。
> >>> >> >
> >>> >> > 以上です、よろしくお願いいたします。
> >>> >> >
> >>> >> > >Asarimaです。
> >>> >> > >
> >>> >> > >　弊社の開発メンバーが以下の記事の通り困っています。
> >>> >> > >　私のほうで何とか原因を突き止め、ソースを修正することで対処できまし
> >>> >> > >た
> >>> >> > >（とはいえ、まだ確認中ですが）のでお知らせします。
> >>> >> > >　しかし本当にこの変更でよいのかが心配です。このような変更に問題がな
> >>> >> > >い
> >>> >> > >か、ご意見を頂戴したく思います。よろしくお願いします。
> >>> >> > >
> >>> >> > >■[Teeda] Gridの幅を調整するするスクリプトが出力されない？
> >>> >> > >http://d.hatena.ne.jp/kazita/20070319/1174287422
> >>> >> > >
> >>> >> > >【不具合の内容】
> >>> >> > >
> >>> >> > >　Teeda 1.0.4を使用しています。
> >>> >> > >　複数のGridを表示する画面があるとします。
> >>> >> > >　複数のクライアントからその画面に対して同時にリクエストを実行する
> >>> >> > >と、
> >>> >> > >HTMLの最後に出力されるGridの幅調整用のスクリプトが、一部出力されなか
> >>> >> > >ったり、余分に出力される不具合が発生します。
> >>> >> > >
> >>> >> > >【原因の発見と対処】
> >>> >> > >
> >>> >> > >　Eclipseのデバッグモードを活用しました。
> >>> >> > >
> >>> >> > >１．THtmlGridRendererのencodeHtmlGridBeginメソッドに入ったところで
> >>> >> > >ブレークポイントを仕掛け、２つのクライアントから同時にリクエストを
> >>> >> > >実行します。２つのスレッドがブレークポイントで止まるので、交互に再開
> >>> >> > >していくと、不具合が確実に再現しました。
> >>> >> > >
> >>> >> > >２．この状態でTBodyRendererのaddRendererListenerメソッドにブレーク
> >>> >> > >ポイントを仕掛けると、２つのスレッドで、２つ目のGridからlistenersが
> >>> >> > >同じインスタンスを指しました。
> >>> >> > >
> >>> >> > >３．listenersを保持している各UIBodyのAttributesの本体である
> >>> >> > >UIComponentBaseのフィールドattributesMapが、２つのスレッドで同じ
> >>> >> > >インスタンスを指しました。
> >>> >> > >
> >>> >> > >４．UIComponentBaseのrestoreAttributeMapメソッドで、２つのスレッドで
> >>> >> > >同じstateインスタンスの参照がComponentAttributeMapのコンストラクタに
> >>> >> > >渡されました。
> >>> >> > >
> >>> >> > >５．TeedaStateManagerImplのrestoreComponentStateFromServerメソッド
> >>> >> > >で、２つのスレッドで同じSerializedViewインスタンスのstateがViewRoot
> >>> >> > >に
> >>> >> > >復元されます。
> >>> >> > >
> >>> >> > >６．私は４．に着目し、stateを復元するときにstateのシャローコピーを
> >>> >> > >渡せばAttributesが同じインスタンスを指さなくなると考え、
> >>> >> > >UIComponentBase
> >>> >> > >のソースを以下のように変更しました。
> >>> >> > >
> >>> >> > >---
> >>> >> > >private void restoreAttributeMap(Object state) {
> >>> >> > >    if (state != null) {
> >>> >> > >//↓
> >>> >> > >//        attributesMap = new ComponentAttributesMap(this, (Map)
> >>> >> > >state);
> >>> >> > >          HashMap map = new HashMap();
> >>> >> > >          map.putAll((Map)state);
> >>> >> > >          attributesMap = new ComponentAttributesMap(this, map);
> >>> >> > >//↑
> >>> >> > >    } else {
> >>> >> > >        clearAttributeMap();
> >>> >> > >    }
> >>> >> > >}
> >>> >> > >---
> >>> >> > >
> >>> >> > >　これでこの問題は発生しなくなりました。
> >>> >> > >
> >>> >> > >【考察】
> >>> >> > >
> >>> >> > >　複数のクライアントから同じ画面に対する要求が同時に起こった場合、コ
> >>> >> > >ン
> >>> >> > >ポーネントツリーにSerializedViewの状態が復元されるようですが、復元さ
> >>> >> > >れる
> >>> >> > >オブジェクトのうちAttributeMapについては参照がコピーされます。
> >>> >> > >　これでBODYコンポーネントのAttributeMapに対してRenderListenerが同時
> >>> >> > >に
> >>> >> > >登録されると、後から登録した方で上書きされるため、上記の不具合が起こ
> >>> >> > >ると
> >>> >> > >考えます。これはGridに限らないとも思われます。
> >>> >> > >
> >>> >> > >　以上です。よろしくお願いいたします。
> _______________________________________________
> Seasar-user mailing list
> [E-MAIL ADDRESS DELETED]
> https://ml.seasar.org/mailman/listinfo/seasar-user
>


-- 
=============================
Shinpei Ohtani
[E-MAIL ADDRESS DELETED]
=============================

Seasar-user メーリングリストの案内