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

Switch to Threaded View
Sqoop, mail # dev - Review Request 15635: SQOOP-1234 Sqoop2: Clean up comments in MSubmission class


Copy link to this message
-
Re: Review Request 15635: SQOOP-1234 Sqoop2: Clean up comments in MSubmission class
Abraham Elmahrek 2013-12-05, 22:02

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15635/#review29839
-----------------------------------------------------------

Ship it!
Thanks for cleaning up the grammar jarcec!

- Abraham Elmahrek
On Nov. 18, 2013, 4:18 p.m., Jarek Cecho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15635/
> -----------------------------------------------------------
>
> (Updated Nov. 18, 2013, 4:18 p.m.)
>
>
> Review request for Sqoop.
>
>
> Bugs: SQOOP-1234
>     https://issues.apache.org/jira/browse/SQOOP-1234
>
>
> Repository: sqoop-sqoop2
>
>
> Description
> -------
>
> I've updated the comments and fixed unused creation date.
>
>
> Diffs
> -----
>
>   common/src/main/java/org/apache/sqoop/model/MSubmission.java a126d404b61483ffb500b8588230813fea66ad2f
>
> Diff: https://reviews.apache.org/r/15635/diff/
>
>
> Testing
> -------
>
> This patch is introducing (almost) no code change, thus there are no new unit tests. The existing are passing though.
>
>
> Thanks,
>
> Jarek Cecho
>
>