diff options
| author | Raimonds Simanovskis <raimonds.simanovskis@gmail.com> | 2010-09-29 13:10:17 +0300 |
|---|---|---|
| committer | Raimonds Simanovskis <raimonds.simanovskis@gmail.com> | 2010-09-29 13:10:17 +0300 |
| commit | e619462165bfab5bee85f9761df7f48a6b8f30f0 (patch) | |
| tree | 480a4ff4d22ff03462613ca27ddf78895b25bd54 /lib/arel/visitors/oracle.rb | |
| parent | 538faf2ebc0e0e52a876f3be59c6219f620c780b (diff) | |
| download | rails-e619462165bfab5bee85f9761df7f48a6b8f30f0.tar.gz rails-e619462165bfab5bee85f9761df7f48a6b8f30f0.tar.bz2 rails-e619462165bfab5bee85f9761df7f48a6b8f30f0.zip | |
use subquery for limit when DISTINCT is used (in Oracle)
Diffstat (limited to 'lib/arel/visitors/oracle.rb')
| -rw-r--r-- | lib/arel/visitors/oracle.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/lib/arel/visitors/oracle.rb b/lib/arel/visitors/oracle.rb index 659b75f16d..c691ce294e 100644 --- a/lib/arel/visitors/oracle.rb +++ b/lib/arel/visitors/oracle.rb @@ -6,7 +6,9 @@ module Arel def visit_Arel_Nodes_SelectStatement o o = order_hacks(o) - if o.limit && o.orders.empty? && !o.offset + # if need to select first records without ORDER BY and GROUP BY and without DISTINCT + # then can use simple ROWNUM in WHERE clause + if o.limit && o.orders.empty? && !o.offset && o.cores.first.projections.first !~ /^DISTINCT / o.cores.last.wheres.push Nodes::LessThanOrEqual.new( Nodes::SqlLiteral.new('ROWNUM'), o.limit ) @@ -31,7 +33,7 @@ module Arel eosql end - if o.limit && !o.orders.empty? + if o.limit o = o.dup limit = o.limit o.limit = nil |