Todo

jacquev6 · Apr 19, 2015 · 05730d1 · 05730d1
1 parent 547761a
commit 05730d1
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/LowVoltage/compounds/batch_delete_item.py b/LowVoltage/compounds/batch_delete_item.py
@@ -21,6 +21,8 @@ def BatchDeleteItem(connection, table, keys):
         if isinstance(r.unprocessed_items, dict) and table in r.unprocessed_items:
             unprocessed_items += r.unprocessed_items[table]
 
+    # @todo Maybe wait a bit before retrying unprocessed items? Same in BatchPutItem and BatchGetItemIterator.
+
     while len(unprocessed_items) != 0:
         r = connection.request(_lv.BatchWriteItem({table: unprocessed_items[:25]}))
         unprocessed_items = unprocessed_items[25:]

diff --git a/LowVoltage/compounds/wait_for_table_activation.py b/LowVoltage/compounds/wait_for_table_activation.py
@@ -13,7 +13,7 @@ def WaitForTableActivation(connection, table):
 
     r = connection.request(_lv.DescribeTable(table))
     while r.table.table_status != "ACTIVE":
-        time.sleep(3)
+        time.sleep(3)  # @todo Use a policy to choose polling interval? Same in WaitForTableDeletion.
         r = connection.request(_lv.DescribeTable(table))
 
 

diff --git a/LowVoltage/testing/dynamodb_local.py b/LowVoltage/testing/dynamodb_local.py
@@ -56,5 +56,3 @@ def setUpClass(cls):
     def assertDateTimeIsReasonable(self, t):
         self.assertGreaterEqual(t, self.before_start)
         self.assertLessEqual(t, self.after_end)
-
-# @todo class TestCaseWithTable to factorize setUp and tearDown
diff --git a/README.rst b/README.rst
@@ -10,10 +10,9 @@ Tenets
 Todo
 ====
 
+- write connected integ tests to reach 100% coverage
 - docs
 - credential provider for AWS's AIM roles
-- ease batch_get more than 100 items
-- ease batch_write more than 25 items
 - create builder for attribute paths
 - improve builder for expressions
 - metrics