[ 
https://issues.apache.org/jira/browse/OOZIE-1575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13841916#comment-13841916
 ] 

Robert Kanter commented on OOZIE-1575:
--------------------------------------

{quote}
Implementing an abstract method is not "Override", so I think the amount of 
"@Override" is sufficient. rebased to the trunk.
{quote}

According to the [Override Annotation 
Javadoc|http://docs.oracle.com/javase/6/docs/api/java/lang/Override.html]:
{quote}
Indicates that a method declaration is intended to override a method 
declaration in a superclass. If a method is annotated with this annotation type 
but does not override a superclass method, compilers are required to generate 
an error message.
{quote}
Abstract classes are still superclasses, so I think you should add it.  My IDE 
flags that it should say {{@Override}} too.

> Add functionality to submit sqoop jobs through http on oozie server side
> ------------------------------------------------------------------------
>
>                 Key: OOZIE-1575
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1575
>             Project: Oozie
>          Issue Type: Sub-task
>          Components: client
>            Reporter: Bowen Zhang
>            Assignee: Bowen Zhang
>             Fix For: trunk
>
>         Attachments: oozie-1575.patch, oozie-1575.patch, oozie-1575.patch, 
> oozie-1575.patch, oozie-1575.patch, oozie-1575.patch, oozie-1575.patch, 
> oozie-1575.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to