Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support atomic writes in the docstore #3500

Merged
merged 3 commits into from
Jan 23, 2025

addressing comments

8d0616d
Select commit
Loading
Failed to load commit list.
Merged

Support atomic writes in the docstore #3500

addressing comments
8d0616d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 23, 2025 in 0s

71.79% of diff hit (target 0.00%)

View this Pull Request on Codecov

71.79% of diff hit (target 0.00%)

Annotations

Check warning on line 627 in docstore/awsdynamodb/dynamo.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

docstore/awsdynamodb/dynamo.go#L626-L627 

Added lines #L626 - L627 were not covered by tests

Check warning on line 631 in docstore/awsdynamodb/dynamo.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

docstore/awsdynamodb/dynamo.go#L631 

Added line #L631 was not covered by tests

Check warning on line 634 in docstore/awsdynamodb/dynamo.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

docstore/awsdynamodb/dynamo.go#L633-L634 

Added lines #L633 - L634 were not covered by tests

Check warning on line 640 in docstore/awsdynamodb/dynamo.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

docstore/awsdynamodb/dynamo.go#L638-L640 

Added lines #L638 - L640 were not covered by tests

Check warning on line 543 in docstore/docstore.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

docstore/docstore.go#L541-L543 

Added lines #L541 - L543 were not covered by tests
View more details on Codecov