[phoenix] branch 5.1 updated: PHOENIX-6482 Fix PherfMainIT#testPherfMain failure and provide better logging in Pherf

[vjasani]

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

vjasani pushed a commit to branch 5.1
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/5.1 by this push:
     new 28df5af  PHOENIX-6482 Fix PherfMainIT#testPherfMain failure and 
provide better logging in Pherf
28df5af is described below

commit 28df5afe114f040258ff7c82ed0e8e7fb58df340
Author: Viraj Jasani <vjas...@apache.org>
AuthorDate: Sat May 29 15:29:18 2021 +0530

    PHOENIX-6482 Fix PherfMainIT#testPherfMain failure and provide better 
logging in Pherf
---
 phoenix-pherf/src/it/java/org/apache/phoenix/pherf/PherfMainIT.java | 4 ++--
 phoenix-pherf/src/main/java/org/apache/phoenix/pherf/Pherf.java     | 2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/phoenix-pherf/src/it/java/org/apache/phoenix/pherf/PherfMainIT.java 
b/phoenix-pherf/src/it/java/org/apache/phoenix/pherf/PherfMainIT.java
index c71fda5..b9cfbcf 100644
--- a/phoenix-pherf/src/it/java/org/apache/phoenix/pherf/PherfMainIT.java
+++ b/phoenix-pherf/src/it/java/org/apache/phoenix/pherf/PherfMainIT.java
@@ -54,8 +54,8 @@ public class PherfMainIT extends ResultBaseTestIT {
     @Test
     public void testPherfMain() throws Exception {
         String[] args = { "-q", "-l",
-                "--schemaFile", ".*create_prod_test_unsalted.sql",
-                "--scenarioFile", ".*prod_test_unsalted_scenario.xml"};
+                "-schemaFile", ".*create_prod_test_unsalted.sql",
+                "-scenarioFile", ".*prod_test_unsalted_scenario.xml"};
         Pherf pherf = new Pherf(args);
         pherf.run();
 
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 48eb0f3..ef3dc00 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
@@ -140,6 +140,7 @@ public class Pherf {
             command = parser.parse(options, args);
         } catch (ParseException e) {
             hf.printHelp("Pherf", options);
+            LOGGER.error("Something went wrong while parsing.", e);
             System.exit(1);
         }
 
@@ -236,6 +237,7 @@ public class Pherf {
             new Pherf(args).run();
         } catch (Exception e) {
             e.printStackTrace();
+            LOGGER.error("Something went wrong.", e);
             System.exit(1);
         }
     }