corrected implementation

Author: ramitg254

ql/src/java/org/apache/hadoop/hive/ql/cache/results/QueryResultsCache.java

@@ -672,14 +672,6 @@ private boolean rewriteFetchWorkForSafeCacheWrite(CacheEntry cacheEntry, FetchWo
     return true;
   }
 
-  /**
-   * Copies each file from the fetch work into {@code resultDir}, preserving relative paths below
-   * the safe-write prefix. Runs under {@link #rwLock} write lock so cache metadata and filesystem
-   * layout stay consistent with other cache mutations.
-   *
-   * @param destFileStatuses output: destination {@link FileStatus} entries for successfully copied files
-   * @return {@code true} if every file was copied and listed; {@code false} otherwise
-   */
   private boolean copyFetchWorkFilesIntoCacheDirUnderWriteLock(FetchWork fetchWork, Path resultDir,
       FileSystem cacheFs, int safeDirAndSepLen, HiveConf queryConf, Set<FileStatus> destFileStatuses) {
     final boolean[] succeeded = { true };
@@ -698,6 +690,7 @@ private boolean copyFetchWorkFilesIntoCacheDirUnderWriteLock(FetchWork fetchWork
         }
       } catch (IOException e) {
         LOG.warn("Failed to write cache entry to {}", resultDir, e);
+        succeeded[0] = false;
       }
     });
     return succeeded[0];