activewarehouse / activewarehouse-etl

Extract-Transform-Load library from ActiveWarehouse
MIT License
240 stars 102 forks source link

Load_cache broken in some case for SQLResolver #120

Closed epinault closed 1 year ago

epinault commented 11 years ago

if specifying a resolver such as

transform :source, :foreign_key_lookup,
{
  :resolver => SQLResolver.new( :sources, :name , :datawarehouse),
  :default => nil
}

the record from the database come back as a form

{"id"=>"10", "name"=>"working"}

and there is a disconnect since the field is a symbol (:name vs "name")

What needs to happen is to match to a string if select_all return only string base hash or symbolize that hash

thbar commented 11 years ago

Thanks for reporting this; strings vs. symbols is often a concern in the way pipeline is processed!

I think we should probably symbolize the hash, right, but then I'll have to add a breaking change note in the next release.