[SQOOP-2659] Sqoop2: Do not stop services that haven't been started in our integration test suite

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.99.6
• Fix Version/s: 1.99.7
• Assignee: Jarek Jarcec Cecho
• Reporter: Jarek Jarcec Cecho
• Created: 04/Nov/15 17:31
• Updated: 09/Nov/15 23:22
• Resolved: 09/Nov/15 22:20

Description

While investigating pre-commit hook failures on SQOOP-2652 I was side tracked at a lot by seeing various exceptions from Hive and Kafka test cases with NullPointerException and spending time investigating them.

Upon further investigation I've noticed that the whole reason is that we're shutting down something that wasn't started and hence the NPE. The problem was that my newly added test case had added an assume statement in @Before* family of methods. The assume statement return false and stopped test execution before the Hive/Kite test cases got initialized. However the testng will execute all tear down methods (@After* and not only those that have been initialized. Hence the tear down methods must be smart enough to not die if the initialization in fact did not happened.

Issue Links

• relates to: SQOOP-2652 Sqoop2: Add test case for S3 incremental import to HDFS