-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64371/
-----------------------------------------------------------
Review request for Sqoop, Boglarka Egyed and Szabolcs Vasas.
Bugs: SQOOP-3233
https://issues.apache.org/jira/browse/SQOOP-3233
Repository: sqoop-trunk
Description
-------
- Corrected the type from VARCHAR to CHAR in
SqoopHCatImportHelper.convertNumberTypes, that the ticket description mentions.
- Found another bug in the convertNumberTypes function: missing parentheses in
the if clause that checks whether the type of val is BigDecimal. Corrected it.
- Slight refactoring: extracted the BigDecimal and Number related conversion
code into two different methods, for better readability.
Diffs
-----
src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatImportHelper.java 1c1ed1e5
src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatImportHelper.java
1c1ed1e58c1e1c9a0364e61627ac1ef725bea8f9
src/test/org/apache/sqoop/hcat/HCatalogImportTest.java 4686493c
src/test/org/apache/sqoop/hcat/HCatalogImportTest.java
4686493c79498d9ca45db227904049736b5ea493
Diff: https://reviews.apache.org/r/64371/diff/1/
Testing
-------
Ran unit tests (ant clean test), and third party tests as well successfully.
Thanks,
Fero Szabo
