SPLCommit::clearCaches

Author: pmbittner

src/main/java/de/variantsync/evolution/Main.java

@@ -107,12 +107,12 @@ public static void main(final String[] args) throws IOException, Resources.Resou
         final VariabilityDatasetLoader datasetLoader = new VariabilityDatasetLoader();
         assert datasetLoader.canLoad(variabilityDatasetDir);
         variabilityDataset = datasetLoader.load(variabilityDatasetDir).getSuccess();
-        final Set<CommitPair<SPLCommit>> commitPairs = variabilityDataset.getCommitPairsForEvolutionStudy();
+        final Set<SPLCommitPair> commitPairs = variabilityDataset.getCommitPairsForEvolutionStudy();
         Logger.info("The dataset contains " + variabilityDataset.getSuccessCommits().size() + " commits for which the variability extraction succeeded.");
         Logger.info("The dataset contains " + variabilityDataset.getErrorCommits().size() + " commits for which the variability extraction failed.");
         Logger.info("The dataset contains " + variabilityDataset.getPartialSuccessCommits().size() + " commits that for which the file presence conditions are missing.");
         Logger.info("The dataset contains " + commitPairs.size() + " usable pairs.");
-        for (final CommitPair<SPLCommit> pair : commitPairs) {
+        for (final SPLCommitPair pair : commitPairs) {
             Logger.debug("<<CHILD> " + pair.child().id() + "> -- <<PARENT> " + pair.parent().id() + ">");
             Logger.debug("<<CHILD> " + pair.child().id() + "> -- <<SPL_COMMIT> " + pair.child().id() + ">");
             Logger.debug("<<PARENT> " + pair.parent().id() + "> -- <<SPL_COMMIT> " + pair.parent().id() + ">");

src/main/java/de/variantsync/evolution/variability/CommitPair.java

@@ -115,6 +115,17 @@ private static Path tryUnzip(final Path path) throws ZipException {
         return path;
     }
 
+    /**
+     * Clears all cached values by calling {@link Lazy#forget()} on all fields.
+     */
+    public void clearCaches() {
+        kernelHavenLog.forget();
+        featureModel.forget();
+        presenceConditions.forget();
+        message.forget();
+        filterCounts.forget();
+    }
+
     /**
      * Return the parents of this commit. As the dataset only contains the data about non-error commits,
      * not all <code>SPLCommit</code> objects are associated with their parents. Therefore, an <code>Optional</code> is returned.

src/main/java/de/variantsync/evolution/variability/SPLCommit.java

@@ -0,0 +1,8 @@
+package de.variantsync.evolution.variability;
+
+public record SPLCommitPair(SPLCommit parent, SPLCommit child) {
+    void clearCaches() {
+        parent.clearCaches();
+        child.clearCaches();
+    }
+}

src/main/java/de/variantsync/evolution/variability/SPLCommitPair.java

@@ -96,9 +96,9 @@ public VariabilityHistory getVariabilityHistory(final Function<Collection<SPLCom
      *
      * @return Set of commit pairs that can be used in a variability evolution study
      */
-    public Set<CommitPair<SPLCommit>> getCommitPairsForEvolutionStudy() {
+    public Set<SPLCommitPair> getCommitPairsForEvolutionStudy() {
         Logger.info("Retrieving commit pairs for study...");
-        var set = successCommits.stream()
+        final var set = successCommits.stream()
                 .map(c -> {
                     if (c.parents().isPresent()) {
                         final SPLCommit[] parents = c.parents().get();
@@ -108,7 +108,7 @@ public Set<CommitPair<SPLCommit>> getCommitPairsForEvolutionStudy() {
                         // We only consider commits that processed an SPL commit whose parent was also processed
                         final boolean parentSuccess = successCommits.contains(p);
                         if (notAMerge && parentSuccess) {
-                            return new CommitPair<>(p, c);
+                            return new SPLCommitPair(p, c);
                         }
                     }
                     return null;