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

Hudson commented on SQOOP-2710:
-------------------------------

SUCCESS: Integrated in Sqoop2 #957 (See 
[https://builds.apache.org/job/Sqoop2/957/])
SQOOP-2710. Sqoop2: Move user from context classes to parent (hshreedharan: 
[https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=e2fc4a75e8ca452187e869e140a92d8acad86197])
* common/src/main/java/org/apache/sqoop/job/etl/PartitionerContext.java
* common/src/main/java/org/apache/sqoop/job/etl/InitializerContext.java
* common/src/main/java/org/apache/sqoop/job/etl/TransferableContext.java
* common/src/main/java/org/apache/sqoop/job/etl/ExtractorContext.java
* common/src/main/java/org/apache/sqoop/job/etl/LoaderContext.java
* common/src/main/java/org/apache/sqoop/job/etl/DestroyerContext.java


> Sqoop2: Move user from context classes to parent TransferableContext
> --------------------------------------------------------------------
>
>                 Key: SQOOP-2710
>                 URL: https://issues.apache.org/jira/browse/SQOOP-2710
>             Project: Sqoop
>          Issue Type: Bug
>            Reporter: Jarek Jarcec Cecho
>            Assignee: Jarek Jarcec Cecho
>             Fix For: 1.99.7
>
>         Attachments: SQOOP-2710.patch
>
>
> It seems that every single child of 
> [{{TransferableContext}}|https://github.com/apache/sqoop/blob/sqoop2/common/src/main/java/org/apache/sqoop/job/etl/TransferableContext.java]
>  defines a property {{user}}, so I would suggest to move that property to the 
> parent class rather then copy-paste the same code everywhere.



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

Reply via email to