You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by xhochy <gi...@git.apache.org> on 2017/09/18 08:00:07 UTC

[GitHub] drill pull request #946: DRILL-5799: native-client: Support alternative buil...

GitHub user xhochy opened a pull request:

    https://github.com/apache/drill/pull/946

    DRILL-5799: native-client: Support alternative build directories

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/xhochy/drill DRILL-5799

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/946.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #946
    
----
commit fef0b3af87b7da01598a4bbaec5a15841ebbc9a6
Author: Korn, Uwe <uw...@blue-yonder.com>
Date:   2017-09-18T07:57:08Z

    DRILL-5799: native-client: Support alternative build directories

----


---

[GitHub] drill issue #946: DRILL-5799: native-client: Support alternative build direc...

Posted by xhochy <gi...@git.apache.org>.
Github user xhochy commented on the issue:

    https://github.com/apache/drill/pull/946
  
    Build failed with unrelated reasons to this PR: 
    
    ```
    [WARNING] Rule 0: org.apache.maven.plugins.enforcer.RequireFilesSize failed with message:
    The file drill-jdbc-all-1.12.0-SNAPSHOT.jar is outside the expected size range.
    
                      This is likely due to you adding new dependencies to a java-exec and not updating the excludes in this module. This is important as it minimizes the size of the dependency of Drill application users.
    /home/travis/build/apache/drill/exec/jdbc-all/target/drill-jdbc-all-1.12.0-SNAPSHOT.jar size (31071836) too large. Max. is 29000000/home/travis/build/apache/drill/exec/jdbc-all/target/drill-jdbc-all-1.12.0-SNAPSHOT.jar
    ```


---

[GitHub] drill pull request #946: DRILL-5799: native-client: Support alternative buil...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/drill/pull/946


---

[GitHub] drill issue #946: DRILL-5799: native-client: Support alternative build direc...

Posted by parthchandra <gi...@git.apache.org>.
Github user parthchandra commented on the issue:

    https://github.com/apache/drill/pull/946
  
    +1. Thanks for the fix. 


---

[GitHub] drill issue #946: DRILL-5799: native-client: Support alternative build direc...

Posted by xhochy <gi...@git.apache.org>.
Github user xhochy commented on the issue:

    https://github.com/apache/drill/pull/946
  
    @paul-rogers Thanks, rebase fixed the build.


---

[GitHub] drill issue #946: DRILL-5799: native-client: Support alternative build direc...

Posted by paul-rogers <gi...@git.apache.org>.
Github user paul-rogers commented on the issue:

    https://github.com/apache/drill/pull/946
  
    @parthchandra or @sohami, can one of you take a quick look at this one since you are familiar with our C builds? 


---

[GitHub] drill issue #946: DRILL-5799: native-client: Support alternative build direc...

Posted by paul-rogers <gi...@git.apache.org>.
Github user paul-rogers commented on the issue:

    https://github.com/apache/drill/pull/946
  
    Build issue has been corrected via another PR.


---