search

rails-sqlserver / activerecord-sqlserver-adapter

SQL Server Adapter For Rails
MIT License
972 stars 559 forks source link

Improve performance of view default function lookup #1072

Closed albus522 closed 1 year ago

albus522 commented 1 year ago

We are reading from a database outside our control that uses some absolutely massive views (over 500 columns). This makes loading the column definition significantly faster.

aidanharan commented 1 year ago

Hi @albus522, what version of Rails are you currently using this patch on? If you are using it on Rails 7.0 then could you make the MR for the 7-0-stable branch? The main branch is being used for Rails 7.1 development.

albus522 commented 1 year ago

I can switch it to 7-0-stable. I figured it would start on main and be patched, but I am good with whatever works best for you.

aidanharan commented 1 year ago

Merged into main in https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/1074