Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 1 commit into from

Conversation

xhochy
Copy link
Member

@xhochy xhochy commented Sep 18, 2017

No description provided.

@xhochy
Copy link
Member Author

xhochy commented Sep 18, 2017

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

@paul-rogers
Copy link
Contributor

Build issue has been corrected via another PR.

@xhochy
Copy link
Member Author

xhochy commented Sep 21, 2017

@paul-rogers Thanks, rebase fixed the build.

@paul-rogers
Copy link
Contributor

@parthchandra or @sohami, can one of you take a quick look at this one since you are familiar with our C builds?

@parthchandra
Copy link
Contributor

+1. Thanks for the fix.

@asfgit asfgit closed this in 68a517b Oct 2, 2017
ilooner pushed a commit to ilooner/drill that referenced this pull request Jan 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants