106 - avoid wraparounds when using ThriftUtils.RANGE_ALL

NewBay · Sep 19, 2012 · f19be8b · f19be8b
1 parent 48a764f
commit f19be8b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/build.gradle b/build.gradle
@@ -1,3 +1,4 @@
+apply plugin: 'idea'
 // Establish version and status
 ext.githubProjectName = rootProject.name // Change if github project name is not the same as the root project's name
 

diff --git a/src/main/java/com/netflix/astyanax/thrift/ThriftColumnFamilyQueryImpl.java b/src/main/java/com/netflix/astyanax/thrift/ThriftColumnFamilyQueryImpl.java
@@ -219,7 +219,10 @@ public ColumnList<C> internalExecute(Client client) throws Exception {
                                     // that will later be dropped
                                     if (firstPage) {
                                         firstPage = false;
-                                        predicate.getSlice_range().setCount(predicate.getSlice_range().getCount() + 1);
+                                        // 106 - avoid wraparounds when using ThriftUtils.RANGE_ALL
+                                        if(! (predicate.getSlice_range().getCount() == Integer.MAX_VALUE))  {
+                                            predicate.getSlice_range().setCount(predicate.getSlice_range().getCount() + 1);
+                                        }
                                     }
                                     else {
                                         if (!columnList.isEmpty())