]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
added semicolon (;) to improve warning message clarity
authorjonathan vanasco <jonathan@2xlp.com>
Tue, 30 Jun 2020 16:31:58 +0000 (12:31 -0400)
committerMike Bayer <mike_mp@zzzcomputing.com>
Mon, 6 Jul 2020 20:48:34 +0000 (16:48 -0400)
### Description

Added a semicolon to improve the clarity of warning message.  I actually had a table named `backend`, and thought it was involved!

While updating the code, I noticed no test that directly tests for this warning message. There are tests for the `Can't sort tables for DROP;` prefix of this message and the `exc.CircularDependencyError`; and some tests for the `exc.CircularDependencyError` message itself. I couldn't find any test for this particular message though.  (Just thought I'd bring that up)

No issue created, because this is minor.

Closes: #5431
Pull-request: https://github.com/sqlalchemy/sqlalchemy/pull/5431
Pull-request-sha: 87fb5733ff4fc1a13dd94277716814ea852f654c

Change-Id: I87a504d30a7dd5155c34f7d7f30b2116d0d3cd3f
(cherry picked from commit b421c51b94fdef2bc542b0a40bfb21490b23ca15)

lib/sqlalchemy/sql/ddl.py

index d7753223117f53f2c03b42c18e9100c3e9932bc1..3b16394dfd35895cc8cf89a981ed7a397450451b 100644 (file)
@@ -903,7 +903,7 @@ class SchemaDropper(DDLBase):
                 util.warn(
                     "Can't sort tables for DROP; an "
                     "unresolvable foreign key "
-                    "dependency exists between tables: %s, and backend does "
+                    "dependency exists between tables: %s; and backend does "
                     "not support ALTER.  To restore at least a partial sort, "
                     "apply use_alter=True to ForeignKey and "
                     "ForeignKeyConstraint "