Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

Add unique "delivery ID" to identify each attempt to process a message. #145

Merged
merged 1 commit into from
Oct 25, 2018

Conversation

jmalloc
Copy link
Owner

@jmalloc jmalloc commented Oct 25, 2018

Partially addresses #68

@codecov-io
Copy link

codecov-io commented Oct 25, 2018

Codecov Report

Merging #145 into master will decrease coverage by 0.03%.
The diff coverage is 31.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #145      +/-   ##
==========================================
- Coverage   29.25%   29.21%   -0.04%     
==========================================
  Files          74       75       +1     
  Lines        2694     2711      +17     
==========================================
+ Hits          788      792       +4     
- Misses       1882     1895      +13     
  Partials       24       24
Impacted Files Coverage Δ
src/axrmq/marshaling.go 0% <0%> (ø) ⬆️
src/ax/endpoint/envelope.go 0% <0%> (ø)
src/ax/observability/logging.go 95% <100%> (+0.55%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f33330...1a45b79. Read the comment docs.

@jmalloc jmalloc merged commit 4af5fed into master Oct 25, 2018
@jmalloc jmalloc deleted the 68-delivery-id branch October 25, 2018 23:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants