Hey Raghu,

After discussing with Aviem Zur, it made more sense to add a readme file at
the kafka directory
<https://github.com/apache/beam/tree/master/sdks/java/io/kafka>.
PR is closed. Thanks

On Tue, Aug 29, 2017, 8:51 PM Raghu Angadi <rang...@google.com.invalid>
wrote:

> Thanks Joey.
> I couldn't find the PR. Do you have a link?
>
>
> On Tue, Aug 29, 2017 at 1:48 AM, Joey Baruch <joey.bar...@gmail.com>
> wrote:
>
> > created pr on the beam site (just reference change for now, as Jean
> > Baptiste wrote)
> >
> > Thanks Aviem!
> >
> > On Tue, Aug 29, 2017 at 11:46 AM Aviem Zur <aviem...@gmail.com> wrote:
> >
> > > Hi Joey.
> > >
> > > This would be great. Also, KafkaIO requires a specific dependency to be
> > > added (beam-sdks-java-io-kafka), we should probably put that as a maven
> > > snippet in the README as well. Feel free to create a PR with this
> README
> > on
> > > GitHub.
> > >
> > > Regarding the long series of links you need to click in the site in
> order
> > > to get to these docs, I agree, perhaps we can reduce this number of
> > clicks,
> > > feel free to create a PR on the beam-site repo on GitHub.
> > >
> > > Welcome to the community!
> > >
> > > On Tue, Aug 29, 2017 at 11:33 AM Jean-Baptiste Onofré <j...@nanthrax.net
> >
> > > wrote:
> > >
> > > > Agree to add the link to javadoc on the I/O list:
> > > >
> > > > https://beam.apache.org/documentation/io/built-in/
> > > >
> > > > Regards
> > > > JB
> > > >
> > > > On 08/29/2017 10:28 AM, Joey Baruch wrote:
> > > > > Hey all,
> > > > >
> > > > > As a new user trying to use a kafkaIO source/sink i couldn't find
> any
> > > > > documentation easily.
> > > > > The documentation page <
> > > > https://beam.apache.org/documentation/io/built-in/>,
> > > > > (which you get to from headder -> doccumentaton -> pipeline i/o ->
> > > built
> > > > in
> > > > > i/o transforms) leads to the kafka class
> > > > > <https://github.com/apache/beam/tree/master/sdks/java/io/kafka>,
> but
> > > > there
> > > > > is no docs there.
> > > > >
> > > > > I will add a simple readme that points to the classe's javadocs.
> > > > >
> > > > > regards
> > > > > Joey Baruch
> > > > >
> > > >
> > > > --
> > > > Jean-Baptiste Onofré
> > > > jbono...@apache.org
> > > > http://blog.nanthrax.net
> > > > Talend - http://www.talend.com
> > > >
> > >
> >
>

Reply via email to