- Fixed bug in INSERT..FROM SELECT construct where selecting from a

UNION would wrap the union in an anonymous (e.g. unlabled) subquery.
fixes #3044

diff --git a/doc/build/changelog/changelog_08.rst b/doc/build/changelog/changelog_08.rst
index c21420b7d6410c4260db3a0ef658b87e8c713e9b..8b00ce1faa780305412ebf1fbd05d24da514b28f 100644 (file)
--- a/doc/build/changelog/changelog_08.rst
+++ b/doc/build/changelog/changelog_08.rst
@@ -11,6 +11,14 @@
 .. changelog::
     :version: 0.8.7
 
+    .. change::
+        :tags: bug, sql
+        :versions: 0.9.5, 1.0.0
+        :tickets: 3044
+
+        Fixed bug in INSERT..FROM SELECT construct where selecting from a
+        UNION would wrap the union in an anonymous (e.g. unlabled) subquery.
+
     .. change::
         :tags: bug, postgresql
         :versions: 0.9.5, 1.0.0

diff --git a/lib/sqlalchemy/sql/expression.py b/lib/sqlalchemy/sql/expression.py
index fadd8dde3cd99b028e61c5baea12fd8350201fc2..444f412e5f026c205680e22ac36a95e8907d346a 100644 (file)
--- a/lib/sqlalchemy/sql/expression.py
+++ b/lib/sqlalchemy/sql/expression.py
@@ -2108,7 +2108,7 @@ def _interpret_as_select(element):
     element = _interpret_as_from(element)
     if isinstance(element, Alias):
         element = element.original
-    if not isinstance(element, Select):
+    if not isinstance(element, SelectBase):
         element = element.select()
     return element
 

diff --git a/test/sql/test_insert.py b/test/sql/test_insert.py
index 8a5c7cd83b4fc165897951a1198070c4991083fc..d9f7b16292f7bfda724ffcacbfab2f18bee9a751 100644 (file)
--- a/test/sql/test_insert.py
+++ b/test/sql/test_insert.py
@@ -196,7 +196,25 @@ class InsertTest(_InsertTestBase, fixtures.TablesTest, AssertsCompiledSQL):
             checkparams={}
         )
 
-
+    def test_insert_from_select_union(self):
+        mytable = self.tables.mytable
+
+        name = 'name'
+        description = 'desc'
+        sel = select(
+            [name, mytable.c.description],
+        ).union(
+            select([name, description])
+        )
+        ins = mytable.insert().\
+                    from_select(
+                            [mytable.c.name, mytable.c.description], sel)
+        self.assert_compile(
+            ins,
+            "INSERT INTO mytable (name, description) "
+                    "SELECT name, mytable.description FROM mytable "
+                    "UNION SELECT name, desc"
+        )
     def test_insert_from_select_col_values(self):
         table1 = self.tables.mytable
         table2 = self.tables.myothertable