You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by "Jarek Jarcec Cecho (JIRA)" <ji...@apache.org> on 2014/03/18 22:12:44 UTC

[jira] [Resolved] (SQOOP-1296) Sqoop2: Revert SQOOP-1233

     [ https://issues.apache.org/jira/browse/SQOOP-1296?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jarek Jarcec Cecho resolved SQOOP-1296.
---------------------------------------

    Resolution: Fixed

Thank you [~hshreedharan]! Reverted.

> Sqoop2: Revert SQOOP-1233
> -------------------------
>
>                 Key: SQOOP-1296
>                 URL: https://issues.apache.org/jira/browse/SQOOP-1296
>             Project: Sqoop
>          Issue Type: Bug
>    Affects Versions: 1.99.3
>            Reporter: Jarek Jarcec Cecho
>            Assignee: Jarek Jarcec Cecho
>            Priority: Minor
>             Fix For: 1.99.4
>
>
> I would like to propose reverting SQOOP-1233 (my own patch). In the JIRA I've removed calls to {{MessageFormat}} and let the formatting propagate to {{IO}} object. However both {{IO}} and {{MessageFormat}} are expecting different format of the formatting string resulting in following ugliness:
> {code}
> sqoop:000> show version --all
> sqoop:000> help
> {0} version:
>   Sqoop {1} revision {2} 
>   Compiled by {3} on {4}{0} version:
>   Sqoop {1} revision {2} 
>   Compiled by {3} on {4}Protocol version:
>   {0}{0} version:
>   Sqoop {1} revision {2} 
>   Compiled by {3} on {4}{0} version:
>   Sqoop {1} revision {2} 
>   Compiled by {3} on {4}{0} version:
>   Sqoop {1} revision {2} 
>   Compiled by {3} on {4}Protocol version:
>   {0}For information about Sqoop, visit: http://sqoop.apache.org/
> Available commands:
>   {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}  {0} ({1}) {2}
> For help on a specific command type: help command
> sqoop:000>
> {code}
> My sincere apologies for not catching this during my testing. I'll go ahead and call {{git revert}} if I'll get +1 from another committer.



--
This message was sent by Atlassian JIRA
(v6.2#6252)