XGTx時に別リクエストでコミットしたEntityにアクセスするとConcurrentModificationExceptionが発生する件

しんめたる @sinmetal

XGTxをcommitしたにも関わらず、別リクエストでConcurrentModificationExceptionが起きてしまっている。何故だ・・・。 #gaeja

しんめたる @sinmetal

XGTranでConcurrentModificationExceptionが出る件、分からないからメーリングリストに投げてみるかー #gaeja

しんめたる @sinmetal

むぅ、朝起きてもConcurrentModificationExceptionのままだった・・・ https://t.co/J3qmikZIX3 #gaeja

Ikeda Daigo @hogedigo

@sinmetal これはひどい・・

しんめたる @sinmetal

@vierjp いやぁ、まさかこうなるとは思ってなかったので、驚きの結果！そして、打開策が思いつかないという！

しんめたる @sinmetal

@hogedigo まさか、朝になってもダメだとは思わなかったorz

Ryo Yamasaki @vierjp

@sinmetal 不思議だね。logEntityはしっかり書き込まれてたからXGTxは完了しているはずだし。やりたいことは「無かったら別のEGのEntityを書き込みたい」で、 「無かったらそのEntityを書き込む」じゃダメなんだよね？

しんめたる @sinmetal

@vierjp そうなのですよ。3つのEntityが全て揃っていた場合、処理を行うが、そうでない場合はLogEntityを出力するって感じです。ロックが残って、まったく解除されないのがきついorz

Ryo Yamasaki @vierjp

@sinmetal ロックが残って消えないのはバグっぽい気がしちゃうね。 人が見るためのログデータならXGTxまで使って厳密にやらなくてもいい、と割りきってしまうのはどうだろうｗ

あおうさ @bluerabbit777jp

@sinmetal @vierjp SDKだとgetOrNullという挙動は元々無くて存在しなくてRuntimeExceptionだったと思うのでSDKとしてはRuntimeException発生したのにそのまま続行して何か処理を進めるというのは考慮してなかったんじゃないかな

Ryo Yamasaki @vierjp

@sinmetal ついでにXGTx中では参照しかしなくてもgetのタイミングで競合するそうだからスケーラビリティが気になるかも。http://t.co/j2FXe60BC2 (Noteのところ)

しんめたる @sinmetal

@bluerabbit777jp @vierjp 確かに存在しないKeyをgetすることが、そもそもおかしいということであれば、分からなくもないですね。ただ、一度ロックがかかると、まったく解除される気配が無いというのがorz

しんめたる @sinmetal

@vierjp 確かにスケールする必要がある場合は別の問題が発生しそうですね。ただ、今のパターンはXGTx内でぶつかるのは、1ユーザのみなので、問題は無さそうです。

あおうさ @bluerabbit777jp

@vierjp @sinmetal lockされっぱなしはバグでしょうね。

Ryo Yamasaki @vierjp

@bluerabbit777jp @sinmetal ですねぇ。

あおうさ @bluerabbit777jp

@sinmetal @vierjp SCN番号が先に進んでしまってるけど、EntityがnullだったからEntityのSCN番号が進めなかったという感じかと

しんめたる @sinmetal

@bluerabbit777jp @vierjp ふむふむ。Entityが無いKeyに対してlockをかけた場合、状態を保存する場所が無いから、tx.commit()もtx.rollbackも効かないということですか・・・。

あおうさ @bluerabbit777jp

@sinmetal tx.commitでException発生してます？getOrNullで発生してます？tx.commitですよね？

しんめたる @sinmetal

@bluerabbit777jp getOrNullで発生してます

あおうさ @bluerabbit777jp

@sinmetal なるほど

あおうさ @bluerabbit777jp

@vierjp @sinmetal こんな感じかなと予想 https://t.co/dzARBAEiUT

あおうさ @bluerabbit777jp

この例だとEntityGroupに対してバージョンNoを持つからSCNじゃなかった

あおうさ @bluerabbit777jp

@vierjp @sinmetal tx.commitが効かないというより誤って処理されっちゃってる感じ

Ryo Yamasaki @vierjp

@bluerabbit777jp @sinmetal ですね。HogeLogはちゃんと書き込まれていたので最初のcommit自体は一応完了しているようですし。

しんめたる @sinmetal

@vierjp @bluerabbit777jp 確かにあおうささんのGist通りな雰囲気がありますね・・・