But is it a duplication of integration with Calcite? On Thu, Jul 21, 2016 at 9:26 AM, Timothy Farkas < [email protected]> wrote:
> I see, cool :) > > On Thu, Jul 21, 2016 at 9:21 AM, Priyanka Gugale <[email protected]> > wrote: > > > Hi Tim, > > > > We are not creating our own DSL, the jooq is just another query > > parser/builder like JsqlParser. I am trying to use one of these query DSL > > libraries to replace the existing code in operator which is written to > > construct the queries. > > > > -Priyanka > > > > On Thu, Jul 21, 2016 at 9:42 PM, Timothy Farkas < > > [email protected]> wrote: > > > > > I don't know the exact context here so please forgive me if I'm > > mistaken. I > > > don't think creating our own DSL is the way to go. Creating a generic > DSL > > > is hard. We should support setting the flavor of SQL being used as a > > > property and then allow standard sql to be specified. There are already > > > mature Apache License SQL parsers which support many different SQL > > > implementations. > > > > > > https://github.com/JSQLParser/JSqlParser > > > > > > Thanks, > > > Tim > > > > > > On Thu, Jul 21, 2016 at 2:19 AM, Priyanka Gugale <[email protected]> > > > wrote: > > > > > > > Looking closely at licensing, it says it *depends but doesn't bundle* > > > those > > > > non ASL license dependencies. As per my understanding those will be > > > > included only if we explicitly include them using our application > pom. > > > > Right away we are not using any of those features which depend of > such > > > > third party licenses. > > > > > > > > Anyone have any suggestion over including this library? > > > > > > > > Dev, > > > > Yes querydsl is an option, but jooq seems more promising. If there we > > see > > > > license is a problem then may be we can go to querydsl. > > > > > > > > -Priyanka > > > > > > > > > > > > > > > > On Wed, Jul 20, 2016 at 8:58 PM, Devendra Tagare < > > > > [email protected]> > > > > wrote: > > > > > > > > > +1 for using DSL constructs that are vendor agnostic. > > > > > > > > > > Checkout https://github.com/querydsl/querydsl (Apache licensed) as > > > well > > > > > in-case it fits better in terms of implementation. > > > > > > > > > > Also, once the DSL work is done, please test and document the > > behavior > > > > > (exactly once, at-least once ..)the operator has with different > > > > databases. > > > > > > > > > > Thanks, > > > > > Dev > > > > > > > > > > On Wed, Jul 20, 2016 at 4:04 AM, Bhupesh Chawda < > [email protected]> > > > > > wrote: > > > > > > > > > > > It is a good idea to get rid of vendor specific implementation > > > > > differences > > > > > > for SQL. > > > > > > > > > > > > However, the licensing does not seem to be straightforward. > Please > > > > check: > > > > > > http://www.jooq.org/legal/licensing. Can this be used as a > > > dependency > > > > in > > > > > > Apex? > > > > > > > > > > > > ~ Bhupesh > > > > > > > > > > > > On Wed, Jul 20, 2016 at 3:06 AM, Priyanka Gugale < > > [email protected]> > > > > > > wrote: > > > > > > > > > > > > > Hi, > > > > > > > > > > > > > > Malhar JDBC operator does lots of string manipulation and other > > > > > handling > > > > > > to > > > > > > > construct sql queries as per user inputs. Instead of > constructing > > > > > queries > > > > > > > on our own we should use some dsl api library which will let us > > > write > > > > > DB > > > > > > > agnostic code and take care of all other complexities. > > > > > > > > > > > > > > I am trying out JOOQ library to write sql query in > > > > > > > AbstractJdbcPollInputOperator. Would like to hear about > > communities > > > > > > > feedback and suggestions. > > > > > > > > > > > > > > -Priyanka > > > > > > > > > > > > > > > > > > > > > > > > > > > >
