Proposed fix for issue 79 - https://github.com/rails/arel/issues/79 #80

Merged
merged 1 commit into from Aug 29, 2011

Conversation

Projects
None yet
2 participants
@jsanders

Proposed fix for issue 79 - #79 - Pass the connection_pool to the ToSql initializer, rather than the engine itself.

tenderlove added a commit that referenced this pull request Aug 29, 2011

Merge pull request #80 from jsanders/patch-1
Proposed fix for issue 79 - #79

@tenderlove tenderlove merged commit 8ab89f5 into rails:master Aug 29, 2011

Proposed fix for issue 79 - #79 - Pass the connection_pool to the ToS…
…ql initializer, rather than the engine itself.
@jsanders

This comment has been minimized.

Show comment Hide comment
@jsanders

jsanders Aug 30, 2011

Thanks Aaron!

Thanks Aaron!

@tenderlove

This comment has been minimized.

Show comment Hide comment
@tenderlove

tenderlove Aug 30, 2011

Owner

No problem! Thank you!

On Tue, Aug 30, 2011 at 9:55 AM, jsanders
reply@reply.github.com
wrote:

Thanks Aaron!

Reply to this email directly or view it on GitHub:
#80 (comment)

Aaron Patterson
http://tenderlovemaking.com/

Owner

tenderlove commented Aug 30, 2011

No problem! Thank you!

On Tue, Aug 30, 2011 at 9:55 AM, jsanders
reply@reply.github.com
wrote:

Thanks Aaron!

Reply to this email directly or view it on GitHub:
#80 (comment)

Aaron Patterson
http://tenderlovemaking.com/

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