You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by bh...@apache.org on 2014/01/07 23:37:40 UTC

[02/13] git commit: ACCUMULO-2138 Backport ACCUMULO-679 to 1.4.x

ACCUMULO-2138 Backport ACCUMULO-679 to 1.4.x

Original commit message: Fixed the [TabletShouldSplit] test to do a more coherent
check and to produce results more consistently that are accurate. WOrks on my machine,
but may need more love later

git-svn-id: https://svn.apache.org/repos/asf/accumulo/trunk@1359426 13f79535-47bb-0310-9956-ffa450edef68
(cherry picked from commit 06db9a008e1fafb821d57f8bd5c35a7065932030)


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/8f4516ea
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/8f4516ea
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/8f4516ea

Branch: refs/heads/master
Commit: 8f4516eac94e093d42de0a3b3c4ac3d2c1d1838b
Parents: d2a1fe7
Author: John Vines <vi...@apache.org>
Authored: Mon Jul 9 22:09:00 2012 +0000
Committer: Bill Havanki <bh...@cloudera.com>
Committed: Tue Jan 7 13:59:48 2014 -0500

----------------------------------------------------------------------
 test/system/auto/simple/split.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/8f4516ea/test/system/auto/simple/split.py
----------------------------------------------------------------------
diff --git a/test/system/auto/simple/split.py b/test/system/auto/simple/split.py
index aa4b033..abc214a 100755
--- a/test/system/auto/simple/split.py
+++ b/test/system/auto/simple/split.py
@@ -38,7 +38,8 @@ class TabletShouldSplit(SunnyDayTest):
         })
     tableSettings = SunnyDayTest.tableSettings.copy()
     tableSettings['test_ingest'] = { 
-    	'table.split.threshold': '5K',
+    	'table.split.threshold': '256K',
+      'table.file.compress.blocksize': '1K',
         }
     def runTest(self):
 
@@ -65,10 +66,10 @@ class TabletShouldSplit(SunnyDayTest):
                     lines.append(line)
         # check that the row values aren't always whole rows, but something shorter
         for line in lines:
-            if len(line) != len(lines[0]):
-                break
+          if len(line) != 14:
+            break
         else:
-            self.fail("The split points are not being shortened")
+          self.fail("The split points are not being shortened")
 
         self.assert_(len(lines) > 10)