[ambari] branch branch-2.7 updated: AMBARI-24257. Tasks fail from time to time due error in file download (aonishuk)

[ao...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

aonishuk pushed a commit to branch branch-2.7
in repository https://gitbox.apache.org/repos/asf/ambari.git


The following commit(s) were added to refs/heads/branch-2.7 by this push:
     new 41ff157  AMBARI-24257. Tasks fail from time to time due error in file download (aonishuk)
41ff157 is described below

commit 41ff1575c56e0b1f3a0f45e16ae6eb1b8e39764b
Author: Andrew Onishuk <ao...@hortonworks.com>
AuthorDate: Fri Jul 6 17:41:34 2018 +0300

    AMBARI-24257. Tasks fail from time to time due error in file download (aonishuk)
---
 ambari-agent/src/main/python/ambari_agent/FileCache.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/ambari-agent/src/main/python/ambari_agent/FileCache.py b/ambari-agent/src/main/python/ambari_agent/FileCache.py
index 780a12b..aec0357 100644
--- a/ambari-agent/src/main/python/ambari_agent/FileCache.py
+++ b/ambari-agent/src/main/python/ambari_agent/FileCache.py
@@ -203,8 +203,9 @@ class FileCache():
       else:
         raise # we are not tolerant to exceptions, command execution will fail
     finally:
-      self.currently_providing[full_path].set()
-      del self.currently_providing[full_path]
+      with self.currently_providing_dict_lock:
+        self.currently_providing[full_path].set()
+        del self.currently_providing[full_path]
 
     return full_path