Hm, I just encountered this again on a branch that based on 5b46b02b49 (top of trunk from this afternoon). Is it definitely supposed to be fixed?
On Thu, Jan 24, 2019 at 9:19 PM Alex Amato <ajam...@google.com> wrote: > Please try rebasing from master, I believe this issue has been resolved. > > On Thu, Jan 24, 2019 at 3:29 PM Ryan Williams <r...@runsascoded.com> > wrote: > >> I'm seeing every ≈third `./gradlew compileJava` fail locally due to this; >> re-running the commit has always succeeded, so far. >> >> Sounds like there is not an immediate fix in the works / no one assigned >> on the JIRA? >> >> On Wed, Jan 23, 2019 at 3:17 PM Kenneth Knowles <k...@apache.org> wrote: >> >>> This might connect to vendoring Calcite. It will be easiest, and have >>> the best incremental build, if we separate the generated code into its own >>> module that has relocation to match the vendored Calcite. >>> >>> Kenn >>> >>> On Wed, Jan 23, 2019 at 11:29 AM Anton Kedin <ke...@google.com> wrote: >>> >>>> We don't pre-generate the code as a separate step. Code gen from the >>>> SQL parser syntax spec and its compilation happens both during the Beam SQL >>>> build task. Splitting the code generation and compilation might not be >>>> trivial. We definitely should look into fixing this though. >>>> >>>> Regards, >>>> Anton >>>> >>>> On Wed, Jan 23, 2019 at 11:13 AM Alex Amato <ajam...@google.com> wrote: >>>> >>>>> Okay, make sense perhaps we can somehow make it fail when it fails to >>>>> generate the dep, rather than when compiling the java code later on >>>>> >>>>> On Wed, Jan 23, 2019 at 11:12 AM Anton Kedin <ke...@google.com> wrote: >>>>> >>>>>> ParserImpl is autogenerated by Calcite at build time. It seems that >>>>>> there's a race condition there and it sometimes fails. Rerunning the >>>>>> build >>>>>> works for me. >>>>>> >>>>>> Regards, >>>>>> Anton >>>>>> >>>>>> On Wed, Jan 23, 2019, 11:06 AM Alex Amato <ajam...@google.com> wrote: >>>>>> >>>>>>> https://jira.apache.org/jira/browse/BEAM-6495?filter=-2 >>>>>>> >>>>>>> Any ideas, how this got through the precommit? >>>>>>> >>>>>>> > Task :beam-sdks-java-extensions-sql:compileJava FAILED >>>>>>> >>>>>>> /usr/local/google/home/ajamato/go/src/ >>>>>>> github.com/apache/beam/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/JdbcFactory.java:29: >>>>>>> error: cannot find symbol >>>>>>> >>>>>>> import >>>>>>> org.apache.beam.sdk.extensions.sql.impl.parser.impl.BeamSqlParserImpl; >>>>>>> >>>>>>> ^ >>>>>>> >>>>>>> symbol: class BeamSqlParserImpl >>>>>>> >>>>>>> location: package >>>>>>> org.apache.beam.sdk.extensions.sql.impl.parser.impl >>>>>>> >>>>>>> 1 error >>>>>>> >>>>>>>
