Thank you @Andrei @Michael. Currently it's a straightforward implementation
of stream ScannableTable, a site page is added for usage. Kindly let me
know if something important is not covered.

Mingmin

On Wed, Apr 24, 2019 at 1:24 PM Michael Mior <mm...@apache.org> wrote:

> I think we'd generally be willing to accept any adapter which is
> well-written, well-tested, and likely to be generally useful. I think
> Kafka meets the final criterion and with some work the current code
> could meet the first two. That said, even if the adapter isn't
> accepted into the main Calcite codebase, I would encourage you to
> publish it as a separate package anyway.
> --
> Michael Mior
> mm...@apache.org
>
> Le mer. 24 avr. 2019 à 08:07, Andrei Sereda <and...@sereda.cc> a écrit :
> >
> > Hi Mingmin,
> >
> > I'm happy to review it.
> >
> > Before would like to confirm with this list that they're OK adding new
> > adapter (kafka) to calcite codebase ?
> >
> > Regards,
> > Andrei.
> >
> > On Tue, Apr 23, 2019 at 10:43 PM Mingmin Xu <ming...@apache.org> wrote:
> >
> > > Hi all,
> > >
> > > I add a new adapter for Apache Kafka, to allow users to query Kafka
> topics
> > > with Calcite STREAM SQL. Can someone take a look at the PR[1]?
> > >
> > > [1]. https://github.com/apache/calcite/pull/1127
> > >
> > > Thanks!
> > > Mingmin
> > >
>


-- 
----
Mingmin

Reply via email to