attr_readonly support + LOB dirty checking #118

Closed
wants to merge 7 commits into
from

Conversation

Projects
None yet
1 participant
@substars
Contributor

substars commented Nov 8, 2011

We ran into some strange deadlocking problems in our app, and it turns out they were caused by OCI streaming CLOB data on every AR save irrespective of whether it had changed. My first thought was to attr_readonly the field, but the adapter didn't look at that setting, either.

So this makes attr_readonly work with LOB data and also doesn't stream LOBs back into the DB unless they've changed.

Should we also make the dirty checking dependent on AR partial updates being turned on? I have mixed feelings about this, but it would be pretty easy to do.

@substars substars closed this Nov 8, 2011

@substars

This comment has been minimized.

Show comment
Hide comment
@substars

substars Nov 8, 2011

Contributor

whoops, i screwed this up, will recreate

Contributor

substars commented Nov 8, 2011

whoops, i screwed this up, will recreate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment