tf5muxserver+tf6muxserver: Remove resource schema caching

Reference: #168 This change removes the `GetProviderSchema` RPC-based caching logic in `NewMuxServer` and directly calls the underlying providers within that RPC. To fully support the removal of resource schema caching, the `PlanResourceChange` logic for checking the `ProposedNewState` data for a destroy plan was optimized to no longer require decoding the data with the type information. For larger scale providers (either number of resources and/or resources with large schemas), this should reduce resident memory utilization with little performance penalty.
hashicorp · Jun 28, 2023 · 111033e · 111033e
1 parent 2d33d1e
commit 111033e
Show file tree

Hide file tree

Showing 37 changed files with 429 additions and 665 deletions.
diff --git a/.changes/unreleased/BUG FIXES-20230627-134839.yaml b/.changes/unreleased/BUG FIXES-20230627-134839.yaml
@@ -0,0 +1,6 @@
+kind: BUG FIXES
+body: 'tf5muxserver: Removed unnecessary resource schema caching, which reduces resident
+  memory utilization'
+time: 2023-06-27T13:48:39.01979-04:00
+custom:
+  Issue: "168"
diff --git a/.changes/unreleased/BUG FIXES-20230627-134907.yaml b/.changes/unreleased/BUG FIXES-20230627-134907.yaml
@@ -0,0 +1,6 @@
+kind: BUG FIXES
+body: 'tf6muxserver: Removed unnecessary resource schema caching, which reduces resident
+  memory utilization'
+time: 2023-06-27T13:49:07.303269-04:00
+custom:
+  Issue: "168"
diff --git a/internal/tf5dynamicvalue/is_null.go b/internal/tf5dynamicvalue/is_null.go
diff --git a/internal/tf5dynamicvalue/is_null_test.go b/internal/tf5dynamicvalue/is_null_test.go
diff --git a/internal/tf6dynamicvalue/is_null.go b/internal/tf6dynamicvalue/is_null.go
diff --git a/internal/tf6dynamicvalue/is_null_test.go b/internal/tf6dynamicvalue/is_null_test.go