Skip to content

[DO NOT MERGE] refactor: CachedOptimizedRevisions#3196

Draft
miparnisari wants to merge 1 commit into
mainfrom
miparnisari/oss-902-optimizedrevision-caching-tech-debt-refactor-proposal
Draft

[DO NOT MERGE] refactor: CachedOptimizedRevisions#3196
miparnisari wants to merge 1 commit into
mainfrom
miparnisari/oss-902-optimizedrevision-caching-tech-debt-refactor-proposal

refactor: CachedOptimizedRevisions

d255d8e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jun 26, 2026 in 1s

77.88% of diff hit (target 76.10%)

View this Pull Request on Codecov

77.88% of diff hit (target 76.10%)

Annotations

Check warning on line 262 in internal/datastore/crdb/crdb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/crdb/crdb.go#L261-L262

Added lines #L261 - L262 were not covered by tests

Check warning on line 266 in internal/datastore/crdb/crdb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/crdb/crdb.go#L265-L266

Added lines #L265 - L266 were not covered by tests

Check warning on line 271 in internal/datastore/crdb/crdb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/crdb/crdb.go#L270-L271

Added lines #L270 - L271 were not covered by tests

Check warning on line 289 in internal/datastore/crdb/crdb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/crdb/crdb.go#L288-L289

Added lines #L288 - L289 were not covered by tests

Check warning on line 297 in internal/datastore/crdb/crdb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/crdb/crdb.go#L296-L297

Added lines #L296 - L297 were not covered by tests

Check warning on line 302 in internal/datastore/crdb/crdb.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/crdb/crdb.go#L301-L302

Added lines #L301 - L302 were not covered by tests

Check warning on line 154 in internal/datastore/proxy/optimized_revision.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/proxy/optimized_revision.go#L153-L154

Added lines #L153 - L154 were not covered by tests

Check warning on line 45 in internal/datastore/revisions/hlc.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/revisions/hlc.go#L44-L45

Added lines #L44 - L45 were not covered by tests

Check warning on line 250 in internal/datastore/spanner/spanner.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/spanner.go#L249-L250

Added lines #L249 - L250 were not covered by tests

Check warning on line 254 in internal/datastore/spanner/spanner.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/spanner.go#L253-L254

Added lines #L253 - L254 were not covered by tests

Check warning on line 259 in internal/datastore/spanner/spanner.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/spanner.go#L258-L259

Added lines #L258 - L259 were not covered by tests

Check warning on line 271 in internal/datastore/spanner/spanner.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/spanner.go#L270-L271

Added lines #L270 - L271 were not covered by tests

Check warning on line 276 in internal/datastore/spanner/spanner.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/spanner.go#L275-L276

Added lines #L275 - L276 were not covered by tests

Check warning on line 284 in internal/datastore/spanner/spanner.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/spanner.go#L283-L284

Added lines #L283 - L284 were not covered by tests

Check warning on line 289 in internal/datastore/spanner/spanner.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/datastore/spanner/spanner.go#L288-L289

Added lines #L288 - L289 were not covered by tests

Check warning on line 503 in pkg/cmd/datastore/datastore.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/cmd/datastore/datastore.go#L498-L503

Added lines #L498 - L503 were not covered by tests