[geode-benchmarks] branch GEODE-6340 updated: add cli options for metadata

[he...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

heybales pushed a commit to branch GEODE-6340
in repository https://gitbox.apache.org/repos/asf/geode-benchmarks.git


The following commit(s) were added to refs/heads/GEODE-6340 by this push:
     new ff4ecaf  add cli options for metadata
ff4ecaf is described below

commit ff4ecaf28ab1567340f2ce80842cd2cd2f722246
Author: Helena A. Bales <hb...@pivotal.io>
AuthorDate: Thu Jan 31 11:40:12 2019 -0800

    add cli options for metadata
---
 harness/src/main/java/org/apache/geode/perftest/jvms/RemoteJVMs.java   | 1 -
 .../main/java/org/apache/geode/perftest/runner/DefaultTestRunner.java  | 3 +--
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/harness/src/main/java/org/apache/geode/perftest/jvms/RemoteJVMs.java b/harness/src/main/java/org/apache/geode/perftest/jvms/RemoteJVMs.java
index 0609f46..337e6ed 100644
--- a/harness/src/main/java/org/apache/geode/perftest/jvms/RemoteJVMs.java
+++ b/harness/src/main/java/org/apache/geode/perftest/jvms/RemoteJVMs.java
@@ -73,7 +73,6 @@ public class RemoteJVMs implements AutoCloseable {
    * Copy results to the provided output directory
    */
   public void copyResults(File benchmarkOutput) throws IOException {
-//    benchmarkOutput.mkdirs();
     for (JVMMapping jvm : jvmMappings) {
       infra.copyFromNode(jvm.getNode(), jvm.getOutputDir(),
           new File(benchmarkOutput, jvm.getRole() + "-" + jvm.getId()));
diff --git a/harness/src/main/java/org/apache/geode/perftest/runner/DefaultTestRunner.java b/harness/src/main/java/org/apache/geode/perftest/runner/DefaultTestRunner.java
index f104fa8..75d33e2 100644
--- a/harness/src/main/java/org/apache/geode/perftest/runner/DefaultTestRunner.java
+++ b/harness/src/main/java/org/apache/geode/perftest/runner/DefaultTestRunner.java
@@ -73,13 +73,12 @@ public class DefaultTestRunner implements TestRunner {
     String metadata = System.getProperty("TEST_METADATA");
     if(!(metadata == null) && !metadata.isEmpty()) {
       File metadataOutput = new File(outputDir, "metadata.json");
-//      metadataOutput.mkdirs();
       metadataOutput.createNewFile();
       FileWriter metadataWriter = new FileWriter(metadataOutput.getAbsoluteFile(), true);
 
       String[] metadataEntries = metadata.split(",");
       for(String data : metadataEntries) {
-        metadataWriter.write(data);
+        metadataWriter.write(data + "\n");
       }
       metadataWriter.flush();
     }