[jira] [Commented] (SQOOP-2867) Sqoop2: ResTiliency: Define one single constant for all

["Sqoop QA bot (JIRA)" <ji...@apache.org>]

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

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

Testing file [SQOOP-2867.patch|https://issues.apache.org/jira/secure/attachment/12805759/SQOOP-2867.patch] against branch sqoop2 took 0:00:13.699676.

{color:red}Overall:{color} -1 due to an error(s), see details below:

{color:green}SUCCESS:{color} Clean was successful
{color:red}ERROR:{color} Failed to apply patch (exit code 1):
{code}error: client/src/main/java/org/apache/sqoop/client/request/RequestConstants.java: No such file or directory
error: server/src/main/java/org/apache/sqoop/handler/HandlerConstants.java: No such file or directory
{code}


Console output is available [here|https://builds.apache.org/job/PreCommit-SQOOP-Build/2371/console].

This message is automatically generated.

> Sqoop2: ResTiliency: Define one single constant for all
> -------------------------------------------------------
>
>                 Key: SQOOP-2867
>                 URL: https://issues.apache.org/jira/browse/SQOOP-2867
>             Project: Sqoop
>          Issue Type: Sub-task
>            Reporter: Jarek Jarcec Cecho
>            Assignee: Boglarka Egyed
>              Labels: newbie
>             Fix For: 1.99.7
>
>         Attachments: SQOOP-2867.patch
>
>
> We're currently using string "all" in all Handlers that are supporting it. As [~abrahamfine] correctly [pointed out|https://reviews.apache.org/r/43783/], we should convert it to a shared constant.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)