This is definitely a typo. Feel free to send PRs to correct these on sight :)
On Tue, Jun 26, 2018 at 8:51 AM Rafael Fernandez <rfern...@google.com> wrote: > Filed https://issues.apache.org/jira/browse/BEAM-4644 for this. I > assigned it to +Ankur Goenka <goe...@google.com> because it's the first > name in history :p (please reroute where appropriate). > > Thanks! > r > > On Tue, Jun 26, 2018 at 8:23 AM Lukasz Cwik <lc...@google.com> wrote: > >> That is an internal class to the Flink runner. Runners are allowed to >> choose whichever logging framework they want to use with the understanding >> that the SDK and shared libraries use SLF4J but most likely its a simple >> typo. >> >> On Tue, Jun 26, 2018 at 7:22 AM Kenneth Knowles <k...@google.com> wrote: >> >>> Seems like a legit bug to me. Perhaps we can adjust checkstyle, or some >>> other more semantic analysis, to forbid it. >>> >>> Kenn >>> >>> On Tue, Jun 26, 2018 at 6:48 AM Rafael Fernandez <rfern...@google.com> >>> wrote: >>> >>>> +Lukasz Cwik <lc...@google.com> , +Henning Rohde <hero...@google.com> >>>> >>>> On Tue, Jun 26, 2018 at 1:25 AM Romain Manni-Bucau < >>>> rmannibu...@gmail.com> wrote: >>>> >>>>> Hi guys, >>>>> >>>>> answering a question on slack i realized flink >>>>> ExecutableStageDoFnOperator.java uses JUL instead of SLF4J, not sure it is >>>>> intended so thought I would mention it here. >>>>> >>>>> Side note: archetype and some test code does as well but it is less an >>>>> issue. >>>>> >>>>> Romain Manni-Bucau >>>>> @rmannibucau <https://twitter.com/rmannibucau> | Blog >>>>> <https://rmannibucau.metawerx.net/> | Old Blog >>>>> <http://rmannibucau.wordpress.com> | Github >>>>> <https://github.com/rmannibucau> | LinkedIn >>>>> <https://www.linkedin.com/in/rmannibucau> | Book >>>>> <https://www.packtpub.com/application-development/java-ee-8-high-performance> >>>>> >>>>
