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> >>>> >>>
smime.p7s
Description: S/MIME Cryptographic Signature
