Skip to content

Commit 8e0d636

Browse files
authored
Merge pull request #67 from n9te9/remove/unnecessary-merge-logic
remove: unnecessary merger.merge logic
2 parents f541695 + b44adde commit 8e0d636

1 file changed

Lines changed: 2 additions & 11 deletions

File tree

federation/executor/merger/merger.go

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -140,18 +140,9 @@ func (m *merger) MergeEntities(rootResult, result map[string]any, step *planner.
140140
return nil, fmt.Errorf("entities data is not an array")
141141
}
142142

143-
// Navigate to the first array
144-
var arrayContainer any = current
145-
arrayPath := mergePath[:index+1] // Include the array field itself
146-
for _, segment := range arrayPath {
147-
if containerMap, ok := arrayContainer.(map[string]any); ok {
148-
arrayContainer = containerMap[segment]
149-
}
150-
}
151-
152-
array, ok := arrayContainer.([]any)
143+
array, ok := current.([]any)
153144
if !ok {
154-
return nil, fmt.Errorf("expected array at merge path %v", arrayPath)
145+
return nil, fmt.Errorf("expected array at merge path %v", mergePath[:index+1])
155146
}
156147

157148
// The remaining path after the array

0 commit comments

Comments
 (0)