We should be killing the backup maps on make permanent.

Author: archaelus

src/logplex_shard.erl

@@ -123,11 +123,11 @@ handle_call({prepare, {new_shard_info, OldNewMap}}, _From, State) ->
 handle_call({make_permanent, new_shard_info}, _From, State) ->
     try
         [ stop_buffer(B)
-          || B <- logplex_shard_info:pid_list(?OLD_WRITE_MAP) ],
-        logplex_shard_info:delete(?OLD_WRITE_MAP),
+          || B <- logplex_shard_info:pid_list(?BACKUP_WRITE_MAP) ],
+        logplex_shard_info:delete(?BACKUP_WRITE_MAP),
         [ stop_pool(P)
-          || P <- logplex_shard_info:pid_list(?OLD_READ_MAP) ],
-        logplex_shard_info:delete(?OLD_READ_MAP),
+          || P <- logplex_shard_info:pid_list(?BACKUP_READ_MAP) ],
+        logplex_shard_info:delete(?BACKUP_READ_MAP),
         {reply, ok, State}
     catch
         C:E ->