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