Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

243 rabbitmq adaptor #298

Merged
merged 1 commit into from
Mar 9, 2017
Merged

243 rabbitmq adaptor #298

merged 1 commit into from
Mar 9, 2017

Conversation

jipperinbham
Copy link
Contributor

@jipperinbham jipperinbham commented Mar 8, 2017

adds a RabbitMQ adaptor for consuming/publishing

  • CHANGELOG.md updated
  • README.md updated
  • add SSL tests

fixes #243

@codecov
Copy link

codecov bot commented Mar 8, 2017

Codecov Report

Merging #298 into master will increase coverage by 0.58%.
The diff coverage is 86.93%.

@@            Coverage Diff             @@
##           master     #298      +/-   ##
==========================================
+ Coverage   80.06%   80.64%   +0.58%     
==========================================
  Files          43       47       +4     
  Lines        2307     2480     +173     
==========================================
+ Hits         1847     2000     +153     
- Misses        326      337      +11     
- Partials      134      143       +9
Impacted Files Coverage Δ
adaptor/rabbitmq/rabbitmq.go 100% <100%> (ø)
adaptor/rabbitmq/writer.go 100% <100%> (ø)
adaptor/rabbitmq/client.go 100% <100%> (ø)
pipeline/node.go 76.28% <58.33%> (-1.27%)
adaptor/rabbitmq/reader.go 73.52% <73.52%> (ø)
pipeline/pipeline.go 58.24% <0%> (+2.19%)

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 b523292...b38ef5e. Read the comment docs.

@jipperinbham jipperinbham force-pushed the 243-rabbitmq-adaptor branch 2 times, most recently from 38f0fb5 to c1671df Compare March 9, 2017 14:32
the rabbitmq adaptor is capable of acting as a consumer or producer and works with JSON data with configurable routing keys

fixes #243
@jipperinbham
Copy link
Contributor Author

publisher confirms was left out for now until we have a way to track messages processed internally

this change also updated to go 1.8 and fixed a bug pipeline/node.go introduced in #290 where the namespace filter function was not getting passed to the Read() function.

@jipperinbham jipperinbham merged commit 42584ed into master Mar 9, 2017
@jipperinbham jipperinbham deleted the 243-rabbitmq-adaptor branch March 9, 2017 15:08
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.

RabbitMQ adaptor
1 participant