fix: Call Release on resource semaphore (#279)

hermanschaaf · web-flow · commit 051e24710b64 · 2022-10-11T11:06:55.000Z
We were missing a `Release` call on the `resource_concurrency` semaphore.
diff --git a/schema/table.go b/schema/table.go
@@ -206,6 +206,7 @@ func (t *Table) Resolve(ctx context.Context, meta ClientMeta, parent *Resource,
 				}
 				wg.Add(1)
 				go func() {
+					defer resourcesSem.Release(1)
 					defer wg.Done()
 					//nolint:all
 					summary.Merge(t.resolveObject(ctx, meta, parent, objects[i], resolvedResources))

Original file line number	Diff line number	Diff line change
`@@ -206,6 +206,7 @@ func (t Table) Resolve(ctx context.Context, meta ClientMeta, parent Resource,`
`206`	`206`	`}`
`207`	`207`	`wg.Add(1)`
`208`	`208`	`go func() {`
	`209`	`+ defer resourcesSem.Release(1)`
`209`	`210`	`defer wg.Done()`
`210`	`211`	`//nolint:all`
`211`	`212`	`summary.Merge(t.resolveObject(ctx, meta, parent, objects[i], resolvedResources))`