sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SQOOP-1858) Sqoop2: Print out warnings from git apply on output of precommit hook
Date Fri, 12 Dec 2014 05:47:14 GMT

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

Hudson commented on SQOOP-1858:
-------------------------------

SUCCESS: Integrated in Sqoop2-hadoop200 #651 (See [https://builds.apache.org/job/Sqoop2-hadoop200/651/])
SQOOP-1858: Sqoop2: Print out warnings from git apply on output of precommit hook (abe: https://git-wip-us.apache.org/repos/asf?p=sqoop.git&a=commit&h=c3900910f8fd0e86913c3e55e8b59ff7acda2b3f)
* dev-support/test-patch.py


> Sqoop2: Print out warnings from git apply on output of precommit hook
> ---------------------------------------------------------------------
>
>                 Key: SQOOP-1858
>                 URL: https://issues.apache.org/jira/browse/SQOOP-1858
>             Project: Sqoop
>          Issue Type: Bug
>    Affects Versions: 1.99.4
>            Reporter: Jarek Jarcec Cecho
>            Assignee: Jarek Jarcec Cecho
>             Fix For: 1.99.5
>
>         Attachments: SQOOP-1858.patch
>
>
> Currently we only check the return value of executing {{git apply}} and printing it's
stdin/stderr only if the return is not 0. However having warnings (for example trailing whitespace)
won't change the return value and hence those get silently ignored. Hence I would suggest
to enhance the precommit hook to print out the output {{git apply}} in case that it's not
empty to highlight the warnings.



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

Mime
View raw message