[HIVE-10768] In QTestGenTask.execute() we should not throw an exception right away if we are unable to clean any old files

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.2.1
• Component/s: Testing Infrastructure
• Assignee: Hari Sankar Sivarama Subramaniyan
• Reporter: Hari Sankar Sivarama Subramaniyan
• Created: 20/May/15 07:30
• Updated: 05/Jun/15 20:57
• Resolved: 27/May/15 07:38

Description

Currently in QTestGenTask.execute(), we have code like :

       if (logFile != null) {
         File lf = new File(logFile);
         if (lf.exists()) {
           if (!lf.delete()) {
             throw new Exception("Could not delete log file " + lf.getCanonicalPath());
         }

If the log file gets locked due to some other process, it throws the above exception and short circuits the entire test suite. We should provide a mechanism to generate alternative new log files/temporary files whenever possible to avoid unit test failures.

Issue Links

• links to: Review board