[CalendarServer-changes] [9608] CalendarServer/branches/users/glyph/q

[source_changes at macosforge.org]

Revision: 9608
          http://trac.macosforge.org/projects/calendarserver/changeset/9608
Author:   glyph at apple.com
Date:     2012-08-11 01:54:48 -0700 (Sat, 11 Aug 2012)
Log Message:
-----------
Actual docstring for the test.

Modified Paths:
--------------
    CalendarServer/branches/users/glyph/q/twext/enterprise/dal/test/test_sqlsyntax.py

Property Changed:
----------------
    CalendarServer/branches/users/glyph/q/

Modified: CalendarServer/branches/users/glyph/q/twext/enterprise/dal/test/test_sqlsyntax.py
===================================================================
--- CalendarServer/branches/users/glyph/q/twext/enterprise/dal/test/test_sqlsyntax.py	2012-08-11 08:54:47 UTC (rev 9607)
+++ CalendarServer/branches/users/glyph/q/twext/enterprise/dal/test/test_sqlsyntax.py	2012-08-11 08:54:48 UTC (rev 9608)
@@ -1025,7 +1025,13 @@
 
     def test_updateReturningSQLite(self):
         """
-        SQLite updates all its rows.
+        Since SQLite does not support the SQL 'returning' syntax extension, in
+        order to preserve the rows that will be modified during an UPDATE
+        statement, we must first find the rows that will be affected, then
+        update them, then return the rows that were affected.  Since we might
+        be changing even part of the primary key, we use the internal 'rowid'
+        column to uniquely and reliably identify rows in the sqlite database
+        that have been modified.
         """
         csql = CatchSQL()
         stmt = Update({self.schema.FOO.BAR: 4321},
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20120811/9cd2aede/attachment-0001.html>