Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-06 Thread Jungtaek Lim
FYI: the patch is ready to be reviewed. https://github.com/apache/storm/pull/2447 2017년 12월 6일 (수) 오후 4:18, Jungtaek Lim 님이 작성: > Received 4 +1 binding and 1 +1 non-binding in a day, which looks consensus > has been made. > I will skip voting process and go ahead since Storm

Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-05 Thread Jungtaek Lim
Received 4 +1 binding and 1 +1 non-binding in a day, which looks consensus has been made. I will skip voting process and go ahead since Storm SQL is experimental. I'm filing issue here https://issues.apache.org/jira/browse/STORM-2845 Thanks for participating! 2017년 12월 6일 (수) 오후 12:40, Xin

Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-05 Thread Xin Wang
+1 - Xin 2017-12-06 7:14 GMT+08:00 Hugo Da Cruz Louro : > +1. It would just be good that once you move onto streams API this is easy > to test. It is highly important that we can quickly run simple tests form > the CLI and IDE. Without that it is very hard to fix bugs,

Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-05 Thread Hugo Da Cruz Louro
+1. It would just be good that once you move onto streams API this is easy to test. It is highly important that we can quickly run simple tests form the CLI and IDE. Without that it is very hard to fix bugs, test new features during early developments, and come up with ways to reproduce users

Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-05 Thread Jungtaek Lim
Arun and Priyank, Yes that's how we have been having tests with trident mode, and same applies on moving on streams API. Btw, it doesn't mean we provide users to execute SQL in local cluster mode. Maybe good to have that CLI feature. Given that we've 3 +1 (binding) including me and 1 +1

Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-05 Thread P. Taylor Goetz
+1 -Taylor > On Dec 5, 2017, at 2:09 PM, Priyank Shah wrote: > > +1 (NB). As Arun mentioned, it will be good to run test it by running > topologies in local mode. > > On 12/5/17, 10:07 AM, "Arun Iyer on behalf of Arun Mahadevan" >

Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-05 Thread Priyank Shah
+1 (NB). As Arun mentioned, it will be good to run test it by running topologies in local mode. On 12/5/17, 10:07 AM, "Arun Iyer on behalf of Arun Mahadevan" wrote: +1, I don’t see much use for standalone mode other than for testing.

Re: [DISCUSS] Drop standalone mode of Storm SQL

2017-12-05 Thread Arun Mahadevan
+1, I don’t see much use for standalone mode other than for testing. Assume we can use the storm-sql in local mode to run topologies locally without deploying to cluster ? Thanks, Arun On 12/4/17, 10:53 PM, "Jungtaek Lim" wrote: >Hi devs, > >We have been exposing

[DISCUSS] Drop standalone mode of Storm SQL

2017-12-04 Thread Jungtaek Lim
Hi devs, We have been exposing "standalone mode" of Storm SQL which leverages Storm SQL in a JVM process rather than composing topology and run. At a start we implemented both standalone and trident modes with same approach, but while we improved Storm SQL by leveraging more features on Calcite,