Deprecate and add warning to remove AtomicFieldOffsets, unnecessary p…

…ublic func Signed-off-by: Bogdan Drutu <[email protected]>
open-telemetry · Dec 13, 2021 · b8e4e70 · b8e4e70
1 parent 6cafa6a
commit b8e4e70
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/sdk/metric/alignment_test.go b/sdk/metric/alignment_test.go
@@ -17,13 +17,17 @@ package metric
 import (
 	"os"
 	"testing"
+	"unsafe"
 
 	ottest "go.opentelemetry.io/otel/internal/internaltest"
 )
 
 // Ensure struct alignment prior to running tests.
 func TestMain(m *testing.M) {
-	offsets := AtomicFieldOffsets()
+	offsets := map[string]uintptr{
+		"record.refMapped.value": unsafe.Offsetof(record{}.refMapped.value),
+		"record.updateCount":     unsafe.Offsetof(record{}.updateCount),
+	}
 	var r []ottest.FieldOffset
 	for name, offset := range offsets {
 		r = append(r, ottest.FieldOffset{

diff --git a/sdk/metric/atomicfields.go b/sdk/metric/atomicfields.go
@@ -16,6 +16,7 @@ package metric // import "go.opentelemetry.io/otel/sdk/metric"
 
 import "unsafe"
 
+// Deprecated: will be removed soon.
 func AtomicFieldOffsets() map[string]uintptr {
 	return map[string]uintptr{
 		"record.refMapped.value": unsafe.Offsetof(record{}.refMapped.value),