Repository: oozie
Updated Branches:
  refs/heads/master 9b6f0c91e -> 9dee9d572


OOZIE-2960 Fix warnings for @return tag has no argument (Artem Ervits via 
gezapeti)


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

Branch: refs/heads/master
Commit: 9dee9d572ac6058b1730962352c1f1a63e1c2114
Parents: 9b6f0c9
Author: Gezapeti Cseh <gezap...@gmail.com>
Authored: Tue Aug 1 10:41:45 2017 +0200
Committer: Gezapeti Cseh <gezap...@gmail.com>
Committed: Tue Aug 1 10:41:45 2017 +0200

----------------------------------------------------------------------
 client/src/main/java/org/apache/oozie/cli/OozieCLI.java |  2 +-
 .../java/org/apache/oozie/CoordinatorActionBean.java    |  2 +-
 .../main/java/org/apache/oozie/CoordinatorEngine.java   |  9 +++++----
 core/src/main/java/org/apache/oozie/DagEngine.java      |  8 ++++----
 .../main/java/org/apache/oozie/WorkflowActionBean.java  |  6 +++---
 .../apache/oozie/action/hadoop/FsActionExecutor.java    |  2 +-
 .../org/apache/oozie/action/hadoop/LauncherHelper.java  |  2 +-
 .../org/apache/oozie/action/ssh/SshActionExecutor.java  |  2 +-
 .../apache/oozie/command/coord/CoordCommandUtils.java   |  2 +-
 .../coord/CoordMaterializeTransitionXCommand.java       |  2 +-
 .../command/coord/CoordPushDependencyCheckXCommand.java |  2 +-
 .../apache/oozie/command/wf/ActionStartXCommand.java    |  2 +-
 .../java/org/apache/oozie/coord/CoordELEvaluator.java   |  6 +++---
 .../main/java/org/apache/oozie/coord/CoordUtils.java    |  4 ++--
 .../apache/oozie/event/messaging/MessageFactory.java    |  8 ++++----
 .../apache/oozie/event/messaging/MessageSerializer.java |  4 ++--
 .../java/org/apache/oozie/jms/ConnectionContext.java    | 12 ++++++------
 .../main/java/org/apache/oozie/service/ELService.java   |  5 ++---
 .../org/apache/oozie/service/HadoopAccessorService.java |  2 +-
 .../java/org/apache/oozie/service/JMSTopicService.java  |  2 +-
 .../main/java/org/apache/oozie/service/Services.java    |  2 +-
 .../main/java/org/apache/oozie/service/UUIDService.java |  2 +-
 .../org/apache/oozie/service/WorkflowAppService.java    |  2 +-
 .../org/apache/oozie/service/XLogStreamingService.java  |  1 -
 .../apache/oozie/service/ZKJobsConcurrencyService.java  |  8 ++++----
 .../java/org/apache/oozie/servlet/BaseJobServlet.java   |  6 +++---
 .../java/org/apache/oozie/servlet/V0JobsServlet.java    |  3 ---
 .../java/org/apache/oozie/servlet/V1JobsServlet.java    |  6 +++---
 .../main/java/org/apache/oozie/sla/SLACalculator.java   | 10 +++++-----
 .../java/org/apache/oozie/sla/SLACalculatorMemory.java  |  4 +++-
 .../main/java/org/apache/oozie/store/OozieSchema.java   |  6 +++---
 .../java/org/apache/oozie/util/JaasConfiguration.java   |  4 ++--
 .../main/java/org/apache/oozie/util/MappingRule.java    |  6 +++---
 .../main/java/org/apache/oozie/util/StatusUtils.java    |  2 +-
 .../src/main/java/org/apache/oozie/util/XLogFilter.java |  5 ++---
 core/src/main/java/org/apache/oozie/util/db/Schema.java |  6 +++---
 .../java/org/apache/oozie/util/db/SqlStatement.java     |  4 ++--
 .../apache/oozie/workflow/lite/DBLiteWorkflowLib.java   |  2 +-
 release-log.txt                                         |  1 +
 39 files changed, 81 insertions(+), 83 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/client/src/main/java/org/apache/oozie/cli/OozieCLI.java
----------------------------------------------------------------------
diff --git a/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 
b/client/src/main/java/org/apache/oozie/cli/OozieCLI.java
index a7983f0..a545754 100644
--- a/client/src/main/java/org/apache/oozie/cli/OozieCLI.java
+++ b/client/src/main/java/org/apache/oozie/cli/OozieCLI.java
@@ -862,7 +862,7 @@ public class OozieCLI {
      * Check if configuration has specified
      * @param wc
      * @param commandLine
-     * @return
+     * @return isConf
      * @throws IOException
      */
     private boolean isConfigurationSpecified(OozieClient wc, CommandLine 
commandLine) throws IOException {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 
b/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java
index b1be7c9..ddec1e6 100644
--- a/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java
+++ b/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java
@@ -370,7 +370,7 @@ public class CoordinatorActionBean implements
 
     /**
      * Return the status in string
-     * @return
+     * @return statusStr
      */
     public String getStatusStr() {
         return statusStr;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/CoordinatorEngine.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/CoordinatorEngine.java 
b/core/src/main/java/org/apache/oozie/CoordinatorEngine.java
index 1eacc92..89fda05 100644
--- a/core/src/main/java/org/apache/oozie/CoordinatorEngine.java
+++ b/core/src/main/java/org/apache/oozie/CoordinatorEngine.java
@@ -912,7 +912,7 @@ public class CoordinatorEngine extends BaseEngine {
      * @param filter Filter for jobs that will be suspended, can be name, 
user, group, status, id or combination of any
      * @param start Offset for the jobs that will be suspended
      * @param length maximum number of jobs that will be suspended
-     * @return
+     * @return coordinatorJobInfo
      * @throws CoordinatorEngineException
      */
     public CoordinatorJobInfo suspendJobs(String filter, int start, int 
length) throws CoordinatorEngineException {
@@ -935,7 +935,7 @@ public class CoordinatorEngine extends BaseEngine {
      * @param filter Filter for jobs that will be resumed, can be name, user, 
group, status, id or combination of any
      * @param start Offset for the jobs that will be resumed
      * @param length maximum number of jobs that will be resumed
-     * @return
+     * @return coordinatorJobInfo returns resumed jobs
      * @throws CoordinatorEngineException
      */
     public CoordinatorJobInfo resumeJobs(String filter, int start, int length) 
throws CoordinatorEngineException {
@@ -957,7 +957,8 @@ public class CoordinatorEngine extends BaseEngine {
      * @param id jobID
      * @param actions action list
      * @param dates nominal time list
-     * @return pair of coord action bean and list of missing input 
dependencies.
+     * @return CoordActionMissingDependenciesXCommand pair of coord action 
bean and
+     * list of missing input dependencies.
      * @throws CommandException
      */
     public List<Pair<CoordinatorActionBean, Map<String, ActionDependency>>> 
getCoordActionMissingDependencies(String id,
@@ -971,7 +972,7 @@ public class CoordinatorEngine extends BaseEngine {
      * @param wfActionName workflow action name
      * @param offset
      * @param len
-     * @return list of CoordinatorWfActionBean in a coordinator
+     * @return CoordWfActionInfoXCommand list of CoordinatorWfActionBean in a 
coordinator
      * @throws CoordinatorEngineException
      */
      public List<CoordinatorWfActionBean> getWfActionByJobIdAndName(String 
jobId, String wfActionName, int offset, int len)

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/DagEngine.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/DagEngine.java 
b/core/src/main/java/org/apache/oozie/DagEngine.java
index 36198b5..01cf449 100644
--- a/core/src/main/java/org/apache/oozie/DagEngine.java
+++ b/core/src/main/java/org/apache/oozie/DagEngine.java
@@ -126,7 +126,7 @@ public class DagEngine extends BaseEngine {
      * Submit a workflow through a coordinator. It validates configuration 
properties.
      * @param conf job conf
      * @param parentId parent of workflow
-     * @return
+     * @return jobId returns jobId of the launched workflow
      * @throws DagEngineException
      */
     public String submitJobFromCoordinator(Configuration conf, String 
parentId) throws DagEngineException {
@@ -586,7 +586,7 @@ public class DagEngine extends BaseEngine {
      * @param filter Jobs that satisfy the filter will be killed
      * @param start start index in the database of jobs
      * @param len maximum number of jobs that will be killed
-     * @return
+     * @return workflowsInfo return the jobs that've been killed
      * @throws DagEngineException
      */
     public WorkflowsInfo killJobs(String filter, int start, int len) throws 
DagEngineException {
@@ -608,7 +608,7 @@ public class DagEngine extends BaseEngine {
      * @param filter Filter for jobs that will be suspended, can be name, 
user, group, status, id or combination of any
      * @param start Offset for the jobs that will be suspended
      * @param len maximum number of jobs that will be suspended
-     * @return
+     * @return workflowsInfo return the jobs that've been suspended
      * @throws DagEngineException
      */
     public WorkflowsInfo suspendJobs(String filter, int start, int len) throws 
DagEngineException {
@@ -630,7 +630,7 @@ public class DagEngine extends BaseEngine {
      * @param filter Filter for jobs that will be resumed, can be name, user, 
group, status, id or combination of any
      * @param start Offset for the jobs that will be resumed
      * @param len maximum number of jobs that will be resumed
-     * @return
+     * @return workflowsInfo returns the jobs that've been resumed
      * @throws DagEngineException
      */
     public WorkflowsInfo resumeJobs(String filter, int start, int len) throws 
DagEngineException {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/WorkflowActionBean.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/WorkflowActionBean.java 
b/core/src/main/java/org/apache/oozie/WorkflowActionBean.java
index 974c52b..60f3b64 100644
--- a/core/src/main/java/org/apache/oozie/WorkflowActionBean.java
+++ b/core/src/main/java/org/apache/oozie/WorkflowActionBean.java
@@ -345,7 +345,7 @@ public class WorkflowActionBean implements Writable, 
WorkflowAction, JsonBean {
     /**
      * Return whether workflow action in terminal state or not
      *
-     * @return
+     * @return isTerminalState Return whether workflow action in terminal 
state or not
      */
     public boolean inTerminalState() {
         boolean isTerminalState = false;
@@ -581,7 +581,7 @@ public class WorkflowActionBean implements Writable, 
WorkflowAction, JsonBean {
     /**
      * Get external ChildIds
      *
-     * @return
+     * @return externalChildIDs Get external ChildIds
      */
     public StringBlob getExternalChildIDsBlob() {
         return externalChildIDs;
@@ -682,7 +682,7 @@ public class WorkflowActionBean implements Writable, 
WorkflowAction, JsonBean {
     /**
      * Get status
      *
-     * @return
+     * @return statusStr Get status
      */
     public String getStatusStr() {
         return statusStr;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java 
b/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java
index c2a6192..bf022d5 100644
--- a/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java
+++ b/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java
@@ -649,7 +649,7 @@ public class FsActionExecutor extends ActionExecutor {
 
     /**
      * @param context
-     * @return
+     * @return Path returns recovery path
      * @throws HadoopAccessorException
      * @throws IOException
      * @throws URISyntaxException

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/action/hadoop/LauncherHelper.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/action/hadoop/LauncherHelper.java 
b/core/src/main/java/org/apache/oozie/action/hadoop/LauncherHelper.java
index f9e0aef..21c9b7e 100644
--- a/core/src/main/java/org/apache/oozie/action/hadoop/LauncherHelper.java
+++ b/core/src/main/java/org/apache/oozie/action/hadoop/LauncherHelper.java
@@ -221,7 +221,7 @@ public class LauncherHelper {
     /**
      * Get the sequence file path storing all action data
      * @param actionDir
-     * @return
+     * @return Path returns sequence file path storing all action data
      */
     public static Path getActionDataSequenceFilePath(Path actionDir) {
         return new Path(actionDir, LauncherAMUtils.ACTION_DATA_SEQUENCE_FILE);

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/action/ssh/SshActionExecutor.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/action/ssh/SshActionExecutor.java 
b/core/src/main/java/org/apache/oozie/action/ssh/SshActionExecutor.java
index 5fd30f7..5097f79 100644
--- a/core/src/main/java/org/apache/oozie/action/ssh/SshActionExecutor.java
+++ b/core/src/main/java/org/apache/oozie/action/ssh/SshActionExecutor.java
@@ -720,7 +720,7 @@ public class SshActionExecutor extends ActionExecutor {
      * @param bytesRead The number of bytes read from this stream to date.
      * @param readAll If true, the stream is drained while their is data 
available in it. Otherwise, only a single chunk
      * of data is read, irrespective of how much is available.
-     * @return
+     * @return bReadSession returns drainBuffer for stream of contents
      * @throws IOException
      */
     private int drainBuffer(BufferedReader br, StringBuffer storageBuf, int 
maxLength, int bytesRead, boolean readAll)

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java 
b/core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java
index d0995a0..fbc6152 100644
--- a/core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java
+++ b/core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java
@@ -596,7 +596,7 @@ public class CoordCommandUtils {
     /**
      * @param eAction the actionXml related element
      * @param actionBean the coordinator action bean
-     * @return
+     * @return actionXml returns actionXml as String
      * @throws Exception
      */
     static String dryRunCoord(Element eAction, CoordinatorActionBean 
actionBean) throws Exception {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java
 
b/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java
index f6c1782..91d3508 100644
--- 
a/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java
+++ 
b/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java
@@ -234,7 +234,7 @@ public class CoordMaterializeTransitionXCommand extends 
MaterializeTransitionXCo
      * case it returns now (to materialize all actions in the past)
      *
      * @param currentMatTime
-     * @return
+     * @return Date returns materialization for window for catch-up jobs
      * @throws CommandException
      * @throws JDOMException
      */

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java
 
b/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java
index 6d8aa0f..0ad3970 100644
--- 
a/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java
+++ 
b/core/src/main/java/org/apache/oozie/command/coord/CoordPushDependencyCheckXCommand.java
@@ -202,7 +202,7 @@ public class CoordPushDependencyCheckXCommand extends 
CoordinatorXCommand<Void>
 
     /**
      * Return the re-queue interval for coord push dependency check
-     * @return
+     * @return requeueInterval returns the requeue interval for coord push 
dependency check
      */
     public long getCoordPushCheckRequeueInterval() {
         long requeueInterval = 
ConfigurationService.getLong(CONF_COORD_PUSH_CHECK_REQUEUE_INTERVAL);

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java 
b/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java
index 5bc0eab..71c29cd 100644
--- a/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java
+++ b/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java
@@ -353,7 +353,7 @@ public class ActionStartXCommand extends 
ActionXCommand<org.apache.oozie.command
      * Get action executor context
      * @param isRetry
      * @param isUserRetry
-     * @return
+     * @return ActionExecutorContext returns action executor context
      */
     protected ActionExecutorContext getContext(boolean isRetry, boolean 
isUserRetry) {
         return new ActionXCommand.ActionExecutorContext(wfJob, wfAction, 
isRetry, isUserRetry);

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/coord/CoordELEvaluator.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/coord/CoordELEvaluator.java 
b/core/src/main/java/org/apache/oozie/coord/CoordELEvaluator.java
index 809c026..4230a20 100644
--- a/core/src/main/java/org/apache/oozie/coord/CoordELEvaluator.java
+++ b/core/src/main/java/org/apache/oozie/coord/CoordELEvaluator.java
@@ -120,7 +120,7 @@ public class CoordELEvaluator {
      * @param eAction
      * @param coordAction
      * @param conf
-     * @return
+     * @return eval returns SLA evaluator to be used during Materialization
      * @throws Exception
      */
     public static ELEvaluator createSLAEvaluator(Element eAction, 
CoordinatorActionBean coordAction, Configuration conf)
@@ -156,7 +156,7 @@ public class CoordELEvaluator {
      * @param conf
      * @param group
      * @param dataNameList
-     * @return
+     * @return eval returns an Evaluator using conf and input/output-data 
(used for sla)
      * @throws Exception
      */
     public static ELEvaluator createELEvaluatorForDataAndConf(Configuration 
conf, String group,
@@ -269,7 +269,7 @@ public class CoordELEvaluator {
      * Create Dataset object using the Dataset XML information
      *
      * @param eData
-     * @return
+     * @return ds returns Dataset object using the Dataset XML information
      * @throws Exception
      */
     private static SyncCoordDataset getDSObject(Element eData) throws 
Exception {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/coord/CoordUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/coord/CoordUtils.java 
b/core/src/main/java/org/apache/oozie/coord/CoordUtils.java
index 5425d86..2874bf6 100644
--- a/core/src/main/java/org/apache/oozie/coord/CoordUtils.java
+++ b/core/src/main/java/org/apache/oozie/coord/CoordUtils.java
@@ -280,7 +280,7 @@ public class CoordUtils {
       * @param actionBean
       * @param coordName
       * @param jobConf
-      * @return
+      * @return true if SLA alert is disabled for action
       * @throws ParseException
       */
     public static boolean isSlaAlertDisabled(CoordinatorActionBean actionBean, 
String coordName, Configuration jobConf)
@@ -332,7 +332,7 @@ public class CoordUtils {
      * @param coordName
      * @param jobConf
      * @param slaAlertType
-     * @return
+     * @return status of coord action SLA alert
      * @throws ParseException
      */
     private static boolean getCoordActionSLAAlertStatus(CoordinatorActionBean 
actionBean, String coordName,

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/event/messaging/MessageFactory.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/event/messaging/MessageFactory.java 
b/core/src/main/java/org/apache/oozie/event/messaging/MessageFactory.java
index 09b110e..9d828c3 100644
--- a/core/src/main/java/org/apache/oozie/event/messaging/MessageFactory.java
+++ b/core/src/main/java/org/apache/oozie/event/messaging/MessageFactory.java
@@ -56,7 +56,7 @@ public class MessageFactory {
     /**
      * Gets the configured serializer
      *
-     * @return
+     * @return MessageSerializer Gets the configured serializer
      */
     public static MessageSerializer getMessageSerializer() {
         return MessageSerializerHolder.INSTANCE;
@@ -66,7 +66,7 @@ public class MessageFactory {
      * Constructs and returns the workflow job message for workflow job event
      *
      * @param wfJobEvent the workflow job event
-     * @return
+     * @return wfJobMessage returns the workflow job message for the workflow 
job event
      */
     public static WorkflowJobMessage createWorkflowJobMessage(WorkflowJobEvent 
wfJobEvent) {
         WorkflowJobMessage wfJobMessage = new 
WorkflowJobMessage(wfJobEvent.getEventStatus(), wfJobEvent.getId(),
@@ -80,7 +80,7 @@ public class MessageFactory {
      * action event
      *
      * @param coordActionEvent the coordinator action event
-     * @return
+     * @return coordActionMessage returns the coordinator action message for 
coordinator
      */
     public static CoordinatorActionMessage 
createCoordinatorActionMessage(CoordinatorActionEvent coordActionEvent) {
         CoordinatorActionMessage coordActionMessage = new 
CoordinatorActionMessage(coordActionEvent.getEventStatus(),
@@ -94,7 +94,7 @@ public class MessageFactory {
     /**
      * Constructs and returns SLA notification message
      * @param event SLA event
-     * @return
+     * @return slaMessage returns SLA notification message
      */
     public static SLAMessage createSLAMessage(SLAEvent event) {
         SLAMessage slaMessage = new SLAMessage(event.getEventStatus(), 
event.getSLAStatus(), event.getAppType(),

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/event/messaging/MessageSerializer.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/event/messaging/MessageSerializer.java 
b/core/src/main/java/org/apache/oozie/event/messaging/MessageSerializer.java
index f480ddb..a3798b8 100644
--- a/core/src/main/java/org/apache/oozie/event/messaging/MessageSerializer.java
+++ b/core/src/main/java/org/apache/oozie/event/messaging/MessageSerializer.java
@@ -30,13 +30,13 @@ public abstract class MessageSerializer {
     /**
      * Get the serialized string from object
      * @param object the java object
-     * @return
+     * @return returns serialized string from object
      */
     public abstract String getSerializedObject(Object object);
 
     /**
      * Get the message format for the serializer
-     * @return
+     * @return returns message format for the serializer
      */
     public abstract String getMessageFormat();
 

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/jms/ConnectionContext.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/jms/ConnectionContext.java 
b/core/src/main/java/org/apache/oozie/jms/ConnectionContext.java
index 6794569..4a76dfb 100644
--- a/core/src/main/java/org/apache/oozie/jms/ConnectionContext.java
+++ b/core/src/main/java/org/apache/oozie/jms/ConnectionContext.java
@@ -50,7 +50,7 @@ public interface ConnectionContext {
     /**
      * Checks whether connection is initialized or not
      *
-     * @return
+     * @return true if connection is initialized
      */
     boolean isConnectionInitialized();
 
@@ -58,7 +58,7 @@ public interface ConnectionContext {
      * Creates session using the specified session opts
      *
      * @param sessionOpts
-     * @return
+     * @return Session returns session using the specified session opts
      * @throws JMSException
      */
     Session createSession(int sessionOpts) throws JMSException;
@@ -68,7 +68,7 @@ public interface ConnectionContext {
      *
      * @param session
      * @param topicName
-     * @return
+     * @return MessageConsumer returns consumer using session and topic name
      * @throws JMSException
      */
     MessageConsumer createConsumer(Session session, String topicName) throws 
JMSException;
@@ -78,7 +78,7 @@ public interface ConnectionContext {
      *
      * @param session
      * @param topicName
-     * @return
+     * @return MessageConsumer returns consumer using session, topic name and 
selector
      * @throws JMSException
      */
     MessageConsumer createConsumer(Session session, String topicName, String 
selector) throws JMSException;
@@ -88,7 +88,7 @@ public interface ConnectionContext {
      *
      * @param session
      * @param topicName
-     * @return
+     * @return MessageProducer returns producer using session and topic name
      * @throws JMSException
      */
     MessageProducer createProducer(Session session, String topicName) throws 
JMSException;
@@ -97,7 +97,7 @@ public interface ConnectionContext {
      * Creates a threadlocal session using session opts
      *
      * @param sessionOpts
-     * @return
+     * @return Session returns a threadlocal session using session opts
      * @throws JMSException
      */
     Session createThreadLocalSession(final int sessionOpts) throws 
JMSException;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/ELService.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/service/ELService.java 
b/core/src/main/java/org/apache/oozie/service/ELService.java
index 4a1a895..5b3ba14 100644
--- a/core/src/main/java/org/apache/oozie/service/ELService.java
+++ b/core/src/main/java/org/apache/oozie/service/ELService.java
@@ -160,9 +160,8 @@ public class ELService implements Service {
      * Return an {@link ELEvaluator} pre-configured with the constants and 
functions for the specific group of
      * EL-functions and variables defined in the configuration. If the group 
name doesn't exist,
      * IllegalArgumentException is thrown
-     *
-     * @param group Name of the group of required EL Evaluator.
-     * @return a preconfigured {@link ELEvaluator}.
+     * @param group: Name of the group of required EL Evaluator.
+     * @return ELEvaluator a preconfigured {@link ELEvaluator}.
      */
     public ELEvaluator createEvaluator(String group) {
         ELEvaluator.Context context = new ELEvaluator.Context();

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java 
b/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java
index 7f1145d..b507c79 100644
--- a/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java
+++ b/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java
@@ -752,7 +752,7 @@ public class HadoopAccessorService implements Service {
      * @param conf The configuration to process
      * @param uri The URI of the remote filesystem (e.g. HDFS)
      * @param dir The directory on the remote filesystem to write the file to
-     * @return
+     * @return localResource
      * @throws IOException A problem occurred writing the file
      * @throws HadoopAccessorException A problem occured with Hadoop
      * @throws URISyntaxException A problem occurred parsing the URI

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/JMSTopicService.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/service/JMSTopicService.java 
b/core/src/main/java/org/apache/oozie/service/JMSTopicService.java
index 35c82c3..a28330b 100644
--- a/core/src/main/java/org/apache/oozie/service/JMSTopicService.java
+++ b/core/src/main/java/org/apache/oozie/service/JMSTopicService.java
@@ -144,7 +144,7 @@ public class JMSTopicService implements Service {
     /**
      * Retrieve topic from Job id
      * @param jobId
-     * @return
+     * @return topicPrefix + topicName Retrieve topic from Job id
      * @throws JPAExecutorException
      */
     public String getTopic(String jobId) throws JPAExecutorException {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/Services.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/service/Services.java 
b/core/src/main/java/org/apache/oozie/service/Services.java
index 7f47f88..1ca3bf5 100644
--- a/core/src/main/java/org/apache/oozie/service/Services.java
+++ b/core/src/main/java/org/apache/oozie/service/Services.java
@@ -148,7 +148,7 @@ public class Services {
     /**
      * Return active system mode. <p> .
      *
-     * @return
+     * @return systemMode returns active system mode
      */
 
     public SYSTEM_MODE getSystemMode() {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/UUIDService.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/service/UUIDService.java 
b/core/src/main/java/org/apache/oozie/service/UUIDService.java
index 313f9dd..faddd6d 100644
--- a/core/src/main/java/org/apache/oozie/service/UUIDService.java
+++ b/core/src/main/java/org/apache/oozie/service/UUIDService.java
@@ -108,7 +108,7 @@ public class UUIDService implements Service {
     /**
      * Create a unique ID.
      *
-     * @param type: Type of Id. Generally 'C' for Coordinator, 'W' for 
Workflow and 'B' for Bundle.
+     * @param type Type of Id. Generally 'C' for Coordinator, 'W' for Workflow 
and 'B' for Bundle.
      * @return unique ID, id = "${sequence}-${systemId}-[C|W|B]" where,
      * sequence is ${padded_counter}-${startTime} whose length is exactly 7 + 
1 + 15 = 23 characters.
      * systemId is the value defined in the {@link #CONF_SYSTEM_ID} 
configuration property.

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/WorkflowAppService.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/service/WorkflowAppService.java 
b/core/src/main/java/org/apache/oozie/service/WorkflowAppService.java
index 841bcd7..c725f49 100644
--- a/core/src/main/java/org/apache/oozie/service/WorkflowAppService.java
+++ b/core/src/main/java/org/apache/oozie/service/WorkflowAppService.java
@@ -263,7 +263,7 @@ public abstract class WorkflowAppService implements Service 
{
      * Parse workflow definition.
      *
      * @param jobConf
-     * @return
+     * @return WorkflowApp
      * @throws WorkflowException
      */
     public abstract WorkflowApp parseDef(Configuration jobConf) throws 
WorkflowException;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/XLogStreamingService.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/service/XLogStreamingService.java 
b/core/src/main/java/org/apache/oozie/service/XLogStreamingService.java
index 40ba46e..3cfbeac 100644
--- a/core/src/main/java/org/apache/oozie/service/XLogStreamingService.java
+++ b/core/src/main/java/org/apache/oozie/service/XLogStreamingService.java
@@ -70,7 +70,6 @@ public class XLogStreamingService implements Service, 
Instrumentable {
      * Stream the log of a job.
      *
      * @param logStreamer the log streamer
-     * @param filter log streamer filter.
      * @param startTime start time for log events to filter.
      * @param endTime end time for log events to filter.
      * @param writer writer to stream the log to.

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/service/ZKJobsConcurrencyService.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/service/ZKJobsConcurrencyService.java 
b/core/src/main/java/org/apache/oozie/service/ZKJobsConcurrencyService.java
index 93f5bf8..a7a902f 100644
--- a/core/src/main/java/org/apache/oozie/service/ZKJobsConcurrencyService.java
+++ b/core/src/main/java/org/apache/oozie/service/ZKJobsConcurrencyService.java
@@ -134,7 +134,7 @@ public class ZKJobsConcurrencyService extends 
JobsConcurrencyService implements
      * index of this server in ZooKeeper's list of servers is equal to the id 
of the job mod the number of servers.
      *
      * @param ids The list of job ids to check
-     * @return a filtered list of job ids that this server should process
+     * @return filteredIds a filtered list of job ids that this server should 
process
      */
     @Override
     public List<String> getJobIdsForThisServer(List<String> ids) {
@@ -173,7 +173,7 @@ public class ZKJobsConcurrencyService extends 
JobsConcurrencyService implements
      * Return a map of instance id to Oozie server URL.  This implementation 
always returns a map with where the key is the instance
      * id and the value is the URL of each Oozie server that we can see in the 
service discovery in ZooKeeper.
      *
-     * @return A map of Oozie instance ids and URLs
+     * @return urls A map of Oozie instance ids and URLs
      */
     @Override
     public Map<String, String> getServerUrls() {
@@ -193,7 +193,7 @@ public class ZKJobsConcurrencyService extends 
JobsConcurrencyService implements
      * where the key is the instance id and the value is the URL of each Oozie 
server that we can see in the service
      * discovery in ZooKeeper.
      *
-     * @return A map of Oozie instance ids and URLs
+     * @return urls A map of Oozie instance ids and URLs
      */
     @Override
     public Map<String, String> getOtherServerUrls() {
@@ -227,7 +227,7 @@ public class ZKJobsConcurrencyService extends 
JobsConcurrencyService implements
     /**
      * Return if it is running in HA mode
      *
-     * @return
+     * @return true Return whether or not it is running in HA mode
      */
     @Override
     public boolean isHighlyAvailableMode() {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java 
b/core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java
index a9ea615..b6a896f 100644
--- a/core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java
+++ b/core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java
@@ -420,7 +420,7 @@ public abstract class BaseJobServlet extends 
JsonRestServlet {
      *
      * @param request
      * @param response
-     * @return
+     * @return JSONObject
      * @throws XServletException
      * @throws IOException TODO
      */
@@ -621,7 +621,7 @@ public abstract class BaseJobServlet extends 
JsonRestServlet {
      *
      * @param request the request
      * @param response the response
-     * @return the JSON object
+     * @return JSONObject the JSON object
      * @throws XServletException the x servlet exception
      * @throws IOException Signals that an I/O exception has occurred.
      */
@@ -629,4 +629,4 @@ public abstract class BaseJobServlet extends 
JsonRestServlet {
             throws XServletException, IOException {
         throw new XServletException(HttpServletResponse.SC_BAD_REQUEST, 
ErrorCode.E0302, NOT_SUPPORTED_MESSAGE);
     }
-}
\ No newline at end of file
+}

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/servlet/V0JobsServlet.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/servlet/V0JobsServlet.java 
b/core/src/main/java/org/apache/oozie/servlet/V0JobsServlet.java
index 8c978fb..ff50f3d 100644
--- a/core/src/main/java/org/apache/oozie/servlet/V0JobsServlet.java
+++ b/core/src/main/java/org/apache/oozie/servlet/V0JobsServlet.java
@@ -121,7 +121,6 @@ public class V0JobsServlet extends BaseJobsServlet {
      * service implementation to bulk kill jobs
      * @param request
      * @param response
-     * @return
      * @throws XServletException
      * @throws IOException
      */
@@ -135,7 +134,6 @@ public class V0JobsServlet extends BaseJobsServlet {
      * service implementation to bulk suspend jobs
      * @param request
      * @param response
-     * @return
      * @throws XServletException
      * @throws IOException
      */
@@ -149,7 +147,6 @@ public class V0JobsServlet extends BaseJobsServlet {
      * service implementation to bulk resume jobs
      * @param request
      * @param response
-     * @return
      * @throws XServletException
      * @throws IOException
      */

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/servlet/V1JobsServlet.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/servlet/V1JobsServlet.java 
b/core/src/main/java/org/apache/oozie/servlet/V1JobsServlet.java
index 9473821..c1ca65f 100644
--- a/core/src/main/java/org/apache/oozie/servlet/V1JobsServlet.java
+++ b/core/src/main/java/org/apache/oozie/servlet/V1JobsServlet.java
@@ -434,7 +434,7 @@ public class V1JobsServlet extends BaseJobsServlet {
      * service implementation to bulk kill jobs
      * @param request
      * @param response
-     * @return
+     * @return bulkModifyJobs implementation to bulk kill jobs
      * @throws XServletException
      * @throws IOException
      */
@@ -448,7 +448,7 @@ public class V1JobsServlet extends BaseJobsServlet {
      * service implementation to bulk suspend jobs
      * @param request
      * @param response
-     * @return
+     * @return bulkModifyJobs implementation to bulk suspend jobs
      * @throws XServletException
      * @throws IOException
      */
@@ -462,7 +462,7 @@ public class V1JobsServlet extends BaseJobsServlet {
      * service implementation to bulk resume jobs
      * @param request
      * @param response
-     * @return
+     * @return bulkModifyJobs implementation to bulk resume jobs
      * @throws XServletException
      * @throws IOException
      */

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/sla/SLACalculator.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/sla/SLACalculator.java 
b/core/src/main/java/org/apache/oozie/sla/SLACalculator.java
index 146cdcb..db71865 100644
--- a/core/src/main/java/org/apache/oozie/sla/SLACalculator.java
+++ b/core/src/main/java/org/apache/oozie/sla/SLACalculator.java
@@ -58,7 +58,7 @@ public interface SLACalculator {
      * Enable jobs sla alert.
      *
      * @param jobId the job ids
-     * @return true, if successful
+     * @return true if successful
      * @throws JPAExecutorException the JPA executor exception
      * @throws ServiceException the service exception
      */
@@ -67,7 +67,7 @@ public interface SLACalculator {
     /**
      * Enable sla alert for child jobs.
      * @param parentJobIds the parent job ids
-     * @return
+     * @return true if successful
      * @throws JPAExecutorException
      * @throws ServiceException
      */
@@ -77,7 +77,7 @@ public interface SLACalculator {
      * Disable jobs Sla alert.
      *
      * @param jobId the job ids
-     * @return true, if successful
+     * @return true if successful
      * @throws JPAExecutorException the JPA executor exception
      * @throws ServiceException the service exception
      */
@@ -87,7 +87,7 @@ public interface SLACalculator {
     /**
      * Disable Sla alert for child jobs.
      * @param parentJobIds the parent job ids
-     * @return
+     * @return true if successful
      * @throws JPAExecutorException
      * @throws ServiceException
      */
@@ -99,7 +99,7 @@ public interface SLACalculator {
      * Support definition are sla-should-start, sla-should-end, 
sla-nominal-time and sla-max-duration.
      *
      * @param jobIdsSLAPair the job ids sla pair
-     * @return true, if successful
+     * @return true if successful
      * @throws JPAExecutorException the JPA executor exception
      * @throws ServiceException the service exception
      */

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 
b/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
index 456440a..3c6b50a 100644
--- a/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
+++ b/core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
@@ -174,7 +174,8 @@ public class SLACalculatorMemory implements SLACalculator {
      * Get SLACalcStatus from map if SLARegistration is not null, else create 
a new SLACalcStatus
      * This function deosn't update  slaMap
      * @param jobId
-     * @return
+     * @return SLACalcStatus returns SLACalcStatus from map if SLARegistration 
is not null,
+     * else create a new SLACalcStatus
      * @throws JPAExecutorException
      */
     private SLACalcStatus getOrCreateSLACalcStatus(String jobId) throws 
JPAExecutorException {
@@ -351,6 +352,7 @@ public class SLACalculatorMemory implements SLACalculator {
 
     /**
      * Register a new job into the map for SLA tracking
+     * @return true if successful
      */
     @Override
     public boolean addRegistration(String jobId, SLARegistrationBean reg) 
throws JPAExecutorException {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/store/OozieSchema.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/store/OozieSchema.java 
b/core/src/main/java/org/apache/oozie/store/OozieSchema.java
index d2fdd28..ea785d3 100644
--- a/core/src/main/java/org/apache/oozie/store/OozieSchema.java
+++ b/core/src/main/java/org/apache/oozie/store/OozieSchema.java
@@ -209,7 +209,7 @@ public class OozieSchema {
      * Gets the query that will be used to validate the connection
      *
      * @param dbName
-     * @return
+     * @return String returns the query that will be used to validate the 
connection
      */
     public static String getValidationQuery(String dbName) {
         return "select count(" + OozieColumn.VER_versionNumber.columnName() + 
") from " + dbName + "."
@@ -220,7 +220,7 @@ public class OozieSchema {
      * Generates the Insert statement to insert the OOZIE_VERSION to table
      *
      * @param dbName
-     * @return
+     * @return String returns the Insert statement for the OOZIE_VERSION to a 
table
      */
     public static String generateInsertVersionScript(String dbName) {
         return "INSERT INTO " + dbName + "." + 
OozieTable.VERSION.name().toUpperCase() + "("
@@ -230,7 +230,7 @@ public class OozieSchema {
     /**
      * Gets the Oozie Schema Version
      *
-     * @return
+     * @return String returns the Oozie Schema Version
      */
     public static String getOozieVersion() {
         return OOZIE_VERSION;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/util/JaasConfiguration.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/util/JaasConfiguration.java 
b/core/src/main/java/org/apache/oozie/util/JaasConfiguration.java
index cb9dd17..e0b500f 100644
--- a/core/src/main/java/org/apache/oozie/util/JaasConfiguration.java
+++ b/core/src/main/java/org/apache/oozie/util/JaasConfiguration.java
@@ -56,7 +56,7 @@ public class JaasConfiguration extends Configuration {
      * <p>
      * 
javax.security.auth.login.Configuration.setConfiguration(JaasConfiguration.getInstance());
      *
-     * @return
+     * @return me return the singleton
      */
     public static Configuration getInstance() {
         if (me == null) {
@@ -112,7 +112,7 @@ public class JaasConfiguration extends Configuration {
     /**
      * Returns the entries map.
      *
-     * @return the entries map
+     * @return entries the entries map
      */
     public static Map<String, AppConfigurationEntry> getEntries() {
         return entries;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/util/MappingRule.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/util/MappingRule.java 
b/core/src/main/java/org/apache/oozie/util/MappingRule.java
index 312ec13..c1f615e 100644
--- a/core/src/main/java/org/apache/oozie/util/MappingRule.java
+++ b/core/src/main/java/org/apache/oozie/util/MappingRule.java
@@ -53,7 +53,7 @@ public class MappingRule {
 
     /**
      * Gets the from rule
-     * @return
+     * @return fromString Gets the from rule
      */
     public String getFromRule() {
         return fromString;
@@ -61,7 +61,7 @@ public class MappingRule {
 
     /**
      * Gets the to rule
-     * @return
+     * @return toString Gets the to rule
      */
     public String getToRule() {
         return toString;
@@ -70,7 +70,7 @@ public class MappingRule {
     /**
      * Applies rules based on the input
      * @param input
-     * @return
+     * @return result Applies rules based on the input
      */
     public String applyRule(String input) {
         if (patternMatch) {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/util/StatusUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/util/StatusUtils.java 
b/core/src/main/java/org/apache/oozie/util/StatusUtils.java
index 24eba21..69d7265 100644
--- a/core/src/main/java/org/apache/oozie/util/StatusUtils.java
+++ b/core/src/main/java/org/apache/oozie/util/StatusUtils.java
@@ -149,7 +149,7 @@ public class StatusUtils {
      * Get the status of coordinator job for Oozie versions (3.2 and before) 
when RUNNINGWITHERROR,
      * SUSPENDEDWITHERROR and PAUSEDWITHERROR are not supported
      * @param currentJobStatus
-     * @return
+     * @return newStatus Get the status of coordinator job
      */
     public static Job.Status getStatusIfBackwardSupportTrue(Job.Status 
currentJobStatus) {
         Job.Status newStatus = currentJobStatus;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/util/XLogFilter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/util/XLogFilter.java 
b/core/src/main/java/org/apache/oozie/util/XLogFilter.java
index 5fa3078..1fba33b 100644
--- a/core/src/main/java/org/apache/oozie/util/XLogFilter.java
+++ b/core/src/main/java/org/apache/oozie/util/XLogFilter.java
@@ -120,9 +120,8 @@ public class XLogFilter {
 
     /**
      * Checks if the logLevel and logMessage goes through the logFilter.
-     *
      * @param logParts ArrayList of logParts
-     * @return
+     * @return true if line contains the permitted logLevel
      */
     public boolean splitsMatches(LogLine logLine) {
         // Check whether logLine matched with filter
@@ -149,7 +148,7 @@ public class XLogFilter {
      * Checks if the logLevel and logMessage goes through the logFilter.
      *
      * @param logParts
-     * @return
+     * @return true if the logLevel and logMessage goes through the logFilter
      */
     public boolean matches(ArrayList<String> logParts) {
         if (getStartDate() != null) {

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/util/db/Schema.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/util/db/Schema.java 
b/core/src/main/java/org/apache/oozie/util/db/Schema.java
index 0ba09fc..cb0e24d 100644
--- a/core/src/main/java/org/apache/oozie/util/db/Schema.java
+++ b/core/src/main/java/org/apache/oozie/util/db/Schema.java
@@ -33,7 +33,7 @@ public class Schema {
         /**
          * Name of the Table
          *
-         * @return
+         * @return Name of the Table
          */
         String name();
     }
@@ -73,7 +73,7 @@ public class Schema {
         /**
          * Returns the length of the column
          *
-         * @return
+         * @return Returns the length of the column
          */
         int getLength();
 
@@ -92,7 +92,7 @@ public class Schema {
         /**
          * Column that is to be indexed
          *
-         * @return
+         * @return Column that is to be indexed
          */
         Column column();
     }

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/util/db/SqlStatement.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/oozie/util/db/SqlStatement.java 
b/core/src/main/java/org/apache/oozie/util/db/SqlStatement.java
index 83b455e..e2e0613 100755
--- a/core/src/main/java/org/apache/oozie/util/db/SqlStatement.java
+++ b/core/src/main/java/org/apache/oozie/util/db/SqlStatement.java
@@ -805,7 +805,7 @@ public abstract class SqlStatement {
          *
          * @param column
          * @param value A temporary place holder which will be replaced while 
preparing
-         * @return
+         * @return retVal Set the VALUES that are to be inserted
          */
         public Insert value(Column column, Object value) {
             Insert retVal = new Insert(this);
@@ -946,7 +946,7 @@ public abstract class SqlStatement {
      * Create the Appropriate SQL Statement with the given values
      *
      * @param temp
-     * @return
+     * @return retVal Create the Appropriate SQL Statement with the given 
values
      */
     private SqlStatement create(ArrayList<Object> temp) {
         SqlStatement retVal = null;

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/core/src/main/java/org/apache/oozie/workflow/lite/DBLiteWorkflowLib.java
----------------------------------------------------------------------
diff --git 
a/core/src/main/java/org/apache/oozie/workflow/lite/DBLiteWorkflowLib.java 
b/core/src/main/java/org/apache/oozie/workflow/lite/DBLiteWorkflowLib.java
index 3c41c83..dd6afa0 100644
--- a/core/src/main/java/org/apache/oozie/workflow/lite/DBLiteWorkflowLib.java
+++ b/core/src/main/java/org/apache/oozie/workflow/lite/DBLiteWorkflowLib.java
@@ -67,7 +67,7 @@ public class DBLiteWorkflowLib extends LiteWorkflowLib {
      * Loads the Workflow instance with the given ID.
      *
      * @param id
-     * @return
+     * @return pInstance returns a workflow instance with the given ID
      * @throws WorkflowException
      */
     @Override

http://git-wip-us.apache.org/repos/asf/oozie/blob/9dee9d57/release-log.txt
----------------------------------------------------------------------
diff --git a/release-log.txt b/release-log.txt
index 4d39c34..aa35c0f 100644
--- a/release-log.txt
+++ b/release-log.txt
@@ -1,5 +1,6 @@
 -- Oozie 5.0.0 release (trunk - unreleased)
 
+OOZIE-2960 Fix warnings for @return tag has no argument (Artem Ervits via 
gezapeti)
 OOZIE-3019 Remove getPasswordMethod in ConfigurationService (gezapeti)
 OOZIE-3009 Number of Oozie tests executed dropped after OOZIE-2854 
(andras.piros via asasvari)
 OOZIE-2961 Build contains multiple warnings for is not a parameter name (Artem 
Ervits via gezapeti)

Reply via email to