プロジェクト

全般

プロフィール

Defect #582

完了

Skillの賛同者数と、Skillポイントのズレ

啓介 大橋 さんが13年以上前に追加. 13年以上前に更新.

ステータス:
終了(Closed)
優先度:
通常(Normal)
担当者:
対象バージョン:
開始日:
2010/10/07
期日:
進捗率:

100%

予定工数:

説明

多分トランザクション制御してないから?

http://github.com/YusukeKokubo/SkillMaps/blob/master/src/com/appspot/skillmaps/server/service/SkillServiceImpl.java

のpullSkillで
Aさん、Bさんが同時に賛同すると、ずれるのでは

GlobalTransactionを使ったほうがいいかも。。。

啓介 大橋 さんが13年以上前に更新

  • 対象バージョンSprint10.10 にセット

yusuke kokubo さんが13年以上前に更新

こんなに早くこんなことが起きるとは^^;

GlobalTransaction使えば大丈夫なんでしたっけ?

啓介 大橋 さんが13年以上前に更新

多分同じSkill更新に行っているので、更新がかぶるとException?で終わる気がします。
キャッチして、リトライする感じっすかね。そうすれば、イケそうな気がします。
※今日slim3本持ってきてるので帰りに見てみます。

次の更新でまた正常に戻りそうですが、、、

啓介 大橋 さんが13年以上前に更新

  • ステータス新規(New) から 解決(Resolved) に変更
  • 担当者啓介 大橋 にセット

いろいろ確認して、
トランザクションだけだとだめなことに気付きました。
pullSkillのskill.getRelation().getModelList()でgetModelList()は一度でも取得していると取得データをキャッシュするため、
UserUIを表示した時点で、現状getModelListをしているため、
このタイミングのデータで件数を取得していることになっていました。

GlobalTransactionの対応の中で一緒に治ったのはずなので、マージをお願いしまーす。

yusuke kokubo さんが13年以上前に更新

  • ステータス解決(Resolved) から 終了(Closed) に変更
  • 進捗率0 から 100 に変更

マージしました。
ありがとうございます。

啓介 大橋 さんが13年以上前に更新

  • ステータス終了(Closed) から 担当(Assigned) に変更

申し訳ないっす。
ツイッター側の修正漏れです。

SkillServiceImpl#putSkillの

        if (sendTwitter) {
            TwitterUtil.tweetSkillAppended(skill);
        }

をcomplete=trueのあとに移動して、
TwitterUtil.tweetSkillAppended(skill)


TwitterUtil.tweetSkillAppended(pullSkill)

に直してください。
すいません。ローカルでツイッターのテストが出来なかったので抜けてました。。。

啓介 大橋 さんが13年以上前に更新

  • ステータス担当(Assigned) から 終了(Closed) に変更
  • 担当者啓介 大橋 から yusuke kokubo に変更

なおしていただいてありがとうございます!
完了とします。

他の形式にエクスポート: Atom PDF