Skip to content

fix: handle DynamoDB creation timestamp precision#633

Open
kiwigitops wants to merge 1 commit into
aws:mainfrom
kiwigitops:fix/dynamodb-creation-time-precision
Open

fix: handle DynamoDB creation timestamp precision#633
kiwigitops wants to merge 1 commit into
aws:mainfrom
kiwigitops:fix/dynamodb-creation-time-precision

Conversation

@kiwigitops

Copy link
Copy Markdown

Summary

  • add ApproximateCreationDateTimePrecision to DynamoDB stream records
  • parse and round-trip millisecond and microsecond creation timestamps when precision is present
  • add a regression test for a microsecond timestamp payload

Testing

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 64.28571% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.81%. Comparing base (87e344b) to head (3c1aeaf).

Files with missing lines Patch % Lines
events/dynamodb.go 64.28% 11 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #633      +/-   ##
==========================================
- Coverage   75.12%   74.81%   -0.32%     
==========================================
  Files          36       37       +1     
  Lines        1419     1461      +42     
==========================================
+ Hits         1066     1093      +27     
- Misses        274      285      +11     
- Partials       79       83       +4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants