Address review comments

Co-authored-by: Florentin Dörre <florentin.dorre@neotechnology.com>
Co-authored-by: Ioannis Panagiotas <ioannis.panagiotas@neotechnology.com>

Author: 3 people

algo/src/main/java/org/neo4j/gds/leiden/LeidenMutateConfig.java

@@ -20,13 +20,10 @@
 package org.neo4j.gds.leiden;
 
 import org.neo4j.gds.annotation.Configuration;
-import org.neo4j.gds.annotation.ValueClass;
 import org.neo4j.gds.config.MutatePropertyConfig;
 import org.neo4j.gds.core.CypherMapWrapper;
 
-@ValueClass
 @Configuration
-@SuppressWarnings("immutables:subtype")
 public interface LeidenMutateConfig extends LeidenBaseConfig, MutatePropertyConfig {
 
     static LeidenMutateConfig of(CypherMapWrapper userInput) {

algo/src/main/java/org/neo4j/gds/leiden/LeidenStatsConfig.java

@@ -21,12 +21,9 @@
 
 
 import org.neo4j.gds.annotation.Configuration;
-import org.neo4j.gds.annotation.ValueClass;
 import org.neo4j.gds.core.CypherMapWrapper;
 
-@ValueClass
 @Configuration
-@SuppressWarnings("immutables:subtype")
 public interface LeidenStatsConfig extends LeidenBaseConfig {
     static LeidenStatsConfig of(CypherMapWrapper userInput) {
         return new LeidenStatsConfigImpl(userInput);

algo/src/main/java/org/neo4j/gds/leiden/LeidenStreamConfig.java

@@ -21,12 +21,9 @@
 
 
 import org.neo4j.gds.annotation.Configuration;
-import org.neo4j.gds.annotation.ValueClass;
 import org.neo4j.gds.core.CypherMapWrapper;
 
-@ValueClass
 @Configuration
-@SuppressWarnings("immutables:subtype")
 public interface LeidenStreamConfig extends LeidenBaseConfig {
     static LeidenStreamConfig of(CypherMapWrapper userInput) {
         return new LeidenStreamConfigImpl(userInput);

proc/community/src/main/java/org/neo4j/gds/leiden/LeidenMutateSpec.java

@@ -77,9 +77,6 @@ private AbstractResultBuilder<MutateResult> resultBuilder(
             .withLevels(leidenResult.ranLevels())
             .withDidConverge(leidenResult.didConverge())
             .withCommunityFunction(leidenResult.communitiesFunction())
-            .withPreProcessingMillis(computationResult.preProcessingMillis())
-            .withComputeMillis(computationResult.computeMillis())
-            .withNodeCount(computationResult.graph().nodeCount())
             .withConfig(computationResult.config());
     }
 }