diff options
| author | Taras Madan <tarasmadan@google.com> | 2024-11-06 20:50:45 +0100 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2024-11-07 08:38:08 +0000 |
| commit | 867e44df36d93e8127938eca6f6a5c339a2ba0b8 (patch) | |
| tree | 5ac045074727eb4b67dc7f2d90414c43317a2dd8 /pkg | |
| parent | 69abf6c7a42942f529f2b3d5b6917f97c5aecc10 (diff) | |
pkg/coveragedb: store hitcount instead of coveredlines
Instrumented lines + hit count gives more information than instrumented + covered lines.
Expected storage cost is at the same level.
Diffstat (limited to 'pkg')
| -rw-r--r-- | pkg/coveragedb/spanner.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/pkg/coveragedb/spanner.go b/pkg/coveragedb/spanner.go index dfab7ff7a..0ff3a3197 100644 --- a/pkg/coveragedb/spanner.go +++ b/pkg/coveragedb/spanner.go @@ -22,7 +22,7 @@ type FilesRecord struct { Instrumented int64 Covered int64 LinesInstrumented []int64 - LinesCovered []int64 + HitCounts []int64 } type FileSubsystems struct { @@ -51,7 +51,7 @@ type Coverage struct { Instrumented int64 Covered int64 LinesInstrumented []int64 - LinesCovered []int64 + HitCounts []int64 } func SaveMergeResult(ctx context.Context, projectID string, covMap map[string]*Coverage, @@ -113,10 +113,10 @@ func fileRecordMutation(session, filePath string, record *Coverage) *spanner.Mut Instrumented: record.Instrumented, Covered: record.Covered, LinesInstrumented: record.LinesInstrumented, - LinesCovered: record.LinesCovered, + HitCounts: record.HitCounts, }) if err != nil { - panic(fmt.Sprintf("failed to fileRecordMutation(): %s", err.Error())) + panic(fmt.Sprintf("failed to fileRecordMutation: %v", err)) } return insert } |
