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

Aditya Kishore updated DRILL-1548:
----------------------------------
    Attachment: DRILL-1548-Remove-jdbc-all-from-default-build.patch

I took liberty to update your patch with the following changes.

# The entire contrib can be excluded directly from the project root pom.
# {{-DskipContrib}} and {{-DskipInterpreter}} should disable their respective 
packaging and not {{drill-jdbc-all}}.
# Always good idea to at least document changes which are unrelated to the JIRA 
but minor enough to sneak in, like removing the duplicate dependencies.
# And lastly, please follow the convention {{DRILL-<issue_number>: 
<issue_summary>}} for patches.

> Remove jdbc-all from default build
> ----------------------------------
>
>                 Key: DRILL-1548
>                 URL: https://issues.apache.org/jira/browse/DRILL-1548
>             Project: Apache Drill
>          Issue Type: Improvement
>            Reporter: Hanifi Gunes
>            Assignee: Steven Phillips
>            Priority: Minor
>             Fix For: 0.7.0
>
>         Attachments: DRILL-1548-Remove-jdbc-all-from-default-build.patch, 
> DRILL-1548.1.patch.txt, DRILL-1548.patch
>
>
> Currently, jdbc-all module is built each and every time maven build is 
> started. Building jdbc-all is time consuming and outputs verbose messages 
> making it hard to track lines of interest. This issue proposes to remove 
> jdbc-all from the default maven build possibly moving it to apache build.



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

Reply via email to