You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by vj...@apache.org on 2021/05/29 10:04:58 UTC
[phoenix] branch 5.1 updated: PHOENIX-6482 Fix
PherfMainIT#testPherfMain failure and provide better logging in Pherf
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 <vj...@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);
}
}