Project

General

Profile

Defect #582

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

Added by 啓介 大橋 almost 9 years ago. Updated almost 9 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Assignee:
Target version:
Start date:
10/07/2010
Due date:
% Done:

100%

Estimated time:

Description

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

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

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

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

Associated revisions

Revision 8219135a (diff)
Added by 啓介 大橋 almost 9 years ago

トランザクションの追加と、skillを再取得(ロックのためと、relationのモデルリストを再取得させるため)#refs 582

Revision 4a706496 (diff)
Added by yusuke kokubo almost 9 years ago

スキルをツイートするときに必ずポイントが1になってしまっていた問題を修正 refs 582

Revision 350d094c (diff)
Added by yusuke kokubo almost 9 years ago

スキルをツイートするときに必ずポイントが1になってしまっていた問題を修正 refs 582

Revision ba23c7b9 (diff)
Added by yusuke kokubo almost 9 years ago

スキルに賛同するイベントを個別にわけるようにした refs 582

Revision 08bf2d44 (diff)
Added by yusuke kokubo almost 9 years ago

スキルに賛同するイベントを個別にわけるようにした refs 582

History

#1

Updated by 啓介 大橋 almost 9 years ago

  • Target version set to Sprint10.10
#2

Updated by yusuke kokubo almost 9 years ago

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

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

#3

Updated by 啓介 大橋 almost 9 years ago

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

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

#4

Updated by 啓介 大橋 almost 9 years ago

  • Status changed from 新規(New) to 解決(Resolved)
  • Assignee set to 啓介 大橋

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

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

#5

Updated by yusuke kokubo almost 9 years ago

  • Status changed from 解決(Resolved) to 終了(Closed)
  • % Done changed from 0 to 100

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

#6

Updated by 啓介 大橋 almost 9 years ago

  • Status changed from 終了(Closed) to 担当(Assigned)

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

SkillServiceImpl#putSkillの

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

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


TwitterUtil.tweetSkillAppended(pullSkill)

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

#7

Updated by 啓介 大橋 almost 9 years ago

  • Status changed from 担当(Assigned) to 終了(Closed)
  • Assignee changed from 啓介 大橋 to yusuke kokubo

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

Also available in: Atom PDF