I had seen some unit test failures, but they seemed unrelated to my changes to the avro reader. This is the reason I didn't merge last night.
The tests now appear to be running, but I haven't been able to figure out what could have changed. I was planning on sending a separate message about the test flakiness if I saw it again. The tests running to completion did allow it to get to the checkstyle in the build, which found whitespace I had left at the end of a line. I fixed that and have updated the branch with the small change to fix that problem. It also looks like I made an error updating a test that didn't need changes after a small modification to our test harness. I will push this small change to the branch soon to fix this as well. I don't know why you would be getting these compilation errors, are you sure you are on a clean branch with no changes in your working copy? On Wed, Mar 9, 2016 at 9:17 AM, Stefán Baxter <ste...@activitystream.com> wrote: > I can not build the project after pulling these: > > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 01:16 min > [INFO] Finished at: 2016-03-09T17:15:05+00:00 > [INFO] Final Memory: 138M/2788M > [INFO] > ------------------------------------------------------------------------ > [ERROR] Failed to execute goal > org.apache.maven.plugins:maven-compiler-plugin:3.2:testCompile > (default-testCompile) on project drill-java-exec: Compilation failure: > Compilation failure: > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[68] > error: illegal start of type > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[68,2] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[68,4] > error: illegal start of type > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[68,6] > error: <identifier> expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[68,7] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[69] > error: illegal start of type > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[69,2] > error: <identifier> expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[69,4] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[69,6] > error: illegal start of type > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[69,7] > error: <identifier> expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[70,3] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[127] > error: illegal start of type > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[127,3] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[127,6] > error: illegal start of type > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[127,7] > error: <identifier> expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroFormatTest.java:[127,14] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[99] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[99,2] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[99,4] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[100,6] > error: > expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[100,7] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[119] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[119,2] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[119,4] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[119,6] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[120] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[120,3] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[120,6] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[120,14] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[120,48] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[125,4] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[125,11] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[125,25] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[125,35] > error: not a statement > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[125,46] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[131,21] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[131,31] > error: not a statement > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[131,42] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[135,4] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[135,29] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[139,4] > error: illegal start of expression > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[139,54] > error: ';' expected > [ERROR] > > /var/as/drill/exec/java-exec/src/test/java/org/apache/drill/exec/store/avro/AvroTestUtil.java:[702,1] > error: reached end of file while parsing > [ERROR] -> [Help 1] > [ERROR] > [ERROR] To see the full stack trace of the errors, re-run Maven with the -e > switch. > [ERROR] Re-run Maven using the -X switch to enable full debug logging. > [ERROR] > [ERROR] For more information about the errors and possible solutions, > please read the following articles: > [ERROR] [Help 1] > http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException > [ERROR] > [ERROR] After correcting the problems, you can resume the build with the > command > [ERROR] mvn <goals> -rf :drill-java-exec > > > On Wed, Mar 9, 2016 at 1:45 AM, StevenMPhillips <g...@git.apache.org> > wrote: > > > Github user StevenMPhillips commented on the pull request: > > > > https://github.com/apache/drill/pull/419#issuecomment-194060258 > > > > +1 > > > > > > --- > > If your project is set up for it, you can reply to this email and have > your > > reply appear on GitHub as well. If your project does not have this > feature > > enabled and wishes so, or if the feature is enabled but not working, > please > > contact infrastructure at infrastruct...@apache.org or file a JIRA > ticket > > with INFRA. > > --- > > > -- Jason Altekruse Software Engineer at Dremio Apache Drill Committer