sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SQOOP-2425) Sqoop2: Improve our pre-commit hook
Date Wed, 05 Aug 2015 02:10:04 GMT

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

ASF subversion and git services commented on SQOOP-2425:

Commit e908d1597715c51f0e098e28ec4ff4b67e2ac9d9 in sqoop's branch refs/heads/sqoop2 from Richard
[ https://git-wip-us.apache.org/repos/asf?p=sqoop.git;h=e908d15 ]

SQOOP-2425: Sqoop2: Precommit: Do not wipe out working directory on finish

(Jarek Jarcec Cecho via Richard Zhou)

> Sqoop2: Improve our pre-commit hook
> -----------------------------------
>                 Key: SQOOP-2425
>                 URL: https://issues.apache.org/jira/browse/SQOOP-2425
>             Project: Sqoop
>          Issue Type: Bug
>    Affects Versions: 1.99.6
>            Reporter: Jarek Jarcec Cecho
>            Assignee: Jarek Jarcec Cecho
> I would like to propose several improvements to our pre-commit hook, so that it's even
more useful then it's now :)
> Couple of ideas from top of my mind:
> * Order the actions that are performed. Right now the report always starts with ERRORS
and I find it confusing. I would like to see order, something like "compilation success, unit
test success, integration tests failed, ...).
> * Dependency checker. Detect if/when somebody adds a dependency and point it out (to
verify that it's desired).
> * Save test log outputs, so that we can get to them. Their absence is making troubleshooting
quite hard.
> * It would be great to add some additional static analysis - perhaps a checkstyle checker
if we would have a rules to follow :)
> I'm sure that there are other ides, so please don't hesitate and comment what you would
like to see!

This message was sent by Atlassian JIRA

View raw message