update Status equals&hashCode

Author: JoeCqupt

api/src/main/java/io/grpc/Status.java

@@ -656,8 +656,14 @@ private static String parseAsciiStringSlow(byte[] value) {
    * additional fields may be added to Status in the future.
    */
   @Override
-  public boolean equals(Object obj) {
-    return super.equals(obj);
+  public boolean equals(Object o) {
+    if (!(o instanceof Status)) {
+      return false;
+    }
+    Status status = (Status) o;
+    return code == status.code
+            && Objects.equal(description, status.description)
+            && Objects.equal(cause, status.cause);
   }
 
   /**
@@ -667,6 +673,6 @@ public boolean equals(Object obj) {
    */
   @Override
   public int hashCode() {
-    return super.hashCode();
+    return Objects.hashCode(code, description, cause);
   }
 }

api/src/test/java/io/grpc/StatusOrTest.java

@@ -61,6 +61,8 @@ public void equals_differentStatuses() {
   @Test
   public void equals_sameStatuses() {
     assertThat(StatusOr.fromStatus(Status.ABORTED)).isEqualTo(StatusOr.fromStatus(Status.ABORTED));
+    assertThat(StatusOr.fromStatus(Status.ABORTED.withDescription("aborted")))
+            .isEqualTo(StatusOr.fromStatus(Status.ABORTED.withDescription("aborted")));
   }
 
   @Test

api/src/test/java/io/grpc/StatusTest.java

@@ -17,6 +17,7 @@
 package io.grpc;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotEquals;
 import static org.junit.Assert.assertSame;
 
 import io.grpc.Status.Code;
@@ -51,15 +52,25 @@ public void sameCauseReturnsSelf() {
     assertSame(Status.CANCELLED, Status.CANCELLED.withCause(null));
   }
 
+  @Test
+  public void equalsStatus() {
+    IllegalStateException ex = new IllegalStateException("The operation was aborted");
+    assertEquals(Status.ABORTED.withDescription("The operation was aborted")
+                    .withCause(ex),
+            Status.ABORTED.withDescription("The operation was aborted")
+                    .withCause(ex));
+    assertNotEquals(Status.ABORTED.withDescription("The operation was aborted"), null);
+  }
+
   @Test
   public void sameDescriptionReturnsSelf() {
     assertSame(Status.CANCELLED, Status.CANCELLED.withDescription(null));
     assertSame(Status.CANCELLED, Status.CANCELLED.augmentDescription(null));
   }
 
   @Test
-  public void useObjectHashCode() {
-    assertEquals(Status.CANCELLED.hashCode(), System.identityHashCode(Status.CANCELLED));
+  public void notUseObjectHashCode() {
+    assertNotEquals(Status.CANCELLED.hashCode(), System.identityHashCode(Status.CANCELLED));
   }
 
   @Test