Hi, I have created a github issue under the following URL [1].
[1] https://github.com/wso2/siddhi/issues/777 -- Thanks, Miyuru Dayarathna Senior Technical Lead Mobile: +94713527783 Blog: http://miyurublog.blogspot.com On Fri, Feb 23, 2018 at 1:55 PM, Srinath Perera <srin...@wso2.com> wrote: > +1 > > reasons > > 1. The mathematical definition of partition includes all elements in a > set. https://en.wikipedia.org/wiki/Partition_of_a_set. IMO it is a bad > idea to break this as it is a well-defined idea. > 2. Often this will happen due to error, and if we ignore it, it is > very hard to debug > 3. If user wants to ignore a partition he can do it as post processing > > > On Fri, Feb 23, 2018 at 1:49 PM, Miyuru Dayarathna <miyu...@wso2.com> > wrote: > >> Hi, >> >> We had several offline discussions on the results returned by the >> following partition by range Siddhi query. This query simply drops events >> which are in the range 100-149 because those events do not fall into either >> small or large partitions. Ideally we should throw an error when parsing >> this Siddhi query since it does not include a partition for the (price>=100 >> and price <150) case. However, currently Siddhi does not throw such error. >> We might think of such scenario as user may not really interested of the >> values between 100-149 and hence we should ignore that range. But it could >> lead to errors which are created unintentionally because user might >> actually forget to address the range 100-149. In such situation, throwing >> an error will make the user to address the range 100-149 properly. >> Therefore, we should throw an error if the user forget to cover the >> complete range of values in a partition by range Siddhi query. WDYT? >> >> @app:name('incrementalPersistenceTest10') >> define stream cseEventStreamOne (symbol string, price float,volume int); >> partition with (price<100 as 'small' or price>=150 as 'large' of >> cseEventStreamOne) >> begin @info(name = 'query1') >> from cseEventStreamOne#window.length(4) >> select symbol,sum(price) as price group by symbol insert into >> OutStockStream ; >> end >> >> >> -- >> Thanks, >> Miyuru Dayarathna >> Senior Technical Lead >> Mobile: +94713527783 <+94%2071%20352%207783> >> Blog: http://miyurublog.blogspot.com >> > > > > -- > ============================ > Srinath Perera, Ph.D. > http://people.apache.org/~hemapani/ > http://srinathsview.blogspot.com/ > -- Thanks, Miyuru Dayarathna Senior Technical Lead Mobile: +94713527783 Blog: http://miyurublog.blogspot.com
_______________________________________________ Architecture mailing list Architecture@wso2.org https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture
