Defect #582
Skillの賛同者数と、Skillポイントのズレ
Start date:
10/07/2010
Due date:
% Done:
100%
Estimated time:
Description
多分トランザクション制御してないから?
のpullSkillで
Aさん、Bさんが同時に賛同すると、ずれるのでは
GlobalTransactionを使ったほうがいいかも。。。
Updated by 啓介 大橋 over 10 years ago
多分同じSkill更新に行っているので、更新がかぶるとException?で終わる気がします。
キャッチして、リトライする感じっすかね。そうすれば、イケそうな気がします。
※今日slim3本持ってきてるので帰りに見てみます。
次の更新でまた正常に戻りそうですが、、、
Updated by 啓介 大橋 over 10 years ago
- Status changed from 新規(New) to 解決(Resolved)
- Assignee set to 啓介 大橋
いろいろ確認して、
トランザクションだけだとだめなことに気付きました。
pullSkillのskill.getRelation().getModelList()でgetModelList()は一度でも取得していると取得データをキャッシュするため、
UserUIを表示した時点で、現状getModelListをしているため、
このタイミングのデータで件数を取得していることになっていました。
GlobalTransactionの対応の中で一緒に治ったのはずなので、マージをお願いしまーす。
Updated by yusuke kokubo over 10 years ago
- Status changed from 解決(Resolved) to 終了(Closed)
- % Done changed from 0 to 100
マージしました。
ありがとうございます。
Updated by 啓介 大橋 over 10 years ago
- Status changed from 終了(Closed) to 担当(Assigned)
申し訳ないっす。
ツイッター側の修正漏れです。
SkillServiceImpl#putSkillの
if (sendTwitter) { TwitterUtil.tweetSkillAppended(skill); }
をcomplete=trueのあとに移動して、
TwitterUtil.tweetSkillAppended(skill)
を
TwitterUtil.tweetSkillAppended(pullSkill)
に直してください。
すいません。ローカルでツイッターのテストが出来なかったので抜けてました。。。
Updated by 啓介 大橋 over 10 years ago
- Status changed from 担当(Assigned) to 終了(Closed)
- Assignee changed from 啓介 大橋 to yusuke kokubo
なおしていただいてありがとうございます!
完了とします。