Adding sqlline tool to Apache Ignite project

classic Classic list List threaded Threaded
36 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Adding sqlline tool to Apache Ignite project

Ilya Suntsov
Hi Igniters,

I've found one command line tool that can be used for execute queries
(DDL/DML) with data from Apache Ignite grid via
*org.apache.ignite.IgniteJdbcThinDriver*.

Here you can find overview of internal commands of this tool:
https://cwiki.apache.org/confluence/display/IGNITE/Overview+sqlline+tool

The most problems with sqlline relate with some issues from our side or
some unsupported features.

Would be nice to add tool like this (sqlline or something else) in our
project.

Have you any objections or other thoughts?


--
Ilya Suntsov
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dmagda
Hi Ilya,

It will be a useful addition to Ignite. I would include the tool in one of the nearest releases. It’s license is BSD-3 which is compatible with ours. However, there is a note on ASF side saying that BSD licenses can be included without advertising clause [1]. What does it mean for us? Can we document the tool and mention on our site?

[1] https://www.apache.org/legal/resolved.html#category-a <https://www.apache.org/legal/resolved.html#category-a>


Denis

> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]> wrote:
>
> Hi Igniters,
>
> I've found one command line tool that can be used for execute queries
> (DDL/DML) with data from Apache Ignite grid via
> *org.apache.ignite.IgniteJdbcThinDriver*.
>
> Here you can find overview of internal commands of this tool:
> https://cwiki.apache.org/confluence/display/IGNITE/Overview+sqlline+tool
>
> The most problems with sqlline relate with some issues from our side or
> some unsupported features.
>
> Would be nice to add tool like this (sqlline or something else) in our
> project.
>
> Have you any objections or other thoughts?
>
>
> --
> Ilya Suntsov

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Ilya Kasnacheev
Hi Denis,

There are two kinds of BSD license, 3-clause and 4-clause. The difference
between them is advertising clause:

"3. All advertising materials mentioning features or use of this software
must display the following acknowledgement: This product includes software
developed by the University of California, Berkeley and its contributors."

which is causing problems with other licenses and which sqlline's license
doesn't contain:
https://github.com/julianhyde/sqlline/blob/master/LICENSE
So it should be all good for inclusion.

--
Ilya Kasnacheev.

2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:

> Hi Ilya,
>
> It will be a useful addition to Ignite. I would include the tool in one of
> the nearest releases. It’s license is BSD-3 which is compatible with ours.
> However, there is a note on ASF side saying that BSD licenses can be
> included without advertising clause [1]. What does it mean for us? Can we
> document the tool and mention on our site?
>
> [1] https://www.apache.org/legal/resolved.html#category-a <
> https://www.apache.org/legal/resolved.html#category-a>
>
> —
> Denis
>
> > On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]> wrote:
> >
> > Hi Igniters,
> >
> > I've found one command line tool that can be used for execute queries
> > (DDL/DML) with data from Apache Ignite grid via
> > *org.apache.ignite.IgniteJdbcThinDriver*.
> >
> > Here you can find overview of internal commands of this tool:
> > https://cwiki.apache.org/confluence/display/IGNITE/Overview+sqlline+tool
> >
> > The most problems with sqlline relate with some issues from our side or
> > some unsupported features.
> >
> > Would be nice to add tool like this (sqlline or something else) in our
> > project.
> >
> > Have you any objections or other thoughts?
> >
> >
> > --
> > Ilya Suntsov
>
>
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dmagda
Hi Ilya,

Thanks for the clarification! Referring to the page shared by you [1] if we need to get author’s consent in a written form:

A permissive license similar to the BSD 2-Clause License, but with a 3rd clause that prohibits others from using the name of the project or its contributors to promote derived products without written consent.

[1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <https://github.com/julianhyde/sqlline/blob/master/LICENSE>

I’ll reach out the author requesting the permit.


Denis

> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <[hidden email]> wrote:
>
> Hi Denis,
>
> There are two kinds of BSD license, 3-clause and 4-clause. The difference
> between them is advertising clause:
>
> "3. All advertising materials mentioning features or use of this software
> must display the following acknowledgement: This product includes software
> developed by the University of California, Berkeley and its contributors."
>
> which is causing problems with other licenses and which sqlline's license
> doesn't contain:
> https://github.com/julianhyde/sqlline/blob/master/LICENSE
> So it should be all good for inclusion.
>
> --
> Ilya Kasnacheev.
>
> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
>
>> Hi Ilya,
>>
>> It will be a useful addition to Ignite. I would include the tool in one of
>> the nearest releases. It’s license is BSD-3 which is compatible with ours.
>> However, there is a note on ASF side saying that BSD licenses can be
>> included without advertising clause [1]. What does it mean for us? Can we
>> document the tool and mention on our site?
>>
>> [1] https://www.apache.org/legal/resolved.html#category-a <
>> https://www.apache.org/legal/resolved.html#category-a>
>>
>> —
>> Denis
>>
>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]> wrote:
>>>
>>> Hi Igniters,
>>>
>>> I've found one command line tool that can be used for execute queries
>>> (DDL/DML) with data from Apache Ignite grid via
>>> *org.apache.ignite.IgniteJdbcThinDriver*.
>>>
>>> Here you can find overview of internal commands of this tool:
>>> https://cwiki.apache.org/confluence/display/IGNITE/Overview+sqlline+tool
>>>
>>> The most problems with sqlline relate with some issues from our side or
>>> some unsupported features.
>>>
>>> Would be nice to add tool like this (sqlline or something else) in our
>>> project.
>>>
>>> Have you any objections or other thoughts?
>>>
>>>
>>> --
>>> Ilya Suntsov
>>
>>

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dmagda
Igniters,

Let me introduce Julian Hyde [1], creator of SQLLine tool and our Apache mate,

Julian,

Please grant that Apache Ignite community a permission to include SQLLine [2] it in every Ignite deliverable (source, binary). It’s planned to suggest the tool as a default command line SQL utility for Ignite clusters. SQLLite and Ignite usage will also be documented on Ignite’s technical documentation.

[1] https://people.apache.org/~jhyde/
[2] https://github.com/julianhyde/sqlline <https://github.com/julianhyde/sqlline>


Denis

> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]> wrote:
>
> Hi Ilya,
>
> Thanks for the clarification! Referring to the page shared by you [1] if we need to get author’s consent in a written form:
>
> A permissive license similar to the BSD 2-Clause License, but with a 3rd clause that prohibits others from using the name of the project or its contributors to promote derived products without written consent.
>
> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <https://github.com/julianhyde/sqlline/blob/master/LICENSE>
>
> I’ll reach out the author requesting the permit.
>
> —
> Denis
>
>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <[hidden email]> wrote:
>>
>> Hi Denis,
>>
>> There are two kinds of BSD license, 3-clause and 4-clause. The difference
>> between them is advertising clause:
>>
>> "3. All advertising materials mentioning features or use of this software
>> must display the following acknowledgement: This product includes software
>> developed by the University of California, Berkeley and its contributors."
>>
>> which is causing problems with other licenses and which sqlline's license
>> doesn't contain:
>> https://github.com/julianhyde/sqlline/blob/master/LICENSE
>> So it should be all good for inclusion.
>>
>> --
>> Ilya Kasnacheev.
>>
>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
>>
>>> Hi Ilya,
>>>
>>> It will be a useful addition to Ignite. I would include the tool in one of
>>> the nearest releases. It’s license is BSD-3 which is compatible with ours.
>>> However, there is a note on ASF side saying that BSD licenses can be
>>> included without advertising clause [1]. What does it mean for us? Can we
>>> document the tool and mention on our site?
>>>
>>> [1] https://www.apache.org/legal/resolved.html#category-a <
>>> https://www.apache.org/legal/resolved.html#category-a>
>>>
>>> —
>>> Denis
>>>
>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]> wrote:
>>>>
>>>> Hi Igniters,
>>>>
>>>> I've found one command line tool that can be used for execute queries
>>>> (DDL/DML) with data from Apache Ignite grid via
>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
>>>>
>>>> Here you can find overview of internal commands of this tool:
>>>> https://cwiki.apache.org/confluence/display/IGNITE/Overview+sqlline+tool
>>>>
>>>> The most problems with sqlline relate with some issues from our side or
>>>> some unsupported features.
>>>>
>>>> Would be nice to add tool like this (sqlline or something else) in our
>>>> project.
>>>>
>>>> Have you any objections or other thoughts?
>>>>
>>>>
>>>> --
>>>> Ilya Suntsov
>>>
>>>
>

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Julian Hyde
Denis,

I’m glad you’re thinking of using SQLLine. Under the BSD license, you don’t need my permission to distribute, but I grant that permission.

Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is in good company.

If you need extensions, please discuss on the dev list, or open a GitHub case or pull request. SQLLine operates in the usual way for a GitHub project. It’s unlikely that you’ll need Ignite-specific extensions — SQLLine just exposes what comes through the JDBC driver — but we can discuss if the need arises. The Hive project forked SQLLine into its own Beeline module and I’d like to avoid a repeat of that.

Julian

> On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]> wrote:
>
> Igniters,
>
> Let me introduce Julian Hyde [1], creator of SQLLine tool and our Apache mate,
>
> Julian,
>
> Please grant that Apache Ignite community a permission to include SQLLine [2] it in every Ignite deliverable (source, binary). It’s planned to suggest the tool as a default command line SQL utility for Ignite clusters. SQLLite and Ignite usage will also be documented on Ignite’s technical documentation.
>
> [1] https://people.apache.org/~jhyde/ <https://people.apache.org/~jhyde/>
> [2] https://github.com/julianhyde/sqlline <https://github.com/julianhyde/sqlline>
>
> —
> Denis
>
>> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email] <mailto:[hidden email]>> wrote:
>>
>> Hi Ilya,
>>
>> Thanks for the clarification! Referring to the page shared by you [1] if we need to get author’s consent in a written form:
>>
>> A permissive license similar to the BSD 2-Clause License, but with a 3rd clause that prohibits others from using the name of the project or its contributors to promote derived products without written consent.
>>
>> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <https://github.com/julianhyde/sqlline/blob/master/LICENSE> <https://github.com/julianhyde/sqlline/blob/master/LICENSE <https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
>>
>> I’ll reach out the author requesting the permit.
>>
>> —
>> Denis
>>
>>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <[hidden email] <mailto:[hidden email]>> wrote:
>>>
>>> Hi Denis,
>>>
>>> There are two kinds of BSD license, 3-clause and 4-clause. The difference
>>> between them is advertising clause:
>>>
>>> "3. All advertising materials mentioning features or use of this software
>>> must display the following acknowledgement: This product includes software
>>> developed by the University of California, Berkeley and its contributors."
>>>
>>> which is causing problems with other licenses and which sqlline's license
>>> doesn't contain:
>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <https://github.com/julianhyde/sqlline/blob/master/LICENSE>
>>> So it should be all good for inclusion.
>>>
>>> --
>>> Ilya Kasnacheev.
>>>
>>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
>>>
>>>> Hi Ilya,
>>>>
>>>> It will be a useful addition to Ignite. I would include the tool in one of
>>>> the nearest releases. It’s license is BSD-3 which is compatible with ours.
>>>> However, there is a note on ASF side saying that BSD licenses can be
>>>> included without advertising clause [1]. What does it mean for us? Can we
>>>> document the tool and mention on our site?
>>>>
>>>> [1] https://www.apache.org/legal/resolved.html#category-a <
>>>> https://www.apache.org/legal/resolved.html#category-a>
>>>>
>>>> —
>>>> Denis
>>>>
>>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]> wrote:
>>>>>
>>>>> Hi Igniters,
>>>>>
>>>>> I've found one command line tool that can be used for execute queries
>>>>> (DDL/DML) with data from Apache Ignite grid via
>>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
>>>>>
>>>>> Here you can find overview of internal commands of this tool:
>>>>> https://cwiki.apache.org/confluence/display/IGNITE/Overview+sqlline+tool
>>>>>
>>>>> The most problems with sqlline relate with some issues from our side or
>>>>> some unsupported features.
>>>>>
>>>>> Would be nice to add tool like this (sqlline or something else) in our
>>>>> project.
>>>>>
>>>>> Have you any objections or other thoughts?
>>>>>
>>>>>
>>>>> --
>>>>> Ilya Suntsov
>>>>
>>>>
>>
>

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Ilya Suntsov
Guys,

I've created ticket [1] for version 2.4.
https://issues.apache.org/jira/browse/IGNITE-6561

2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:

> Denis,
>
> I’m glad you’re thinking of using SQLLine. Under the BSD license, you
> don’t need my permission to distribute, but I grant that permission.
>
> Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is in
> good company.
>
> If you need extensions, please discuss on the dev list, or open a GitHub
> case or pull request. SQLLine operates in the usual way for a GitHub
> project. It’s unlikely that you’ll need Ignite-specific extensions —
> SQLLine just exposes what comes through the JDBC driver — but we can
> discuss if the need arises. The Hive project forked SQLLine into its own
> Beeline module and I’d like to avoid a repeat of that.
>
> Julian
>
> > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]> wrote:
> >
> > Igniters,
> >
> > Let me introduce Julian Hyde [1], creator of SQLLine tool and our Apache
> mate,
> >
> > Julian,
> >
> > Please grant that Apache Ignite community a permission to include
> SQLLine [2] it in every Ignite deliverable (source, binary). It’s planned
> to suggest the tool as a default command line SQL utility for Ignite
> clusters. SQLLite and Ignite usage will also be documented on Ignite’s
> technical documentation.
> >
> > [1] https://people.apache.org/~jhyde/ <https://people.apache.org/~jhyde/
> >
> > [2] https://github.com/julianhyde/sqlline <https://github.com/
> julianhyde/sqlline>
> >
> > —
> > Denis
> >
> >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email] <mailto:
> [hidden email]>> wrote:
> >>
> >> Hi Ilya,
> >>
> >> Thanks for the clarification! Referring to the page shared by you [1]
> if we need to get author’s consent in a written form:
> >>
> >> A permissive license similar to the BSD 2-Clause License, but with a
> 3rd clause that prohibits others from using the name of the project or its
> contributors to promote derived products without written consent.
> >>
> >> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> >>
> >> I’ll reach out the author requesting the permit.
> >>
> >> —
> >> Denis
> >>
> >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> [hidden email] <mailto:[hidden email]>> wrote:
> >>>
> >>> Hi Denis,
> >>>
> >>> There are two kinds of BSD license, 3-clause and 4-clause. The
> difference
> >>> between them is advertising clause:
> >>>
> >>> "3. All advertising materials mentioning features or use of this
> software
> >>> must display the following acknowledgement: This product includes
> software
> >>> developed by the University of California, Berkeley and its
> contributors."
> >>>
> >>> which is causing problems with other licenses and which sqlline's
> license
> >>> doesn't contain:
> >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> >>> So it should be all good for inclusion.
> >>>
> >>> --
> >>> Ilya Kasnacheev.
> >>>
> >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> >>>
> >>>> Hi Ilya,
> >>>>
> >>>> It will be a useful addition to Ignite. I would include the tool in
> one of
> >>>> the nearest releases. It’s license is BSD-3 which is compatible with
> ours.
> >>>> However, there is a note on ASF side saying that BSD licenses can be
> >>>> included without advertising clause [1]. What does it mean for us?
> Can we
> >>>> document the tool and mention on our site?
> >>>>
> >>>> [1] https://www.apache.org/legal/resolved.html#category-a <
> >>>> https://www.apache.org/legal/resolved.html#category-a>
> >>>>
> >>>> —
> >>>> Denis
> >>>>
> >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]>
> wrote:
> >>>>>
> >>>>> Hi Igniters,
> >>>>>
> >>>>> I've found one command line tool that can be used for execute queries
> >>>>> (DDL/DML) with data from Apache Ignite grid via
> >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> >>>>>
> >>>>> Here you can find overview of internal commands of this tool:
> >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> Overview+sqlline+tool
> >>>>>
> >>>>> The most problems with sqlline relate with some issues from our side
> or
> >>>>> some unsupported features.
> >>>>>
> >>>>> Would be nice to add tool like this (sqlline or something else) in
> our
> >>>>> project.
> >>>>>
> >>>>> Have you any objections or other thoughts?
> >>>>>
> >>>>>
> >>>>> --
> >>>>> Ilya Suntsov
> >>>>
> >>>>
> >>
> >
>
>


--
Ilya Suntsov
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Ilya Suntsov
In reply to this post by Julian Hyde
Guys,

I've created the ticket for 2.4 release:
https://issues.apache.org/jira/browse/IGNITE-6561

2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:

> Denis,
>
> I’m glad you’re thinking of using SQLLine. Under the BSD license, you
> don’t need my permission to distribute, but I grant that permission.
>
> Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is in
> good company.
>
> If you need extensions, please discuss on the dev list, or open a GitHub
> case or pull request. SQLLine operates in the usual way for a GitHub
> project. It’s unlikely that you’ll need Ignite-specific extensions —
> SQLLine just exposes what comes through the JDBC driver — but we can
> discuss if the need arises. The Hive project forked SQLLine into its own
> Beeline module and I’d like to avoid a repeat of that.
>
> Julian
>
> > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]> wrote:
> >
> > Igniters,
> >
> > Let me introduce Julian Hyde [1], creator of SQLLine tool and our Apache
> mate,
> >
> > Julian,
> >
> > Please grant that Apache Ignite community a permission to include
> SQLLine [2] it in every Ignite deliverable (source, binary). It’s planned
> to suggest the tool as a default command line SQL utility for Ignite
> clusters. SQLLite and Ignite usage will also be documented on Ignite’s
> technical documentation.
> >
> > [1] https://people.apache.org/~jhyde/ <https://people.apache.org/~jhyde/
> >
> > [2] https://github.com/julianhyde/sqlline <https://github.com/
> julianhyde/sqlline>
> >
> > —
> > Denis
> >
> >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email] <mailto:
> [hidden email]>> wrote:
> >>
> >> Hi Ilya,
> >>
> >> Thanks for the clarification! Referring to the page shared by you [1]
> if we need to get author’s consent in a written form:
> >>
> >> A permissive license similar to the BSD 2-Clause License, but with a
> 3rd clause that prohibits others from using the name of the project or its
> contributors to promote derived products without written consent.
> >>
> >> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> >>
> >> I’ll reach out the author requesting the permit.
> >>
> >> —
> >> Denis
> >>
> >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> [hidden email] <mailto:[hidden email]>> wrote:
> >>>
> >>> Hi Denis,
> >>>
> >>> There are two kinds of BSD license, 3-clause and 4-clause. The
> difference
> >>> between them is advertising clause:
> >>>
> >>> "3. All advertising materials mentioning features or use of this
> software
> >>> must display the following acknowledgement: This product includes
> software
> >>> developed by the University of California, Berkeley and its
> contributors."
> >>>
> >>> which is causing problems with other licenses and which sqlline's
> license
> >>> doesn't contain:
> >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> >>> So it should be all good for inclusion.
> >>>
> >>> --
> >>> Ilya Kasnacheev.
> >>>
> >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> >>>
> >>>> Hi Ilya,
> >>>>
> >>>> It will be a useful addition to Ignite. I would include the tool in
> one of
> >>>> the nearest releases. It’s license is BSD-3 which is compatible with
> ours.
> >>>> However, there is a note on ASF side saying that BSD licenses can be
> >>>> included without advertising clause [1]. What does it mean for us?
> Can we
> >>>> document the tool and mention on our site?
> >>>>
> >>>> [1] https://www.apache.org/legal/resolved.html#category-a <
> >>>> https://www.apache.org/legal/resolved.html#category-a>
> >>>>
> >>>> —
> >>>> Denis
> >>>>
> >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]>
> wrote:
> >>>>>
> >>>>> Hi Igniters,
> >>>>>
> >>>>> I've found one command line tool that can be used for execute queries
> >>>>> (DDL/DML) with data from Apache Ignite grid via
> >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> >>>>>
> >>>>> Here you can find overview of internal commands of this tool:
> >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> Overview+sqlline+tool
> >>>>>
> >>>>> The most problems with sqlline relate with some issues from our side
> or
> >>>>> some unsupported features.
> >>>>>
> >>>>> Would be nice to add tool like this (sqlline or something else) in
> our
> >>>>> project.
> >>>>>
> >>>>> Have you any objections or other thoughts?
> >>>>>
> >>>>>
> >>>>> --
> >>>>> Ilya Suntsov
> >>>>
> >>>>
> >>
> >
>
>


--
Ilya Suntsov
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dsetrakyan
Would be nice to get it in 2.3. This is critical functionality for our
users and 2.4 seems too far to give anyone comfort.

On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov <[hidden email]> wrote:

> Guys,
>
> I've created the ticket for 2.4 release:
> https://issues.apache.org/jira/browse/IGNITE-6561
>
> 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
>
> > Denis,
> >
> > I’m glad you’re thinking of using SQLLine. Under the BSD license, you
> > don’t need my permission to distribute, but I grant that permission.
> >
> > Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is in
> > good company.
> >
> > If you need extensions, please discuss on the dev list, or open a GitHub
> > case or pull request. SQLLine operates in the usual way for a GitHub
> > project. It’s unlikely that you’ll need Ignite-specific extensions —
> > SQLLine just exposes what comes through the JDBC driver — but we can
> > discuss if the need arises. The Hive project forked SQLLine into its own
> > Beeline module and I’d like to avoid a repeat of that.
> >
> > Julian
> >
> > > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]> wrote:
> > >
> > > Igniters,
> > >
> > > Let me introduce Julian Hyde [1], creator of SQLLine tool and our
> Apache
> > mate,
> > >
> > > Julian,
> > >
> > > Please grant that Apache Ignite community a permission to include
> > SQLLine [2] it in every Ignite deliverable (source, binary). It’s planned
> > to suggest the tool as a default command line SQL utility for Ignite
> > clusters. SQLLite and Ignite usage will also be documented on Ignite’s
> > technical documentation.
> > >
> > > [1] https://people.apache.org/~jhyde/ <https://people.apache.org/~jh
> yde/
> > >
> > > [2] https://github.com/julianhyde/sqlline <https://github.com/
> > julianhyde/sqlline>
> > >
> > > —
> > > Denis
> > >
> > >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email] <mailto:
> > [hidden email]>> wrote:
> > >>
> > >> Hi Ilya,
> > >>
> > >> Thanks for the clarification! Referring to the page shared by you [1]
> > if we need to get author’s consent in a written form:
> > >>
> > >> A permissive license similar to the BSD 2-Clause License, but with a
> > 3rd clause that prohibits others from using the name of the project or
> its
> > contributors to promote derived products without written consent.
> > >>
> > >> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> > https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> > >>
> > >> I’ll reach out the author requesting the permit.
> > >>
> > >> —
> > >> Denis
> > >>
> > >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> > [hidden email] <mailto:[hidden email]>> wrote:
> > >>>
> > >>> Hi Denis,
> > >>>
> > >>> There are two kinds of BSD license, 3-clause and 4-clause. The
> > difference
> > >>> between them is advertising clause:
> > >>>
> > >>> "3. All advertising materials mentioning features or use of this
> > software
> > >>> must display the following acknowledgement: This product includes
> > software
> > >>> developed by the University of California, Berkeley and its
> > contributors."
> > >>>
> > >>> which is causing problems with other licenses and which sqlline's
> > license
> > >>> doesn't contain:
> > >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> > >>> So it should be all good for inclusion.
> > >>>
> > >>> --
> > >>> Ilya Kasnacheev.
> > >>>
> > >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> > >>>
> > >>>> Hi Ilya,
> > >>>>
> > >>>> It will be a useful addition to Ignite. I would include the tool in
> > one of
> > >>>> the nearest releases. It’s license is BSD-3 which is compatible with
> > ours.
> > >>>> However, there is a note on ASF side saying that BSD licenses can be
> > >>>> included without advertising clause [1]. What does it mean for us?
> > Can we
> > >>>> document the tool and mention on our site?
> > >>>>
> > >>>> [1] https://www.apache.org/legal/resolved.html#category-a <
> > >>>> https://www.apache.org/legal/resolved.html#category-a>
> > >>>>
> > >>>> —
> > >>>> Denis
> > >>>>
> > >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]>
> > wrote:
> > >>>>>
> > >>>>> Hi Igniters,
> > >>>>>
> > >>>>> I've found one command line tool that can be used for execute
> queries
> > >>>>> (DDL/DML) with data from Apache Ignite grid via
> > >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> > >>>>>
> > >>>>> Here you can find overview of internal commands of this tool:
> > >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> > Overview+sqlline+tool
> > >>>>>
> > >>>>> The most problems with sqlline relate with some issues from our
> side
> > or
> > >>>>> some unsupported features.
> > >>>>>
> > >>>>> Would be nice to add tool like this (sqlline or something else) in
> > our
> > >>>>> project.
> > >>>>>
> > >>>>> Have you any objections or other thoughts?
> > >>>>>
> > >>>>>
> > >>>>> --
> > >>>>> Ilya Suntsov
> > >>>>
> > >>>>
> > >>
> > >
> >
> >
>
>
> --
> Ilya Suntsov
>
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Sergey Kozlov
Dmitriy, Denis

We're in progress to add sqlline in upcoming 2.3

On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan <[hidden email]>
wrote:

> Would be nice to get it in 2.3. This is critical functionality for our
> users and 2.4 seems too far to give anyone comfort.
>
> On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov <[hidden email]>
> wrote:
>
> > Guys,
> >
> > I've created the ticket for 2.4 release:
> > https://issues.apache.org/jira/browse/IGNITE-6561
> >
> > 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
> >
> > > Denis,
> > >
> > > I’m glad you’re thinking of using SQLLine. Under the BSD license, you
> > > don’t need my permission to distribute, but I grant that permission.
> > >
> > > Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is in
> > > good company.
> > >
> > > If you need extensions, please discuss on the dev list, or open a
> GitHub
> > > case or pull request. SQLLine operates in the usual way for a GitHub
> > > project. It’s unlikely that you’ll need Ignite-specific extensions —
> > > SQLLine just exposes what comes through the JDBC driver — but we can
> > > discuss if the need arises. The Hive project forked SQLLine into its
> own
> > > Beeline module and I’d like to avoid a repeat of that.
> > >
> > > Julian
> > >
> > > > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]> wrote:
> > > >
> > > > Igniters,
> > > >
> > > > Let me introduce Julian Hyde [1], creator of SQLLine tool and our
> > Apache
> > > mate,
> > > >
> > > > Julian,
> > > >
> > > > Please grant that Apache Ignite community a permission to include
> > > SQLLine [2] it in every Ignite deliverable (source, binary). It’s
> planned
> > > to suggest the tool as a default command line SQL utility for Ignite
> > > clusters. SQLLite and Ignite usage will also be documented on Ignite’s
> > > technical documentation.
> > > >
> > > > [1] https://people.apache.org/~jhyde/ <https://people.apache.org/~jh
> > yde/
> > > >
> > > > [2] https://github.com/julianhyde/sqlline <https://github.com/
> > > julianhyde/sqlline>
> > > >
> > > > —
> > > > Denis
> > > >
> > > >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
> <mailto:
> > > [hidden email]>> wrote:
> > > >>
> > > >> Hi Ilya,
> > > >>
> > > >> Thanks for the clarification! Referring to the page shared by you
> [1]
> > > if we need to get author’s consent in a written form:
> > > >>
> > > >> A permissive license similar to the BSD 2-Clause License, but with a
> > > 3rd clause that prohibits others from using the name of the project or
> > its
> > > contributors to promote derived products without written consent.
> > > >>
> > > >> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> > > >>
> > > >> I’ll reach out the author requesting the permit.
> > > >>
> > > >> —
> > > >> Denis
> > > >>
> > > >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> > > [hidden email] <mailto:[hidden email]>> wrote:
> > > >>>
> > > >>> Hi Denis,
> > > >>>
> > > >>> There are two kinds of BSD license, 3-clause and 4-clause. The
> > > difference
> > > >>> between them is advertising clause:
> > > >>>
> > > >>> "3. All advertising materials mentioning features or use of this
> > > software
> > > >>> must display the following acknowledgement: This product includes
> > > software
> > > >>> developed by the University of California, Berkeley and its
> > > contributors."
> > > >>>
> > > >>> which is causing problems with other licenses and which sqlline's
> > > license
> > > >>> doesn't contain:
> > > >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> > > >>> So it should be all good for inclusion.
> > > >>>
> > > >>> --
> > > >>> Ilya Kasnacheev.
> > > >>>
> > > >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> > > >>>
> > > >>>> Hi Ilya,
> > > >>>>
> > > >>>> It will be a useful addition to Ignite. I would include the tool
> in
> > > one of
> > > >>>> the nearest releases. It’s license is BSD-3 which is compatible
> with
> > > ours.
> > > >>>> However, there is a note on ASF side saying that BSD licenses can
> be
> > > >>>> included without advertising clause [1]. What does it mean for us?
> > > Can we
> > > >>>> document the tool and mention on our site?
> > > >>>>
> > > >>>> [1] https://www.apache.org/legal/resolved.html#category-a <
> > > >>>> https://www.apache.org/legal/resolved.html#category-a>
> > > >>>>
> > > >>>> —
> > > >>>> Denis
> > > >>>>
> > > >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]
> >
> > > wrote:
> > > >>>>>
> > > >>>>> Hi Igniters,
> > > >>>>>
> > > >>>>> I've found one command line tool that can be used for execute
> > queries
> > > >>>>> (DDL/DML) with data from Apache Ignite grid via
> > > >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> > > >>>>>
> > > >>>>> Here you can find overview of internal commands of this tool:
> > > >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> > > Overview+sqlline+tool
> > > >>>>>
> > > >>>>> The most problems with sqlline relate with some issues from our
> > side
> > > or
> > > >>>>> some unsupported features.
> > > >>>>>
> > > >>>>> Would be nice to add tool like this (sqlline or something else)
> in
> > > our
> > > >>>>> project.
> > > >>>>>
> > > >>>>> Have you any objections or other thoughts?
> > > >>>>>
> > > >>>>>
> > > >>>>> --
> > > >>>>> Ilya Suntsov
> > > >>>>
> > > >>>>
> > > >>
> > > >
> > >
> > >
> >
> >
> > --
> > Ilya Suntsov
> >
>



--
Sergey Kozlov
GridGain Systems
www.gridgain.com
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dmagda
Here is the original ticket [1]. Ilya, closed the one created by you as a duplicate.

In addition to the tool’s jar inclusion in Ignite’s binary releases let’s create a shell script to simplify the connectivity phase:
name the script as ignitedb.sh for Unix and ignitedb.bat for Windows.

the script uses the following connection string by default: .sqlline -d org.apache.ignite.IgniteJdbcThinDriver --color=true --verbose=true --showWarnings=true --showNestedErrs=true -u jdbc:ignite:thin://127.0.0.1/ <thin://127.0.0.1/>

make up parameters list to adjust Ignite specific part of the connection string: Ignite IP and port, streaming mode, etc. The full list of supported parameters is here: https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver <https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver>


[1] https://issues.apache.org/jira/browse/IGNITE-5608


Denis

> On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]> wrote:
>
> Dmitriy, Denis
>
> We're in progress to add sqlline in upcoming 2.3
>
> On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan <[hidden email]> wrote:
> Would be nice to get it in 2.3. This is critical functionality for our
> users and 2.4 seems too far to give anyone comfort.
>
> On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov <[hidden email]> wrote:
>
> > Guys,
> >
> > I've created the ticket for 2.4 release:
> > https://issues.apache.org/jira/browse/IGNITE-6561
> >
> > 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
> >
> > > Denis,
> > >
> > > I’m glad you’re thinking of using SQLLine. Under the BSD license, you
> > > don’t need my permission to distribute, but I grant that permission.
> > >
> > > Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is in
> > > good company.
> > >
> > > If you need extensions, please discuss on the dev list, or open a GitHub
> > > case or pull request. SQLLine operates in the usual way for a GitHub
> > > project. It’s unlikely that you’ll need Ignite-specific extensions —
> > > SQLLine just exposes what comes through the JDBC driver — but we can
> > > discuss if the need arises. The Hive project forked SQLLine into its own
> > > Beeline module and I’d like to avoid a repeat of that.
> > >
> > > Julian
> > >
> > > > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]> wrote:
> > > >
> > > > Igniters,
> > > >
> > > > Let me introduce Julian Hyde [1], creator of SQLLine tool and our
> > Apache
> > > mate,
> > > >
> > > > Julian,
> > > >
> > > > Please grant that Apache Ignite community a permission to include
> > > SQLLine [2] it in every Ignite deliverable (source, binary). It’s planned
> > > to suggest the tool as a default command line SQL utility for Ignite
> > > clusters. SQLLite and Ignite usage will also be documented on Ignite’s
> > > technical documentation.
> > > >
> > > > [1] https://people.apache.org/~jhyde/ <https://people.apache.org/~jh
> > yde/
> > > >
> > > > [2] https://github.com/julianhyde/sqlline <https://github.com/
> > > julianhyde/sqlline>
> > > >
> > > > —
> > > > Denis
> > > >
> > > >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email] <mailto:
> > > [hidden email]>> wrote:
> > > >>
> > > >> Hi Ilya,
> > > >>
> > > >> Thanks for the clarification! Referring to the page shared by you [1]
> > > if we need to get author’s consent in a written form:
> > > >>
> > > >> A permissive license similar to the BSD 2-Clause License, but with a
> > > 3rd clause that prohibits others from using the name of the project or
> > its
> > > contributors to promote derived products without written consent.
> > > >>
> > > >> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> > > >>
> > > >> I’ll reach out the author requesting the permit.
> > > >>
> > > >> —
> > > >> Denis
> > > >>
> > > >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> > > [hidden email] <mailto:[hidden email]>> wrote:
> > > >>>
> > > >>> Hi Denis,
> > > >>>
> > > >>> There are two kinds of BSD license, 3-clause and 4-clause. The
> > > difference
> > > >>> between them is advertising clause:
> > > >>>
> > > >>> "3. All advertising materials mentioning features or use of this
> > > software
> > > >>> must display the following acknowledgement: This product includes
> > > software
> > > >>> developed by the University of California, Berkeley and its
> > > contributors."
> > > >>>
> > > >>> which is causing problems with other licenses and which sqlline's
> > > license
> > > >>> doesn't contain:
> > > >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> > > >>> So it should be all good for inclusion.
> > > >>>
> > > >>> --
> > > >>> Ilya Kasnacheev.
> > > >>>
> > > >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> > > >>>
> > > >>>> Hi Ilya,
> > > >>>>
> > > >>>> It will be a useful addition to Ignite. I would include the tool in
> > > one of
> > > >>>> the nearest releases. It’s license is BSD-3 which is compatible with
> > > ours.
> > > >>>> However, there is a note on ASF side saying that BSD licenses can be
> > > >>>> included without advertising clause [1]. What does it mean for us?
> > > Can we
> > > >>>> document the tool and mention on our site?
> > > >>>>
> > > >>>> [1] https://www.apache.org/legal/resolved.html#category-a <
> > > >>>> https://www.apache.org/legal/resolved.html#category-a>
> > > >>>>
> > > >>>> —
> > > >>>> Denis
> > > >>>>
> > > >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]>
> > > wrote:
> > > >>>>>
> > > >>>>> Hi Igniters,
> > > >>>>>
> > > >>>>> I've found one command line tool that can be used for execute
> > queries
> > > >>>>> (DDL/DML) with data from Apache Ignite grid via
> > > >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> > > >>>>>
> > > >>>>> Here you can find overview of internal commands of this tool:
> > > >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> > > Overview+sqlline+tool
> > > >>>>>
> > > >>>>> The most problems with sqlline relate with some issues from our
> > side
> > > or
> > > >>>>> some unsupported features.
> > > >>>>>
> > > >>>>> Would be nice to add tool like this (sqlline or something else) in
> > > our
> > > >>>>> project.
> > > >>>>>
> > > >>>>> Have you any objections or other thoughts?
> > > >>>>>
> > > >>>>>
> > > >>>>> --
> > > >>>>> Ilya Suntsov
> > > >>>>
> > > >>>>
> > > >>
> > > >
> > >
> > >
> >
> >
> > --
> > Ilya Suntsov
> >
>
>
>
> --
> Sergey Kozlov
> GridGain Systems
> www.gridgain.com

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Sergey Kozlov
Denis

The link below has included sqlline. Please take a look:
https://ci.ignite.apache.org/viewLog.html?buildId=875441&buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3PrepareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz

On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]> wrote:

> Here is the original ticket [1]. Ilya, closed the one created by you as a
> duplicate.
>
> In addition to the tool’s jar inclusion in Ignite’s binary releases let’s
> create a shell script to simplify the connectivity phase:
>
>    - name the script as ignitedb.sh for Unix and ignitedb.bat for Windows.
>    -
>    - the script uses the following connection string by default: .sqlline
>    -d org.apache.ignite.IgniteJdbcThinDriver --color=true --verbose=true
>    --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
>    thin://127.0.0.1/
>
>
>
>    - make up parameters list to adjust Ignite specific part of the
>    connection string: Ignite IP and port, streaming mode, etc. The full list
>    of supported parameters is here: https://apacheignite-
>    sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>    <https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver>
>
>
>
> [1] https://issues.apache.org/jira/browse/IGNITE-5608
>
> —
> Denis
>
> On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]> wrote:
>
> Dmitriy, Denis
>
> We're in progress to add sqlline in upcoming 2.3
>
> On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan <[hidden email]
> > wrote:
> Would be nice to get it in 2.3. This is critical functionality for our
> users and 2.4 seems too far to give anyone comfort.
>
> On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov <[hidden email]>
> wrote:
>
> > Guys,
> >
> > I've created the ticket for 2.4 release:
> > https://issues.apache.org/jira/browse/IGNITE-6561
> >
> > 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
> >
> > > Denis,
> > >
> > > I’m glad you’re thinking of using SQLLine. Under the BSD license, you
> > > don’t need my permission to distribute, but I grant that permission.
> > >
> > > Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is in
> > > good company.
> > >
> > > If you need extensions, please discuss on the dev list, or open a
> GitHub
> > > case or pull request. SQLLine operates in the usual way for a GitHub
> > > project. It’s unlikely that you’ll need Ignite-specific extensions —
> > > SQLLine just exposes what comes through the JDBC driver — but we can
> > > discuss if the need arises. The Hive project forked SQLLine into its
> own
> > > Beeline module and I’d like to avoid a repeat of that.
> > >
> > > Julian
> > >
> > > > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]> wrote:
> > > >
> > > > Igniters,
> > > >
> > > > Let me introduce Julian Hyde [1], creator of SQLLine tool and our
> > Apache
> > > mate,
> > > >
> > > > Julian,
> > > >
> > > > Please grant that Apache Ignite community a permission to include
> > > SQLLine [2] it in every Ignite deliverable (source, binary). It’s
> planned
> > > to suggest the tool as a default command line SQL utility for Ignite
> > > clusters. SQLLite and Ignite usage will also be documented on Ignite’s
> > > technical documentation.
> > > >
> > > > [1] https://people.apache.org/~jhyde/ <https://people.apache.org/~jh
> > yde/
> > > >
> > > > [2] https://github.com/julianhyde/sqlline <https://github.com/
> > > julianhyde/sqlline>
> > > >
> > > > —
> > > > Denis
> > > >
> > > >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
>  <mailto:
> > > [hidden email]>> wrote:
> > > >>
> > > >> Hi Ilya,
> > > >>
> > > >> Thanks for the clarification! Referring to the page shared by you
> [1]
> > > if we need to get author’s consent in a written form:
> > > >>
> > > >> A permissive license similar to the BSD 2-Clause License, but with a
> > > 3rd clause that prohibits others from using the name of the project or
> > its
> > > contributors to promote derived products without written consent.
> > > >>
> > > >> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> > > >>
> > > >> I’ll reach out the author requesting the permit.
> > > >>
> > > >> —
> > > >> Denis
> > > >>
> > > >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> > > [hidden email] <mailto:[hidden email]>> wrote:
> > > >>>
> > > >>> Hi Denis,
> > > >>>
> > > >>> There are two kinds of BSD license, 3-clause and 4-clause. The
> > > difference
> > > >>> between them is advertising clause:
> > > >>>
> > > >>> "3. All advertising materials mentioning features or use of this
> > > software
> > > >>> must display the following acknowledgement: This product includes
> > > software
> > > >>> developed by the University of California, Berkeley and its
> > > contributors."
> > > >>>
> > > >>> which is causing problems with other licenses and which sqlline's
> > > license
> > > >>> doesn't contain:
> > > >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> > > >>> So it should be all good for inclusion.
> > > >>>
> > > >>> --
> > > >>> Ilya Kasnacheev.
> > > >>>
> > > >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> > > >>>
> > > >>>> Hi Ilya,
> > > >>>>
> > > >>>> It will be a useful addition to Ignite. I would include the tool
> in
> > > one of
> > > >>>> the nearest releases. It’s license is BSD-3 which is compatible
> with
> > > ours.
> > > >>>> However, there is a note on ASF side saying that BSD licenses can
> be
> > > >>>> included without advertising clause [1]. What does it mean for us?
> > > Can we
> > > >>>> document the tool and mention on our site?
> > > >>>>
> > > >>>> [1] https://www.apache.org/legal/resolved.html#category-a <
> > > >>>> https://www.apache.org/legal/resolved.html#category-a>
> > > >>>>
> > > >>>> —
> > > >>>> Denis
> > > >>>>
> > > >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <[hidden email]
> >
> > > wrote:
> > > >>>>>
> > > >>>>> Hi Igniters,
> > > >>>>>
> > > >>>>> I've found one command line tool that can be used for execute
> > queries
> > > >>>>> (DDL/DML) with data from Apache Ignite grid via
> > > >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> > > >>>>>
> > > >>>>> Here you can find overview of internal commands of this tool:
> > > >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> > > Overview+sqlline+tool
> > > >>>>>
> > > >>>>> The most problems with sqlline relate with some issues from our
> > side
> > > or
> > > >>>>> some unsupported features.
> > > >>>>>
> > > >>>>> Would be nice to add tool like this (sqlline or something else)
> in
> > > our
> > > >>>>> project.
> > > >>>>>
> > > >>>>> Have you any objections or other thoughts?
> > > >>>>>
> > > >>>>>
> > > >>>>> --
> > > >>>>> Ilya Suntsov
> > > >>>>
> > > >>>>
> > > >>
> > > >
> > >
> > >
> >
> >
> > --
> > Ilya Suntsov
> >
>
>
>
> --
> Sergey Kozlov
> GridGain Systems
> www.gridgain.com
>
>
>


--
Sergey Kozlov
GridGain Systems
www.gridgain.com
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Vladimir Ozerov
Denis,

Setting default host to 127.0.0.1 is bad idea, because it mean that in
practice users would have to change the script always. Instead, we should
accept host name as argument. This is perfectly fine from usability
perspective, most tools work this way (i.e. throw error when started
without arguments).

Also IMO "ignitedb" is misleading name. Users would like think that it is a
kind of script to start database, rather than to connect to it. We should
think on other names. E.g. "ignitesql".

On Fri, Oct 6, 2017 at 5:23 PM, Sergey Kozlov <[hidden email]> wrote:

> Denis
>
> The link below has included sqlline. Please take a look:
> https://ci.ignite.apache.org/viewLog.html?buildId=875441&
> buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3Pre
> pareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz
>
> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]> wrote:
>
> > Here is the original ticket [1]. Ilya, closed the one created by you as a
> > duplicate.
> >
> > In addition to the tool’s jar inclusion in Ignite’s binary releases let’s
> > create a shell script to simplify the connectivity phase:
> >
> >    - name the script as ignitedb.sh for Unix and ignitedb.bat for
> Windows.
> >    -
> >    - the script uses the following connection string by default: .sqlline
> >    -d org.apache.ignite.IgniteJdbcThinDriver --color=true --verbose=true
> >    --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
> >    thin://127.0.0.1/
> >
> >
> >
> >    - make up parameters list to adjust Ignite specific part of the
> >    connection string: Ignite IP and port, streaming mode, etc. The full
> list
> >    of supported parameters is here: https://apacheignite-
> >    sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
> >    <https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
> >
> >
> >
> >
> > [1] https://issues.apache.org/jira/browse/IGNITE-5608
> >
> > —
> > Denis
> >
> > On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]> wrote:
> >
> > Dmitriy, Denis
> >
> > We're in progress to add sqlline in upcoming 2.3
> >
> > On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan <[hidden email]
> > > wrote:
> > Would be nice to get it in 2.3. This is critical functionality for our
> > users and 2.4 seems too far to give anyone comfort.
> >
> > On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov <[hidden email]>
> > wrote:
> >
> > > Guys,
> > >
> > > I've created the ticket for 2.4 release:
> > > https://issues.apache.org/jira/browse/IGNITE-6561
> > >
> > > 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
> > >
> > > > Denis,
> > > >
> > > > I’m glad you’re thinking of using SQLLine. Under the BSD license, you
> > > > don’t need my permission to distribute, but I grant that permission.
> > > >
> > > > Drill, Phoenix and Calcite already distribute SQLLine, so Ignite is
> in
> > > > good company.
> > > >
> > > > If you need extensions, please discuss on the dev list, or open a
> > GitHub
> > > > case or pull request. SQLLine operates in the usual way for a GitHub
> > > > project. It’s unlikely that you’ll need Ignite-specific extensions —
> > > > SQLLine just exposes what comes through the JDBC driver — but we can
> > > > discuss if the need arises. The Hive project forked SQLLine into its
> > own
> > > > Beeline module and I’d like to avoid a repeat of that.
> > > >
> > > > Julian
> > > >
> > > > > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]>
> wrote:
> > > > >
> > > > > Igniters,
> > > > >
> > > > > Let me introduce Julian Hyde [1], creator of SQLLine tool and our
> > > Apache
> > > > mate,
> > > > >
> > > > > Julian,
> > > > >
> > > > > Please grant that Apache Ignite community a permission to include
> > > > SQLLine [2] it in every Ignite deliverable (source, binary). It’s
> > planned
> > > > to suggest the tool as a default command line SQL utility for Ignite
> > > > clusters. SQLLite and Ignite usage will also be documented on
> Ignite’s
> > > > technical documentation.
> > > > >
> > > > > [1] https://people.apache.org/~jhyde/ <
> https://people.apache.org/~jh
> > > yde/
> > > > >
> > > > > [2] https://github.com/julianhyde/sqlline <https://github.com/
> > > > julianhyde/sqlline>
> > > > >
> > > > > —
> > > > > Denis
> > > > >
> > > > >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
> >  <mailto:
> > > > [hidden email]>> wrote:
> > > > >>
> > > > >> Hi Ilya,
> > > > >>
> > > > >> Thanks for the clarification! Referring to the page shared by you
> > [1]
> > > > if we need to get author’s consent in a written form:
> > > > >>
> > > > >> A permissive license similar to the BSD 2-Clause License, but
> with a
> > > > 3rd clause that prohibits others from using the name of the project
> or
> > > its
> > > > contributors to promote derived products without written consent.
> > > > >>
> > > > >> [1] https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> > > > >>
> > > > >> I’ll reach out the author requesting the permit.
> > > > >>
> > > > >> —
> > > > >> Denis
> > > > >>
> > > > >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> > > > [hidden email] <mailto:[hidden email]>> wrote:
> > > > >>>
> > > > >>> Hi Denis,
> > > > >>>
> > > > >>> There are two kinds of BSD license, 3-clause and 4-clause. The
> > > > difference
> > > > >>> between them is advertising clause:
> > > > >>>
> > > > >>> "3. All advertising materials mentioning features or use of this
> > > > software
> > > > >>> must display the following acknowledgement: This product includes
> > > > software
> > > > >>> developed by the University of California, Berkeley and its
> > > > contributors."
> > > > >>>
> > > > >>> which is causing problems with other licenses and which sqlline's
> > > > license
> > > > >>> doesn't contain:
> > > > >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> > > > >>> So it should be all good for inclusion.
> > > > >>>
> > > > >>> --
> > > > >>> Ilya Kasnacheev.
> > > > >>>
> > > > >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> > > > >>>
> > > > >>>> Hi Ilya,
> > > > >>>>
> > > > >>>> It will be a useful addition to Ignite. I would include the tool
> > in
> > > > one of
> > > > >>>> the nearest releases. It’s license is BSD-3 which is compatible
> > with
> > > > ours.
> > > > >>>> However, there is a note on ASF side saying that BSD licenses
> can
> > be
> > > > >>>> included without advertising clause [1]. What does it mean for
> us?
> > > > Can we
> > > > >>>> document the tool and mention on our site?
> > > > >>>>
> > > > >>>> [1] https://www.apache.org/legal/resolved.html#category-a <
> > > > >>>> https://www.apache.org/legal/resolved.html#category-a>
> > > > >>>>
> > > > >>>> —
> > > > >>>> Denis
> > > > >>>>
> > > > >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <
> [hidden email]
> > >
> > > > wrote:
> > > > >>>>>
> > > > >>>>> Hi Igniters,
> > > > >>>>>
> > > > >>>>> I've found one command line tool that can be used for execute
> > > queries
> > > > >>>>> (DDL/DML) with data from Apache Ignite grid via
> > > > >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> > > > >>>>>
> > > > >>>>> Here you can find overview of internal commands of this tool:
> > > > >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> > > > Overview+sqlline+tool
> > > > >>>>>
> > > > >>>>> The most problems with sqlline relate with some issues from our
> > > side
> > > > or
> > > > >>>>> some unsupported features.
> > > > >>>>>
> > > > >>>>> Would be nice to add tool like this (sqlline or something else)
> > in
> > > > our
> > > > >>>>> project.
> > > > >>>>>
> > > > >>>>> Have you any objections or other thoughts?
> > > > >>>>>
> > > > >>>>>
> > > > >>>>> --
> > > > >>>>> Ilya Suntsov
> > > > >>>>
> > > > >>>>
> > > > >>
> > > > >
> > > >
> > > >
> > >
> > >
> > > --
> > > Ilya Suntsov
> > >
> >
> >
> >
> > --
> > Sergey Kozlov
> > GridGain Systems
> > www.gridgain.com
> >
> >
> >
>
>
> --
> Sergey Kozlov
> GridGain Systems
> www.gridgain.com
>
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dsetrakyan
I like ignitesql.

⁣D.​

On Oct 6, 2017, 4:49 PM, at 4:49 PM, Vladimir Ozerov <[hidden email]> wrote:

>Denis,
>
>Setting default host to 127.0.0.1 is bad idea, because it mean that in
>practice users would have to change the script always. Instead, we
>should
>accept host name as argument. This is perfectly fine from usability
>perspective, most tools work this way (i.e. throw error when started
>without arguments).
>
>Also IMO "ignitedb" is misleading name. Users would like think that it
>is a
>kind of script to start database, rather than to connect to it. We
>should
>think on other names. E.g. "ignitesql".
>
>On Fri, Oct 6, 2017 at 5:23 PM, Sergey Kozlov <[hidden email]>
>wrote:
>
>> Denis
>>
>> The link below has included sqlline. Please take a look:
>> https://ci.ignite.apache.org/viewLog.html?buildId=875441&
>> buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3Pre
>> pareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz
>>
>> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]>
>wrote:
>>
>> > Here is the original ticket [1]. Ilya, closed the one created by
>you as a
>> > duplicate.
>> >
>> > In addition to the tool’s jar inclusion in Ignite’s binary releases
>let’s
>> > create a shell script to simplify the connectivity phase:
>> >
>> >    - name the script as ignitedb.sh for Unix and ignitedb.bat for
>> Windows.
>> >    -
>> >    - the script uses the following connection string by default:
>.sqlline
>> >    -d org.apache.ignite.IgniteJdbcThinDriver --color=true
>--verbose=true
>> >    --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
>> >    thin://127.0.0.1/
>> >
>> >
>> >
>> >    - make up parameters list to adjust Ignite specific part of the
>> >    connection string: Ignite IP and port, streaming mode, etc. The
>full
>> list
>> >    of supported parameters is here: https://apacheignite-
>> >    sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>> >
><https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>> >
>> >
>> >
>> >
>> > [1] https://issues.apache.org/jira/browse/IGNITE-5608
>> >
>> > —
>> > Denis
>> >
>> > On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]>
>wrote:
>> >
>> > Dmitriy, Denis
>> >
>> > We're in progress to add sqlline in upcoming 2.3
>> >
>> > On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan
><[hidden email]
>> > > wrote:
>> > Would be nice to get it in 2.3. This is critical functionality for
>our
>> > users and 2.4 seems too far to give anyone comfort.
>> >
>> > On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov
><[hidden email]>
>> > wrote:
>> >
>> > > Guys,
>> > >
>> > > I've created the ticket for 2.4 release:
>> > > https://issues.apache.org/jira/browse/IGNITE-6561
>> > >
>> > > 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
>> > >
>> > > > Denis,
>> > > >
>> > > > I’m glad you’re thinking of using SQLLine. Under the BSD
>license, you
>> > > > don’t need my permission to distribute, but I grant that
>permission.
>> > > >
>> > > > Drill, Phoenix and Calcite already distribute SQLLine, so
>Ignite is
>> in
>> > > > good company.
>> > > >
>> > > > If you need extensions, please discuss on the dev list, or open
>a
>> > GitHub
>> > > > case or pull request. SQLLine operates in the usual way for a
>GitHub
>> > > > project. It’s unlikely that you’ll need Ignite-specific
>extensions —
>> > > > SQLLine just exposes what comes through the JDBC driver — but
>we can
>> > > > discuss if the need arises. The Hive project forked SQLLine
>into its
>> > own
>> > > > Beeline module and I’d like to avoid a repeat of that.
>> > > >
>> > > > Julian
>> > > >
>> > > > > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]>
>> wrote:
>> > > > >
>> > > > > Igniters,
>> > > > >
>> > > > > Let me introduce Julian Hyde [1], creator of SQLLine tool and
>our
>> > > Apache
>> > > > mate,
>> > > > >
>> > > > > Julian,
>> > > > >
>> > > > > Please grant that Apache Ignite community a permission to
>include
>> > > > SQLLine [2] it in every Ignite deliverable (source, binary).
>It’s
>> > planned
>> > > > to suggest the tool as a default command line SQL utility for
>Ignite
>> > > > clusters. SQLLite and Ignite usage will also be documented on
>> Ignite’s
>> > > > technical documentation.
>> > > > >
>> > > > > [1] https://people.apache.org/~jhyde/ <
>> https://people.apache.org/~jh
>> > > yde/
>> > > > >
>> > > > > [2] https://github.com/julianhyde/sqlline
><https://github.com/
>> > > > julianhyde/sqlline>
>> > > > >
>> > > > > —
>> > > > > Denis
>> > > > >
>> > > > >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
>> >  <mailto:
>> > > > [hidden email]>> wrote:
>> > > > >>
>> > > > >> Hi Ilya,
>> > > > >>
>> > > > >> Thanks for the clarification! Referring to the page shared
>by you
>> > [1]
>> > > > if we need to get author’s consent in a written form:
>> > > > >>
>> > > > >> A permissive license similar to the BSD 2-Clause License,
>but
>> with a
>> > > > 3rd clause that prohibits others from using the name of the
>project
>> or
>> > > its
>> > > > contributors to promote derived products without written
>consent.
>> > > > >>
>> > > > >> [1]
>https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
>> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
>> > > > >>
>> > > > >> I’ll reach out the author requesting the permit.
>> > > > >>
>> > > > >> —
>> > > > >> Denis
>> > > > >>
>> > > > >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
>> > > > [hidden email] <mailto:[hidden email]>>
>wrote:
>> > > > >>>
>> > > > >>> Hi Denis,
>> > > > >>>
>> > > > >>> There are two kinds of BSD license, 3-clause and 4-clause.
>The
>> > > > difference
>> > > > >>> between them is advertising clause:
>> > > > >>>
>> > > > >>> "3. All advertising materials mentioning features or use of
>this
>> > > > software
>> > > > >>> must display the following acknowledgement: This product
>includes
>> > > > software
>> > > > >>> developed by the University of California, Berkeley and its
>> > > > contributors."
>> > > > >>>
>> > > > >>> which is causing problems with other licenses and which
>sqlline's
>> > > > license
>> > > > >>> doesn't contain:
>> > > > >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>
>> > > > >>> So it should be all good for inclusion.
>> > > > >>>
>> > > > >>> --
>> > > > >>> Ilya Kasnacheev.
>> > > > >>>
>> > > > >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
>> > > > >>>
>> > > > >>>> Hi Ilya,
>> > > > >>>>
>> > > > >>>> It will be a useful addition to Ignite. I would include
>the tool
>> > in
>> > > > one of
>> > > > >>>> the nearest releases. It’s license is BSD-3 which is
>compatible
>> > with
>> > > > ours.
>> > > > >>>> However, there is a note on ASF side saying that BSD
>licenses
>> can
>> > be
>> > > > >>>> included without advertising clause [1]. What does it mean
>for
>> us?
>> > > > Can we
>> > > > >>>> document the tool and mention on our site?
>> > > > >>>>
>> > > > >>>> [1] https://www.apache.org/legal/resolved.html#category-a
><
>> > > > >>>> https://www.apache.org/legal/resolved.html#category-a>
>> > > > >>>>
>> > > > >>>> —
>> > > > >>>> Denis
>> > > > >>>>
>> > > > >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <
>> [hidden email]
>> > >
>> > > > wrote:
>> > > > >>>>>
>> > > > >>>>> Hi Igniters,
>> > > > >>>>>
>> > > > >>>>> I've found one command line tool that can be used for
>execute
>> > > queries
>> > > > >>>>> (DDL/DML) with data from Apache Ignite grid via
>> > > > >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
>> > > > >>>>>
>> > > > >>>>> Here you can find overview of internal commands of this
>tool:
>> > > > >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
>> > > > Overview+sqlline+tool
>> > > > >>>>>
>> > > > >>>>> The most problems with sqlline relate with some issues
>from our
>> > > side
>> > > > or
>> > > > >>>>> some unsupported features.
>> > > > >>>>>
>> > > > >>>>> Would be nice to add tool like this (sqlline or something
>else)
>> > in
>> > > > our
>> > > > >>>>> project.
>> > > > >>>>>
>> > > > >>>>> Have you any objections or other thoughts?
>> > > > >>>>>
>> > > > >>>>>
>> > > > >>>>> --
>> > > > >>>>> Ilya Suntsov
>> > > > >>>>
>> > > > >>>>
>> > > > >>
>> > > > >
>> > > >
>> > > >
>> > >
>> > >
>> > > --
>> > > Ilya Suntsov
>> > >
>> >
>> >
>> >
>> > --
>> > Sergey Kozlov
>> > GridGain Systems
>> > www.gridgain.com
>> >
>> >
>> >
>>
>>
>> --
>> Sergey Kozlov
>> GridGain Systems
>> www.gridgain.com
>>
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

Anton Vinogradov
How about sqlconsole.sh or sqlcmd.sh ?

On Fri, Oct 6, 2017 at 6:04 PM, <[hidden email]> wrote:

> I like ignitesql.
>
> ⁣D.​
>
> On Oct 6, 2017, 4:49 PM, at 4:49 PM, Vladimir Ozerov <[hidden email]>
> wrote:
> >Denis,
> >
> >Setting default host to 127.0.0.1 is bad idea, because it mean that in
> >practice users would have to change the script always. Instead, we
> >should
> >accept host name as argument. This is perfectly fine from usability
> >perspective, most tools work this way (i.e. throw error when started
> >without arguments).
> >
> >Also IMO "ignitedb" is misleading name. Users would like think that it
> >is a
> >kind of script to start database, rather than to connect to it. We
> >should
> >think on other names. E.g. "ignitesql".
> >
> >On Fri, Oct 6, 2017 at 5:23 PM, Sergey Kozlov <[hidden email]>
> >wrote:
> >
> >> Denis
> >>
> >> The link below has included sqlline. Please take a look:
> >> https://ci.ignite.apache.org/viewLog.html?buildId=875441&
> >> buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3Pre
> >> pareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz
> >>
> >> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]>
> >wrote:
> >>
> >> > Here is the original ticket [1]. Ilya, closed the one created by
> >you as a
> >> > duplicate.
> >> >
> >> > In addition to the tool’s jar inclusion in Ignite’s binary releases
> >let’s
> >> > create a shell script to simplify the connectivity phase:
> >> >
> >> >    - name the script as ignitedb.sh for Unix and ignitedb.bat for
> >> Windows.
> >> >    -
> >> >    - the script uses the following connection string by default:
> >.sqlline
> >> >    -d org.apache.ignite.IgniteJdbcThinDriver --color=true
> >--verbose=true
> >> >    --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
> >> >    thin://127.0.0.1/
> >> >
> >> >
> >> >
> >> >    - make up parameters list to adjust Ignite specific part of the
> >> >    connection string: Ignite IP and port, streaming mode, etc. The
> >full
> >> list
> >> >    of supported parameters is here: https://apacheignite-
> >> >    sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
> >> >
> ><https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
> >> >
> >> >
> >> >
> >> >
> >> > [1] https://issues.apache.org/jira/browse/IGNITE-5608
> >> >
> >> > —
> >> > Denis
> >> >
> >> > On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]>
> >wrote:
> >> >
> >> > Dmitriy, Denis
> >> >
> >> > We're in progress to add sqlline in upcoming 2.3
> >> >
> >> > On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan
> ><[hidden email]
> >> > > wrote:
> >> > Would be nice to get it in 2.3. This is critical functionality for
> >our
> >> > users and 2.4 seems too far to give anyone comfort.
> >> >
> >> > On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov
> ><[hidden email]>
> >> > wrote:
> >> >
> >> > > Guys,
> >> > >
> >> > > I've created the ticket for 2.4 release:
> >> > > https://issues.apache.org/jira/browse/IGNITE-6561
> >> > >
> >> > > 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
> >> > >
> >> > > > Denis,
> >> > > >
> >> > > > I’m glad you’re thinking of using SQLLine. Under the BSD
> >license, you
> >> > > > don’t need my permission to distribute, but I grant that
> >permission.
> >> > > >
> >> > > > Drill, Phoenix and Calcite already distribute SQLLine, so
> >Ignite is
> >> in
> >> > > > good company.
> >> > > >
> >> > > > If you need extensions, please discuss on the dev list, or open
> >a
> >> > GitHub
> >> > > > case or pull request. SQLLine operates in the usual way for a
> >GitHub
> >> > > > project. It’s unlikely that you’ll need Ignite-specific
> >extensions —
> >> > > > SQLLine just exposes what comes through the JDBC driver — but
> >we can
> >> > > > discuss if the need arises. The Hive project forked SQLLine
> >into its
> >> > own
> >> > > > Beeline module and I’d like to avoid a repeat of that.
> >> > > >
> >> > > > Julian
> >> > > >
> >> > > > > On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]>
> >> wrote:
> >> > > > >
> >> > > > > Igniters,
> >> > > > >
> >> > > > > Let me introduce Julian Hyde [1], creator of SQLLine tool and
> >our
> >> > > Apache
> >> > > > mate,
> >> > > > >
> >> > > > > Julian,
> >> > > > >
> >> > > > > Please grant that Apache Ignite community a permission to
> >include
> >> > > > SQLLine [2] it in every Ignite deliverable (source, binary).
> >It’s
> >> > planned
> >> > > > to suggest the tool as a default command line SQL utility for
> >Ignite
> >> > > > clusters. SQLLite and Ignite usage will also be documented on
> >> Ignite’s
> >> > > > technical documentation.
> >> > > > >
> >> > > > > [1] https://people.apache.org/~jhyde/ <
> >> https://people.apache.org/~jh
> >> > > yde/
> >> > > > >
> >> > > > > [2] https://github.com/julianhyde/sqlline
> ><https://github.com/
> >> > > > julianhyde/sqlline>
> >> > > > >
> >> > > > > —
> >> > > > > Denis
> >> > > > >
> >> > > > >> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
> >> >  <mailto:
> >> > > > [hidden email]>> wrote:
> >> > > > >>
> >> > > > >> Hi Ilya,
> >> > > > >>
> >> > > > >> Thanks for the clarification! Referring to the page shared
> >by you
> >> > [1]
> >> > > > if we need to get author’s consent in a written form:
> >> > > > >>
> >> > > > >> A permissive license similar to the BSD 2-Clause License,
> >but
> >> with a
> >> > > > 3rd clause that prohibits others from using the name of the
> >project
> >> or
> >> > > its
> >> > > > contributors to promote derived products without written
> >consent.
> >> > > > >>
> >> > > > >> [1]
> >https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> >> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> >> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> >> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> >> > > > >>
> >> > > > >> I’ll reach out the author requesting the permit.
> >> > > > >>
> >> > > > >> —
> >> > > > >> Denis
> >> > > > >>
> >> > > > >>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> >> > > > [hidden email] <mailto:[hidden email]>>
> >wrote:
> >> > > > >>>
> >> > > > >>> Hi Denis,
> >> > > > >>>
> >> > > > >>> There are two kinds of BSD license, 3-clause and 4-clause.
> >The
> >> > > > difference
> >> > > > >>> between them is advertising clause:
> >> > > > >>>
> >> > > > >>> "3. All advertising materials mentioning features or use of
> >this
> >> > > > software
> >> > > > >>> must display the following acknowledgement: This product
> >includes
> >> > > > software
> >> > > > >>> developed by the University of California, Berkeley and its
> >> > > > contributors."
> >> > > > >>>
> >> > > > >>> which is causing problems with other licenses and which
> >sqlline's
> >> > > > license
> >> > > > >>> doesn't contain:
> >> > > > >>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> >> > > > https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> >> > > > >>> So it should be all good for inclusion.
> >> > > > >>>
> >> > > > >>> --
> >> > > > >>> Ilya Kasnacheev.
> >> > > > >>>
> >> > > > >>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> >> > > > >>>
> >> > > > >>>> Hi Ilya,
> >> > > > >>>>
> >> > > > >>>> It will be a useful addition to Ignite. I would include
> >the tool
> >> > in
> >> > > > one of
> >> > > > >>>> the nearest releases. It’s license is BSD-3 which is
> >compatible
> >> > with
> >> > > > ours.
> >> > > > >>>> However, there is a note on ASF side saying that BSD
> >licenses
> >> can
> >> > be
> >> > > > >>>> included without advertising clause [1]. What does it mean
> >for
> >> us?
> >> > > > Can we
> >> > > > >>>> document the tool and mention on our site?
> >> > > > >>>>
> >> > > > >>>> [1] https://www.apache.org/legal/resolved.html#category-a
> ><
> >> > > > >>>> https://www.apache.org/legal/resolved.html#category-a>
> >> > > > >>>>
> >> > > > >>>> —
> >> > > > >>>> Denis
> >> > > > >>>>
> >> > > > >>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <
> >> [hidden email]
> >> > >
> >> > > > wrote:
> >> > > > >>>>>
> >> > > > >>>>> Hi Igniters,
> >> > > > >>>>>
> >> > > > >>>>> I've found one command line tool that can be used for
> >execute
> >> > > queries
> >> > > > >>>>> (DDL/DML) with data from Apache Ignite grid via
> >> > > > >>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> >> > > > >>>>>
> >> > > > >>>>> Here you can find overview of internal commands of this
> >tool:
> >> > > > >>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> >> > > > Overview+sqlline+tool
> >> > > > >>>>>
> >> > > > >>>>> The most problems with sqlline relate with some issues
> >from our
> >> > > side
> >> > > > or
> >> > > > >>>>> some unsupported features.
> >> > > > >>>>>
> >> > > > >>>>> Would be nice to add tool like this (sqlline or something
> >else)
> >> > in
> >> > > > our
> >> > > > >>>>> project.
> >> > > > >>>>>
> >> > > > >>>>> Have you any objections or other thoughts?
> >> > > > >>>>>
> >> > > > >>>>>
> >> > > > >>>>> --
> >> > > > >>>>> Ilya Suntsov
> >> > > > >>>>
> >> > > > >>>>
> >> > > > >>
> >> > > > >
> >> > > >
> >> > > >
> >> > >
> >> > >
> >> > > --
> >> > > Ilya Suntsov
> >> > >
> >> >
> >> >
> >> >
> >> > --
> >> > Sergey Kozlov
> >> > GridGain Systems
> >> > www.gridgain.com
> >> >
> >> >
> >> >
> >>
> >>
> >> --
> >> Sergey Kozlov
> >> GridGain Systems
> >> www.gridgain.com
> >>
>
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dmagda
Really nice, could click through the getting started [1] in a minute!

+1 to rename the script to “ignitesql”. Vladimir’s point makes total sense.

However, tend to disagree that the host has to be requested all the times. We never request a configuration or host name for ignite.sh, visor or web agent scripts. I would follow this approach that’s excellent for dev time.

But, we need to support “help” (-h, -help) argument listing all the parameters accepted by the tools.

Please consider our feedback and share the next build once it’s ready.


[1] https://apacheignite-sql.readme.io/v2.1/docs/getting-started <https://apacheignite-sql.readme.io/v2.1/docs/getting-started>


Denis

> On Oct 6, 2017, at 9:04 AM, Anton Vinogradov <[hidden email]> wrote:
>
> How about sqlconsole.sh or sqlcmd.sh ?
>
> On Fri, Oct 6, 2017 at 6:04 PM, <[hidden email]> wrote:
>
>> I like ignitesql.
>>
>> ⁣D.​
>>
>> On Oct 6, 2017, 4:49 PM, at 4:49 PM, Vladimir Ozerov <[hidden email]>
>> wrote:
>>> Denis,
>>>
>>> Setting default host to 127.0.0.1 is bad idea, because it mean that in
>>> practice users would have to change the script always. Instead, we
>>> should
>>> accept host name as argument. This is perfectly fine from usability
>>> perspective, most tools work this way (i.e. throw error when started
>>> without arguments).
>>>
>>> Also IMO "ignitedb" is misleading name. Users would like think that it
>>> is a
>>> kind of script to start database, rather than to connect to it. We
>>> should
>>> think on other names. E.g. "ignitesql".
>>>
>>> On Fri, Oct 6, 2017 at 5:23 PM, Sergey Kozlov <[hidden email]>
>>> wrote:
>>>
>>>> Denis
>>>>
>>>> The link below has included sqlline. Please take a look:
>>>> https://ci.ignite.apache.org/viewLog.html?buildId=875441&
>>>> buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3Pre
>>>> pareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz
>>>>
>>>> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]>
>>> wrote:
>>>>
>>>>> Here is the original ticket [1]. Ilya, closed the one created by
>>> you as a
>>>>> duplicate.
>>>>>
>>>>> In addition to the tool’s jar inclusion in Ignite’s binary releases
>>> let’s
>>>>> create a shell script to simplify the connectivity phase:
>>>>>
>>>>>   - name the script as ignitedb.sh for Unix and ignitedb.bat for
>>>> Windows.
>>>>>   -
>>>>>   - the script uses the following connection string by default:
>>> .sqlline
>>>>>   -d org.apache.ignite.IgniteJdbcThinDriver --color=true
>>> --verbose=true
>>>>>   --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
>>>>>   thin://127.0.0.1/
>>>>>
>>>>>
>>>>>
>>>>>   - make up parameters list to adjust Ignite specific part of the
>>>>>   connection string: Ignite IP and port, streaming mode, etc. The
>>> full
>>>> list
>>>>>   of supported parameters is here: https://apacheignite-
>>>>>   sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>>>>>
>>> <https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-5608
>>>>>
>>>>> —
>>>>> Denis
>>>>>
>>>>> On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]>
>>> wrote:
>>>>>
>>>>> Dmitriy, Denis
>>>>>
>>>>> We're in progress to add sqlline in upcoming 2.3
>>>>>
>>>>> On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan
>>> <[hidden email]
>>>>>> wrote:
>>>>> Would be nice to get it in 2.3. This is critical functionality for
>>> our
>>>>> users and 2.4 seems too far to give anyone comfort.
>>>>>
>>>>> On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov
>>> <[hidden email]>
>>>>> wrote:
>>>>>
>>>>>> Guys,
>>>>>>
>>>>>> I've created the ticket for 2.4 release:
>>>>>> https://issues.apache.org/jira/browse/IGNITE-6561
>>>>>>
>>>>>> 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
>>>>>>
>>>>>>> Denis,
>>>>>>>
>>>>>>> I’m glad you’re thinking of using SQLLine. Under the BSD
>>> license, you
>>>>>>> don’t need my permission to distribute, but I grant that
>>> permission.
>>>>>>>
>>>>>>> Drill, Phoenix and Calcite already distribute SQLLine, so
>>> Ignite is
>>>> in
>>>>>>> good company.
>>>>>>>
>>>>>>> If you need extensions, please discuss on the dev list, or open
>>> a
>>>>> GitHub
>>>>>>> case or pull request. SQLLine operates in the usual way for a
>>> GitHub
>>>>>>> project. It’s unlikely that you’ll need Ignite-specific
>>> extensions —
>>>>>>> SQLLine just exposes what comes through the JDBC driver — but
>>> we can
>>>>>>> discuss if the need arises. The Hive project forked SQLLine
>>> into its
>>>>> own
>>>>>>> Beeline module and I’d like to avoid a repeat of that.
>>>>>>>
>>>>>>> Julian
>>>>>>>
>>>>>>>> On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]>
>>>> wrote:
>>>>>>>>
>>>>>>>> Igniters,
>>>>>>>>
>>>>>>>> Let me introduce Julian Hyde [1], creator of SQLLine tool and
>>> our
>>>>>> Apache
>>>>>>> mate,
>>>>>>>>
>>>>>>>> Julian,
>>>>>>>>
>>>>>>>> Please grant that Apache Ignite community a permission to
>>> include
>>>>>>> SQLLine [2] it in every Ignite deliverable (source, binary).
>>> It’s
>>>>> planned
>>>>>>> to suggest the tool as a default command line SQL utility for
>>> Ignite
>>>>>>> clusters. SQLLite and Ignite usage will also be documented on
>>>> Ignite’s
>>>>>>> technical documentation.
>>>>>>>>
>>>>>>>> [1] https://people.apache.org/~jhyde/ <
>>>> https://people.apache.org/~jh
>>>>>> yde/
>>>>>>>>
>>>>>>>> [2] https://github.com/julianhyde/sqlline
>>> <https://github.com/
>>>>>>> julianhyde/sqlline>
>>>>>>>>
>>>>>>>> —
>>>>>>>> Denis
>>>>>>>>
>>>>>>>>> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
>>>>> <mailto:
>>>>>>> [hidden email]>> wrote:
>>>>>>>>>
>>>>>>>>> Hi Ilya,
>>>>>>>>>
>>>>>>>>> Thanks for the clarification! Referring to the page shared
>>> by you
>>>>> [1]
>>>>>>> if we need to get author’s consent in a written form:
>>>>>>>>>
>>>>>>>>> A permissive license similar to the BSD 2-Clause License,
>>> but
>>>> with a
>>>>>>> 3rd clause that prohibits others from using the name of the
>>> project
>>>> or
>>>>>> its
>>>>>>> contributors to promote derived products without written
>>> consent.
>>>>>>>>>
>>>>>>>>> [1]
>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
>>>>>>>>>
>>>>>>>>> I’ll reach out the author requesting the permit.
>>>>>>>>>
>>>>>>>>> —
>>>>>>>>> Denis
>>>>>>>>>
>>>>>>>>>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
>>>>>>> [hidden email] <mailto:[hidden email]>>
>>> wrote:
>>>>>>>>>>
>>>>>>>>>> Hi Denis,
>>>>>>>>>>
>>>>>>>>>> There are two kinds of BSD license, 3-clause and 4-clause.
>>> The
>>>>>>> difference
>>>>>>>>>> between them is advertising clause:
>>>>>>>>>>
>>>>>>>>>> "3. All advertising materials mentioning features or use of
>>> this
>>>>>>> software
>>>>>>>>>> must display the following acknowledgement: This product
>>> includes
>>>>>>> software
>>>>>>>>>> developed by the University of California, Berkeley and its
>>>>>>> contributors."
>>>>>>>>>>
>>>>>>>>>> which is causing problems with other licenses and which
>>> sqlline's
>>>>>>> license
>>>>>>>>>> doesn't contain:
>>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>
>>>>>>>>>> So it should be all good for inclusion.
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> Ilya Kasnacheev.
>>>>>>>>>>
>>>>>>>>>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
>>>>>>>>>>
>>>>>>>>>>> Hi Ilya,
>>>>>>>>>>>
>>>>>>>>>>> It will be a useful addition to Ignite. I would include
>>> the tool
>>>>> in
>>>>>>> one of
>>>>>>>>>>> the nearest releases. It’s license is BSD-3 which is
>>> compatible
>>>>> with
>>>>>>> ours.
>>>>>>>>>>> However, there is a note on ASF side saying that BSD
>>> licenses
>>>> can
>>>>> be
>>>>>>>>>>> included without advertising clause [1]. What does it mean
>>> for
>>>> us?
>>>>>>> Can we
>>>>>>>>>>> document the tool and mention on our site?
>>>>>>>>>>>
>>>>>>>>>>> [1] https://www.apache.org/legal/resolved.html#category-a
>>> <
>>>>>>>>>>> https://www.apache.org/legal/resolved.html#category-a>
>>>>>>>>>>>
>>>>>>>>>>> —
>>>>>>>>>>> Denis
>>>>>>>>>>>
>>>>>>>>>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <
>>>> [hidden email]
>>>>>>
>>>>>>> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>> Hi Igniters,
>>>>>>>>>>>>
>>>>>>>>>>>> I've found one command line tool that can be used for
>>> execute
>>>>>> queries
>>>>>>>>>>>> (DDL/DML) with data from Apache Ignite grid via
>>>>>>>>>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
>>>>>>>>>>>>
>>>>>>>>>>>> Here you can find overview of internal commands of this
>>> tool:
>>>>>>>>>>>> https://cwiki.apache.org/confluence/display/IGNITE/
>>>>>>> Overview+sqlline+tool
>>>>>>>>>>>>
>>>>>>>>>>>> The most problems with sqlline relate with some issues
>>> from our
>>>>>> side
>>>>>>> or
>>>>>>>>>>>> some unsupported features.
>>>>>>>>>>>>
>>>>>>>>>>>> Would be nice to add tool like this (sqlline or something
>>> else)
>>>>> in
>>>>>>> our
>>>>>>>>>>>> project.
>>>>>>>>>>>>
>>>>>>>>>>>> Have you any objections or other thoughts?
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> --
>>>>>>>>>>>> Ilya Suntsov
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Ilya Suntsov
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Sergey Kozlov
>>>>> GridGain Systems
>>>>> www.gridgain.com
>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Sergey Kozlov
>>>> GridGain Systems
>>>> www.gridgain.com
>>>>
>>

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dmagda
> But, we need to support “help” (-h, -help) argument listing all the parameters accepted by the tools.

Meant accepted by the ignitesql script only such as host name.


Denis

> On Oct 6, 2017, at 12:20 PM, Denis Magda <[hidden email]> wrote:
>
> Really nice, could click through the getting started [1] in a minute!
>
> +1 to rename the script to “ignitesql”. Vladimir’s point makes total sense.
>
> However, tend to disagree that the host has to be requested all the times. We never request a configuration or host name for ignite.sh, visor or web agent scripts. I would follow this approach that’s excellent for dev time.
>
> But, we need to support “help” (-h, -help) argument listing all the parameters accepted by the tools.
>
> Please consider our feedback and share the next build once it’s ready.
>
>
> [1] https://apacheignite-sql.readme.io/v2.1/docs/getting-started <https://apacheignite-sql.readme.io/v2.1/docs/getting-started>
>
> —
> Denis
>
>> On Oct 6, 2017, at 9:04 AM, Anton Vinogradov <[hidden email]> wrote:
>>
>> How about sqlconsole.sh or sqlcmd.sh ?
>>
>> On Fri, Oct 6, 2017 at 6:04 PM, <[hidden email]> wrote:
>>
>>> I like ignitesql.
>>>
>>> ⁣D.​
>>>
>>> On Oct 6, 2017, 4:49 PM, at 4:49 PM, Vladimir Ozerov <[hidden email]>
>>> wrote:
>>>> Denis,
>>>>
>>>> Setting default host to 127.0.0.1 is bad idea, because it mean that in
>>>> practice users would have to change the script always. Instead, we
>>>> should
>>>> accept host name as argument. This is perfectly fine from usability
>>>> perspective, most tools work this way (i.e. throw error when started
>>>> without arguments).
>>>>
>>>> Also IMO "ignitedb" is misleading name. Users would like think that it
>>>> is a
>>>> kind of script to start database, rather than to connect to it. We
>>>> should
>>>> think on other names. E.g. "ignitesql".
>>>>
>>>> On Fri, Oct 6, 2017 at 5:23 PM, Sergey Kozlov <[hidden email]>
>>>> wrote:
>>>>
>>>>> Denis
>>>>>
>>>>> The link below has included sqlline. Please take a look:
>>>>> https://ci.ignite.apache.org/viewLog.html?buildId=875441&
>>>>> buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3Pre
>>>>> pareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz
>>>>>
>>>>> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]>
>>>> wrote:
>>>>>
>>>>>> Here is the original ticket [1]. Ilya, closed the one created by
>>>> you as a
>>>>>> duplicate.
>>>>>>
>>>>>> In addition to the tool’s jar inclusion in Ignite’s binary releases
>>>> let’s
>>>>>> create a shell script to simplify the connectivity phase:
>>>>>>
>>>>>>  - name the script as ignitedb.sh for Unix and ignitedb.bat for
>>>>> Windows.
>>>>>>  -
>>>>>>  - the script uses the following connection string by default:
>>>> .sqlline
>>>>>>  -d org.apache.ignite.IgniteJdbcThinDriver --color=true
>>>> --verbose=true
>>>>>>  --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
>>>>>>  thin://127.0.0.1/
>>>>>>
>>>>>>
>>>>>>
>>>>>>  - make up parameters list to adjust Ignite specific part of the
>>>>>>  connection string: Ignite IP and port, streaming mode, etc. The
>>>> full
>>>>> list
>>>>>>  of supported parameters is here: https://apacheignite-
>>>>>>  sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>>>>>>
>>>> <https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-5608
>>>>>>
>>>>>> —
>>>>>> Denis
>>>>>>
>>>>>> On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]>
>>>> wrote:
>>>>>>
>>>>>> Dmitriy, Denis
>>>>>>
>>>>>> We're in progress to add sqlline in upcoming 2.3
>>>>>>
>>>>>> On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan
>>>> <[hidden email]
>>>>>>> wrote:
>>>>>> Would be nice to get it in 2.3. This is critical functionality for
>>>> our
>>>>>> users and 2.4 seems too far to give anyone comfort.
>>>>>>
>>>>>> On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov
>>>> <[hidden email]>
>>>>>> wrote:
>>>>>>
>>>>>>> Guys,
>>>>>>>
>>>>>>> I've created the ticket for 2.4 release:
>>>>>>> https://issues.apache.org/jira/browse/IGNITE-6561
>>>>>>>
>>>>>>> 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
>>>>>>>
>>>>>>>> Denis,
>>>>>>>>
>>>>>>>> I’m glad you’re thinking of using SQLLine. Under the BSD
>>>> license, you
>>>>>>>> don’t need my permission to distribute, but I grant that
>>>> permission.
>>>>>>>>
>>>>>>>> Drill, Phoenix and Calcite already distribute SQLLine, so
>>>> Ignite is
>>>>> in
>>>>>>>> good company.
>>>>>>>>
>>>>>>>> If you need extensions, please discuss on the dev list, or open
>>>> a
>>>>>> GitHub
>>>>>>>> case or pull request. SQLLine operates in the usual way for a
>>>> GitHub
>>>>>>>> project. It’s unlikely that you’ll need Ignite-specific
>>>> extensions —
>>>>>>>> SQLLine just exposes what comes through the JDBC driver — but
>>>> we can
>>>>>>>> discuss if the need arises. The Hive project forked SQLLine
>>>> into its
>>>>>> own
>>>>>>>> Beeline module and I’d like to avoid a repeat of that.
>>>>>>>>
>>>>>>>> Julian
>>>>>>>>
>>>>>>>>> On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]>
>>>>> wrote:
>>>>>>>>>
>>>>>>>>> Igniters,
>>>>>>>>>
>>>>>>>>> Let me introduce Julian Hyde [1], creator of SQLLine tool and
>>>> our
>>>>>>> Apache
>>>>>>>> mate,
>>>>>>>>>
>>>>>>>>> Julian,
>>>>>>>>>
>>>>>>>>> Please grant that Apache Ignite community a permission to
>>>> include
>>>>>>>> SQLLine [2] it in every Ignite deliverable (source, binary).
>>>> It’s
>>>>>> planned
>>>>>>>> to suggest the tool as a default command line SQL utility for
>>>> Ignite
>>>>>>>> clusters. SQLLite and Ignite usage will also be documented on
>>>>> Ignite’s
>>>>>>>> technical documentation.
>>>>>>>>>
>>>>>>>>> [1] https://people.apache.org/~jhyde/ <
>>>>> https://people.apache.org/~jh
>>>>>>> yde/
>>>>>>>>>
>>>>>>>>> [2] https://github.com/julianhyde/sqlline
>>>> <https://github.com/
>>>>>>>> julianhyde/sqlline>
>>>>>>>>>
>>>>>>>>> —
>>>>>>>>> Denis
>>>>>>>>>
>>>>>>>>>> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
>>>>>> <mailto:
>>>>>>>> [hidden email]>> wrote:
>>>>>>>>>>
>>>>>>>>>> Hi Ilya,
>>>>>>>>>>
>>>>>>>>>> Thanks for the clarification! Referring to the page shared
>>>> by you
>>>>>> [1]
>>>>>>>> if we need to get author’s consent in a written form:
>>>>>>>>>>
>>>>>>>>>> A permissive license similar to the BSD 2-Clause License,
>>>> but
>>>>> with a
>>>>>>>> 3rd clause that prohibits others from using the name of the
>>>> project
>>>>> or
>>>>>>> its
>>>>>>>> contributors to promote derived products without written
>>>> consent.
>>>>>>>>>>
>>>>>>>>>> [1]
>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
>>>>>>>>>>
>>>>>>>>>> I’ll reach out the author requesting the permit.
>>>>>>>>>>
>>>>>>>>>> —
>>>>>>>>>> Denis
>>>>>>>>>>
>>>>>>>>>>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
>>>>>>>> [hidden email] <mailto:[hidden email]>>
>>>> wrote:
>>>>>>>>>>>
>>>>>>>>>>> Hi Denis,
>>>>>>>>>>>
>>>>>>>>>>> There are two kinds of BSD license, 3-clause and 4-clause.
>>>> The
>>>>>>>> difference
>>>>>>>>>>> between them is advertising clause:
>>>>>>>>>>>
>>>>>>>>>>> "3. All advertising materials mentioning features or use of
>>>> this
>>>>>>>> software
>>>>>>>>>>> must display the following acknowledgement: This product
>>>> includes
>>>>>>>> software
>>>>>>>>>>> developed by the University of California, Berkeley and its
>>>>>>>> contributors."
>>>>>>>>>>>
>>>>>>>>>>> which is causing problems with other licenses and which
>>>> sqlline's
>>>>>>>> license
>>>>>>>>>>> doesn't contain:
>>>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>
>>>>>>>>>>> So it should be all good for inclusion.
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> Ilya Kasnacheev.
>>>>>>>>>>>
>>>>>>>>>>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
>>>>>>>>>>>
>>>>>>>>>>>> Hi Ilya,
>>>>>>>>>>>>
>>>>>>>>>>>> It will be a useful addition to Ignite. I would include
>>>> the tool
>>>>>> in
>>>>>>>> one of
>>>>>>>>>>>> the nearest releases. It’s license is BSD-3 which is
>>>> compatible
>>>>>> with
>>>>>>>> ours.
>>>>>>>>>>>> However, there is a note on ASF side saying that BSD
>>>> licenses
>>>>> can
>>>>>> be
>>>>>>>>>>>> included without advertising clause [1]. What does it mean
>>>> for
>>>>> us?
>>>>>>>> Can we
>>>>>>>>>>>> document the tool and mention on our site?
>>>>>>>>>>>>
>>>>>>>>>>>> [1] https://www.apache.org/legal/resolved.html#category-a
>>>> <
>>>>>>>>>>>> https://www.apache.org/legal/resolved.html#category-a>
>>>>>>>>>>>>
>>>>>>>>>>>> —
>>>>>>>>>>>> Denis
>>>>>>>>>>>>
>>>>>>>>>>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <
>>>>> [hidden email]
>>>>>>>
>>>>>>>> wrote:
>>>>>>>>>>>>>
>>>>>>>>>>>>> Hi Igniters,
>>>>>>>>>>>>>
>>>>>>>>>>>>> I've found one command line tool that can be used for
>>>> execute
>>>>>>> queries
>>>>>>>>>>>>> (DDL/DML) with data from Apache Ignite grid via
>>>>>>>>>>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Here you can find overview of internal commands of this
>>>> tool:
>>>>>>>>>>>>> https://cwiki.apache.org/confluence/display/IGNITE/
>>>>>>>> Overview+sqlline+tool
>>>>>>>>>>>>>
>>>>>>>>>>>>> The most problems with sqlline relate with some issues
>>>> from our
>>>>>>> side
>>>>>>>> or
>>>>>>>>>>>>> some unsupported features.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Would be nice to add tool like this (sqlline or something
>>>> else)
>>>>>> in
>>>>>>>> our
>>>>>>>>>>>>> project.
>>>>>>>>>>>>>
>>>>>>>>>>>>> Have you any objections or other thoughts?
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> --
>>>>>>>>>>>>> Ilya Suntsov
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Ilya Suntsov
>>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Sergey Kozlov
>>>>>> GridGain Systems
>>>>>> www.gridgain.com
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Sergey Kozlov
>>>>> GridGain Systems
>>>>> www.gridgain.com
>>>>>
>>>
>

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dmagda
Thought over 127.0.0.1 as a default host once again. The bad thing about it is that the user gets a lengthy exception stack trace if Ignite  is not running locally and not a small error message.

What are the other opinions on this? Do we want to follow Vladimir’s suggestion forcing to set the host name/IP (port is optional) for the sake of usability or leaver 127.0.0.1 as default?


Denis
 

> On Oct 6, 2017, at 12:21 PM, Denis Magda <[hidden email]> wrote:
>
>> But, we need to support “help” (-h, -help) argument listing all the parameters accepted by the tools.
>
> Meant accepted by the ignitesql script only such as host name.
>
> —
> Denis
>
>> On Oct 6, 2017, at 12:20 PM, Denis Magda <[hidden email]> wrote:
>>
>> Really nice, could click through the getting started [1] in a minute!
>>
>> +1 to rename the script to “ignitesql”. Vladimir’s point makes total sense.
>>
>> However, tend to disagree that the host has to be requested all the times. We never request a configuration or host name for ignite.sh, visor or web agent scripts. I would follow this approach that’s excellent for dev time.
>>
>> But, we need to support “help” (-h, -help) argument listing all the parameters accepted by the tools.
>>
>> Please consider our feedback and share the next build once it’s ready.
>>
>>
>> [1] https://apacheignite-sql.readme.io/v2.1/docs/getting-started <https://apacheignite-sql.readme.io/v2.1/docs/getting-started>
>>
>> —
>> Denis
>>
>>> On Oct 6, 2017, at 9:04 AM, Anton Vinogradov <[hidden email]> wrote:
>>>
>>> How about sqlconsole.sh or sqlcmd.sh ?
>>>
>>> On Fri, Oct 6, 2017 at 6:04 PM, <[hidden email]> wrote:
>>>
>>>> I like ignitesql.
>>>>
>>>> ⁣D.​
>>>>
>>>> On Oct 6, 2017, 4:49 PM, at 4:49 PM, Vladimir Ozerov <[hidden email]>
>>>> wrote:
>>>>> Denis,
>>>>>
>>>>> Setting default host to 127.0.0.1 is bad idea, because it mean that in
>>>>> practice users would have to change the script always. Instead, we
>>>>> should
>>>>> accept host name as argument. This is perfectly fine from usability
>>>>> perspective, most tools work this way (i.e. throw error when started
>>>>> without arguments).
>>>>>
>>>>> Also IMO "ignitedb" is misleading name. Users would like think that it
>>>>> is a
>>>>> kind of script to start database, rather than to connect to it. We
>>>>> should
>>>>> think on other names. E.g. "ignitesql".
>>>>>
>>>>> On Fri, Oct 6, 2017 at 5:23 PM, Sergey Kozlov <[hidden email]>
>>>>> wrote:
>>>>>
>>>>>> Denis
>>>>>>
>>>>>> The link below has included sqlline. Please take a look:
>>>>>> https://ci.ignite.apache.org/viewLog.html?buildId=875441&
>>>>>> buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3Pre
>>>>>> pareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz
>>>>>>
>>>>>> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]>
>>>>> wrote:
>>>>>>
>>>>>>> Here is the original ticket [1]. Ilya, closed the one created by
>>>>> you as a
>>>>>>> duplicate.
>>>>>>>
>>>>>>> In addition to the tool’s jar inclusion in Ignite’s binary releases
>>>>> let’s
>>>>>>> create a shell script to simplify the connectivity phase:
>>>>>>>
>>>>>>> - name the script as ignitedb.sh for Unix and ignitedb.bat for
>>>>>> Windows.
>>>>>>> -
>>>>>>> - the script uses the following connection string by default:
>>>>> .sqlline
>>>>>>> -d org.apache.ignite.IgniteJdbcThinDriver --color=true
>>>>> --verbose=true
>>>>>>> --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
>>>>>>> thin://127.0.0.1/
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> - make up parameters list to adjust Ignite specific part of the
>>>>>>> connection string: Ignite IP and port, streaming mode, etc. The
>>>>> full
>>>>>> list
>>>>>>> of supported parameters is here: https://apacheignite-
>>>>>>> sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>>>>>>>
>>>>> <https://apacheignite-sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-5608
>>>>>>>
>>>>>>> —
>>>>>>> Denis
>>>>>>>
>>>>>>> On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]>
>>>>> wrote:
>>>>>>>
>>>>>>> Dmitriy, Denis
>>>>>>>
>>>>>>> We're in progress to add sqlline in upcoming 2.3
>>>>>>>
>>>>>>> On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan
>>>>> <[hidden email]
>>>>>>>> wrote:
>>>>>>> Would be nice to get it in 2.3. This is critical functionality for
>>>>> our
>>>>>>> users and 2.4 seems too far to give anyone comfort.
>>>>>>>
>>>>>>> On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov
>>>>> <[hidden email]>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> Guys,
>>>>>>>>
>>>>>>>> I've created the ticket for 2.4 release:
>>>>>>>> https://issues.apache.org/jira/browse/IGNITE-6561
>>>>>>>>
>>>>>>>> 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
>>>>>>>>
>>>>>>>>> Denis,
>>>>>>>>>
>>>>>>>>> I’m glad you’re thinking of using SQLLine. Under the BSD
>>>>> license, you
>>>>>>>>> don’t need my permission to distribute, but I grant that
>>>>> permission.
>>>>>>>>>
>>>>>>>>> Drill, Phoenix and Calcite already distribute SQLLine, so
>>>>> Ignite is
>>>>>> in
>>>>>>>>> good company.
>>>>>>>>>
>>>>>>>>> If you need extensions, please discuss on the dev list, or open
>>>>> a
>>>>>>> GitHub
>>>>>>>>> case or pull request. SQLLine operates in the usual way for a
>>>>> GitHub
>>>>>>>>> project. It’s unlikely that you’ll need Ignite-specific
>>>>> extensions —
>>>>>>>>> SQLLine just exposes what comes through the JDBC driver — but
>>>>> we can
>>>>>>>>> discuss if the need arises. The Hive project forked SQLLine
>>>>> into its
>>>>>>> own
>>>>>>>>> Beeline module and I’d like to avoid a repeat of that.
>>>>>>>>>
>>>>>>>>> Julian
>>>>>>>>>
>>>>>>>>>> On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]>
>>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>> Igniters,
>>>>>>>>>>
>>>>>>>>>> Let me introduce Julian Hyde [1], creator of SQLLine tool and
>>>>> our
>>>>>>>> Apache
>>>>>>>>> mate,
>>>>>>>>>>
>>>>>>>>>> Julian,
>>>>>>>>>>
>>>>>>>>>> Please grant that Apache Ignite community a permission to
>>>>> include
>>>>>>>>> SQLLine [2] it in every Ignite deliverable (source, binary).
>>>>> It’s
>>>>>>> planned
>>>>>>>>> to suggest the tool as a default command line SQL utility for
>>>>> Ignite
>>>>>>>>> clusters. SQLLite and Ignite usage will also be documented on
>>>>>> Ignite’s
>>>>>>>>> technical documentation.
>>>>>>>>>>
>>>>>>>>>> [1] https://people.apache.org/~jhyde/ <
>>>>>> https://people.apache.org/~jh
>>>>>>>> yde/
>>>>>>>>>>
>>>>>>>>>> [2] https://github.com/julianhyde/sqlline
>>>>> <https://github.com/
>>>>>>>>> julianhyde/sqlline>
>>>>>>>>>>
>>>>>>>>>> —
>>>>>>>>>> Denis
>>>>>>>>>>
>>>>>>>>>>> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
>>>>>>> <mailto:
>>>>>>>>> [hidden email]>> wrote:
>>>>>>>>>>>
>>>>>>>>>>> Hi Ilya,
>>>>>>>>>>>
>>>>>>>>>>> Thanks for the clarification! Referring to the page shared
>>>>> by you
>>>>>>> [1]
>>>>>>>>> if we need to get author’s consent in a written form:
>>>>>>>>>>>
>>>>>>>>>>> A permissive license similar to the BSD 2-Clause License,
>>>>> but
>>>>>> with a
>>>>>>>>> 3rd clause that prohibits others from using the name of the
>>>>> project
>>>>>> or
>>>>>>>> its
>>>>>>>>> contributors to promote derived products without written
>>>>> consent.
>>>>>>>>>>>
>>>>>>>>>>> [1]
>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
>>>>>>>>>>>
>>>>>>>>>>> I’ll reach out the author requesting the permit.
>>>>>>>>>>>
>>>>>>>>>>> —
>>>>>>>>>>> Denis
>>>>>>>>>>>
>>>>>>>>>>>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
>>>>>>>>> [hidden email] <mailto:[hidden email]>>
>>>>> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>> Hi Denis,
>>>>>>>>>>>>
>>>>>>>>>>>> There are two kinds of BSD license, 3-clause and 4-clause.
>>>>> The
>>>>>>>>> difference
>>>>>>>>>>>> between them is advertising clause:
>>>>>>>>>>>>
>>>>>>>>>>>> "3. All advertising materials mentioning features or use of
>>>>> this
>>>>>>>>> software
>>>>>>>>>>>> must display the following acknowledgement: This product
>>>>> includes
>>>>>>>>> software
>>>>>>>>>>>> developed by the University of California, Berkeley and its
>>>>>>>>> contributors."
>>>>>>>>>>>>
>>>>>>>>>>>> which is causing problems with other licenses and which
>>>>> sqlline's
>>>>>>>>> license
>>>>>>>>>>>> doesn't contain:
>>>>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>
>>>>>>>>>>>> So it should be all good for inclusion.
>>>>>>>>>>>>
>>>>>>>>>>>> --
>>>>>>>>>>>> Ilya Kasnacheev.
>>>>>>>>>>>>
>>>>>>>>>>>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
>>>>>>>>>>>>
>>>>>>>>>>>>> Hi Ilya,
>>>>>>>>>>>>>
>>>>>>>>>>>>> It will be a useful addition to Ignite. I would include
>>>>> the tool
>>>>>>> in
>>>>>>>>> one of
>>>>>>>>>>>>> the nearest releases. It’s license is BSD-3 which is
>>>>> compatible
>>>>>>> with
>>>>>>>>> ours.
>>>>>>>>>>>>> However, there is a note on ASF side saying that BSD
>>>>> licenses
>>>>>> can
>>>>>>> be
>>>>>>>>>>>>> included without advertising clause [1]. What does it mean
>>>>> for
>>>>>> us?
>>>>>>>>> Can we
>>>>>>>>>>>>> document the tool and mention on our site?
>>>>>>>>>>>>>
>>>>>>>>>>>>> [1] https://www.apache.org/legal/resolved.html#category-a
>>>>> <
>>>>>>>>>>>>> https://www.apache.org/legal/resolved.html#category-a>
>>>>>>>>>>>>>
>>>>>>>>>>>>> —
>>>>>>>>>>>>> Denis
>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <
>>>>>> [hidden email]
>>>>>>>>
>>>>>>>>> wrote:
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Hi Igniters,
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> I've found one command line tool that can be used for
>>>>> execute
>>>>>>>> queries
>>>>>>>>>>>>>> (DDL/DML) with data from Apache Ignite grid via
>>>>>>>>>>>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Here you can find overview of internal commands of this
>>>>> tool:
>>>>>>>>>>>>>> https://cwiki.apache.org/confluence/display/IGNITE/
>>>>>>>>> Overview+sqlline+tool
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> The most problems with sqlline relate with some issues
>>>>> from our
>>>>>>>> side
>>>>>>>>> or
>>>>>>>>>>>>>> some unsupported features.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Would be nice to add tool like this (sqlline or something
>>>>> else)
>>>>>>> in
>>>>>>>>> our
>>>>>>>>>>>>>> project.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Have you any objections or other thoughts?
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> --
>>>>>>>>>>>>>> Ilya Suntsov
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>> Ilya Suntsov
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Sergey Kozlov
>>>>>>> GridGain Systems
>>>>>>> www.gridgain.com
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Sergey Kozlov
>>>>>> GridGain Systems
>>>>>> www.gridgain.com
>>>>>>
>>>>
>>
>

Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dsetrakyan
In reply to this post by Anton Vinogradov
On Fri, Oct 6, 2017 at 9:04 AM, Anton Vinogradov <[hidden email]>
wrote:

> How about sqlconsole.sh or sqlcmd.sh ?
>

Shouldn't we have "ignite" in the name?
Reply | Threaded
Open this post in threaded view
|

Re: Adding sqlline tool to Apache Ignite project

dsetrakyan
In reply to this post by dmagda
How does the binding happen? Can we bind to everything, like we do in
Ignite?

On Fri, Oct 6, 2017 at 2:51 PM, Denis Magda <[hidden email]> wrote:

> Thought over 127.0.0.1 as a default host once again. The bad thing about
> it is that the user gets a lengthy exception stack trace if Ignite  is not
> running locally and not a small error message.
>
> What are the other opinions on this? Do we want to follow Vladimir’s
> suggestion forcing to set the host name/IP (port is optional) for the sake
> of usability or leaver 127.0.0.1 as default?
>
> —
> Denis
>
> > On Oct 6, 2017, at 12:21 PM, Denis Magda <[hidden email]> wrote:
> >
> >> But, we need to support “help” (-h, -help) argument listing all the
> parameters accepted by the tools.
> >
> > Meant accepted by the ignitesql script only such as host name.
> >
> > —
> > Denis
> >
> >> On Oct 6, 2017, at 12:20 PM, Denis Magda <[hidden email]> wrote:
> >>
> >> Really nice, could click through the getting started [1] in a minute!
> >>
> >> +1 to rename the script to “ignitesql”. Vladimir’s point makes total
> sense.
> >>
> >> However, tend to disagree that the host has to be requested all the
> times. We never request a configuration or host name for ignite.sh, visor
> or web agent scripts. I would follow this approach that’s excellent for dev
> time.
> >>
> >> But, we need to support “help” (-h, -help) argument listing all the
> parameters accepted by the tools.
> >>
> >> Please consider our feedback and share the next build once it’s ready.
> >>
> >>
> >> [1] https://apacheignite-sql.readme.io/v2.1/docs/getting-started <
> https://apacheignite-sql.readme.io/v2.1/docs/getting-started>
> >>
> >> —
> >> Denis
> >>
> >>> On Oct 6, 2017, at 9:04 AM, Anton Vinogradov <[hidden email]>
> wrote:
> >>>
> >>> How about sqlconsole.sh or sqlcmd.sh ?
> >>>
> >>> On Fri, Oct 6, 2017 at 6:04 PM, <[hidden email]> wrote:
> >>>
> >>>> I like ignitesql.
> >>>>
> >>>> ⁣D.​
> >>>>
> >>>> On Oct 6, 2017, 4:49 PM, at 4:49 PM, Vladimir Ozerov <
> [hidden email]>
> >>>> wrote:
> >>>>> Denis,
> >>>>>
> >>>>> Setting default host to 127.0.0.1 is bad idea, because it mean that
> in
> >>>>> practice users would have to change the script always. Instead, we
> >>>>> should
> >>>>> accept host name as argument. This is perfectly fine from usability
> >>>>> perspective, most tools work this way (i.e. throw error when started
> >>>>> without arguments).
> >>>>>
> >>>>> Also IMO "ignitedb" is misleading name. Users would like think that
> it
> >>>>> is a
> >>>>> kind of script to start database, rather than to connect to it. We
> >>>>> should
> >>>>> think on other names. E.g. "ignitesql".
> >>>>>
> >>>>> On Fri, Oct 6, 2017 at 5:23 PM, Sergey Kozlov <[hidden email]>
> >>>>> wrote:
> >>>>>
> >>>>>> Denis
> >>>>>>
> >>>>>> The link below has included sqlline. Please take a look:
> >>>>>> https://ci.ignite.apache.org/viewLog.html?buildId=875441&
> >>>>>> buildTypeId=IgniteRelease_XxxFromMirrorIgniteRelease3Pre
> >>>>>> pareVote&tab=artifacts#!1rrb2,-wpvx2aopzexz
> >>>>>>
> >>>>>> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda <[hidden email]>
> >>>>> wrote:
> >>>>>>
> >>>>>>> Here is the original ticket [1]. Ilya, closed the one created by
> >>>>> you as a
> >>>>>>> duplicate.
> >>>>>>>
> >>>>>>> In addition to the tool’s jar inclusion in Ignite’s binary releases
> >>>>> let’s
> >>>>>>> create a shell script to simplify the connectivity phase:
> >>>>>>>
> >>>>>>> - name the script as ignitedb.sh for Unix and ignitedb.bat for
> >>>>>> Windows.
> >>>>>>> -
> >>>>>>> - the script uses the following connection string by default:
> >>>>> .sqlline
> >>>>>>> -d org.apache.ignite.IgniteJdbcThinDriver --color=true
> >>>>> --verbose=true
> >>>>>>> --showWarnings=true --showNestedErrs=true -u jdbc:ignite:
> >>>>>>> thin://127.0.0.1/
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> - make up parameters list to adjust Ignite specific part of the
> >>>>>>> connection string: Ignite IP and port, streaming mode, etc. The
> >>>>> full
> >>>>>> list
> >>>>>>> of supported parameters is here: https://apacheignite-
> >>>>>>> sql.readme.io/docs/jdbc-driver#jdbc-thin-driver
> >>>>>>>
> >>>>> <https://apacheignite-sql.readme.io/docs/jdbc-driver#
> jdbc-thin-driver
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> [1] https://issues.apache.org/jira/browse/IGNITE-5608
> >>>>>>>
> >>>>>>> —
> >>>>>>> Denis
> >>>>>>>
> >>>>>>> On Oct 5, 2017, at 9:02 AM, Sergey Kozlov <[hidden email]>
> >>>>> wrote:
> >>>>>>>
> >>>>>>> Dmitriy, Denis
> >>>>>>>
> >>>>>>> We're in progress to add sqlline in upcoming 2.3
> >>>>>>>
> >>>>>>> On Thu, Oct 5, 2017 at 5:30 PM, Dmitriy Setrakyan
> >>>>> <[hidden email]
> >>>>>>>> wrote:
> >>>>>>> Would be nice to get it in 2.3. This is critical functionality for
> >>>>> our
> >>>>>>> users and 2.4 seems too far to give anyone comfort.
> >>>>>>>
> >>>>>>> On Thu, Oct 5, 2017 at 11:33 AM, Ilya Suntsov
> >>>>> <[hidden email]>
> >>>>>>> wrote:
> >>>>>>>
> >>>>>>>> Guys,
> >>>>>>>>
> >>>>>>>> I've created the ticket for 2.4 release:
> >>>>>>>> https://issues.apache.org/jira/browse/IGNITE-6561
> >>>>>>>>
> >>>>>>>> 2017-08-30 22:21 GMT+03:00 Julian Hyde <[hidden email]>:
> >>>>>>>>
> >>>>>>>>> Denis,
> >>>>>>>>>
> >>>>>>>>> I’m glad you’re thinking of using SQLLine. Under the BSD
> >>>>> license, you
> >>>>>>>>> don’t need my permission to distribute, but I grant that
> >>>>> permission.
> >>>>>>>>>
> >>>>>>>>> Drill, Phoenix and Calcite already distribute SQLLine, so
> >>>>> Ignite is
> >>>>>> in
> >>>>>>>>> good company.
> >>>>>>>>>
> >>>>>>>>> If you need extensions, please discuss on the dev list, or open
> >>>>> a
> >>>>>>> GitHub
> >>>>>>>>> case or pull request. SQLLine operates in the usual way for a
> >>>>> GitHub
> >>>>>>>>> project. It’s unlikely that you’ll need Ignite-specific
> >>>>> extensions —
> >>>>>>>>> SQLLine just exposes what comes through the JDBC driver — but
> >>>>> we can
> >>>>>>>>> discuss if the need arises. The Hive project forked SQLLine
> >>>>> into its
> >>>>>>> own
> >>>>>>>>> Beeline module and I’d like to avoid a repeat of that.
> >>>>>>>>>
> >>>>>>>>> Julian
> >>>>>>>>>
> >>>>>>>>>> On Aug 29, 2017, at 6:35 PM, Denis Magda <[hidden email]>
> >>>>>> wrote:
> >>>>>>>>>>
> >>>>>>>>>> Igniters,
> >>>>>>>>>>
> >>>>>>>>>> Let me introduce Julian Hyde [1], creator of SQLLine tool and
> >>>>> our
> >>>>>>>> Apache
> >>>>>>>>> mate,
> >>>>>>>>>>
> >>>>>>>>>> Julian,
> >>>>>>>>>>
> >>>>>>>>>> Please grant that Apache Ignite community a permission to
> >>>>> include
> >>>>>>>>> SQLLine [2] it in every Ignite deliverable (source, binary).
> >>>>> It’s
> >>>>>>> planned
> >>>>>>>>> to suggest the tool as a default command line SQL utility for
> >>>>> Ignite
> >>>>>>>>> clusters. SQLLite and Ignite usage will also be documented on
> >>>>>> Ignite’s
> >>>>>>>>> technical documentation.
> >>>>>>>>>>
> >>>>>>>>>> [1] https://people.apache.org/~jhyde/ <
> >>>>>> https://people.apache.org/~jh
> >>>>>>>> yde/
> >>>>>>>>>>
> >>>>>>>>>> [2] https://github.com/julianhyde/sqlline
> >>>>> <https://github.com/
> >>>>>>>>> julianhyde/sqlline>
> >>>>>>>>>>
> >>>>>>>>>> —
> >>>>>>>>>> Denis
> >>>>>>>>>>
> >>>>>>>>>>> On Aug 25, 2017, at 9:17 AM, Denis Magda <[hidden email]
> >>>>>>> <mailto:
> >>>>>>>>> [hidden email]>> wrote:
> >>>>>>>>>>>
> >>>>>>>>>>> Hi Ilya,
> >>>>>>>>>>>
> >>>>>>>>>>> Thanks for the clarification! Referring to the page shared
> >>>>> by you
> >>>>>>> [1]
> >>>>>>>>> if we need to get author’s consent in a written form:
> >>>>>>>>>>>
> >>>>>>>>>>> A permissive license similar to the BSD 2-Clause License,
> >>>>> but
> >>>>>> with a
> >>>>>>>>> 3rd clause that prohibits others from using the name of the
> >>>>> project
> >>>>>> or
> >>>>>>>> its
> >>>>>>>>> contributors to promote derived products without written
> >>>>> consent.
> >>>>>>>>>>>
> >>>>>>>>>>> [1]
> >>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> >>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE> <
> >>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> >>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>>
> >>>>>>>>>>>
> >>>>>>>>>>> I’ll reach out the author requesting the permit.
> >>>>>>>>>>>
> >>>>>>>>>>> —
> >>>>>>>>>>> Denis
> >>>>>>>>>>>
> >>>>>>>>>>>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev <
> >>>>>>>>> [hidden email] <mailto:[hidden email]>>
> >>>>> wrote:
> >>>>>>>>>>>>
> >>>>>>>>>>>> Hi Denis,
> >>>>>>>>>>>>
> >>>>>>>>>>>> There are two kinds of BSD license, 3-clause and 4-clause.
> >>>>> The
> >>>>>>>>> difference
> >>>>>>>>>>>> between them is advertising clause:
> >>>>>>>>>>>>
> >>>>>>>>>>>> "3. All advertising materials mentioning features or use of
> >>>>> this
> >>>>>>>>> software
> >>>>>>>>>>>> must display the following acknowledgement: This product
> >>>>> includes
> >>>>>>>>> software
> >>>>>>>>>>>> developed by the University of California, Berkeley and its
> >>>>>>>>> contributors."
> >>>>>>>>>>>>
> >>>>>>>>>>>> which is causing problems with other licenses and which
> >>>>> sqlline's
> >>>>>>>>> license
> >>>>>>>>>>>> doesn't contain:
> >>>>>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE <
> >>>>>>>>> https://github.com/julianhyde/sqlline/blob/master/LICENSE>
> >>>>>>>>>>>> So it should be all good for inclusion.
> >>>>>>>>>>>>
> >>>>>>>>>>>> --
> >>>>>>>>>>>> Ilya Kasnacheev.
> >>>>>>>>>>>>
> >>>>>>>>>>>> 2017-08-25 18:48 GMT+03:00 Denis Magda <[hidden email]>:
> >>>>>>>>>>>>
> >>>>>>>>>>>>> Hi Ilya,
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> It will be a useful addition to Ignite. I would include
> >>>>> the tool
> >>>>>>> in
> >>>>>>>>> one of
> >>>>>>>>>>>>> the nearest releases. It’s license is BSD-3 which is
> >>>>> compatible
> >>>>>>> with
> >>>>>>>>> ours.
> >>>>>>>>>>>>> However, there is a note on ASF side saying that BSD
> >>>>> licenses
> >>>>>> can
> >>>>>>> be
> >>>>>>>>>>>>> included without advertising clause [1]. What does it mean
> >>>>> for
> >>>>>> us?
> >>>>>>>>> Can we
> >>>>>>>>>>>>> document the tool and mention on our site?
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> [1] https://www.apache.org/legal/resolved.html#category-a
> >>>>> <
> >>>>>>>>>>>>> https://www.apache.org/legal/resolved.html#category-a>
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> —
> >>>>>>>>>>>>> Denis
> >>>>>>>>>>>>>
> >>>>>>>>>>>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov <
> >>>>>> [hidden email]
> >>>>>>>>
> >>>>>>>>> wrote:
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Hi Igniters,
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> I've found one command line tool that can be used for
> >>>>> execute
> >>>>>>>> queries
> >>>>>>>>>>>>>> (DDL/DML) with data from Apache Ignite grid via
> >>>>>>>>>>>>>> *org.apache.ignite.IgniteJdbcThinDriver*.
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Here you can find overview of internal commands of this
> >>>>> tool:
> >>>>>>>>>>>>>> https://cwiki.apache.org/confluence/display/IGNITE/
> >>>>>>>>> Overview+sqlline+tool
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> The most problems with sqlline relate with some issues
> >>>>> from our
> >>>>>>>> side
> >>>>>>>>> or
> >>>>>>>>>>>>>> some unsupported features.
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Would be nice to add tool like this (sqlline or something
> >>>>> else)
> >>>>>>> in
> >>>>>>>>> our
> >>>>>>>>>>>>>> project.
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Have you any objections or other thoughts?
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> --
> >>>>>>>>>>>>>> Ilya Suntsov
> >>>>>>>>>>>>>
> >>>>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> --
> >>>>>>>> Ilya Suntsov
> >>>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>> --
> >>>>>>> Sergey Kozlov
> >>>>>>> GridGain Systems
> >>>>>>> www.gridgain.com
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>
> >>>>>>
> >>>>>> --
> >>>>>> Sergey Kozlov
> >>>>>> GridGain Systems
> >>>>>> www.gridgain.com
> >>>>>>
> >>>>
> >>
> >
>
>
12