Rename remove to removeByField

servers/src/main/java/tachyon/master/next/IndexedSet.java

@@ -144,10 +144,9 @@ public boolean remove(T object) {
    *
    * @param fieldName the field name
    * @param value the field value
-   * @return the set of objects that are removed, if no object is removed, the returned set is empty
    * @return true if the objects are removed, otherwise false
    */
-  public boolean remove(String fieldName, Object value) {
+  public boolean removeByField(String fieldName, Object value) {
     Set<T> toRemove = mSetIndexedByFieldValue.get(fieldName).remove(value);
     boolean success = true;
     if (toRemove != null) {

servers/src/main/java/tachyon/master/next/filesystem/meta/InodeDirectory.java

@@ -166,7 +166,7 @@ public synchronized boolean removeChild(Inode child) {
    * @return true if the inode was removed, false otherwise.
    */
   public synchronized boolean removeChild(String name) {
-    return mChildren.remove("mName", name);
+    return mChildren.removeByField("mName", name);
   }
 
   @Override

servers/src/test/java/tachyon/master/next/IndexedSetTest.java

@@ -95,17 +95,17 @@ public void getTest() {
   }
 
   @Test
-  public void removeTest1() {
+  public void removeTest() {
     Pair toRemove = mSet.all().iterator().next();
     Assert.assertEquals(3, mSet.getByField("mDouble", toRemove.mDouble).size());
     Assert.assertTrue(mSet.remove(toRemove));
     Assert.assertEquals(2, mSet.getByField("mDouble", toRemove.mDouble).size());
   }
 
   @Test
-  public void removeTest2() {
+  public void removeByFieldTest() {
     Assert.assertEquals(3, mSet.getByField("mInt", 1).size());
-    Assert.assertTrue(mSet.remove("mInt", 1));
+    Assert.assertTrue(mSet.removeByField("mInt", 1));
     Assert.assertEquals(0, mSet.getByField("mInt", 1).size());
   }
 }