[phoenix] branch 4.x-HBase-1.2 updated: PHOENIX-5199 Pherf overrides user provided properties like dataloader threadpool, monitor frequency etc with pherf.properties

[karanmehta93]

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

karanmehta93 pushed a commit to branch 4.x-HBase-1.2
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/4.x-HBase-1.2 by this push:
     new 746a8ef  PHOENIX-5199 Pherf overrides user provided properties like 
dataloader threadpool, monitor frequency etc with pherf.properties
746a8ef is described below

commit 746a8efa76dd8884db2873647750aa8a79fb6516
Author: Monani Mihir <mmon...@salesforce.com>
AuthorDate: Fri Mar 15 16:48:12 2019 +0530

    PHOENIX-5199 Pherf overrides user provided properties like dataloader 
threadpool, monitor frequency etc with pherf.properties
---
 phoenix-pherf/src/main/java/org/apache/phoenix/pherf/Pherf.java       | 2 +-
 .../main/java/org/apache/phoenix/pherf/workload/WriteWorkload.java    | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/Pherf.java 
b/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/Pherf.java
index 43061e0..d92ffde 100644
--- a/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/Pherf.java
+++ b/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/Pherf.java
@@ -156,7 +156,7 @@ public class Pherf {
                 writerThreadPoolSize =
                 command.getOptionValue("writerThreadSize",
                         
properties.getProperty("pherf.default.dataloader.threadpool"));
-        properties.setProperty("pherf. default.dataloader.threadpool", 
writerThreadPoolSize);
+        properties.setProperty("pherf.default.dataloader.threadpool", 
writerThreadPoolSize);
         label = command.getOptionValue("label", null);
         compareResults = command.getOptionValue("compare", null);
         compareType = command.hasOption("useAverageCompareType") ? 
CompareType.AVERAGE : CompareType.MINIMUM;
diff --git 
a/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/workload/WriteWorkload.java
 
b/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/workload/WriteWorkload.java
index 019c326..0a05dc8 100644
--- 
a/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/workload/WriteWorkload.java
+++ 
b/phoenix-pherf/src/main/java/org/apache/phoenix/pherf/workload/WriteWorkload.java
@@ -84,8 +84,8 @@ public class WriteWorkload implements Workload {
 
     public WriteWorkload(PhoenixUtil phoenixUtil, XMLConfigParser parser, 
Scenario scenario, GeneratePhoenixStats generateStatistics)
             throws Exception {
-        this(phoenixUtil, 
PherfConstants.create().getProperties(PherfConstants.PHERF_PROPERTIES,
-                false),
+        this(phoenixUtil,
+                
PherfConstants.create().getProperties(PherfConstants.PHERF_PROPERTIES, true),
                 parser, scenario, generateStatistics);
     }