Skip to content

Commit

Permalink
json objects should be merged with deep merge
Browse files Browse the repository at this point in the history
See http://vertx.io/docs/apidocs/io/vertx/core/json/JsonObject.html#mergeIn-io.vertx.core.json.JsonObject-boolean- 
With parameter deep set to *true*, we have a real merge of the values. So it is possible to add different stores with e.g. json files and merge all the contents together.
  • Loading branch information
stundzig committed Jun 14, 2017
1 parent b6d8dce commit 1d33cf9
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ private void compute(Handler<AsyncResult<JsonObject>> completionHandler) {
} else {
// Merge the different futures
JsonObject json = new JsonObject();
futures.forEach(future -> json.mergeIn((JsonObject) future.result()));
futures.forEach(future -> json.mergeIn((JsonObject) future.result(), true));
try {
completionHandler.handle(Future.succeededFuture(json));
} catch (Throwable e) {
Expand Down

0 comments on commit 1d33cf9

Please sign in to comment.