Merge branch 'master' into collector

* master:
  removing unused join_sql method

3 files changed, 0 insertions, 27 deletions

diff --git a/lib/arel/select_manager.rb b/lib/arel/select_manager.rb
index db34ffb946..fe0d26a16e 100644
--- a/lib/arel/select_manager.rb
+++ b/lib/arel/select_manager.rb
@@ -215,13 +215,6 @@ module Arel
     end
     alias limit= take
 
-    def join_sql
-      return nil if @ctx.source.right.empty?
-
-      sql = visitor.dup.extend(Visitors::JoinSql).accept @ctx
-      Nodes::SqlLiteral.new sql
-    end
-
     def join_sources
       @ctx.source.right
     end
diff --git a/lib/arel/visitors.rb b/lib/arel/visitors.rb
index 3986ad9ce5..4a8d254ba7 100644
--- a/lib/arel/visitors.rb
+++ b/lib/arel/visitors.rb
@@ -6,7 +6,6 @@ require 'arel/visitors/postgresql'
 require 'arel/visitors/mysql'
 require 'arel/visitors/mssql'
 require 'arel/visitors/oracle'
-require 'arel/visitors/join_sql'
 require 'arel/visitors/where_sql'
 require 'arel/visitors/dot'
 require 'arel/visitors/ibm_db'
diff --git a/lib/arel/visitors/join_sql.rb b/lib/arel/visitors/join_sql.rb
deleted file mode 100644
index 1cdd7eb5ca..0000000000
--- a/lib/arel/visitors/join_sql.rb
+++ /dev/null
@@ -1,19 +0,0 @@
-module Arel
-  module Visitors
-    ###
-    # This class produces SQL for JOIN clauses but omits the "single-source"
-    # part of the Join grammar:
-    #
-    #   http://www.sqlite.org/syntaxdiagrams.html#join-source
-    #
-    # This visitor is used in SelectManager#join_sql and is for backwards
-    # compatibility with Arel V1.0
-    module JoinSql
-      private
-
-      def visit_Arel_Nodes_SelectCore o
-        o.source.right.map { |j| visit j }.join ' '
-      end
-    end
-  end
-end