Defect #582
Skillの賛同者数と、Skillポイントのズレ
Start date:
10/07/2010
Due date:
% Done:
100%
Estimated time:
Description
多分トランザクション制御してないから？
のpullSkillで
Aさん、Bさんが同時に賛同すると、ずれるのでは
GlobalTransactionを使ったほうがいいかも。。。
Associated revisions
スキルをツイートするときに必ずポイントが1になってしまっていた問題を修正 refs 582
スキルをツイートするときに必ずポイントが1になってしまっていた問題を修正 refs 582
スキルに賛同するイベントを個別にわけるようにした refs 582
スキルに賛同するイベントを個別にわけるようにした refs 582
History
Updated by 
Updated by
Updated by 啓介 大橋 almost 9 years ago
多分同じSkill更新に行っているので、更新がかぶるとException？で終わる気がします。
キャッチして、リトライする感じっすかね。そうすれば、イケそうな気がします。
※今日slim3本持ってきてるので帰りに見てみます。
次の更新でまた正常に戻りそうですが、、、
Updated by 啓介 大橋 almost 9 years ago
- Status changed from 新規（New） to 解決（Resolved）
- Assignee set to 啓介 大橋
いろいろ確認して、
トランザクションだけだとだめなことに気付きました。
pullSkillのskill.getRelation().getModelList()でgetModelList()は一度でも取得していると取得データをキャッシュするため、
UserUIを表示した時点で、現状getModelListをしているため、
このタイミングのデータで件数を取得していることになっていました。
GlobalTransactionの対応の中で一緒に治ったのはずなので、マージをお願いしまーす。
Updated by yusuke kokubo almost 9 years ago
- Status changed from 解決（Resolved） to 終了（Closed）
- % Done changed from 0 to 100
マージしました。
ありがとうございます。
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)
に直してください。
すいません。ローカルでツイッターのテストが出来なかったので抜けてました。。。
Updated by 啓介 大橋 almost 9 years ago
- Status changed from 担当（Assigned） to 終了（Closed）
- Assignee changed from 啓介 大橋 to yusuke kokubo
なおしていただいてありがとうございます！
完了とします。
トランザクションの追加と、skillを再取得（ロックのためと、relationのモデルリストを再取得させるため)#refs 582