zabetak commented on a change in pull request #2726:
URL: https://github.com/apache/hive/pull/2726#discussion_r731772394
##########
File path:
itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CoreCliDriver.java
##########
@@ -67,91 +67,60 @@ public void beforeClass() {
String cleanupScript = cliConfig.getCleanupScript();
Set<QTestExternalDB> externalDBs = cliConfig.getExternalDBs();
- try {
- qt = new ElapsedTimeLoggingWrapper<QTestUtil>() {
- @Override
- public QTestUtil invokeInternal() throws Exception {
- return new QTestUtil(
- QTestArguments.QTestArgumentsBuilder.instance()
- .withOutDir(cliConfig.getResultsDir())
- .withLogDir(cliConfig.getLogDir())
- .withClusterType(miniMR)
- .withConfDir(hiveConfDir)
- .withInitScript(initScript)
- .withCleanupScript(cleanupScript)
- .withExternalDBs(externalDBs)
- .withLlapIo(true)
- .withFsType(cliConfig.getFsType())
- .build());
- }
- }.invoke("QtestUtil instance created", LOG, true);
- } catch (Exception e) {
- System.err.println("Exception: " + e.getMessage());
- e.printStackTrace();
- System.err.flush();
- throw new RuntimeException("Unexpected exception in static
initialization", e);
- }
+ qt = new ElapsedTimeLoggingWrapper<QTestUtil>() {
+ @Override
+ public QTestUtil invokeInternal() throws Exception {
+ return new QTestUtil(
+ QTestArguments.QTestArgumentsBuilder.instance()
+ .withOutDir(cliConfig.getResultsDir())
+ .withLogDir(cliConfig.getLogDir())
+ .withClusterType(miniMR)
+ .withConfDir(hiveConfDir)
+ .withInitScript(initScript)
+ .withCleanupScript(cleanupScript)
+ .withExternalDBs(externalDBs)
+ .withLlapIo(true)
+ .withFsType(cliConfig.getFsType())
+ .build());
+ }
+ }.invoke("QtestUtil instance created", LOG, true);
}
@Override
@Before
- public void setUp() {
- try {
- new ElapsedTimeLoggingWrapper<Void>() {
- @Override
- public Void invokeInternal() throws Exception {
- qt.newSession();
- return null;
- }
- }.invoke("PerTestSetup done.", LOG, false);
-
- } catch (Exception e) {
- System.err.println("Exception: " + e.getMessage());
- e.printStackTrace();
- System.err.flush();
- fail("Unexpected exception in setup");
- }
+ public void setUp() throws Exception {
+ new ElapsedTimeLoggingWrapper<Void>() {
+ @Override
+ public Void invokeInternal() throws Exception {
+ qt.newSession();
+ return null;
+ }
+ }.invoke("PerTestSetup done.", LOG, false);
}
@Override
@After
- public void tearDown() {
- try {
- new ElapsedTimeLoggingWrapper<Void>() {
- @Override
- public Void invokeInternal() throws Exception {
- qt.clearPostTestEffects();
- qt.clearTestSideEffects();
- return null;
- }
- }.invoke("PerTestTearDown done.", LOG, false);
-
- } catch (Exception e) {
- System.err.println("Exception: " + e.getMessage());
- e.printStackTrace();
- System.err.flush();
- fail("Unexpected exception in tearDown");
- }
+ public void tearDown() throws Exception {
+ new ElapsedTimeLoggingWrapper<Void>() {
Review comment:
I totally agree. I will log a separate JIRA again for keeping things
more self-contained and giving a heads-up to others who may have a different
opinion.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
