TritonDataCenter#46 --dry-run does unnecessary work

Author: bcantrill

bin/statemap

@@ -795,6 +795,9 @@ var main = function ()
 		config.tags = [];
 	}
 
+	if (opts.dry_run)
+		config.dryrun = 1;
+
 	file = opts._args[0];
 
 	stream = mod_fs.createReadStream(file).pipe(mod_concatjson.parse());
@@ -810,6 +813,9 @@ var main = function ()
 		obj.data = [];
 
 		config.coalesced = mod_statemap.ingest(file, function (datum) {
+			if (opts.dry_run)
+				return;
+
 			if (!datum.hasOwnProperty('entity')) {
 				addTag(config, datum);
 				return;

src/glue.cc

@@ -141,6 +141,7 @@ loadConfig(Isolate *isolate, statemap_config_t *config, Local<Object> obj)
 	LOADCONFIG_INTFIELD(begin);
 	LOADCONFIG_INTFIELD(end);
 	LOADCONFIG_INTFIELD(notags);
+	LOADCONFIG_INTFIELD(dryrun);
 
 	return (0);
 }
@@ -202,12 +203,14 @@ ingest(const FunctionCallbackInfo<Value>& args)
 		return;
 	}
 
-	ingestEmitTags(args, statemap);
+	if (!config.smc_dryrun) {
+		ingestEmitTags(args, statemap);
 
-	for (entity = statemap->sm_entities; entity != NULL;
-	    entity = entity->sme_next) {
-		if (ingestEmitEntity(args, statemap, entity) != 0)
-			break;
+		for (entity = statemap->sm_entities; entity != NULL;
+		    entity = entity->sme_next) {
+			if (ingestEmitEntity(args, statemap, entity) != 0)
+				break;
+		}
 	}
 
 	args.GetReturnValue().Set(v8::Number::New(isolate,