Overall looks good! I have the following comments:
i) Question on the Type of IME + data partition:
How do we enforce that user adds IME w/ the expected partition id to the corresponding sub-collection?
ii) In the architecture graph, what's the difference between SSP queues and Data source/sink? What is the layer exposed to the user (I.e. programmer)?
ii) Agree w/ the approach to use a customized queues managed by the admin. However, the reason not to use BEM is not very clear. For the matter of fact, BEM is just one optional base class for SystemConsumer implementation. Not sure why we necessarily need to be limited by BEM.
iii) In the code examples,
A) what's the difference between durable state vs non-durable state in highlevel API? I don't see any difference. Also, the SEP has clearly described that the design is only for InMemory input/output/intermediate streams. I noticed that you added changelog as inputs in low-level API. But it is not clear how this changelog is defined and why it is an input to the application???
B) the code example for checkpoint is empty and we have stated that we won't support checkpoint in this SEP. Can we remove it? Thanks! -Yi
On Wed, Sep 6, 2017 at 2:06 PM, xinyu liu <[EMAIL PROTECTED]> wrote:
> +1 on the overall design. This will make testing a lot easier! > > Thanks, > Xinyu > > On Wed, Sep 6, 2017 at 10:45 AM, Bharath Kumara Subramanian < > [EMAIL PROTECTED]> wrote: > > > Hi all, > > > > Can you please vote for SEP-8? > > You can find the design document here > > <https://cwiki.apache.org/confluence/pages/viewpage. > action?pageId=71013043 > > >. > > > > Thanks, > > Bharath > > >
NEW: Monitor These Apps!
Apache Lucene, Apache Solr and all other Apache Software Foundation project and their respective logos are trademarks of the Apache Software Foundation.
Elasticsearch, Kibana, Logstash, and Beats are trademarks of Elasticsearch BV, registered in the U.S. and in other countries. This site and Sematext Group is in no way affiliated with Elasticsearch BV.
Service operated by Sematext