[Enhancement] Comparison using reference equality instead of value equality.[Operation] #4023 #4276
Merged
mxsm merged 1 commit intoapache:masterfrom Jul 24, 2023
Merged
[Enhancement] Comparison using reference equality instead of value equality.[Operation] #4023 #4276mxsm merged 1 commit intoapache:masterfrom
mxsm merged 1 commit intoapache:masterfrom
Conversation
mxsm
approved these changes
Jul 23, 2023
Codecov Report
@@ Coverage Diff @@
## master #4276 +/- ##
============================================
- Coverage 16.87% 16.86% -0.01%
+ Complexity 1427 1426 -1
============================================
Files 593 593
Lines 26073 26073
Branches 2401 2401
============================================
- Hits 4400 4398 -2
- Misses 21233 21234 +1
- Partials 440 441 +1
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Alonexc
approved these changes
Jul 24, 2023
pandaapo
approved these changes
Jul 24, 2023
fabian4
pushed a commit
to fabian4/eventmesh
that referenced
this pull request
Jul 25, 2023
…ue equality.[Operation] (apache#4276)
xwm1992
pushed a commit
that referenced
this pull request
Jul 27, 2023
* feat: add redis connector. * feat: add redis connector. * feat: add redis connector. * feat: add redis connector. * feat: add redis connector. * [ISSUE #4023]Comparison using reference equality instead of value equality.[Operation] (#4276) * [ISSUE #4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] (#4270) * [ISSUE #4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE #4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE #4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE #4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE #4099]Refactor re-used constant's usage from dedicated file (#4257) * [ISSUE #4099]: Refactor re-used constant's usage from dedicated file * [ISSUE #4099] Rename some constants for mongodb storage plugin * [ISSUE #4099] Refactor constant imports in mongodb cloud event util * [ISSUE #4099]: Replace contants imports to class import in mongo cloud event util * fix import order. * fix import order. * fix import order. * fix import order. --------- Co-authored-by: Fabian Bao <fabian.bao@agfa.com> Co-authored-by: kyooosukedn <87076947+kyooosukedn@users.noreply.github.com> Co-authored-by: Sam V Jose <124816912+devCod3r@users.noreply.github.com> Co-authored-by: ruhshan <ruhshan.ahmed@gmail.com>
xuhongjia
pushed a commit
to Deckers-Ohana/eventmesh
that referenced
this pull request
Mar 13, 2025
…ue equality.[Operation] (apache#4276)
xuhongjia
pushed a commit
to Deckers-Ohana/eventmesh
that referenced
this pull request
Mar 13, 2025
* feat: add redis connector. * feat: add redis connector. * feat: add redis connector. * feat: add redis connector. * feat: add redis connector. * [ISSUE apache#4023]Comparison using reference equality instead of value equality.[Operation] (apache#4276) * [ISSUE apache#4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] (apache#4270) * [ISSUE apache#4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE apache#4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE apache#4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE apache#4269] Used switch to replace the if-else. [MeshMessageProtocolAdaptor] * [ISSUE apache#4099]Refactor re-used constant's usage from dedicated file (apache#4257) * [ISSUE apache#4099]: Refactor re-used constant's usage from dedicated file * [ISSUE apache#4099] Rename some constants for mongodb storage plugin * [ISSUE apache#4099] Refactor constant imports in mongodb cloud event util * [ISSUE apache#4099]: Replace contants imports to class import in mongo cloud event util * fix import order. * fix import order. * fix import order. * fix import order. --------- Co-authored-by: Fabian Bao <fabian.bao@agfa.com> Co-authored-by: kyooosukedn <87076947+kyooosukedn@users.noreply.github.com> Co-authored-by: Sam V Jose <124816912+devCod3r@users.noreply.github.com> Co-authored-by: ruhshan <ruhshan.ahmed@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #4023 .
Motivation
Changed value equality to reference equality
Modifications
Changed == to equals()
Documentation