BRILLIANT!  Thank you very much Jeremy.  That works like a champ!

I spent a bunch of time going thru the changelog and completed missed that 
note.  

Thank you for the brilliant work with the Sequel gem and your support.

You rock!  Cheers!


On Monday, April 9, 2018 at 8:31:56 PM UTC-7, Jeremy Evans wrote:
>
> On Monday, April 9, 2018 at 4:57:15 PM UTC-7, Joe Rzepiejewski wrote:
>>
>> We're using Redshift with Sequel gem *4.45* and are able to override the 
>> methods for `insert_returning_sql` and `supports_returning?`.
>> When we upgraded to Sequel *>= 4.46* our overridden methods are *not* 
>> called.  (We have the code to override the Database and Dataset -- akin to 
>> what's in the sequel-redshift 0.0.1 gem)
>>
>> It appears that our overridden Redshift dataset method is not being 
>> called.  e.g. db[...].insert(...) and returns the standard 
>> Postgres::Dataset not our Sequel::Redshift::Dataset.  I am trying to 
>> determine what has changed such that the overridden Dataset is not being 
>> called.
>>
>
> Not sure how the Redshift database adapter is setup, but if it works by 
> inherited from the postgres adapter and setting a DatasetClass constant, 
> it's probably the result of this change mentioned in the release notes:
>
> * Automatically looking up the dataset class for a Database
>   instance by looking for a DatasetClass constant in the Database's
>   class is now deprecated.  All adapters that ship with Sequel have
>   been converted, but external adapters should now define the
>   Database#dataset_class_default private method appropriately
>   to return the correct dataset class
>
> Implement the Database#dataset_class_default method in the redshift 
> adapter return the appropriate dataset class in that case.
>
> If that is not it, you'll want to bisect the changes between 4.45.0 and 
> 4.46.0 and see what caused it, and try to fix the problem.
>
> Thanks,
> Jeremy
>

-- 
You received this message because you are subscribed to the Google Groups 
"sequel-talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sequel-talk+unsubscr...@googlegroups.com.
To post to this group, send email to sequel-talk@googlegroups.com.
Visit this group at https://groups.google.com/group/sequel-talk.
For more options, visit https://groups.google.com/d/optout.

Reply via email to