Skip to content
This repository has been archived by the owner on Nov 11, 2020. It is now read-only.

Commit

Permalink
Merge pull request #95 from conveyal/fix-modifications-checksum
Browse files Browse the repository at this point in the history
refactor(comment): clarify modifications checksum
  • Loading branch information
ansoncfit authored Dec 11, 2017
2 parents 65b4b19 + f2803c1 commit 3ca245f
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/main/java/com/conveyal/taui/models/AnalysisRequest.java
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,9 @@ public AnalysisTask populateTask (AnalysisTask task, Project project) {
modifications = modificationsForProject(project.accessGroup, projectId, variantIndex);
}

// Generate a checksum of the modifications to uniquely identify this dataset for the broker and the workers.
// The CRC is appended to the scenario ID to identify a unique revision of the scenario (still denoted here
// as variant) allowing the worker to cache and reuse networks built by applying that exact revision of the
// scenario to a base network.
CRC32 crc = new CRC32();
crc.update(JsonUtilities.objectToJsonBytes(modifications));
long crcValue = crc.getValue();
Expand Down

0 comments on commit 3ca245f

Please sign in to comment.