Throw SHUTDOWN error received from Redis (#2613)

src/main/java/redis/clients/jedis/BinaryJedis.java

@@ -3442,16 +3442,19 @@ public Long lastsave() {
    * is switched off without the lost of any data. This is not guaranteed if the client uses simply
    * {@link #save() SAVE} and then {@link #quit() QUIT} because other clients may alter the DB data
    * between the two commands.
-   * @return Status code reply on error. On success nothing is returned since the server quits and
-   *         the connection is closed.
+   * @return {@code null}
+   * @throws JedisException with the status code reply on error. On success nothing is thrown since
+   *         the server quits and the connection is closed.
    */
   @Override
-  public String shutdown() {
+  public String shutdown() throws JedisException {
     client.shutdown();
     String status;
     try {
       status = client.getStatusCodeReply();
-    } catch (JedisException ex) {
+      throw new JedisException(status);
+    } catch (JedisConnectionException jce) {
+      // expected
       status = null;
     }
     return status;
@@ -3461,8 +3464,8 @@ public String shutdown() {
   public void shutdown(final SaveMode saveMode) throws JedisException {
     client.shutdown(saveMode);
     try {
-      throw new JedisDataException(client.getStatusCodeReply());
-    } catch (JedisConnectionException ex) {
+      throw new JedisException(client.getStatusCodeReply());
+    } catch (JedisConnectionException jce) {
       // expected
     }
   }

src/main/java/redis/clients/jedis/commands/BasicCommands.java

@@ -134,9 +134,10 @@ public interface BasicCommands {
   /**
    * Stop all the client. Perform a SAVE (if one save point is configured). Flush the append only
    * file if AOF is enabled quit the server
-   * @return only in case of error.
+   * @return {@code null}
+   * @throws JedisException only in case of error.
    */
-  String shutdown();
+  String shutdown() throws JedisException;
 
   /**
    * @see SaveMode