> On April 3, 2015, 2:32 p.m., Andrew Onischuk wrote:
> > ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/configuration/sqoop-env.xml,
> > line 57
> > <https://reviews.apache.org/r/32810/diff/1/?file=914634#file914634line57>
> >
> > We don't have such a property-type. Please remove this it can result in
> > failures later.
Please take a look
public enum PropertyType {
PASSWORD,
USER,
GROUP,
ADDITIONAL_USER_PROPERTY
}
- Andrew
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32810/#review78773
-----------------------------------------------------------
On April 3, 2015, 2:28 p.m., Vitalyi Brodetskyi wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32810/
> -----------------------------------------------------------
>
> (Updated April 3, 2015, 2:28 p.m.)
>
>
> Review request for Ambari, Andrew Onischuk and Dmitro Lisnichenko.
>
>
> Bugs: AMBARI-10346
> https://issues.apache.org/jira/browse/AMBARI-10346
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Configured the Jenkins job to install HDP through Ambari and run tests with
> Oracle DB.
> I see mysql-connector instead of oracle connector in Sqoop lib directory.
> Ojdbc.jar should be in the Sqoop lib directory.
>
>
> Diffs
> -----
>
> ambari-server/src/main/python/ambari-server.py 8b23d83
> ambari-server/src/main/python/ambari_server/dbConfiguration.py 2913e3f
>
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/configuration/sqoop-env.xml
> 9a740be
>
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/params.py
> e12acbe
>
> ambari-server/src/main/resources/common-services/SQOOP/1.4.4.2.0/package/scripts/sqoop.py
> 99ad575
> ambari-server/src/test/python/stacks/2.0.6/SQOOP/test_sqoop.py 1fe98db
> ambari-server/src/test/python/stacks/2.0.6/configs/default_sqoop.json
> PRE-CREATION
>
> Diff: https://reviews.apache.org/r/32810/diff/
>
>
> Testing
> -------
>
> mvn clean test
>
>
> Thanks,
>
> Vitalyi Brodetskyi
>
>
