aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases/arel/attributes
diff options
context:
space:
mode:
authorVladimir Kochnev <hashtable@yandex.ru>2018-09-25 14:54:03 +0400
committerVladimir Kochnev <hashtable@yandex.ru>2018-09-25 15:39:14 +0400
commit41b92914f89be855cc6af768f13fd5fc53c967fa (patch)
tree7ee8310a3727a51a8a7f5e2bcf95427488930057 /activerecord/test/cases/arel/attributes
parent4f3eb86ffec8ebc4faea3143dda8ebb5aad74d08 (diff)
downloadrails-41b92914f89be855cc6af768f13fd5fc53c967fa.tar.gz
rails-41b92914f89be855cc6af768f13fd5fc53c967fa.tar.bz2
rails-41b92914f89be855cc6af768f13fd5fc53c967fa.zip
Abandon TOP support.
Initially, `TOP` was introduced to support `limit` for MSSQL database. Unlike PostgreSQL/MySQL/SQLite, MSSQL does not have native `LIMIT`/`OFFSET` support. The commit adding `TOP` is 1a246f71616cf246a75ef6cbdb56032e43d4e643. However, it figured out that `TOP` implementation was weak and it's not sufficient to also support `OFFSET`, then `TOP` was substituted with `ROW_NUMBER()` subquery in be48ed3071fd6524d0145c4ad3faeb4aafe3eda3. This is a well known trick in MSSQL - https://stackoverflow.com/questions/2135418/equivalent-of-limit-and-offset-for-sql-server. So now we don't need this `visit_Arel_Nodes_Top` at all. It does nothing useful but also adds an extra space after `SELECT` when `LIMIT` is being used for **any** database.
Diffstat (limited to 'activerecord/test/cases/arel/attributes')
0 files changed, 0 insertions, 0 deletions