Clean up GDLFactory

Co-authored-by: Sören Reichardt <soren.reichardt@neotechnology.com>

Author: s1ck

test-utils/src/main/java/org/neo4j/gds/gdl/GdlFactory.java

@@ -22,7 +22,6 @@
 import org.immutables.builder.Builder;
 import org.jetbrains.annotations.NotNull;
 import org.neo4j.gds.NodeLabel;
-import org.neo4j.gds.PropertyMapping;
 import org.neo4j.gds.RelationshipType;
 import org.neo4j.gds.api.CSRGraphStoreFactory;
 import org.neo4j.gds.api.DatabaseId;
@@ -31,7 +30,6 @@
 import org.neo4j.gds.api.IdMap;
 import org.neo4j.gds.api.PropertyState;
 import org.neo4j.gds.api.nodeproperties.ValueType;
-import org.neo4j.gds.api.properties.nodes.NodePropertyValues;
 import org.neo4j.gds.api.schema.Direction;
 import org.neo4j.gds.api.schema.GraphSchema;
 import org.neo4j.gds.api.schema.RelationshipSchema;
@@ -49,7 +47,6 @@
 import org.neo4j.gds.core.loading.construction.NodeLabelTokens;
 import org.neo4j.gds.core.loading.construction.PropertyValues;
 import org.neo4j.gds.core.loading.construction.RelationshipsBuilder;
-import org.neo4j.gds.core.loading.nodeproperties.NodePropertiesFromStoreBuilder;
 import org.neo4j.gds.core.utils.mem.MemoryEstimation;
 import org.neo4j.gds.core.utils.mem.MemoryEstimations;
 import org.neo4j.gds.core.utils.progress.tasks.ProgressTracker;
@@ -165,34 +162,6 @@ protected ProgressTracker initProgressTracker() {
 
     @Override
     protected GraphSchema computeGraphSchema(Nodes nodes, RelationshipImportResult relationshipImportResult) {
-//        var nodeProperties = nodes.properties();
-//        var nodeSchema = NodeSchema.empty();
-//        gdlHandler
-//            .getVertices()
-//            .forEach(vertex -> {
-//                var labels = vertex.getLabels().stream().map(NodeLabel::of).collect(Collectors.toList());
-//                if (labels.isEmpty()) {
-//                    labels = List.of(NodeLabel.ALL_NODES);
-//                }
-//
-//                labels.forEach(label -> vertex
-//                    .getProperties()
-//                    .forEach((propertyKey, propertyValue) -> nodeSchema
-//                        .getOrCreateLabel(label)
-//                        .addProperty(
-//                            propertyKey,
-//                            PropertySchema.of(
-//                                propertyKey,
-//                                nodeProperties.get(propertyKey).valueType(),
-//                                nodeProperties.get(propertyKey).defaultValue(),
-//                                nodeProperties.get(propertyKey).propertyState()
-//                            )
-//                    )
-//                ));
-//            });
-//        // in case there were no properties add all labels
-//        nodes.idMap().availableNodeLabels().forEach(nodeSchema::getOrCreateLabel);
-
         var relationshipSchema = relationshipImportResult.importResults().entrySet().stream().reduce(
             RelationshipSchema.empty(),
             (unionSchema, entry) -> {
@@ -259,31 +228,7 @@ private Nodes loadNodes() {
             );
         });
 
-        var nodes = nodesBuilder.build();
-        return Nodes.of(nodes.schema(), nodes.idMap(), loadNodeProperties(nodes.idMap()), graphProjectConfig.propertyState());
-    }
-
-    private Map<PropertyMapping, NodePropertyValues> loadNodeProperties(IdMap idMap) {
-        var propertyBuilders = new HashMap<PropertyMapping, NodePropertiesFromStoreBuilder>();
-
-        gdlHandler.getVertices().forEach(vertex -> vertex
-            .getProperties()
-            .forEach((propertyKey, propertyValue) -> {
-                if (propertyValue instanceof List) {
-                    propertyValue = convertListProperty((List<?>) propertyValue);
-                }
-
-                propertyBuilders.computeIfAbsent(PropertyMapping.of(propertyKey), (key) ->
-                    NodePropertiesFromStoreBuilder.of(
-                        DefaultValue.DEFAULT,
-                        1
-                    )).set(vertex.getId(), Values.of(propertyValue));
-            }));
-
-        return propertyBuilders
-            .entrySet()
-            .stream()
-            .collect(Collectors.toMap(Map.Entry::getKey, entry -> entry.getValue().build(idMap)));
+        return nodesBuilder.build();
     }
 
     @NotNull