[TACHYON-220] WIP Comments describing issues and possible solutions 

Some initial comments to help improve the performance of master info. From what I gather there is locking on stack local objects that are never really put back into the map. I would appreciate suggestions before implementing it, hence the WIP tag.
Can you point to a example that does that? That is not something you see in
java often.  If you know of a example we should fix that example to comply
with java convention
On Nov 20, 2014 9:48 AM, "Manu Goyal" <notifications@github.com> wrote:

> In core/src/main/java/tachyon/master/MasterInfo.java:
>
> > @@ -1527,6 +1525,24 @@ public TachyonURI getPath(int fileId) throws FileDoesNotExistException {
> >    }
> >
> >    /**
> > +   * Get the path specified by a given inode. Should only be called
> > +   * within a synchronized call e.g. {@link MasterInfo#getPath(tachyon.master.Inode)}.
> > +   * This is because calling sync repeatedly within a recursive method results in
> > +   * poor performance. So this method must be called in a synchronized way.
> > +   * @param inode The inode
> > +   * @return the path of the inode
> > +   */
> > +  private TachyonURI getPathUsingRecursion(Inode inode) {
>
> A name like _getPath is probably more in line with the naming convention.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amplab/tachyon/pull/527/files#r20663692>.
>