-
Notifications
You must be signed in to change notification settings - Fork 641
[WIP][ISSUE #482]Add protocol adaptor plugin #572
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
Conversation
fe34a63 to
986edf8
Compare
...sage/src/main/java/org/apache/eventmesh/protocol/openmessage/OpenMessageProtocolAdaptor.java
Outdated
Show resolved
Hide resolved
...sage/src/main/java/org/apache/eventmesh/protocol/openmessage/OpenMessageProtocolAdaptor.java
Outdated
Show resolved
Hide resolved
...vents/src/main/java/org/apache/eventmesh/protocol/cloudevents/CloudEventProtocolAdaptor.java
Outdated
Show resolved
Hide resolved
...vents/src/main/java/org/apache/eventmesh/protocol/cloudevents/CloudEventProtocolAdaptor.java
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## develop #572 +/- ##
=============================================
+ Coverage 10.97% 11.00% +0.03%
- Complexity 391 395 +4
=============================================
Files 263 267 +4
Lines 12498 12509 +11
Branches 1047 1047
=============================================
+ Hits 1372 1377 +5
- Misses 11008 11014 +6
Partials 118 118
Continue to review full report at Codecov.
|
6ceabb5 to
47d54e0
Compare
ea89e50 to
11e5480
Compare
xwm1992
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Add protocol adaptor plugin * ProtocolAdaptor deal with Package * Remove java doc check
* Add protocol adaptor plugin * ProtocolAdaptor deal with Package * Remove java doc check
close #482