Merge pull request #2148 from Netflix/too-many-requests

split 429 Too Many Requests into its own error type

Author: iuliiasobolevska

graphql-error-types/src/main/java/com/netflix/graphql/types/errors/ErrorDetail.java

@@ -22,7 +22,7 @@
 import static com.netflix.graphql.types.errors.ErrorType.*;
 
 /**
- * The ErrorDetail is an optional field which will provide more fine grained information on the error condition.
+ * The ErrorDetail is an optional field which will provide more fine-grained information on the error condition.
  * This allows the ErrorType enumeration to be small and mostly static so that application branching logic
  * can depend on it. The ErrorDetail provides a more specific cause for the error. This enumeration will
  * be much larger and likely change/grow over time.
@@ -59,11 +59,20 @@ enum Common implements ErrorDetail {
          * The server detected that the client is exhibiting a behavior that
          * might be generating excessive load.
          * <p>
-         * HTTP Mapping: 429 Too Many Requests or 420 Enhance Your Calm
+         * HTTP Mapping: 420 Enhance Your Calm
          * Error Type: UNAVAILABLE
          */
         ENHANCE_YOUR_CALM(UNAVAILABLE),
 
+        /**
+         * The server detected that the client is exhibiting a behavior that
+         * might be generating excessive load.
+         * <p>
+         * HTTP Mapping: 429 Too Many Requests
+         * Error Type: UNAVAILABLE
+         */
+        TOO_MANY_REQUESTS(UNAVAILABLE),
+
         /**
          * The requested field is not found in the schema.
          * <p>

graphql-error-types/src/main/resources/META-INF/schema/errordetail.graphqls

@@ -107,11 +107,20 @@ enum ErrorDetail {
     The server detected that the client is exhibiting a behavior that
     might be generating excessive load.
 
-    HTTP Mapping: 429 Too Many Requests or 420 Enhance Your Calm
+    HTTP Mapping: 420 Enhance Your Calm
     Error Type: UNAVAILABLE
     """
     ENHANCE_YOUR_CALM
 
+    """
+    The server detected that the client is exhibiting a behavior that
+    might be generating excessive load.
+
+    HTTP Mapping: 429 Too Many Requests
+    Error Type: UNAVAILABLE
+    """
+    TOO_MANY_REQUESTS
+
     """
     Request failed due to network errors.