Revert "enable multivalues insert (pandas-dev#19664)" (pandas-dev#21355)

This reverts commit 7c7bd56.

(cherry picked from commit c460710)

Author: jorisvandenbossche

doc/source/io.rst

@@ -4719,14 +4719,6 @@ writes ``data`` to the database in batches of 1000 rows at a time:
 
     data.to_sql('data_chunked', engine, chunksize=1000)
 
-.. note::
-
-    The function :func:`~pandas.DataFrame.to_sql` will perform a multivalue
-    insert if the engine dialect ``supports_multivalues_insert``. This will
-    greatly speed up the insert in some cases.
-
-SQL data types
-++++++++++++++
 
 :func:`~pandas.DataFrame.to_sql` will try to map your data to an appropriate
 SQL data type based on the dtype of the data. When you have columns of dtype

doc/source/whatsnew/v0.23.1.txt

@@ -10,19 +10,22 @@ and bug fixes. We recommend that all users upgrade to this version.
     :local:
     :backlinks: none
 
-.. _whatsnew_0231.enhancements:
-
-New features
-~~~~~~~~~~~~
-
-
-.. _whatsnew_0231.deprecations:
-
-Deprecations
-~~~~~~~~~~~~
-
--
--
+.. _whatsnew_0231.fixed_regressions:
+
+Fixed Regressions
+
+- Reverted the ability of :func:`~DataFrame.to_sql` to perform multivalue
+  inserts as this caused regression in certain cases (:issue:`21103`).
+  In the future this will be made configurable.
+- Fixed regression in the :attr:`DatetimeIndex.date` and :attr:`DatetimeIndex.time`
+  attributes in case of timezone-aware data: :attr:`DatetimeIndex.time` returned
+  a tz-aware time instead of tz-naive (:issue:`21267`) and :attr:`DatetimeIndex.date`
+  returned incorrect date when the input date has a non-UTC timezone (:issue:`21230`).
+- Fixed regression in :meth:`pandas.io.json.json_normalize` when called with ``None`` values
+  in nested levels in JSON (:issue:`21158`).
+- Bug in :meth:`~DataFrame.to_csv` causes encoding error when compression and encoding are specified (:issue:`21241`, :issue:`21118`)
+- Bug preventing pandas from being importable with -OO optimization (:issue:`21071`)
+- Bug in :meth:`Categorical.fillna` incorrectly raising a ``TypeError`` when `value` the individual categories are iterable and `value` is an iterable (:issue:`21097`, :issue:`19788`)
 
 .. _whatsnew_0231.performance:
 

pandas/io/sql.py

@@ -572,29 +572,8 @@ def create(self):
         else:
             self._execute_create()
 
-    def insert_statement(self, data, conn):
-        """
-        Generate tuple of SQLAlchemy insert statement and any arguments
-        to be executed by connection (via `_execute_insert`).
-
-        Parameters
-        ----------
-        conn : SQLAlchemy connectable(engine/connection)
-            Connection to recieve the data
-        data : list of dict
-            The data to be inserted
-
-        Returns
-        -------
-        SQLAlchemy statement
-            insert statement
-        *, optional
-            Additional parameters to be passed when executing insert statement
-        """
-        dialect = getattr(conn, 'dialect', None)
-        if dialect and getattr(dialect, 'supports_multivalues_insert', False):
-            return self.table.insert(data),
-        return self.table.insert(), data
+    def insert_statement(self):
+        return self.table.insert()
 
     def insert_data(self):
         if self.index is not None:
@@ -633,9 +612,8 @@ def insert_data(self):
         return column_names, data_list
 
     def _execute_insert(self, conn, keys, data_iter):
-        """Insert data into this table with database connection"""
         data = [{k: v for k, v in zip(keys, row)} for row in data_iter]
-        conn.execute(*self.insert_statement(data, conn))
+        conn.execute(self.insert_statement(), data)
 
     def insert(self, chunksize=None):
         keys, data_list = self.insert_data()

pandas/tests/io/test_sql.py

@@ -1665,29 +1665,6 @@ class Temporary(Base):
 
         tm.assert_frame_equal(df, expected)
 
-    def test_insert_multivalues(self):
-        # issues addressed
-        # https://github.com/pandas-dev/pandas/issues/14315
-        # https://github.com/pandas-dev/pandas/issues/8953
-
-        db = sql.SQLDatabase(self.conn)
-        df = DataFrame({'A': [1, 0, 0], 'B': [1.1, 0.2, 4.3]})
-        table = sql.SQLTable("test_table", db, frame=df)
-        data = [
-            {'A': 1, 'B': 0.46},
-            {'A': 0, 'B': -2.06}
-        ]
-        statement = table.insert_statement(data, conn=self.conn)[0]
-
-        if self.supports_multivalues_insert:
-            assert statement.parameters == data, (
-                'insert statement should be multivalues'
-            )
-        else:
-            assert statement.parameters is None, (
-                'insert statement should not be multivalues'
-            )
-
 
 class _TestSQLAlchemyConn(_EngineToConnMixin, _TestSQLAlchemy):
 
@@ -1702,7 +1679,6 @@ class _TestSQLiteAlchemy(object):
 
     """
     flavor = 'sqlite'
-    supports_multivalues_insert = True
 
     @classmethod
     def connect(cls):
@@ -1751,7 +1727,6 @@ class _TestMySQLAlchemy(object):
 
     """
     flavor = 'mysql'
-    supports_multivalues_insert = True
 
     @classmethod
     def connect(cls):
@@ -1821,7 +1796,6 @@ class _TestPostgreSQLAlchemy(object):
 
     """
     flavor = 'postgresql'
-    supports_multivalues_insert = True
 
     @classmethod
     def connect(cls):