Improve C comment about why we return "0 0" for some tags.

diff --git a/src/backend/tcop/pquery.c b/src/backend/tcop/pquery.c
index 05068fa406c9176a00a5bd20de5054b12082b3e1..42960b82250a65b2ae112e89728e6ef461e8bdae 100644 (file)
--- a/src/backend/tcop/pquery.c
+++ b/src/backend/tcop/pquery.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/tcop/pquery.c,v 1.134 2010/01/02 16:57:52 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/tcop/pquery.c,v 1.135 2010/02/13 22:45:41 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1318,9 +1318,14 @@ PortalRunMulti(Portal portal, bool isTopLevel,
         * If a command completion tag was supplied, use it.  Otherwise use the
         * portal's commandTag as the default completion tag.
         *
-        * Exception: clients will expect INSERT/UPDATE/DELETE tags to have
-        * counts, so fake something up if necessary.  (This could happen if the
-        * original query was replaced by a DO INSTEAD rule.)
+        * Exception: Clients expect INSERT/UPDATE/DELETE tags to have
+        * counts, so fake them with zeros.  This can happen with DO INSTEAD
+        * rules if there is no replacement query of the same type as the
+        * original.  We print "0 0" here because technically there is no
+        * query of the matching tag type, and printing a non-zero count for
+        * a different query type seems wrong, e.g.  an INSERT that does
+        * an UPDATE instead should not print "0 1" if one row
+        * was updated.  See QueryRewrite(), step 3, for details.
         */
        if (completionTag && completionTag[0] == '\0')
        {