DBCreator: Fix ScriptRunner to strip comments, reformat code

Signed-off-by: Rohit Yadav <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
Commit: 
http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/ef07cde4
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/ef07cde4
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/ef07cde4

Branch: refs/heads/javelin
Commit: ef07cde449029e8d2255e467a81369ad6b131347
Parents: 6d15541
Author: Rohit Yadav <[email protected]>
Authored: Thu Jan 17 16:08:51 2013 -0800
Committer: Rohit Yadav <[email protected]>
Committed: Thu Jan 17 16:10:07 2013 -0800

----------------------------------------------------------------------
 server/src/com/cloud/upgrade/DatabaseCreator.java |   16 +++++++---------
 utils/src/com/cloud/utils/db/ScriptRunner.java    |    8 ++++++--
 2 files changed, 13 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/ef07cde4/server/src/com/cloud/upgrade/DatabaseCreator.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/upgrade/DatabaseCreator.java 
b/server/src/com/cloud/upgrade/DatabaseCreator.java
index f0a8c5a..4bc2038 100755
--- a/server/src/com/cloud/upgrade/DatabaseCreator.java
+++ b/server/src/com/cloud/upgrade/DatabaseCreator.java
@@ -31,17 +31,15 @@ import com.cloud.utils.component.SystemIntegrityChecker;
 import com.cloud.utils.db.ScriptRunner;
 import com.cloud.utils.db.Transaction;
 
-/**
- * Creates the CloudStack Database by using the 4.0 schema and apply 
- * upgrade steps to it.
- */
+// Creates the CloudStack Database by using the 4.0 schema and apply
+// upgrade steps to it.
 public class DatabaseCreator {
     protected static void printHelp(String cmd) {
         System.out.println(
-                "DatabaseCreator creates the database schema by removing the 
\n" + 
-                        "previous schema, creating the schema, and running \n" 
+
+                "DatabaseCreator creates the database schema by removing the 
\n" +
+                "previous schema, creating the schema, and running \n" +
                 "through the database updaters.");
-        System.out.println("Usage: " + cmd + " [initial schema file] [database 
upgrade class]");
+        System.out.println("Usage: " + cmd + " [schema files] [database 
upgrade class]");
     }
 
     public static void main(String[] args) {
@@ -73,7 +71,7 @@ public class DatabaseCreator {
                     } catch (FileNotFoundException e) {
                         System.err.println("Unable to read " + args[i] + ": " 
+ e.getMessage());
                         System.exit(1);
-                    } 
+                    }
                     try {
                         runner.runScript(reader);
                     } catch (IOException e) {
@@ -111,4 +109,4 @@ public class DatabaseCreator {
             }
         }
     }
-}
\ No newline at end of file
+}

http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/ef07cde4/utils/src/com/cloud/utils/db/ScriptRunner.java
----------------------------------------------------------------------
diff --git a/utils/src/com/cloud/utils/db/ScriptRunner.java 
b/utils/src/com/cloud/utils/db/ScriptRunner.java
index 5a67a89..5690fa1 100644
--- a/utils/src/com/cloud/utils/db/ScriptRunner.java
+++ b/utils/src/com/cloud/utils/db/ScriptRunner.java
@@ -46,7 +46,7 @@ public class ScriptRunner {
 
     private String delimiter = DEFAULT_DELIMITER;
     private boolean fullLineDelimiter = false;
-    
+
     private StringBuffer _logBuffer = new StringBuffer();
 
     /**
@@ -170,7 +170,11 @@ public class ScriptRunner {
                     }
                     Thread.yield();
                 } else {
-                    command.append(line);
+                    int idx = line.indexOf("--");
+                    if (idx != -1)
+                        command.append(line.substring(0, idx));
+                    else
+                        command.append(line);
                     command.append(" ");
                 }
             }

Reply via email to