You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by "Sqoop QA bot (JIRA)" <ji...@apache.org> on 2013/07/12 19:15:49 UTC

[jira] [Commented] (SQOOP-1105) Sqoop2: Remove our implementation of StringUtils

    [ https://issues.apache.org/jira/browse/SQOOP-1105?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13707098#comment-13707098 ] 

Sqoop QA bot commented on SQOOP-1105:
-------------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12592028/sqoop-1105.1.txt against branch sqoop2.

{color:green}Overall:{color} +1 all checks pass

{color:green}SUCCESS:{color} Clean was successful
{color:green}SUCCESS:{color} Patch applied correctly
{color:green}SUCCESS:{color} Patch compiled
{color:green}SUCCESS:{color} All tests passed

Console output: https://builds.apache.org/job/PreCommit-SQOOP-Build/61/console

This message is automatically generated.
                
> Sqoop2: Remove our implementation of StringUtils
> ------------------------------------------------
>
>                 Key: SQOOP-1105
>                 URL: https://issues.apache.org/jira/browse/SQOOP-1105
>             Project: Sqoop
>          Issue Type: Bug
>    Affects Versions: 1.99.2
>            Reporter: Jarek Jarcec Cecho
>            Assignee: Raghav Kumar Gautam
>            Priority: Trivial
>         Attachments: sqoop-1105.1.txt
>
>
> We do have our own implementation of [StringUtils|https://github.com/apache/sqoop/blob/sqoop2/common/src/main/java/org/apache/sqoop/utils/StringUtils.java] class. I think that we should remove it and use the one available in {{commons-lang}} (which is already our dependency).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira