Repository: oozie
Updated Branches:
  refs/heads/branch-4.3 c2d4a14a2 -> 3db0cf4c3


OOZIE-3127 Remove redundant check for user (satishsaley)

(cherry picked from commit 417894d6abe196e3ab172fb4fb5708d938f63c71)


Project: http://git-wip-us.apache.org/repos/asf/oozie/repo
Commit: http://git-wip-us.apache.org/repos/asf/oozie/commit/3db0cf4c
Tree: http://git-wip-us.apache.org/repos/asf/oozie/tree/3db0cf4c
Diff: http://git-wip-us.apache.org/repos/asf/oozie/diff/3db0cf4c

Branch: refs/heads/branch-4.3
Commit: 3db0cf4c31b65f6d8c983019bc30af470adb10a6
Parents: c2d4a14
Author: satishsaley <satishsa...@apache.org>
Authored: Fri Dec 8 17:22:15 2017 -0800
Committer: satishsaley <satishsa...@apache.org>
Committed: Fri Dec 8 17:23:27 2017 -0800

----------------------------------------------------------------------
 .../main/java/org/apache/oozie/servlet/BaseJobsServlet.java   | 7 -------
 release-log.txt                                               | 1 +
 2 files changed, 1 insertion(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/oozie/blob/3db0cf4c/core/src/main/java/org/apache/oozie/servlet/BaseJobsServlet.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/servlet/BaseJobsServlet.java 
b/core/src/main/java/org/apache/oozie/servlet/BaseJobsServlet.java
index 8dfd160..41abeb1 100644
--- a/core/src/main/java/org/apache/oozie/servlet/BaseJobsServlet.java
+++ b/core/src/main/java/org/apache/oozie/servlet/BaseJobsServlet.java
@@ -91,7 +91,6 @@ public abstract class BaseJobsServlet extends JsonRestServlet 
{
         conf = conf.trim();
         conf = conf.resolve();
 
-        validateJobConfiguration(conf);
         String requestUser = getUser(request);
         if (!requestUser.equals(UNDEF)) {
             conf.set(OozieClient.USER_NAME, requestUser);
@@ -249,10 +248,4 @@ public abstract class BaseJobsServlet extends 
JsonRestServlet {
     abstract JSONObject getJobs(HttpServletRequest request)
     throws XServletException, IOException;
 
-    static void validateJobConfiguration(Configuration conf) throws 
XServletException {
-        if (conf.get(OozieClient.USER_NAME) == null) {
-            throw new XServletException(HttpServletResponse.SC_BAD_REQUEST, 
ErrorCode.E0401,
-                    OozieClient.USER_NAME);
-        }
-    }
 }

http://git-wip-us.apache.org/repos/asf/oozie/blob/3db0cf4c/release-log.txt
----------------------------------------------------------------------
diff --git a/release-log.txt b/release-log.txt
index 729d27c..455f527 100644
--- a/release-log.txt
+++ b/release-log.txt
@@ -1,5 +1,6 @@
 -- Oozie 4.3.1 release
 
+OOZIE-3127 Remove redundant check for user (satishsaley)
 OOZIE-2885 Running Spark actions should not need Hive on the classpath 
(satishsaley)
 OOZIE-3058 nocleanup option is missing in oozie-coordinator-0.5.xsd 
(satishsaley)
 OOZIE-3036 Spark 2.2.0 support: tell Spark not to get any delegation tokens 
(andras.piros via gezapeti)

Reply via email to