Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Sqoop, mail # dev - Review Request: SQOOP-484: Update troubleshooting guide section about MySQL and TINYINT(1) columns


Copy link to this message
-
Re: Review Request: SQOOP-484: Update troubleshooting guide section about MySQL and TINYINT(1) columns
Kathleen Ting 2012-05-09, 23:02

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5065/#review7751
-----------------------------------------------------------
Jarcec - good work. Some feedback.
1.) "This is because by default MySQL" needs a 'the', i.e. "This is because by default the MySQL"
2.) How about "A more clean solution" instead of "Most cleaner solution"?
3.) How about expanding on "Another solution would be to use sqoop ability to override default mapping." - e.g. Another solution would be to explicitly override the column mapping for the datatype TINYINT(1) column. For example, if the column name is foo, then pass the following option to Sqoop during import: --map-column-hive foo=tinyint. In the case of non-Hive imports to HDFS, use --map-column-java foo=integer.

- Kathleen
On 2012-05-08 11:21:33, Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5065/
> -----------------------------------------------------------
>
> (Updated 2012-05-08 11:21:33)
>
>
> Review request for Sqoop and Kathleen Ting.
>
>
> Summary
> -------
>
> I've updated corresponding section in Troubleshooting guide.
>
>
> This addresses bug SQOOP-484.
>     https://issues.apache.org/jira/browse/SQOOP-484
>
>
> Diffs
> -----
>
>   /src/docs/user/troubleshooting.txt 1335407
>
> Diff: https://reviews.apache.org/r/5065/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Jarek
>
>