aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/test/abstract_unit.rb
diff options
context:
space:
mode:
authorSean Griffin <sean@seantheprogrammer.com>2015-10-15 09:40:21 -0700
committerSean Griffin <sean@seantheprogrammer.com>2015-10-15 09:50:37 -0700
commit34321e4a433bb7eef48fd743286601403f8f7d82 (patch)
tree45ffbd21ae35cb5002566a4327e5addb9d005d47 /activesupport/test/abstract_unit.rb
parentde92b06bca9e744525452dd83e13a784bcfb9d3e (diff)
downloadrails-34321e4a433bb7eef48fd743286601403f8f7d82.tar.gz
rails-34321e4a433bb7eef48fd743286601403f8f7d82.tar.bz2
rails-34321e4a433bb7eef48fd743286601403f8f7d82.zip
Add an immutable string type to opt out of string duping
This type adds an escape hatch to apps for which string duping causes unacceptable memory growth. The reason we are duping them is in order to detect mutation, which was a feature added to 4.2 in #15674. The string type was modified to support this behavior in #15788. Memory growth is really only a concern for string types, as it's the only mutable type where the act of coersion does not create a new object regardless (as we're usually returning an object of a different class). I do feel strongly that if we are going to support detecting mutation, we should do it universally for any type which is mutable. While it is less common and ideomatic to mutate strings than arrays or hashes, there shouldn't be rules or gotchas to understanding our behavior. However, I also appreciate that for apps which are using a lot of string columns, this would increase the number of allocations by a large factor. To ensure that we keep our contract, if you'd like to opt out of mutation detection on strings, you'll also be option out of mutation of those strings. I'm not completely married to the thought that strings coming out of this actually need to be frozen -- and I think the name is correct either way, as the purpose of this is to provide a string type which does not detect mutation. In the new implementation, I'm only overriding `cast_value`. I did not port over the duping in `serialize`. I cannot think of a reason we'd need to dup the string there, and the tests pass without it. Unfortunately that line was introduced at a time where I was not nearly as good about writing my commit messages, so I have no context as to why I added it. Thanks past Sean. You are a jerk.
Diffstat (limited to 'activesupport/test/abstract_unit.rb')
0 files changed, 0 insertions, 0 deletions