Adding a missing else block for throwing a store def mismatch exception

jcustenborder · Apr 24, 2013 · 9991605 · 9991605
1 parent 626ea52
commit 9991605
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/...-store-builder/src/java/voldemort/store/readonly/mr/azkaban/VoldemortBuildAndPushJob.java b/...-store-builder/src/java/voldemort/store/readonly/mr/azkaban/VoldemortBuildAndPushJob.java
@@ -411,6 +411,11 @@ public void verifySchema(String url) throws Exception {
                                                            + "\nBut expected: "
                                                            + remoteStoreDef);
                             }
+                        } else {
+                            throw new RuntimeException("Your store definition does not match the store definition that is already in the cluster. Have: "
+                                                       + newStoreDef
+                                                       + "\nBut expected: "
+                                                       + remoteStoreDef);
                         }
                     }
 
@@ -788,6 +793,11 @@ public void verifyAvroSchemaAndVersions(String url, boolean isVersioned) throws
                                                            + "\nBut expected: "
                                                            + remoteStoreDef);
                             }
+                        } else {
+                            throw new RuntimeException("Your store definition does not match the store definition that is already in the cluster. Have: "
+                                                       + newStoreDef
+                                                       + "\nBut expected: "
+                                                       + remoteStoreDef);
                         }
                     }