-
Notifications
You must be signed in to change notification settings - Fork 83
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.
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
feat: Add Pipeline per Topic capability #938
feat: Add Pipeline per Topic capability #938
Conversation
9d8c31f
to
314f718
Compare
closes #575 & #713 Signed-off-by: Leonard Goodell <[email protected]>
314f718
to
82dd4a4
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #938 +/- ##
==========================================
+ Coverage 65.34% 66.16% +0.81%
==========================================
Files 33 33
Lines 2346 2503 +157
==========================================
+ Hits 1533 1656 +123
- Misses 722 748 +26
- Partials 91 99 +8 ☔ View full report in Codecov by Sentry. |
In my experience using multiple pipelines, an MQTT client with a specific client ID should be reusable. |
…tionsPipeline Signed-off-by: Leonard Goodell <[email protected]>
@AlexCuse , I added SetDefaultFunctionsPipeline and deprecated SetFunctionsPipeline |
This should make it clearer I think @lenny-intel - would give a chance to get any feedback before 3 too. |
Signed-off-by: Leonard Goodell <[email protected]>
* feat: Add Pipeline per Topic capability closes edgexfoundry#575 & edgexfoundry#713 Signed-off-by: Leonard Goodell <[email protected]>
Dependent on App SDK PR: #938
PR Checklist
Please check if your PR fulfills the following requirements:
Related Docs PR now required (if applicable)
Related Docs PR: edgexfoundry/edgex-docs#551
What is the current behavior?
Only a single functions pipeline can be created
Issue Number: #575 & #713
What is the new behavior?
Multiple functions pipeline can now created.
Does this PR introduce a breaking change?
Are there any new imports or modules? If so, what are they used for and why?
Are there any specific instructions or things that should be known prior to reviewing?
Other information