Skip to content

HOTFIX: Fix equality semantics of KeyValue#1294

Closed
miguno wants to merge 2 commits into
apache:trunkfrom
miguno:KeyValue-equality-hotfix
Closed

HOTFIX: Fix equality semantics of KeyValue#1294
miguno wants to merge 2 commits into
apache:trunkfrom
miguno:KeyValue-equality-hotfix

Conversation

@miguno
Copy link
Copy Markdown
Contributor

@miguno miguno commented Apr 29, 2016

Fixes wrong KeyValue equals logic when keys not equal but values equal.

Original hotfix PR at #1293 (/cc @enothereska)

Please review: @ewencp @ijuma @guozhangwang

@ewencp
Copy link
Copy Markdown
Contributor

ewencp commented Apr 29, 2016

LGTM

@asfgit asfgit closed this in 60380e3 Apr 29, 2016
asfgit pushed a commit that referenced this pull request Apr 29, 2016
Fixes wrong KeyValue equals logic when keys not equal but values equal.

Original hotfix PR at #1293 (/cc enothereska)

Please review: ewencp ijuma guozhangwang

Author: Eno Thereska <eno.thereska@gmail.com>
Author: Michael G. Noll <michael@confluent.io>

Reviewers: Michael G. Noll <michael@confluent.io>, Ewen Cheslack-Postava <ewen@confluent.io>

Closes #1294 from miguno/KeyValue-equality-hotfix

(cherry picked from commit 60380e3)
Signed-off-by: Ewen Cheslack-Postava <me@ewencp.org>
gfodor pushed a commit to AltspaceVR/kafka that referenced this pull request Jun 3, 2016
Fixes wrong KeyValue equals logic when keys not equal but values equal.

Original hotfix PR at apache#1293 (/cc enothereska)

Please review: ewencp ijuma guozhangwang

Author: Eno Thereska <eno.thereska@gmail.com>
Author: Michael G. Noll <michael@confluent.io>

Reviewers: Michael G. Noll <michael@confluent.io>, Ewen Cheslack-Postava <ewen@confluent.io>

Closes apache#1294 from miguno/KeyValue-equality-hotfix
@miguno miguno deleted the KeyValue-equality-hotfix branch February 9, 2021 10:54
efeg added a commit to efeg/kafka that referenced this pull request May 29, 2024
mumrah pushed a commit to mumrah/kafka that referenced this pull request Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants