storage: EncodeMVCCValueForExport does not inline

Epic: none Release note: None
cockroachdb · Mar 11, 2024 · b8ae5f0 · b8ae5f0
1 parent 8454009
commit b8ae5f0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/pkg/sql/importer/import_mvcc_test.go b/pkg/sql/importer/import_mvcc_test.go
@@ -70,8 +70,9 @@ func TestMVCCValueHeaderImportEpoch(t *testing.T) {
 			Key:    startKey,
 			EndKey: endKey,
 		},
-		MVCCFilter: kvpb.MVCCFilter_All,
-		StartTime:  hlc.Timestamp{},
+		MVCCFilter:             kvpb.MVCCFilter_All,
+		StartTime:              hlc.Timestamp{},
+		IncludeMVCCValueHeader: true,
 	}
 
 	header := kvpb.Header{Timestamp: s.Clock().Now()}

diff --git a/pkg/storage/mvcc_value.go b/pkg/storage/mvcc_value.go
@@ -136,8 +136,6 @@ func (v MVCCValue) SafeFormat(w redact.SafePrinter, _ rune) {
 
 // EncodeMVCCValueForExport strips fields from the MVCCValueHeader that
 // should not get exported out of the cluster.
-//
-//gcassert:inline
 func EncodeMVCCValueForExport(mvccValue MVCCValue) ([]byte, error) {
 	// Consider a fast path, where only the roachpb.Value gets exported.
 	// Currently, this only occurs if the value was not imported.