Make sure exclude [__rnt] table names form relation reflection.

* Fixes #219 and #221.
* Thanks @sphogan.

Author: metaskills

CHANGELOG

@@ -4,6 +4,7 @@
 * The #remove_default_constraint uses #execute_procedure now. Fixes #223. Thanks @gicappa and @clintmiller.
 * Mimic other adapters quoting for empty strings passed to integer columns. Fixes #164.
 * Allow named parameters in stored procedures. Fixes #216. Thanks @sixfeetover.
+* Make sure exclude [__rnt] table names form relation reflection. Fixes #219 and #221. Thanks @sphogan.
 
 
 * 3.2.8 *

lib/active_record/connection_adapters/sqlserver/core_ext/relation.rb

@@ -0,0 +1,19 @@
+module ActiveRecord
+  module ConnectionAdapters
+    module Sqlserver
+      module CoreExt
+        module Relation
+          
+          private
+          
+          def tables_in_string(string)
+            super - ['__rnt']
+          end
+          
+        end
+      end
+    end
+  end
+end
+
+ActiveRecord::Relation.send :include, ActiveRecord::ConnectionAdapters::Sqlserver::CoreExt::Relation

lib/active_record/connection_adapters/sqlserver_adapter.rb

@@ -8,6 +8,7 @@
 require 'active_record/connection_adapters/sqlserver/core_ext/active_record'
 require 'active_record/connection_adapters/sqlserver/core_ext/database_statements'
 require 'active_record/connection_adapters/sqlserver/core_ext/explain'
+require 'active_record/connection_adapters/sqlserver/core_ext/relation'
 require 'active_record/connection_adapters/sqlserver/database_limits'
 require 'active_record/connection_adapters/sqlserver/database_statements'
 require 'active_record/connection_adapters/sqlserver/errors'

test/cases/offset_and_limit_test_sqlserver.rb

@@ -4,13 +4,14 @@
 require 'models/reference'
 require 'models/book'
 require 'models/author'
+require 'models/subscription'
 require 'models/post'
 require 'models/comment'
 require 'models/categorization'
 
 class OffsetAndLimitTestSqlserver < ActiveRecord::TestCase
 
-  fixtures :jobs, :people, :references, 
+  fixtures :jobs, :people, :references, :subscriptions,
            :authors, :posts, :comments, :categorizations
 
   setup     :create_10_books
@@ -94,6 +95,13 @@ class OffsetAndLimitTestSqlserver < ActiveRecord::TestCase
       assert_equal [8, 9, 10, 12], david.ordered_uniq_comments.limit(5).offset(6).map(&:id)
       assert_equal 4, david.ordered_uniq_comments.limit(5).offset(6).count
     end
+
+    should 'remove [__rnt] table names from relation reflection and hence do not eager loading' do
+      create_10_books
+      create_10_books
+      assert_queries(1) { Book.limit(10).offset(10).includes(:subscriptions).all }
+    end
+    
 
     context 'with count' do