You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/11/13 11:21:00 UTC
[jira] [Commented] (BROOKLYN-461) Merging config keys:
CollectionMerger.Ref.equals always return true
[ https://issues.apache.org/jira/browse/BROOKLYN-461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16249413#comment-16249413 ]
ASF GitHub Bot commented on BROOKLYN-461:
-----------------------------------------
GitHub user aledsage opened a pull request:
https://github.com/apache/brooklyn-server/pull/893
BROOKLYN-461: coerce empty string to URI as null
This fixes https://issues.apache.org/jira/browse/BROOKLYN-561
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/aledsage/brooklyn-server BROOKLYN-561
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/brooklyn-server/pull/893.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #893
----
----
> Merging config keys: CollectionMerger.Ref.equals always return true
> -------------------------------------------------------------------
>
> Key: BROOKLYN-461
> URL: https://issues.apache.org/jira/browse/BROOKLYN-461
> Project: Brooklyn
> Issue Type: Bug
> Reporter: JC
> Assignee: Aled Sage
> Priority: Trivial
> Fix For: 0.11.0
>
>
> Hi
> In recent github mirror of broolyn-server, I've found the following issue in equals implementation.
> Path: utils/common/src/main/java/org/apache/brooklyn/util/collections/CollectionMerger.java
> {code}
> 196
> 197 @Override
> 198 public boolean equals(Object o) {
> 199 if (!(o instanceof Ref)) {
> 200 return false;
> 201 }
> 202 return ((Ref)o).obj == ((Ref)o).obj;
> 203 }
> {code}
> Line 202 should be like this?
> {code}
> 202 return this.obj == ((Ref)o).obj;
> {code}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)