azkaban-aplcache

Details

diff --git a/azkaban-common/src/main/java/azkaban/project/ProjectManager.java b/azkaban-common/src/main/java/azkaban/project/ProjectManager.java
index 2d0619e..9bcbf05 100644
--- a/azkaban-common/src/main/java/azkaban/project/ProjectManager.java
+++ b/azkaban-common/src/main/java/azkaban/project/ProjectManager.java
@@ -411,21 +411,17 @@ public class ProjectManager {
     // the validators are using different values for the
     // PROJECT_ARCHIVE_FILE_PATH configuration key.
     // In addition, we want to reload the validator objects for each upload, so
-    // that
-    // we can change the validator configuration files without having to restart
-    // Azkaban web server.
-    // If the XmlValidatorManager is an instance variable, 2 consecutive
-    // invocations to the uploadProject
+    // that we can change the validator configuration files without having to
+    // restart Azkaban web server. If the XmlValidatorManager is an instance
+    // variable, 2 consecutive invocations to the uploadProject
     // method might cause the second one to overwrite the
     // PROJECT_ARCHIVE_FILE_PATH configuration parameter
     // of the first, thus causing a wrong archive file path to be passed to the
-    // validators. Creating a
-    // separate XmlValidatorManager object for each upload will prevent this
-    // issue without having to add
+    // validators. Creating a separate XmlValidatorManager object for each
+    // upload will prevent this issue without having to add
     // synchronization between uploads. Since we're already reloading the XML
-    // config file and creating
-    // validator objects for each upload, this does not add too much additional
-    // overhead.
+    // config file and creating validator objects for each upload, this does
+    // not add too much additional overhead.
     ValidatorManager validatorManager = new XmlValidatorManager(prop);
     logger.info("Validating project " + archive.getName()
         + " using the registered validators "
diff --git a/azkaban-webserver/src/main/java/azkaban/webapp/servlet/ProjectManagerServlet.java b/azkaban-webserver/src/main/java/azkaban/webapp/servlet/ProjectManagerServlet.java
index f839a80..b286241 100644
--- a/azkaban-webserver/src/main/java/azkaban/webapp/servlet/ProjectManagerServlet.java
+++ b/azkaban-webserver/src/main/java/azkaban/webapp/servlet/ProjectManagerServlet.java
@@ -87,7 +87,7 @@ public class ProjectManagerServlet extends LoginAbstractAzkabanServlet {
   private static final String LOCKDOWN_CREATE_PROJECTS_KEY =
       "lockdown.create.projects";
 
-  private static final String PROJECT_DOWNLOAD_BUFFER_SIZE =
+  private static final String PROJECT_DOWNLOAD_BUFFER_SIZE_IN_BYTE =
       "project.download.bufffer.size";
 
   private ProjectManager projectManager;
@@ -121,7 +121,8 @@ public class ProjectManagerServlet extends LoginAbstractAzkabanServlet {
     }
 
     downloadBufferSize =
-        server.getServerProps().getInt(PROJECT_DOWNLOAD_BUFFER_SIZE, 8192);
+        server.getServerProps().getInt(PROJECT_DOWNLOAD_BUFFER_SIZE_IN_BYTE,
+            8192);
 
     logger.info("downloadBufferSize: " + downloadBufferSize);
   }
@@ -499,6 +500,9 @@ public class ProjectManagerServlet extends LoginAbstractAzkabanServlet {
       }
 
     } catch (Throwable e) {
+      logger.error(
+          "Encountered error while downloading project zip file for project: "
+              + projectName + " by user: " + user.getUserId(), e);
       throw new ServletException(e);
     } finally {
       IOUtils.closeQuietly(inStream);