Hacker News new | ask | show | jobs
by haimez 2393 days ago
Because rails is extremely eager to redefine classes and monkey patch instances by design. This is hostile to almost all JIT models. V8 spends a very large amount of engineering time optimizing specific cases of handling the common cases of this, but it’s non trivial.
1 comments
hsm3 2393 days ago
Aren't most of those shenanigans done early in the VM's lifespan (as Rails boots, actions are called the first time,...), after which the world would be stable, and JIT-related caches wouldn't have to be lost?
link
haimez 2393 days ago
Generally yes, and I won’t get into the weeds about how inlined methods can have their inlinedness invalided if you look too hard at or breathe on them- but this isn’t a deterministic process. The first time you touch an ActiveRecord::Model, the database is queried and your classes ARE redefined. It’s not even close to the same ballpark of loading a new class on the JVM, and the stabilizing assumption you’re hoping exists often doesn’t. Death by 1,000 cuts
link
t-writescode 2393 days ago
New Rails dev here: I believe the entire ActiveRecord model is constant monkey patching.
link
FooBarWidget 2393 days ago
Long time Rails dev here. I believe not. The Rails core devs have long known that monkey patching is not thread safe, and blows Ruby method caches, so they already reserve monkey patching for initialization time.
link
thomasfedb 2393 days ago
I feel that the definition of "monkey patching" doesn't (or shouldn't) cover what Rails does in AR. Monkey patch suggests a change of behaviour over what was initially intended, however dynamic method generation in AR is very-much part of the intended API.

That said, there are plenty of Rails apps that do monkey patch Rails internals, for better or worse.

link
toasterlovin 2393 days ago
Do you have something specific in mind? From my understanding of ActiveRecord, most of the method generation should happen the first time the class is defined.
link
viraptor 2393 days ago
Closer to the first time it's used. Schema gets queried and methods get added the moment you do the first query on the model normally.
link
mperham 2393 days ago
This is not true in production, where eager loading is always on.
link