[ISSUE #3892] SendMessageRequestHeader, SendMessageBatchRequestHeader and SendMessageBatchV2RequestHeader have almost same body #3893
Conversation
…rmock-powermock-api-mockito2-2.0.9 Bump org.powermock:powermock-api-mockito2 from 2.0.2 to 2.0.9
[ISSUE apache#862] Add some unit test for [evenmesh-storage-rocketmq] module
Update README.md
…orrect order: expected value, actual value.[EventMeshHTTPConfigurationTest]
…eResourceServiceKnativeImpl when start runtime with start.sh
[ISSUE apache#3586] Refactor CloudEventCodec singleton instantiation
[ISSUE apache#3597]Fix load RabbitmqStorageResourceService failure
[ISSUE apache#3595]Fix throw StorageResourceServiceKnativeImpl StorageResourceServiceKnativeImpl when start runtime with start.sh
[ISSUE apache#3484] Did code optimization in PulsarClientWrapper. Closes apache#3484
[ISSUE apache#3521] Swapped the actual and expected values
Bumps [com.mebigfatguy.fb-contrib:fb-contrib](https://github.com/mebigfatguy/fb-contrib) from 7.4.7 to 7.6.0. - [Release notes](https://github.com/mebigfatguy/fb-contrib/releases) - [Commits](mebigfatguy/fb-contrib@v7.4.7...v7.6.0) --- updated-dependencies: - dependency-name: com.mebigfatguy.fb-contrib:fb-contrib dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Update greetings.yml
Fix patch 3401
…gfatguy.fb-contrib-fb-contrib-7.6.0 Bump com.mebigfatguy.fb-contrib:fb-contrib from 7.4.7 to 7.6.0
Removed the parameter 'defaultValue' at line 256. Replaced 'defaultvalue' with "" at line 258 . I have also removed the third parameter at lines 196 and 197.
Swapped actual and expected values in lines 47,48,63,64,75,76.
I have swapped the actual and expected values in the lines 137-155.
Replaced "replaceAll()" with "replace()"
Replaced "replaceAll()" to "replace()"
[ISSUE apache#3964] Catch Exception instead of Throwable.
…ify header to 'checkstyle-header.txt'
* [ISSUE apache#4013] rename the connect sdk module * rename to open connect
* Resolves apache#3962 * Updated EventMeshConstants.java
* Resolves apache#3961 * Updated EventMeshConstants.java
* Resolves apache#3960 * Updated EventMeshConstants.java
* Resolves apache#3957 * Updated EventMeshConstants.java
* Resolves ISSUE apache#3956 * Updated EventMeshConstants.java
* Resolves apache#3959 * Updated EventMeshConstants.java
* Resolves apache#3958 * Resolves-3958 * Fixed Checkmark
Signed-off-by: Yixiang Zhao <[email protected]>
… classes[RejectClientBySubSystemHandler] Signed-off-by: Yixiang Zhao <[email protected]>
|
It has been 60 days since the last activity on this pull request. I am reaching out here to gently remind you that the Apache EventMesh community values every pull request, and please feel free to get in touch with the reviewers at any time. They are available to assist you in advancing the progress of your pull request and offering the latest feedback. If you encounter any challenges during development, seeking support within the community is encouraged. We sincerely appreciate your contributions to Apache EventMesh. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3893 +/- ##
============================================
+ Coverage 14.24% 14.55% +0.31%
- Complexity 1320 1350 +30
============================================
Files 579 580 +1
Lines 28967 28811 -156
Branches 2802 2799 -3
============================================
+ Hits 4125 4194 +69
+ Misses 24450 24225 -225
Partials 392 392 ☔ View full report in Codecov by Sentry. |
| */ | ||
| public abstract class BaseSendMsgRequestHeader extends Header { | ||
|
|
||
| //request code | ||
| private String code; |
There was a problem hiding this comment.
How about using the AbstractSendMsgRequestHeader naming, to match the naming of other classes?
| public void setCode(String code) { | ||
| this.code = code; | ||
| } | ||
|
|
||
| public String getLanguage() { | ||
| return language; | ||
| } | ||
|
|
||
| public void setLanguage(String language) { | ||
| this.language = language; | ||
| } |
There was a problem hiding this comment.
How about using the @Data annotation to save space on those getters and setters?
Pil0tXia
changed the title
|
It has been 60 days since the last activity on this pull request. I am reaching out here to gently remind you that the Apache EventMesh community values every pull request, and please feel free to get in touch with the reviewers at any time. They are available to assist you in advancing the progress of your pull request and offering the latest feedback. If you encounter any challenges during development, seeking support within the community is encouraged. We sincerely appreciate your contributions to Apache EventMesh. |
|
It has been 60 days since the last activity on this pull request. I am reaching out here to gently remind you that the Apache EventMesh community values every pull request, and please feel free to get in touch with the reviewers at any time. They are available to assist you in advancing the progress of your pull request and offering the latest feedback. If you encounter any challenges during development, seeking support within the community is encouraged. We sincerely appreciate your contributions to Apache EventMesh. |
pandaapo
dismissed
Alonexc’s stale review
The merge-base changed after approval.
Fixes #3892.
Motivation
See "Enhancement Request" in issue.
Modifications
See "Describe the solution you'd like" in issue.
Documentation