[ 
https://issues.apache.org/jira/browse/OAK-3404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15374857#comment-15374857
 ] 

Chetan Mehrotra commented on OAK-3404:
--------------------------------------

+1. Just a minor note

{code}
+    @Override
+    public boolean equals(Object obj) {
+        if (this == obj)
+            return true;
+        if (obj == null)
+            return false;
+        if (getClass() != obj.getClass())
+            return false;
+        MountInfo other = (MountInfo) obj;
+        if (name == null) {
+            if (other.name != null)
+                return false;
+        } else if (!name.equals(other.name))
+            return false;
+        return true;
     }
{code}

{{name}} cannot be null so equals can be simplified.

Thanks for taking care of this!

> Path to logical store mapping
> -----------------------------
>
>                 Key: OAK-3404
>                 URL: https://issues.apache.org/jira/browse/OAK-3404
>             Project: Jackrabbit Oak
>          Issue Type: Technical task
>          Components: core
>            Reporter: Chetan Mehrotra
>            Assignee: Chetan Mehrotra
>         Attachments: OAK-3404-v1.patch, ismounted.patch, mounts-v2.patch, 
> mounts.patch
>
>
> Need to provide an API {{MountInfoProvider}} which would be responsible for 
> mapping a given path to logical store name 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to