On Thursday, July 23, 2015 at 8:43:23 AM UTC-7, [email protected] wrote: > > > > On Thursday, July 23, 2015 at 4:58:23 PM UTC+2, Jeremy Evans wrote: > > >> Does this fix it: >> >> diff --git a/lib/sequel/adapters/shared/mssql.rb >> b/lib/sequel/adapters/shared/mssql.rb >> index 3993930..b7acc4e 100644 >> --- a/lib/sequel/adapters/shared/mssql.rb >> +++ b/lib/sequel/adapters/shared/mssql.rb >> @@ -433,7 +433,7 @@ module Sequel >> inf_sch_qual = lambda{|s| info_sch_sch ? >> Sequel.qualify(info_sch_sch, s) : Sequel.expr(s)} >> sys_qual = lambda{|s| info_sch_sch ? >> Sequel.qualify(info_sch_sch, Sequel.qualify(Sequel.lit(''), s)) : >> Sequel.expr(s)} >> >> - identity_cols = metadata_dataset.from(:sys__columns). >> + identity_cols = >> metadata_dataset.from(Sequel.lit('[sys].[columns]')). >> where(:object_id=>table_id, :is_identity=>true). >> select_map(:name) >> >> >> This patch only changes sys.columns, not the information_schema stuff, so >> it should hopefully not have this problem. >> >> > No it is continuing with TinyTds::Error: Invalid object name 'SYSINDEXES'. > > In my example code I replaced some other table: > > sql = > sql.gsub(/\[SYS\].\[COLUMNS\]|SYSINDEXES|SYSINDEXKEYS|SYSCOLUMNS/){|x| > > But your change looks promising. I will try if I can build a pull request > next week - I think I can find all places on my own. >
Sounds good. If you can get the change to me by Thursday next week, the change can make 4.25.0. 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 [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sequel-talk. For more options, visit https://groups.google.com/d/optout.
