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

Veena Basavaraj edited comment on SQOOP-1870 at 12/24/14 12:14 AM:
-------------------------------------------------------------------

2 things I noticed

[~abec]

1. package org.apache.sqoop.job.Constants. java has come constants that are not 
really used, May be we can nuke this? 

2. Do we still want to keep the package as package org.apache.sqoop.job.etl; ? 
or change it to connector.etl or something more relevant since it is NOW in 
connector-sdk


was (Author: vybs):
2 things I noticed

[~abec]

1. package org.apache.sqoop.job.Constants. java has come constants that are not 
really used, May be we can nuke this? 

2. Do we still want to keep the package as job.etl ? or change it to 
connector.etl or something more relevant since it is NOW in connector-sdk

> IDF API and impementation package depednency
> --------------------------------------------
>
>                 Key: SQOOP-1870
>                 URL: https://issues.apache.org/jira/browse/SQOOP-1870
>             Project: Sqoop
>          Issue Type: Sub-task
>          Components: sqoop2-framework
>            Reporter: Veena Basavaraj
>            Assignee: Veena Basavaraj
>             Fix For: 1.99.5
>
>
> Currently the IDF API is in connector SDK package, 
> The right place for IDF API is sqoop-spi package since custom implementations 
> of the IDF can be done in future.
> Instead today sqoop SPI that holds connectors depends on SDK which in fact 
> should be the other way round, the connector API is the root and should not 
> depend on its implemenation package.
> connector SDK was intended to hold concrete implementations and some extent 
> utility classes that  the connectors use, there is obvioulsy some leaks and 
> execution engine even depends on connecto-rsdk



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

Reply via email to