sequel 4.49.0 → 5.0.0

data/lib/sequel/exceptions.rb

@@ -75,7 +75,7 @@ module Sequel
 
   (
   # Error raised on an invalid operation, such as trying to update or delete
-  # a joined or grouped dataset.
+  # a joined or grouped dataset when the database does not support that.
   InvalidOperation = Class.new(Error)
   ).name
 
@@ -113,20 +113,4 @@ module Sequel
   # and won't reraise it (unless a reraise is requested).
   Rollback = Class.new(Error)
   ).name
-
-  # SEQUEL5: Remove
-  (UnbindDuplicate = Class.new(Error)).name
-
-  Error::AdapterNotFound = AdapterNotFound
-  Error::InvalidOperation = InvalidOperation
-  Error::InvalidValue = InvalidValue
-  Error::PoolTimeoutError = PoolTimeout
-  Error::Rollback = Rollback
-
-  Sequel::Deprecation.deprecate_constant(self, :UnbindDuplicate)
-  Sequel::Deprecation.deprecate_constant(Error, :AdapterNotFound)
-  Sequel::Deprecation.deprecate_constant(Error, :InvalidOperation)
-  Sequel::Deprecation.deprecate_constant(Error, :InvalidValue)
-  Sequel::Deprecation.deprecate_constant(Error, :PoolTimeoutError)
-  Sequel::Deprecation.deprecate_constant(Error, :Rollback)
 end

data/lib/sequel/extensions/_model_pg_row.rb

@@ -4,13 +4,6 @@ module Sequel
   module Plugins
     module PgRow
       module DatabaseMethods
-        ESCAPE_RE = /("|\\)/.freeze
-        Sequel::Deprecation.deprecate_constant(self, :ESCAPE_RE)
-        ESCAPE_REPLACEMENT = '\\\\\1'.freeze
-        Sequel::Deprecation.deprecate_constant(self, :ESCAPE_REPLACEMENT)
-        COMMA = ','
-        Sequel::Deprecation.deprecate_constant(self, :COMMA)
-
         # Handle Sequel::Model instances in bound variables.
         def bound_variable_arg(arg, conn)
           case arg

data/lib/sequel/extensions/_pretty_table.rb

@@ -27,7 +27,7 @@ module Sequel
 
     # Return the string that #print will print via puts.
     def self.string(records, columns = nil) # records is an array of hashes
-      columns ||= records.first.keys.sort_by(&:to_s)
+      columns ||= records.first.keys.sort
       sizes = column_sizes(records, columns)
       sep_line = separator_line(columns, sizes)
 
@@ -37,8 +37,6 @@ module Sequel
       array.join("\n")
     end
 
-    ### Private Module Methods ###
-
     # Hash of the maximum size of the value for each column 
     def self.column_sizes(records, columns) # :nodoc:
       sizes = Hash.new {0}

data/lib/sequel/extensions/arbitrary_servers.rb

@@ -9,34 +9,34 @@
 # Then you can pass arbitrary connection options for the server/shard
 # to use as a hash:
 #
-#   DB[:table].server(:host=>'...', :database=>'...').all
+#   DB[:table].server(host: '...', database: '...').all
 #
 # Because Sequel can never be sure that the connection will be reused,
 # arbitrary connections are disconnected as soon as the outermost block
 # that uses them exits.  So this example uses the same connection:
 #
-#   DB.transaction(:server=>{:host=>'...', :database=>'...'}) do |c|
-#     DB.transaction(:server=>{:host=>'...', :database=>'...'}) do |c2|
+#   DB.transaction(server: {host: '...', database: '...'}) do |c|
+#     DB.transaction(server: {host: '...', database: '...'}) do |c2|
 #       # c == c2
 #     end
 #   end
 #
 # But this example does not:
 #
-#   DB.transaction(:server=>{:host=>'...', :database=>'...'}) do |c|
+#   DB.transaction(server: {host: '...', database: '...'}) do |c|
 #   end
-#   DB.transaction(:server=>{:host=>'...', :database=>'...'}) do |c2|
+#   DB.transaction(server: {host: '...', database: '...'}) do |c2|
 #     # c != c2
 #   end
 #
 # You can use this extension in conjunction with the server_block
 # extension:
 #
-#   DB.with_server(:host=>'...', :database=>'...') do
+#   DB.with_server(host: '...', database: '...') do
 #     DB.synchronize do
 #       # All of these use the host/database given to with_server
-#       DB[:table].insert(:c=>1)
-#       DB[:table].update(:c=>2)
+#       DB[:table].insert(c: 1)
+#       DB[:table].update(c: 2)
 #       DB.tables
 #       DB[:table].all
 #     end
@@ -46,8 +46,8 @@
 # extension may want to do the following to so that you don't need
 # to call synchronize separately:
 #
-#   def DB.with_server(*)
-#     super{synchronize{yield}}
+#   def DB.with_server(*a)
+#     super(*a){synchronize{yield}}
 #   end
 #
 # Note that this extension only works with the sharded threaded connection

data/lib/sequel/extensions/connection_expiration.rb

@@ -40,7 +40,7 @@ module Sequel
 
     # Initialize the data structures used by this extension.
     def self.extended(pool)
-      pool.instance_eval do
+      pool.instance_exec do
         sync do
           @connection_expiration_timestamps ||= {}
           @connection_expiration_timeout ||= 14400

data/lib/sequel/extensions/connection_validator.rb

@@ -60,7 +60,7 @@ module Sequel
 
     # Initialize the data structures used by this extension.
     def self.extended(pool)
-      pool.instance_eval do
+      pool.instance_exec do
         sync do
           @connection_timestamps ||= {}
           @connection_validation_timeout ||= 3600

data/lib/sequel/extensions/constraint_validations.rb

@@ -22,7 +22,7 @@
 # generally want to create it first, before creating any other application
 # tables.
 #
-# Because migrations instance_eval the up and down blocks on a database,
+# Because migrations instance_exec the up and down blocks on a database,
 # using this extension in a migration can be done via:
 #
 #   Sequel.migration do
@@ -43,7 +43,7 @@
 # similar to the validation_helpers model plugin API.  However,
 # instead of having separate validates_* methods, it just adds a validate
 # method that accepts a block to the schema generators.  Like the
-# create_table and alter_table blocks, this block is instance_evaled and
+# create_table and alter_table blocks, this block is instance_execed and
 # offers its own DSL. Example:
 #
 #   DB.create_table(:table) do
@@ -56,8 +56,8 @@
 #     end
 #   end
 #
-# instance_eval is used in this case because create_table and alter_table
-# already use instance_eval, so losing access to the surrounding receiver
+# instance_exec is used in this case because create_table and alter_table
+# already use instance_exec, so losing access to the surrounding receiver
 # is not an issue.
 #
 # Here's a breakdown of the constraints created for each constraint validation
@@ -196,9 +196,9 @@ module Sequel
         @generator.validation({:type=>:drop, :name=>constraint})
       end
 
-      # Alias of instance_eval for a nicer API.
+      # Alias of instance_exec for a nicer API.
       def process(&block)
-        instance_eval(&block)
+        instance_exec(&block)
       end
     end
 
@@ -260,7 +260,7 @@ module Sequel
       super do
         extend CreateTableGeneratorMethods
         @validations = []
-        instance_eval(&block) if block
+        instance_exec(&block) if block
       end
     end
 
@@ -308,7 +308,7 @@ module Sequel
       super do
         extend AlterTableGeneratorMethods
         @validations = []
-        instance_eval(&block) if block
+        instance_exec(&block) if block
       end
     end
 
@@ -374,10 +374,10 @@ module Sequel
         when :max_length
           generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| Sequel.char_length(c) <= arg}))
         when *REVERSE_OPERATOR_MAP.keys
-          generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| Sequel.identifier(c).send(REVERSE_OPERATOR_MAP[validation_type], arg)}))
+          generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| Sequel.identifier(c).public_send(REVERSE_OPERATOR_MAP[validation_type], arg)}))
         when :length_range
           op = arg.exclude_end? ? :< : :<=
-          generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| (Sequel.char_length(c) >= arg.begin) & Sequel.char_length(c).send(op, arg.end)}))
+          generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| (Sequel.char_length(c) >= arg.begin) & Sequel.char_length(c).public_send(op, arg.end)}))
           arg = "#{arg.begin}..#{'.' if arg.exclude_end?}#{arg.end}"
         when :format
           generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| {c => arg}}))
@@ -407,7 +407,7 @@ module Sequel
             raise Error, "validates includes only supports arrays and ranges currently, cannot handle: #{arg.inspect}"
           end
         when :like, :ilike
-          generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| Sequel.send(validation_type, c, arg)}))
+          generator_add_constraint_from_validation(generator, val, Sequel.&(*columns.map{|c| Sequel.public_send(validation_type, c, arg)}))
         when :unique
           generator.unique(columns, :name=>constraint)
           columns = [columns.join(',')]

data/lib/sequel/extensions/core_extensions.rb

@@ -21,8 +21,8 @@ class Array
   # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this array, not matching all of the
   # conditions.
   #
-  #   ~[[:a, true]] # SQL: a IS NOT TRUE
-  #   ~[[:a, 1], [:b, [2, 3]]] # SQL: a != 1 OR b NOT IN (2, 3)
+  #   ~[[:a, true]] # SQL: (a IS NOT TRUE)
+  #   ~[[:a, 1], [:b, [2, 3]]] # SQL: ((a != 1) OR (b NOT IN (2, 3)))
   def ~
     Sequel.~(self)
   end
@@ -30,7 +30,7 @@ class Array
   # Return a <tt>Sequel::SQL::CaseExpression</tt> with this array as the conditions and the given
   # default value and expression.
   #
-  #   [[{:a=>[2,3]}, 1]].case(0) # SQL: CASE WHEN a IN (2, 3) THEN 1 ELSE 0 END
+  #   [[{a: [2,3]}, 1]].case(0) # SQL: CASE WHEN (a IN (2, 3)) THEN 1 ELSE 0 END
   #   [[:a, 1], [:b, 2]].case(:d, :c) # SQL: CASE c WHEN a THEN 1 WHEN b THEN 2 ELSE d END
   def case(*args)
     ::Sequel::SQL::CaseExpression.new(self, *args)
@@ -42,16 +42,12 @@ class Array
   # this array as a value in a filter, but may be necessary if you are using it as a
   # value with placeholder SQL:
   #
-  #   DB[:a].where([:a, :b]=>[[1, 2], [3, 4]]) # SQL: (a, b) IN ((1, 2), (3, 4))
-  #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]]) # SQL: (a, b) IN ((1 = 2) AND (3 = 4))
-  #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]].sql_value_list) # SQL: (a, b) IN ((1, 2), (3, 4))
+  #   DB[:a].where([:a, :b]=>[[1, 2], [3, 4]]) # SQL: ((a, b) IN ((1, 2), (3, 4)))
+  #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]]) # SQL: ((a, b) IN ((1 = 2) AND (3 = 4)))
+  #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]].sql_value_list) # SQL: ((a, b) IN ((1, 2), (3, 4)))
   def sql_value_list
     ::Sequel::SQL::ValueList.new(self)
   end
-  def sql_array
-    Sequel::Deprecation.deprecate("Array#sql_array in the core_extensions extension", "Use Array#sql_value_list instead")
-    sql_value_list
-  end
 
   # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this array, matching all of the
   # conditions.  Rarely do you need to call this explicitly, as Sequel generally
@@ -60,8 +56,8 @@ class Array
   # and want to use the = operator instead of the IN operator (which is used by default for
   # arrays of two element arrays).
   #
-  #   [[:a, true]].sql_expr # SQL: a IS TRUE
-  #   [[:a, 1], [:b, [2, 3]]].sql_expr # SQL: a = 1 AND b IN (2, 3)
+  #   [[:a, true]].sql_expr # SQL: (a IS TRUE)
+  #   [[:a, 1], [:b, [2, 3]]].sql_expr # SQL: ((a = 1) AND (b IN (2, 3)))
   def sql_expr
     Sequel[self]
   end
@@ -69,8 +65,8 @@ class Array
   # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this array, matching none
   # of the conditions.
   #
-  #   [[:a, true]].sql_negate # SQL: a IS NOT TRUE
-  #   [[:a, 1], [:b, [2, 3]]].sql_negate # SQL: a != 1 AND b NOT IN (2, 3)
+  #   [[:a, true]].sql_negate # SQL: (a IS NOT TRUE)
+  #   [[:a, 1], [:b, [2, 3]]].sql_negate # SQL: ((a != 1) AND (b NOT IN (2, 3)))
   def sql_negate
     Sequel.negate(self)
   end
@@ -78,8 +74,8 @@ class Array
   # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this array, matching any of the
   # conditions.
   #
-  #   [[:a, true]].sql_or # SQL: a IS TRUE
-  #   [[:a, 1], [:b, [2, 3]]].sql_or # SQL: a = 1 OR b IN (2, 3)
+  #   [[:a, true]].sql_or # SQL: (a IS TRUE)
+  #   [[:a, 1], [:b, [2, 3]]].sql_or # SQL: ((a = 1) OR (b IN (2, 3)))
   def sql_or
     Sequel.or(self)
   end
@@ -90,9 +86,9 @@ class Array
   # concatenation.
   #
   #   [:a].sql_string_join # SQL: a
-  #   [:a, :b].sql_string_join # SQL: a || b
-  #   [:a, 'b'].sql_string_join # SQL: a || 'b'
-  #   ['a', :b].sql_string_join(' ') # SQL: 'a' || ' ' || b
+  #   [:a, :b].sql_string_join # SQL: (a || b)
+  #   [:a, 'b'].sql_string_join # SQL: (a || 'b')
+  #   ['a', :b].sql_string_join(' ') # SQL: ('a' || ' ' || b)
   def sql_string_join(joiner=nil)
     Sequel.join(self, joiner)
   end
@@ -104,8 +100,8 @@ class Hash
   # all of the conditions in this hash and the condition specified by
   # the given argument.
   #
-  #   {:a=>1} & :b # SQL: a = 1 AND b
-  #   {:a=>true} & ~:b # SQL: a IS TRUE AND NOT b
+  #   {a: 1} & :b # SQL: ((a = 1) AND b)
+  #   {a: true} & ~:b # SQL: ((a IS TRUE) AND NOT b)
   def &(ce)
     ::Sequel::SQL::BooleanExpression.new(:AND, self, ce)
   end
@@ -114,8 +110,8 @@ class Hash
   # all of the conditions in this hash or the condition specified by
   # the given argument.
   #
-  #   {:a=>1} | :b # SQL: a = 1 OR b
-  #   {:a=>true} | ~:b # SQL: a IS TRUE OR NOT b
+  #   {a: 1} | :b # SQL: ((a = 1) OR b)
+  #   {a: true} | ~:b # SQL: ((a IS TRUE) OR NOT b)
   def |(ce)
     ::Sequel::SQL::BooleanExpression.new(:OR, self, ce)
   end
@@ -123,19 +119,17 @@ class Hash
   # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this hash, not matching all of the
   # conditions.
   #
-  #   ~{:a=>true} # SQL: a IS NOT TRUE
-  #   ~{:a=>1, :b=>[2, 3]} # SQL: a != 1 OR b NOT IN (2, 3)
+  #   ~{a: true} # SQL: (a IS NOT TRUE)
+  #   ~{a: 1, b: [2, 3]} # SQL: ((a != 1) OR (b NOT IN (2, 3)))
   def ~
     ::Sequel::SQL::BooleanExpression.from_value_pairs(self, :OR, true)
   end
 
   # Return a <tt>Sequel::SQL::CaseExpression</tt> with this hash as the conditions and the given
-  # default value.  Note that the order of the conditions will be arbitrary on ruby 1.8, so all
-  # conditions should be orthogonal.
+  # default value.
   #
-  #   {{:a=>[2,3]}=>1}.case(0) # SQL: CASE WHEN a IN (2, 3) THEN 1 ELSE 0 END
-  #   {:a=>1, :b=>2}.case(:d, :c) # SQL: CASE c WHEN a THEN 1 WHEN b THEN 2 ELSE d END
-  #                                 #  or: CASE c WHEN b THEN 2 WHEN a THEN 1 ELSE d END
+  #   {{a: [2,3]}=>1}.case(0) # SQL: CASE WHEN (a IN (2, 3)) THEN 1 ELSE 0 END
+  #   {a: 1, b: 2}.case(:d, :c) # SQL: CASE c WHEN a THEN 1 WHEN b THEN 2 ELSE d END
   def case(*args)
     ::Sequel::SQL::CaseExpression.new(to_a, *args)
   end
@@ -144,8 +138,8 @@ class Hash
   # conditions.  Rarely do you need to call this explicitly, as Sequel generally
   # assumes that hashes specify this type of condition.
   #
-  #   {:a=>true}.sql_expr # SQL: a IS TRUE
-  #   {:a=>1, :b=>[2, 3]}.sql_expr # SQL: a = 1 AND b IN (2, 3)
+  #   {a: true}.sql_expr # SQL: (a IS TRUE)
+  #   {a: 1, b: [2, 3]}.sql_expr # SQL: ((a = 1) AND (b IN (2, 3)))
   def sql_expr
     ::Sequel::SQL::BooleanExpression.from_value_pairs(self)
   end
@@ -153,8 +147,8 @@ class Hash
   # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this hash, matching none
   # of the conditions.
   #
-  #   {:a=>true}.sql_negate # SQL: a IS NOT TRUE
-  #   {:a=>1, :b=>[2, 3]}.sql_negate # SQL: a != 1 AND b NOT IN (2, 3)
+  #   {a: true}.sql_negate # SQL: (a IS NOT TRUE)
+  #   {a: 1, b: [2, 3]}.sql_negate # SQL: ((a != 1) AND (b NOT IN (2, 3)))
   def sql_negate
     ::Sequel::SQL::BooleanExpression.from_value_pairs(self, :AND, true)
   end
@@ -162,8 +156,8 @@ class Hash
   # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this hash, matching any of the
   # conditions.
   #
-  #   {:a=>true}.sql_or # SQL: a IS TRUE
-  #   {:a=>1, :b=>[2, 3]}.sql_or # SQL: a = 1 OR b IN (2, 3)
+  #   {a: true}.sql_or # SQL: (a IS TRUE)
+  #   {a: 1, b: [2, 3]}.sql_or # SQL: ((a = 1) OR (b IN (2, 3)))
   def sql_or
     ::Sequel::SQL::BooleanExpression.from_value_pairs(self, :OR)
   end
@@ -177,16 +171,16 @@ class String
   # Converts a string into a <tt>Sequel::LiteralString</tt>, in order to override string
   # literalization, e.g.:
   #
-  #   DB[:items].where(:abc => 'def').sql #=>
-  #     "SELECT * FROM items WHERE (abc = 'def')"
+  #   DB[:items].where(abc: 'def')
+  #   # "SELECT * FROM items WHERE (abc = 'def')"
   #
-  #   DB[:items].where(:abc => 'def'.lit).sql #=>
-  #     "SELECT * FROM items WHERE (abc = def)"
+  #   DB[:items].where(abc: 'def'.lit)
+  #   # "SELECT * FROM items WHERE (abc = def)"
   #
   # You can also provide arguments, to create a <tt>Sequel::SQL::PlaceholderLiteralString</tt>:
   #
-  #    DB[:items].select{|o| o.count('DISTINCT ?'.lit(:a))}.sql #=>
-  #      "SELECT count(DISTINCT a) FROM items"
+  #   DB[:items].select{|o| o.count('DISTINCT ?'.lit(:a))}
+  #   # "SELECT count(DISTINCT a) FROM items"
   def lit(*args)
     args.empty? ? Sequel::LiteralString.new(self) : Sequel::SQL::PlaceholderLiteralString.new(self, args)
   end
@@ -209,19 +203,15 @@ class Symbol
   include Sequel::SQL::SubscriptMethods
   include Sequel::SQL::ComplexExpressionMethods
 
-  # Returns receiver wrapped in an <tt>Sequel::SQL::Identifier</tt>.  Usually used to
-  # prevent splitting the symbol.
+  # Returns receiver wrapped in an <tt>Sequel::SQL::Identifier</tt>.
   #
-  #   :a__b # SQL: "a"."b"
-  #   :a__b.identifier # SQL: "a__b"
+  #   :a.identifier # SQL: "a"
   def identifier
     Sequel::SQL::Identifier.new(self)
   end
 
   # Returns a <tt>Sequel::SQL::Function</tt> with this as the function name,
-  # and the given arguments. This is aliased as <tt>Symbol#[]</tt> if the RUBY_VERSION
-  # is less than 1.9.0. Ruby 1.9 defines <tt>Symbol#[]</tt>, and Sequel
-  # doesn't override methods defined by ruby itself.
+  # and the given arguments.
   #
   #   :now.sql_function # SQL: now()
   #   :sum.sql_function(:a) # SQL: sum(a)

data/lib/sequel/extensions/core_refinements.rb

@@ -17,8 +17,8 @@ module Sequel::CoreRefinements
     # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this array, not matching all of the
     # conditions.
     #
-    #   ~[[:a, true]] # SQL: a IS NOT TRUE
-    #   ~[[:a, 1], [:b, [2, 3]]] # SQL: a != 1 OR b NOT IN (2, 3)
+    #   ~[[:a, true]] # SQL: (a IS NOT TRUE)
+    #   ~[[:a, 1], [:b, [2, 3]]] # SQL: ((a != 1) OR (b NOT IN (2, 3)))
     def ~
       Sequel.~(self)
     end
@@ -26,7 +26,7 @@ module Sequel::CoreRefinements
     # Return a <tt>Sequel::SQL::CaseExpression</tt> with this array as the conditions and the given
     # default value and expression.
     #
-    #   [[{:a=>[2,3]}, 1]].case(0) # SQL: CASE WHEN a IN (2, 3) THEN 1 ELSE 0 END
+    #   [[{a: [2,3]}, 1]].case(0) # SQL: CASE WHEN (a IN (2, 3)) THEN 1 ELSE 0 END
     #   [[:a, 1], [:b, 2]].case(:d, :c) # SQL: CASE c WHEN a THEN 1 WHEN b THEN 2 ELSE d END
     def case(*args)
       ::Sequel::SQL::CaseExpression.new(self, *args)
@@ -38,9 +38,9 @@ module Sequel::CoreRefinements
     # this array as a value in a filter, but may be necessary if you are using it as a
     # value with placeholder SQL:
     #
-    #   DB[:a].where([:a, :b]=>[[1, 2], [3, 4]]) # SQL: (a, b) IN ((1, 2), (3, 4))
-    #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]]) # SQL: (a, b) IN ((1 = 2) AND (3 = 4))
-    #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]].sql_value_list) # SQL: (a, b) IN ((1, 2), (3, 4))
+    #   DB[:a].where([:a, :b]=>[[1, 2], [3, 4]]) # SQL: ((a, b) IN ((1, 2), (3, 4)))
+    #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]]) # SQL: ((a, b) IN ((1 = 2) AND (3 = 4)))
+    #   DB[:a].where('(a, b) IN ?', [[1, 2], [3, 4]].sql_value_list) # SQL: ((a, b) IN ((1, 2), (3, 4)))
     def sql_value_list
       ::Sequel::SQL::ValueList.new(self)
     end
@@ -52,8 +52,8 @@ module Sequel::CoreRefinements
     # and want to use the = operator instead of the IN operator (which is used by default for
     # arrays of two element arrays).
     #
-    #   [[:a, true]].sql_expr # SQL: a IS TRUE
-    #   [[:a, 1], [:b, [2, 3]]].sql_expr # SQL: a = 1 AND b IN (2, 3)
+    #   [[:a, true]].sql_expr # SQL: (a IS TRUE)
+    #   [[:a, 1], [:b, [2, 3]]].sql_expr # SQL: ((a = 1) AND (b IN (2, 3)))
     def sql_expr
       Sequel[self]
     end
@@ -61,8 +61,8 @@ module Sequel::CoreRefinements
     # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this array, matching none
     # of the conditions.
     #
-    #   [[:a, true]].sql_negate # SQL: a IS NOT TRUE
-    #   [[:a, 1], [:b, [2, 3]]].sql_negate # SQL: a != 1 AND b NOT IN (2, 3)
+    #   [[:a, true]].sql_negate # SQL: (a IS NOT TRUE)
+    #   [[:a, 1], [:b, [2, 3]]].sql_negate # SQL: ((a != 1) AND (b NOT IN (2, 3)))
     def sql_negate
       Sequel.negate(self)
     end
@@ -70,8 +70,8 @@ module Sequel::CoreRefinements
     # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this array, matching any of the
     # conditions.
     #
-    #   [[:a, true]].sql_or # SQL: a IS TRUE
-    #   [[:a, 1], [:b, [2, 3]]].sql_or # SQL: a = 1 OR b IN (2, 3)
+    #   [[:a, true]].sql_or # SQL: (a IS TRUE)
+    #   [[:a, 1], [:b, [2, 3]]].sql_or # SQL: ((a = 1) OR (b IN (2, 3)))
     def sql_or
       Sequel.or(self)
     end
@@ -82,9 +82,9 @@ module Sequel::CoreRefinements
     # concatenation.
     #
     #   [:a].sql_string_join # SQL: a
-    #   [:a, :b].sql_string_join # SQL: a || b
-    #   [:a, 'b'].sql_string_join # SQL: a || 'b'
-    #   ['a', :b].sql_string_join(' ') # SQL: 'a' || ' ' || b
+    #   [:a, :b].sql_string_join # SQL: (a || b)
+    #   [:a, 'b'].sql_string_join # SQL: (a || 'b')
+    #   ['a', :b].sql_string_join(' ') # SQL: ('a' || ' ' || b)
     def sql_string_join(joiner=nil)
       Sequel.join(self, joiner)
     end
@@ -95,8 +95,8 @@ module Sequel::CoreRefinements
     # all of the conditions in this hash and the condition specified by
     # the given argument.
     #
-    #   {:a=>1} & :b # SQL: a = 1 AND b
-    #   {:a=>true} & ~:b # SQL: a IS TRUE AND NOT b
+    #   {a: 1} & :b # SQL: ((a = 1) AND b)
+    #   {a: true} & ~:b # SQL: ((a IS TRUE) AND NOT b)
     def &(ce)
       ::Sequel::SQL::BooleanExpression.new(:AND, self, ce)
     end
@@ -105,8 +105,8 @@ module Sequel::CoreRefinements
     # all of the conditions in this hash or the condition specified by
     # the given argument.
     #
-    #   {:a=>1} | :b # SQL: a = 1 OR b
-    #   {:a=>true} | ~:b # SQL: a IS TRUE OR NOT b
+    #   {a: 1} | :b # SQL: ((a = 1) OR b)
+    #   {a: true} | ~:b # SQL: ((a IS TRUE) OR NOT b)
     def |(ce)
       ::Sequel::SQL::BooleanExpression.new(:OR, self, ce)
     end
@@ -114,19 +114,17 @@ module Sequel::CoreRefinements
     # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this hash, not matching all of the
     # conditions.
     #
-    #   ~{:a=>true} # SQL: a IS NOT TRUE
-    #   ~{:a=>1, :b=>[2, 3]} # SQL: a != 1 OR b NOT IN (2, 3)
+    #   ~{a: true} # SQL: (a IS NOT TRUE)
+    #   ~{a: 1, b: [2, 3]} # SQL: ((a != 1) OR (b NOT IN (2, 3)))
     def ~
       ::Sequel::SQL::BooleanExpression.from_value_pairs(self, :OR, true)
     end
 
     # Return a <tt>Sequel::SQL::CaseExpression</tt> with this hash as the conditions and the given
-    # default value.  Note that the order of the conditions will be arbitrary on ruby 1.8, so all
-    # conditions should be orthogonal.
+    # default value.
     #
-    #   {{:a=>[2,3]}=>1}.case(0) # SQL: CASE WHEN a IN (2, 3) THEN 1 ELSE 0 END
-    #   {:a=>1, :b=>2}.case(:d, :c) # SQL: CASE c WHEN a THEN 1 WHEN b THEN 2 ELSE d END
-    #                                 #  or: CASE c WHEN b THEN 2 WHEN a THEN 1 ELSE d END
+    #   {{a: [2,3]}=>1}.case(0) # SQL: CASE WHEN (a IN (2, 3)) THEN 1 ELSE 0 END
+    #   {a: 1, b: 2}.case(:d, :c) # SQL: CASE c WHEN a THEN 1 WHEN b THEN 2 ELSE d END
     def case(*args)
       ::Sequel::SQL::CaseExpression.new(to_a, *args)
     end
@@ -135,8 +133,8 @@ module Sequel::CoreRefinements
     # conditions.  Rarely do you need to call this explicitly, as Sequel generally
     # assumes that hashes specify this type of condition.
     #
-    #   {:a=>true}.sql_expr # SQL: a IS TRUE
-    #   {:a=>1, :b=>[2, 3]}.sql_expr # SQL: a = 1 AND b IN (2, 3)
+    #   {a: true}.sql_expr # SQL: (a IS TRUE)
+    #   {a: 1, b: [2, 3]}.sql_expr # SQL: ((a = 1) AND (b IN (2, 3)))
     def sql_expr
       ::Sequel::SQL::BooleanExpression.from_value_pairs(self)
     end
@@ -144,8 +142,8 @@ module Sequel::CoreRefinements
     # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this hash, matching none
     # of the conditions.
     #
-    #   {:a=>true}.sql_negate # SQL: a IS NOT TRUE
-    #   {:a=>1, :b=>[2, 3]}.sql_negate # SQL: a != 1 AND b NOT IN (2, 3)
+    #   {a: true}.sql_negate # SQL: (a IS NOT TRUE)
+    #   {a: 1, b: [2, 3]}.sql_negate # SQL: ((a != 1) AND (b NOT IN (2, 3)))
     def sql_negate
       ::Sequel::SQL::BooleanExpression.from_value_pairs(self, :AND, true)
     end
@@ -153,8 +151,8 @@ module Sequel::CoreRefinements
     # Return a <tt>Sequel::SQL::BooleanExpression</tt> created from this hash, matching any of the
     # conditions.
     #
-    #   {:a=>true}.sql_or # SQL: a IS TRUE
-    #   {:a=>1, :b=>[2, 3]}.sql_or # SQL: a = 1 OR b IN (2, 3)
+    #   {a: true}.sql_or # SQL: (a IS TRUE)
+    #   {a: 1, b: [2, 3]}.sql_or # SQL: ((a = 1) OR (b IN (2, 3)))
     def sql_or
       ::Sequel::SQL::BooleanExpression.from_value_pairs(self, :OR)
     end
@@ -167,16 +165,16 @@ module Sequel::CoreRefinements
     # Converts a string into a <tt>Sequel::LiteralString</tt>, in order to override string
     # literalization, e.g.:
     #
-    #   DB[:items].where(:abc => 'def').sql #=>
-    #     "SELECT * FROM items WHERE (abc = 'def')"
+    #   DB[:items].where(abc: 'def')
+    #   # "SELECT * FROM items WHERE (abc = 'def')"
     #
-    #   DB[:items].where(:abc => 'def'.lit).sql #=>
-    #     "SELECT * FROM items WHERE (abc = def)"
+    #   DB[:items].where(abc: 'def'.lit)
+    #   # "SELECT * FROM items WHERE (abc = def)"
     #
     # You can also provide arguments, to create a <tt>Sequel::SQL::PlaceholderLiteralString</tt>:
     #
-    #    DB[:items].select{|o| o.count('DISTINCT ?'.lit(:a))}.sql #=>
-    #      "SELECT count(DISTINCT a) FROM items"
+    #   DB[:items].select{|o| o.count('DISTINCT ?'.lit(:a))}
+    #   # "SELECT count(DISTINCT a) FROM items"
     def lit(*args)
       args.empty? ? Sequel::LiteralString.new(self) : Sequel::SQL::PlaceholderLiteralString.new(self, args)
     end
@@ -199,19 +197,15 @@ module Sequel::CoreRefinements
     include Sequel::SQL::SubscriptMethods
     include Sequel::SQL::ComplexExpressionMethods
 
-    # Returns receiver wrapped in an <tt>Sequel::SQL::Identifier</tt>.  Usually used to
-    # prevent splitting the symbol.
+    # Returns receiver wrapped in an <tt>Sequel::SQL::Identifier</tt>.
     #
-    #   :a__b # SQL: "a"."b"
-    #   :a__b.identifier # SQL: "a__b"
+    #   :ab.identifier # SQL: "a"
     def identifier
       Sequel::SQL::Identifier.new(self)
     end
 
     # Returns a <tt>Sequel::SQL::Function</tt> with this as the function name,
-    # and the given arguments. This is aliased as <tt>Symbol#[]</tt> if the RUBY_VERSION
-    # is less than 1.9.0. Ruby 1.9 defines <tt>Symbol#[]</tt>, and Sequel
-    # doesn't override methods defined by ruby itself.
+    # and the given arguments.
     #
     #   :now.sql_function # SQL: now()
     #   :sum.sql_function(:a) # SQL: sum(a)