Merge pull request #9702 from lassewesth/rena

renaming side effect timing
neo4j · Oct 8, 2024 · 6b09941 · 6b09941
2 parents f8dfddb + d44c23c
commit 6b09941
Show file tree

Hide file tree

Showing 120 changed files with 129 additions and 129 deletions.
diff --git a/...main/java/org/neo4j/gds/applications/algorithms/machinery/AlgorithmProcessingTimings.java b/...main/java/org/neo4j/gds/applications/algorithms/machinery/AlgorithmProcessingTimings.java
@@ -25,11 +25,11 @@
 public class AlgorithmProcessingTimings {
     public final long preProcessingMillis;
     public final long computeMillis;
-    public final long mutateOrWriteMillis;
+    public final long sideEffectMillis; // rename
 
-    AlgorithmProcessingTimings(long preProcessingMillis, long computeMillis, long mutateOrWriteMillis) {
+    AlgorithmProcessingTimings(long preProcessingMillis, long computeMillis, long sideEffectMillis) {
         this.preProcessingMillis = preProcessingMillis;
         this.computeMillis = computeMillis;
-        this.mutateOrWriteMillis = mutateOrWriteMillis;
+        this.sideEffectMillis = sideEffectMillis;
     }
 }
diff --git a/...va/org/neo4j/gds/applications/algorithms/machinery/AlgorithmProcessingTimingsBuilder.java b/...va/org/neo4j/gds/applications/algorithms/machinery/AlgorithmProcessingTimingsBuilder.java
@@ -29,7 +29,7 @@ class AlgorithmProcessingTimingsBuilder {
     // timings
     protected long preProcessingMillis = NOT_AVAILABLE;
     protected long computeMillis = NOT_AVAILABLE;
-    protected long mutateOrWriteMillis = NOT_AVAILABLE;
+    protected long sideEffectMillis = NOT_AVAILABLE; // rename
 
     public void withPreProcessingMillis(long preProcessingMillis) {
         this.preProcessingMillis = preProcessingMillis;
@@ -39,15 +39,15 @@ public void withComputeMillis(long computeMillis) {
         this.computeMillis = computeMillis;
     }
 
-    public void withMutateOrWriteMillis(long mutateOrWriteMillis) {
-        this.mutateOrWriteMillis = mutateOrWriteMillis;
+    public void withSideEffectMillis(long sideEffectMillis) {
+        this.sideEffectMillis = sideEffectMillis;
     }
 
     AlgorithmProcessingTimings build() {
         return new AlgorithmProcessingTimings(
             preProcessingMillis,
             computeMillis,
-            mutateOrWriteMillis
+            sideEffectMillis
         );
     }
 }
diff --git a/...a/org/neo4j/gds/applications/algorithms/machinery/DefaultAlgorithmProcessingTemplate.java b/...a/org/neo4j/gds/applications/algorithms/machinery/DefaultAlgorithmProcessingTemplate.java
@@ -293,7 +293,7 @@ private <RESULT_FROM_ALGORITHM, METADATA> Optional<METADATA> processSideEffect(
     ) {
         if (sideEffect.isEmpty()) return Optional.empty();
 
-        try (var ignored = ProgressTimer.start(timingsBuilder::withMutateOrWriteMillis)) {
+        try (var ignored = ProgressTimer.start(timingsBuilder::withSideEffectMillis)) { // rename
             return sideEffect.get().process(graphResources, result);
         }
     }

diff --git a/...g/neo4j/gds/applications/algorithms/machinery/DefaultAlgorithmProcessingTemplateTest.java b/...g/neo4j/gds/applications/algorithms/machinery/DefaultAlgorithmProcessingTemplateTest.java
@@ -78,7 +78,7 @@ void shouldDoFourStepProcess() {
                 assertThat(result).hasValue("some result");
                 assertThat(timings.preProcessingMillis).isGreaterThan(-1);
                 assertThat(timings.computeMillis).isGreaterThan(-1);
-                assertThat(timings.mutateOrWriteMillis).isGreaterThan(-1);
+                assertThat(timings.sideEffectMillis).isGreaterThan(-1);
                 assertThat(metadata).hasValue("metadata from some side effect");
 
                 return "some rendered result";
@@ -129,7 +129,7 @@ void shouldSkipSideEffect() {
                 assertThat(result).hasValue("some other result");
                 assertThat(timings.preProcessingMillis).isGreaterThan(-1);
                 assertThat(timings.computeMillis).isGreaterThan(-1);
-                assertThat(timings.mutateOrWriteMillis).isEqualTo(-1); // no side effect, no timing
+                assertThat(timings.sideEffectMillis).isEqualTo(-1); // no side effect, no timing
                 assertThat(metadata).isEmpty();
 
                 return "some other rendered result";

diff --git a/...ds/procedures/algorithms/centrality/AlphaHarmonicCentralityResultBuilderForWriteMode.java b/...ds/procedures/algorithms/centrality/AlphaHarmonicCentralityResultBuilderForWriteMode.java
@@ -59,7 +59,7 @@ public Stream<AlphaHarmonicWriteResult> build(
             graph.nodeCount(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             configuration.writeProperty(),
             centralityDistributionAndTiming.getLeft()
         );

diff --git a/...o4j/gds/procedures/algorithms/centrality/ArticulationPointsResultBuilderForWriteMode.java b/...o4j/gds/procedures/algorithms/centrality/ArticulationPointsResultBuilderForWriteMode.java
@@ -48,7 +48,7 @@ public Stream<ArticulationPointsWriteResult> build(
             new ArticulationPointsWriteResult(
                 bitSet.cardinality(),
                 metadata.map(NodePropertiesWritten::value).orElseThrow(),
-                timings.mutateOrWriteMillis,
+                timings.sideEffectMillis,
                 timings.computeMillis,
                 configuration.toMap()
             )

diff --git a/...in/java/org/neo4j/gds/procedures/algorithms/centrality/CelfResultBuilderForWriteMode.java b/...in/java/org/neo4j/gds/procedures/algorithms/centrality/CelfResultBuilderForWriteMode.java
@@ -47,7 +47,7 @@ public Stream<CELFWriteResult> build(
 
         return Stream.of(
             new CELFWriteResult(
-                timings.mutateOrWriteMillis,
+                timings.sideEffectMillis,
                 nodePropertiesWritten.orElseThrow().value(),
                 timings.computeMillis,
                 celfResult.totalSpread(),

diff --git a/...eo4j/gds/procedures/algorithms/centrality/GenericCentralityResultBuilderForWriteMode.java b/...eo4j/gds/procedures/algorithms/centrality/GenericCentralityResultBuilderForWriteMode.java
@@ -55,7 +55,7 @@ <CONFIGURATION extends ConcurrencyConfig & ToMapConvertible> CentralityWriteResu
             timings.preProcessingMillis,
             timings.computeMillis,
             centralityDistributionAndTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             centralityDistributionAndTiming.getLeft(),
             configurationMap
         );

diff --git a/...ava/org/neo4j/gds/procedures/algorithms/centrality/PageRankResultBuilderForWriteMode.java b/...ava/org/neo4j/gds/procedures/algorithms/centrality/PageRankResultBuilderForWriteMode.java
@@ -65,7 +65,7 @@ public Stream<PageRankWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             pageRankDistribution.postProcessingMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configurationMap
         );

diff --git a/.../procedures/algorithms/centrality/stubs/ArticulationPointsResultBuilderForMutateMode.java b/.../procedures/algorithms/centrality/stubs/ArticulationPointsResultBuilderForMutateMode.java
@@ -46,7 +46,7 @@ public ArticulationPointsMutateResult build(
         return new ArticulationPointsMutateResult(
                 bitSet.cardinality(),
                 metadata.map(NodePropertiesWritten::value).orElseThrow(),
-                timings.mutateOrWriteMillis,
+                timings.sideEffectMillis,
                 timings.computeMillis,
                 configuration.toMap()
         );

diff --git a/.../org/neo4j/gds/procedures/algorithms/centrality/stubs/CelfResultBuilderForMutateMode.java b/.../org/neo4j/gds/procedures/algorithms/centrality/stubs/CelfResultBuilderForMutateMode.java
@@ -47,7 +47,7 @@ public CELFMutateResult build(
             .withNodeCount(graph.nodeCount())
             .withPreProcessingMillis(timings.preProcessingMillis)
             .withComputeMillis(timings.computeMillis)
-            .withMutateMillis(timings.mutateOrWriteMillis)
+            .withMutateMillis(timings.sideEffectMillis)
             .withNodePropertiesWritten(metadata.orElseThrow().value())
             .withConfig(configuration)
             .build();

diff --git a/...s/procedures/algorithms/centrality/stubs/GenericCentralityResultBuilderForMutateMode.java b/...s/procedures/algorithms/centrality/stubs/GenericCentralityResultBuilderForMutateMode.java
@@ -59,7 +59,7 @@ <CONFIGURATION extends ConcurrencyConfig & ToMapConvertible> CentralityMutateRes
             timings.preProcessingMillis,
             timings.computeMillis,
             centralityDistributionAndTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             centralityDistributionAndTiming.getLeft(),
             configurationMap
         );

diff --git a/...ds/procedures/algorithms/centrality/stubs/IndirectExposureResultBuilderForMutateMode.java b/...ds/procedures/algorithms/centrality/stubs/IndirectExposureResultBuilderForMutateMode.java
@@ -50,7 +50,7 @@ public IndirectExposureMutateResult build(
             indirectExposureResult.didConverge(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/.../neo4j/gds/procedures/algorithms/centrality/stubs/PageRankResultBuilderForMutateMode.java b/.../neo4j/gds/procedures/algorithms/centrality/stubs/PageRankResultBuilderForMutateMode.java
@@ -65,7 +65,7 @@ public PageRankMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             pageRankDistribution.postProcessingMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configurationMap
         );

diff --git a/...va/org/neo4j/gds/procedures/algorithms/community/K1ColoringResultBuilderForWriteMode.java b/...va/org/neo4j/gds/procedures/algorithms/community/K1ColoringResultBuilderForWriteMode.java
@@ -50,7 +50,7 @@ public Stream<K1ColoringWriteResult> build(
         var k1ColoringWriteResult = new K1ColoringWriteResult(
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             k1ColoringResult.colors().size(),
             usedColors,
             k1ColoringResult.ranIterations(),

diff --git a/...in/java/org/neo4j/gds/procedures/algorithms/community/KCoreResultBuilderForWriteMode.java b/...in/java/org/neo4j/gds/procedures/algorithms/community/KCoreResultBuilderForWriteMode.java
@@ -48,7 +48,7 @@ public Stream<KCoreDecompositionWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             0,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             configuration.toMap()
         );
 

diff --git a/...n/java/org/neo4j/gds/procedures/algorithms/community/KMeansResultBuilderForWriteMode.java b/...n/java/org/neo4j/gds/procedures/algorithms/community/KMeansResultBuilderForWriteMode.java
@@ -75,7 +75,7 @@ public Stream<KmeansWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatistics.computeMilliseconds(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             communitySummary,
             centroids,

diff --git a/.../neo4j/gds/procedures/algorithms/community/LabelPropagationResultBuilderForWriteMode.java b/.../neo4j/gds/procedures/algorithms/community/LabelPropagationResultBuilderForWriteMode.java
@@ -73,7 +73,7 @@ public Stream<LabelPropagationWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatistics.computeMilliseconds(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             nodePropertiesWrittenAndConvertedNodePropertyValues.getLeft().value(),
             configuration.toMap()
         );

diff --git a/...main/java/org/neo4j/gds/procedures/algorithms/community/LccResultBuilderForWriteMode.java b/...main/java/org/neo4j/gds/procedures/algorithms/community/LccResultBuilderForWriteMode.java
@@ -50,7 +50,7 @@ public Stream<LocalClusteringCoefficientWriteResult> build(
             localClusteringCoefficientResult.localClusteringCoefficients().size(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...n/java/org/neo4j/gds/procedures/algorithms/community/LeidenResultBuilderForWriteMode.java b/...n/java/org/neo4j/gds/procedures/algorithms/community/LeidenResultBuilderForWriteMode.java
@@ -75,7 +75,7 @@ public Stream<LeidenWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatistics.computeMilliseconds(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             nodePropertiesWrittenAndConvertedNodePropertyValues.getLeft().value(),
             communitySummary,
             Arrays.stream(leidenResult.modularities()).boxed().collect(Collectors.toList()),

diff --git a/.../java/org/neo4j/gds/procedures/algorithms/community/LouvainResultBuilderForWriteMode.java b/.../java/org/neo4j/gds/procedures/algorithms/community/LouvainResultBuilderForWriteMode.java
@@ -75,7 +75,7 @@ public Stream<LouvainWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatistics.computeMilliseconds(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/.../gds/procedures/algorithms/community/ModularityOptimizationResultBuilderForWriteMode.java b/.../gds/procedures/algorithms/community/ModularityOptimizationResultBuilderForWriteMode.java
@@ -68,7 +68,7 @@ public Stream<ModularityOptimizationWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatistics.computeMilliseconds(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             modularityOptimizationResult.asNodeProperties().nodeCount(),
             modularityOptimizationResult.didConverge(),
             modularityOptimizationResult.ranIterations(),

diff --git a/...java/org/neo4j/gds/procedures/algorithms/community/SccAlphaResultBuilderForWriteMode.java b/...java/org/neo4j/gds/procedures/algorithms/community/SccAlphaResultBuilderForWriteMode.java
@@ -67,7 +67,7 @@ public Stream<AlphaSccWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             hugeLongArray.size(),
             communityStatisticsWithTiming.getLeft(),
             (long) communityDistribution.get("max"),

diff --git a/...main/java/org/neo4j/gds/procedures/algorithms/community/SccResultBuilderForWriteMode.java b/...main/java/org/neo4j/gds/procedures/algorithms/community/SccResultBuilderForWriteMode.java
@@ -73,7 +73,7 @@ public Stream<SccWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...org/neo4j/gds/procedures/algorithms/community/TriangleCountResultBuilderForWriteMode.java b/...org/neo4j/gds/procedures/algorithms/community/TriangleCountResultBuilderForWriteMode.java
@@ -47,7 +47,7 @@ public Stream<TriangleCountWriteResult> build(
             graph.nodeCount(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...main/java/org/neo4j/gds/procedures/algorithms/community/WccResultBuilderForWriteMode.java b/...main/java/org/neo4j/gds/procedures/algorithms/community/WccResultBuilderForWriteMode.java
@@ -71,7 +71,7 @@ public Stream<WccWriteResult> build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatistics.computeMilliseconds(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             nodePropertiesWrittenAndConvertedNodePropertyValues.getLeft().value(),
             configuration.toMap()
         );

diff --git a/...4j/gds/procedures/algorithms/community/stubs/ApproxMaxKCutResultBuilderForMutateMode.java b/...4j/gds/procedures/algorithms/community/stubs/ApproxMaxKCutResultBuilderForMutateMode.java
@@ -50,7 +50,7 @@ public ApproxMaxKCutMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             0,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             configurationMap
         );
     }

diff --git a/...neo4j/gds/procedures/algorithms/community/stubs/K1ColoringResultBuilderForMutateMode.java b/...neo4j/gds/procedures/algorithms/community/stubs/K1ColoringResultBuilderForMutateMode.java
@@ -52,7 +52,7 @@ public K1ColoringMutateResult build(
         return new K1ColoringMutateResult(
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             k1ColoringResult.colors().size(),
             usedColors,
             k1ColoringResult.ranIterations(),

diff --git a/.../org/neo4j/gds/procedures/algorithms/community/stubs/KCoreResultBuilderForMutateMode.java b/.../org/neo4j/gds/procedures/algorithms/community/stubs/KCoreResultBuilderForMutateMode.java
@@ -48,7 +48,7 @@ public KCoreDecompositionMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             0,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             configuration.toMap()
         );
     }

diff --git a/...org/neo4j/gds/procedures/algorithms/community/stubs/KMeansResultBuilderForMutateMode.java b/...org/neo4j/gds/procedures/algorithms/community/stubs/KMeansResultBuilderForMutateMode.java
@@ -74,7 +74,7 @@ public KmeansMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             communityStatisticsWithTiming.getMiddle(),
             centroids,

diff --git a/...gds/procedures/algorithms/community/stubs/LabelPropagationResultBuilderForMutateMode.java b/...gds/procedures/algorithms/community/stubs/LabelPropagationResultBuilderForMutateMode.java
@@ -69,7 +69,7 @@ public LabelPropagationMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...va/org/neo4j/gds/procedures/algorithms/community/stubs/LccResultBuilderForMutateMode.java b/...va/org/neo4j/gds/procedures/algorithms/community/stubs/LccResultBuilderForMutateMode.java
@@ -47,7 +47,7 @@ public LocalClusteringCoefficientMutateResult build(
             localClusteringCoefficientResult.localClusteringCoefficients().size(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...org/neo4j/gds/procedures/algorithms/community/stubs/LeidenResultBuilderForMutateMode.java b/...org/neo4j/gds/procedures/algorithms/community/stubs/LeidenResultBuilderForMutateMode.java
@@ -72,7 +72,7 @@ public LeidenMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             nodePropertiesWrittenAndConvertedNodePropertyValues.getLeft().value(),
             communityStatisticsWithTiming.getMiddle(),
             Arrays.stream(leidenResult.modularities()).boxed().collect(Collectors.toList()),

diff --git a/...rg/neo4j/gds/procedures/algorithms/community/stubs/LouvainResultBuilderForMutateMode.java b/...rg/neo4j/gds/procedures/algorithms/community/stubs/LouvainResultBuilderForMutateMode.java
@@ -70,7 +70,7 @@ public LouvainMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...ocedures/algorithms/community/stubs/ModularityOptimizationResultBuilderForMutateMode.java b/...ocedures/algorithms/community/stubs/ModularityOptimizationResultBuilderForMutateMode.java
@@ -65,7 +65,7 @@ public ModularityOptimizationMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             modularityOptimizationResult.asNodeProperties().nodeCount(),
             modularityOptimizationResult.didConverge(),
             modularityOptimizationResult.ranIterations(),

diff --git a/...va/org/neo4j/gds/procedures/algorithms/community/stubs/SccResultBuilderForMutateMode.java b/...va/org/neo4j/gds/procedures/algorithms/community/stubs/SccResultBuilderForMutateMode.java
@@ -72,7 +72,7 @@ public SccMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...4j/gds/procedures/algorithms/community/stubs/TriangleCountResultBuilderForMutateMode.java b/...4j/gds/procedures/algorithms/community/stubs/TriangleCountResultBuilderForMutateMode.java
@@ -47,7 +47,7 @@ public TriangleCountMutateResult build(
             graph.nodeCount(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/...va/org/neo4j/gds/procedures/algorithms/community/stubs/WccResultBuilderForMutateMode.java b/...va/org/neo4j/gds/procedures/algorithms/community/stubs/WccResultBuilderForMutateMode.java
@@ -65,7 +65,7 @@ public WccMutateResult build(
             timings.preProcessingMillis,
             timings.computeMillis,
             communityStatisticsWithTiming.getRight(),
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             metadata.orElseThrow().value(),
             configuration.toMap()
         );

diff --git a/.../java/org/neo4j/gds/procedures/algorithms/embeddings/FastRPResultBuilderForWriteMode.java b/.../java/org/neo4j/gds/procedures/algorithms/embeddings/FastRPResultBuilderForWriteMode.java
@@ -48,7 +48,7 @@ public Stream<DefaultNodeEmbeddingsWriteResult> build(
             nodePropertiesWritten.orElseThrow().value(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             configuration.toMap()
         );
 

diff --git a/...va/org/neo4j/gds/procedures/algorithms/embeddings/GraphSageResultBuilderForWriteMode.java b/...va/org/neo4j/gds/procedures/algorithms/embeddings/GraphSageResultBuilderForWriteMode.java
@@ -48,7 +48,7 @@ public Stream<DefaultNodeEmbeddingsWriteResult> build(
             nodePropertiesWritten.orElseThrow().value(),
             timings.preProcessingMillis,
             timings.computeMillis,
-            timings.mutateOrWriteMillis,
+            timings.sideEffectMillis,
             configuration.toMap()
         );