[GitHub] drill issue #945: DRILL-5798 Fixed Unstable Tests

2017-09-18 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/945 @paul-rogers I had added a dependency that was being pulled into the jdbc-all jar. I excluded that dependency, but the size of the jar had still grown to be 100 kb over 29 Mb, so I bumped the allowed

[GitHub] drill issue #945: DRILL-5798 Fixed Unstable Tests

2017-09-18 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/945 @paul-rogers Yeah it fixes jdbc-all. Sorry for the mix up this mistake mainly stemmed from me being ignorant of the jenkins build and being unaware of jenkins failures until the batch commit was

[GitHub] drill issue #945: DRILL-5798 Fixed Unstable Tests

2017-09-18 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/945 @ilooner, thanks for the fixes. In general, would be great to avoid posting a PR until it is ready for prime time, that way we'll know when a fix is actually ready for review and commit. (Of

[GitHub] drill issue #945: DRILL-5798 Fixed Unstable Tests

2017-09-18 Thread ilooner
Github user ilooner commented on the issue: https://github.com/apache/drill/pull/945 @paul-rogers PR https://github.com/apache/drill/pull/923 accidentally got merged before I finished fixing jenkins test issues. This PR fixes the tests that were failing on jenkins as a result of the