SqlLiteral does not serialize/de-serailize correctly

rails / arel

A Relational Algebra

2.06k stars 390 forks source link

SqlLiteral does not serialize/de-serailize correctly #509

Closed bobf closed 6 years ago

bobf commented 6 years ago

require 'psych'
require 'arel'
yaml = Arel::Nodes::SqlLiteral.new('*').to_yaml
puts Psych.load(yaml).is_a?(Arel::Nodes::SqlLiteral) # false

I have submitted a PR (https://github.com/rails/arel/pull/503) to fix but haven't had any feedback (posted 20 days ago) so hoping to gain a bit of traction by raising a bug as well.

matthewd commented 6 years ago

Per #523, Arel development is moving to rails/rails.

If this issue is still relevant, please consider reopening it over there. (Note that the Rails repository does not accept feature request issues, and requires reproduction steps that rely on Active Record's documented API.)