* [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
@ 2011-03-08 5:37 Masaki Matsushita
2011-03-08 5:47 ` [ruby-dev:43328] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Shota Fukumori
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: Masaki Matsushita @ 2011-03-08 5:37 UTC (permalink / raw)
To: ruby developers list, "undisclosed-recipients:"
Issue #4483 has been reported by Masaki Matsushita.
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
http://redmine.ruby-lang.org/issues/4483
Author: Masaki Matsushita
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version:
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
--
http://redmine.ruby-lang.org
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:43328] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
@ 2011-03-08 5:47 ` Shota Fukumori
2011-03-08 5:59 ` [ruby-dev:43329] " Shyouhei Urabe
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Shota Fukumori @ 2011-03-08 5:47 UTC (permalink / raw)
To: ruby developers list, "undisclosed-recipients:"
Issue #4483 has been updated by Shota Fukumori.
互換性の面が懸念されます。
たとえばnested transaction例外を想定しているコード (たとえばtest_thread_safeとか) はどうなるのでしょう。
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
http://redmine.ruby-lang.org/issues/4483
Author: Masaki Matsushita
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version:
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
--
http://redmine.ruby-lang.org
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:43329] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
2011-03-08 5:47 ` [ruby-dev:43328] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Shota Fukumori
@ 2011-03-08 5:59 ` Shyouhei Urabe
2011-03-08 6:23 ` [ruby-dev:43330] " Shota Fukumori
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Shyouhei Urabe @ 2011-03-08 5:59 UTC (permalink / raw)
To: ruby developers list, "undisclosed-recipients:"
Issue #4483 has been updated by Shyouhei Urabe.
デフォルトがブロックするかしないかどっち向きに倒れておくべきかには議論のある所だとは思いますが、いきなり第二引数をなくしてしまうというのはやりすぎではないでしょうか。あなたの嬉しさは他者を排斥する理由としては弱いです。
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
http://redmine.ruby-lang.org/issues/4483
Author: Masaki Matsushita
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version:
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
--
http://redmine.ruby-lang.org
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:43330] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
2011-03-08 5:47 ` [ruby-dev:43328] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Shota Fukumori
2011-03-08 5:59 ` [ruby-dev:43329] " Shyouhei Urabe
@ 2011-03-08 6:23 ` Shota Fukumori
2011-03-08 7:01 ` [ruby-dev:43331] " Masaki Matsushita
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Shota Fukumori @ 2011-03-08 6:23 UTC (permalink / raw)
To: ruby developers list, "undisclosed-recipients:"
Issue #4483 has been updated by Shota Fukumori.
Shyouhei Urabe wrote:
> デフォルトがブロックするかしないかどっち向きに倒れておくべきかには議論のある所だとは思いますが、いきなり第二引数をなくしてしまうというのはやりすぎではないでしょうか。あなたの嬉しさは他者を排斥する理由としては弱いです。
あまりパッチみてませんけどそれもありましたね。同意します。
せめてthread_safe = trueにしておいて、今の動作と逆転するような感じだと良いですかね?
無視するか動作を逆転させるかどちらかは別として、引数を削るのは賛成しかねます。
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
http://redmine.ruby-lang.org/issues/4483
Author: Masaki Matsushita
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version:
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
--
http://redmine.ruby-lang.org
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:43331] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
` (2 preceding siblings ...)
2011-03-08 6:23 ` [ruby-dev:43330] " Shota Fukumori
@ 2011-03-08 7:01 ` Masaki Matsushita
2011-03-08 7:11 ` [ruby-dev:43332] " Shota Fukumori
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Masaki Matsushita @ 2011-03-08 7:01 UTC (permalink / raw)
To: ruby developers list, "undisclosed-recipients:"
Issue #4483 has been updated by Masaki Matsushita.
PStoreはRubyのオブジェクトを外部ファイルに格納する為のクラスな訳ですから、あるスレッドがトランザクションを実行中に別のスレッドがトランザクションに入ろうとした場合に、整合性を保つ為に同期する理由はあっても、例外を投げる理由はないのではと考えました。
そこで第2引数を削除したパッチを添付したのですが、確かにいきなり第2引数をなくしてしまうというのはやりすぎだったかも知れません。
それと、私が嬉しいかどうかをこういう場で引き合いに出したのは正しくなかったと思います。
Shota Fukumoriさんの案に賛成します。
ただ、thread_safe = trueとした場合でも、PStore.newに第2引数を与えずにnested transaction例外を想定しているコードには影響してしまいますね。
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
http://redmine.ruby-lang.org/issues/4483
Author: Masaki Matsushita
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version:
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
--
http://redmine.ruby-lang.org
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:43332] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
` (3 preceding siblings ...)
2011-03-08 7:01 ` [ruby-dev:43331] " Masaki Matsushita
@ 2011-03-08 7:11 ` Shota Fukumori
2012-03-18 10:08 ` [ruby-dev:45395] [ruby-trunk - Feature #4483][Assigned] PStoreをデフォルトで複数のスレッドから扱えるようにしたい nahi
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Shota Fukumori @ 2011-03-08 7:11 UTC (permalink / raw)
To: ruby developers list, "undisclosed-recipients:"
Issue #4483 has been updated by Shota Fukumori.
sora_hです
考えてみればどうせblockされるわけで必ずブロック内は実行されるわけで、nested transactionはもしダメなら諦める的な感じで捕捉されることを想定しているコードなら
この仕様でも良いかもしれません。transactionが起きているかどうかのコードはin_transactionで調べるべきだと思いますしね。
とりあえず周りの反応次第ですが。その辺の問題が解決できたら入れても良いかもしれないです。
ということで現状は賛成しかね、解決ししだい賛成という立場になります
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
http://redmine.ruby-lang.org/issues/4483
Author: Masaki Matsushita
Status: Open
Priority: Normal
Assignee:
Category: lib
Target version:
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
--
http://redmine.ruby-lang.org
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:45395] [ruby-trunk - Feature #4483][Assigned] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
` (4 preceding siblings ...)
2011-03-08 7:11 ` [ruby-dev:43332] " Shota Fukumori
@ 2012-03-18 10:08 ` nahi
2012-11-20 12:27 ` [ruby-dev:46558] [ruby-trunk - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい mame (Yusuke Endoh)
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: nahi @ 2012-03-18 10:08 UTC (permalink / raw)
To: ruby developers list
Issue #4483 has been updated by nahi.
Description updated
Status changed from Open to Assigned
Assignee set to nahi
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
https://bugs.ruby-lang.org/issues/4483#change-24908
Author: Glass_saga
Status: Assigned
Priority: Normal
Assignee: nahi
Category: lib
Target version:
=begin
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
=end
--
http://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:46558] [ruby-trunk - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
` (5 preceding siblings ...)
2012-03-18 10:08 ` [ruby-dev:45395] [ruby-trunk - Feature #4483][Assigned] PStoreをデフォルトで複数のスレッドから扱えるようにしたい nahi
@ 2012-11-20 12:27 ` mame (Yusuke Endoh)
2018-03-08 8:46 ` [ruby-dev:50492] [Ruby trunk Feature#4483][Feedback] PStoreをデフォルトで複数のスレッドから扱えるようにしたい mame
2018-03-08 14:04 ` [ruby-dev:50495] [Ruby trunk Feature#4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい glass.saga
8 siblings, 0 replies; 10+ messages in thread
From: mame (Yusuke Endoh) @ 2012-11-20 12:27 UTC (permalink / raw)
To: ruby developers list
Issue #4483 has been updated by mame (Yusuke Endoh).
Target version set to next minor
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
https://bugs.ruby-lang.org/issues/4483#change-33230
Author: Glass_saga (Masaki Matsushita)
Status: Assigned
Priority: Normal
Assignee: nahi (Hiroshi Nakamura)
Category: lib
Target version: next minor
=begin
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
=end
--
http://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:50492] [Ruby trunk Feature#4483][Feedback] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
` (6 preceding siblings ...)
2012-11-20 12:27 ` [ruby-dev:46558] [ruby-trunk - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい mame (Yusuke Endoh)
@ 2018-03-08 8:46 ` mame
2018-03-08 14:04 ` [ruby-dev:50495] [Ruby trunk Feature#4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい glass.saga
8 siblings, 0 replies; 10+ messages in thread
From: mame @ 2018-03-08 8:46 UTC (permalink / raw)
To: ruby-dev
Issue #4483 has been updated by mame (Yusuke Endoh).
Status changed from Assigned to Feedback
Assignee changed from nahi (Hiroshi Nakamura) to Glass_saga (Masaki Matsushita)
これ、どうしますか?まだやる気があったら、ハンドルをお願いします。ないなら、close しましょう。(現在、PStore はメンテナ不在です)
以下は個人的意見です。互換性のため、「いきなり消す」はないと思います。どうしても消したいなら、警告期間を用意すべきでしょう。「デフォルトを thread_safe = true にする」はありうるかもしれませんが、慎重な調査や検討が必要かもしれません。
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
https://bugs.ruby-lang.org/issues/4483#change-70897
* Author: Glass_saga (Masaki Matsushita)
* Status: Feedback
* Priority: Normal
* Assignee: Glass_saga (Masaki Matsushita)
* Target version:
----------------------------------------
=begin
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
=end
---Files--------------------------------
patch.diff (1.61 KB)
--
https://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 10+ messages in thread
* [ruby-dev:50495] [Ruby trunk Feature#4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
` (7 preceding siblings ...)
2018-03-08 8:46 ` [ruby-dev:50492] [Ruby trunk Feature#4483][Feedback] PStoreをデフォルトで複数のスレッドから扱えるようにしたい mame
@ 2018-03-08 14:04 ` glass.saga
8 siblings, 0 replies; 10+ messages in thread
From: glass.saga @ 2018-03-08 14:04 UTC (permalink / raw)
To: ruby-dev
Issue #4483 has been updated by Glass_saga (Masaki Matsushita).
今となっては僕もどういう状況で困っていたのか忘れてしまったので、他に要望がなければクローズしようと思います。
----------------------------------------
Feature #4483: PStoreをデフォルトで複数のスレッドから扱えるようにしたい
https://bugs.ruby-lang.org/issues/4483#change-70915
* Author: Glass_saga (Masaki Matsushita)
* Status: Feedback
* Priority: Normal
* Assignee: Glass_saga (Masaki Matsushita)
* Target version:
----------------------------------------
=begin
PStoreは、デフォルトでは他のスレッドがトランザクションを実行中にトランザクションに入ろうとすると例外nested transactionを発生させます。
initialize時に第2引数が真であれば、別のスレッドがトランザクションから抜けるまで待つようになりますが、
PStoreの用途から考えると、わざわざ引数にそのような指定をしなくても例外を投げずに待ってくれた方が嬉しいと思うのですが、いかがでしょうか。
PStore.newの第2引数を削除し、別のスレッドがトランザクションを実行中でもデフォルトで例外を投げずに待つようにしたpstore.rbと、
それに合わせて改変したtest_pstore.rbのパッチを添付します。
=end
---Files--------------------------------
patch.diff (1.61 KB)
--
https://bugs.ruby-lang.org/
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2018-03-08 14:04 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-08 5:37 [ruby-dev:43327] [Ruby 1.9 - Feature #4483][Open] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Masaki Matsushita
2011-03-08 5:47 ` [ruby-dev:43328] [Ruby 1.9 - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい Shota Fukumori
2011-03-08 5:59 ` [ruby-dev:43329] " Shyouhei Urabe
2011-03-08 6:23 ` [ruby-dev:43330] " Shota Fukumori
2011-03-08 7:01 ` [ruby-dev:43331] " Masaki Matsushita
2011-03-08 7:11 ` [ruby-dev:43332] " Shota Fukumori
2012-03-18 10:08 ` [ruby-dev:45395] [ruby-trunk - Feature #4483][Assigned] PStoreをデフォルトで複数のスレッドから扱えるようにしたい nahi
2012-11-20 12:27 ` [ruby-dev:46558] [ruby-trunk - Feature #4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい mame (Yusuke Endoh)
2018-03-08 8:46 ` [ruby-dev:50492] [Ruby trunk Feature#4483][Feedback] PStoreをデフォルトで複数のスレッドから扱えるようにしたい mame
2018-03-08 14:04 ` [ruby-dev:50495] [Ruby trunk Feature#4483] PStoreをデフォルトで複数のスレッドから扱えるようにしたい glass.saga
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).