commit f99e30c38997e33538493204d5d58ea1a0f62de8
author Robert Newson <rnewson@apache.org> Mon Mar 06 14:06:09 2017 +0000
committer Robert Newson <rnewson@apache.org> Mon Mar 06 14:06:09 2017 +0000
tree 649795e5311a06189a9ff05f839be020295c2d51
parent 63ef337475a266379c7208f52f827980e69d1d1b
parent f706bb87be71006875c2fbaed3a14a18c79396f8

Merge remote-tracking branch 'cloudant/3318-bypass-vhosts'

src/couch_btree.erl

diff --git a/src/couch_btree.erl b/src/couch_btree.erl
index e9ff7f5..adbc92b 100644
--- a/src/couch_btree.erl
+++ b/src/couch_btree.erl
@@ -342,11 +342,9 @@
 % it's probably really inefficient.
 
 chunkify(InList) ->
-    BaseChunkSize = get_chunk_size(),
+    ChunkThreshold = get_chunk_size(),
     case ?term_size(InList) of
-    Size when Size > BaseChunkSize ->
-        NumberOfChunksLikely = ((Size div BaseChunkSize) + 1),
-        ChunkThreshold = Size div NumberOfChunksLikely,
+    Size when Size > ChunkThreshold ->
         chunkify(InList, ChunkThreshold, [], 0, []);
     _Else ->
         [InList]
@@ -354,6 +352,9 @@
 
 chunkify([], _ChunkThreshold, [], 0, OutputChunks) ->
     lists:reverse(OutputChunks);
+chunkify([], _ChunkThreshold, [Item], _OutListSize, [PrevChunk | RestChunks]) ->
+    NewPrevChunk = PrevChunk ++ [Item],
+    lists:reverse(RestChunks, [NewPrevChunk]);
 chunkify([], _ChunkThreshold, OutList, _OutListSize, OutputChunks) ->
     lists:reverse([lists:reverse(OutList) | OutputChunks]);
 chunkify([InElement | RestInList], ChunkThreshold, OutList, OutListSize, OutputChunks) ->

src/couch_db.erl

diff --git a/src/couch_db.erl b/src/couch_db.erl
index 8005e6d..d01a3e0 100644
--- a/src/couch_db.erl
+++ b/src/couch_db.erl
@@ -391,7 +391,7 @@
         case couch_btree:size(T) of
             _ when Acc == null ->
                 null;
-            undefined ->
+            nil ->
                 null;
             Size ->
                 Acc + Size