Thread.current local-variables behavior
Hi, I stumbled upon something which for me is a bug and wanted to check if it is working as intended or not:
Thread.current[:a] = 1 p Thread.current[:a] # => 1 Fiber.new do p Thread.current[:a] # => nil end.resume
There is clearly a problem in either the documentation or the implementation for me there, we are in the same thread yet the returned values are different which is completely counter intuitive...
Why not add a fiber-variables store to allow the following and keep things separated between fibers and threads
Fiber.current[:a] = 1 p Fiber.current[:a] # => 1 Fiber.new do p Fiber.current[:a] # => nil end.resume
Which is now the behavior I would expect.
#1 [ruby-core:41607] Updated by mame (Yusuke Endoh) over 5 years ago
- Status changed from Open to Rejected
Hello, thank you for your reporting.
I am not surprised that you are surprised, but it is intentional.
The rationale is because most of legacy libraries that uses thread-
local storage will expect the storage to be also fiber-local.
We cannot change this behavior until Ruby 3.0 (unless matz decides
to change it).
Yusuke Endoh email@example.com
#2 [ruby-core:41608] Updated by schmurfy (Julien A) over 5 years ago
Although I consider this behavior (and the reason why it was done) completely absurd, how about at least changing the documentation to mention the real behavior of the supposedly thread-local storage ?
Replacing thread-local by fiber-local in the description of Thread and Thread= makes a lot more sense for me since every thread has its own root fiber, by reading the current documentation I expect my second call to Thread.current[:a] to return 1 not nil which is clearly misleading and could create nice bugs...
The current text is ( for Thread[sym] ):
Attribute Reference—Returns the value of a thread-local variable, using either a symbol or a string name. If the specified variable does not exist, returns nil.
#3 [ruby-core:41610] Updated by mame (Yusuke Endoh) over 5 years ago
- Status changed from Rejected to Open
- Assignee set to akr (Akira Tanaka)
Sorry, I was stupid. Indeed the documatation has room for improvement.
If I recall correctly, akira tanaka suggested the current behavior.
So I'm reopening and assigning this ticket to him.
Yusuke Endoh firstname.lastname@example.org
#4 [ruby-core:41697] Updated by schmurfy (Julien A) over 5 years ago
any news on this ?
This documentation bug is here since 1.9 was first released from what I understand so I think it would be rather urgent to fix it, for me a documentation is even worse than a bug in the codebase since it leads you to think wrongly about how things works.
#7 Updated by akr (Akira Tanaka) almost 5 years ago
- Status changed from Assigned to Closed
- % Done changed from 0 to 100
This issue was solved with changeset r36269.
Julien, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.
- thread.c (rb_thread_aref): add explanation for why Thread# and Thread#= are fiber-local and not thread-local. reported by Julien A. [ruby-trunk - Bug #5750]