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

Hudson commented on SQOOP-2386:
-------------------------------

SUCCESS: Integrated in Sqoop2-hadoop200 #859 (See 
[https://builds.apache.org/job/Sqoop2-hadoop200/859/])
SQOOP-2386: Sqoop2: refactor code in HandlerUtils (abe: 
https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=99698deba9f942244fbb416093f77b4b20c09f5e)
* server/src/main/java/org/apache/sqoop/handler/HandlerUtils.java


> Sqoop2: refactor code in HandlerUtils
> -------------------------------------
>
>                 Key: SQOOP-2386
>                 URL: https://issues.apache.org/jira/browse/SQOOP-2386
>             Project: Sqoop
>          Issue Type: Bug
>            Reporter: Dian Fu
>            Assignee: Dian Fu
>             Fix For: 1.99.7
>
>         Attachments: SQOOP-2386.1.patch, SQOOP-2386.2.patch
>
>
> In {{HandlerUtils}}, there is code like this:
> {code}
>  if (repository.findJob(identifier) != null) {
>       jobId = repository.findJob(identifier).getPersistenceId();
>       ...
>  }
> {code}
> This will execute {{repository.findJob}} twice, this is unnecessary.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to