Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id C914C200D45 for ; Thu, 19 Oct 2017 01:27:01 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id C78A1160BEB; Wed, 18 Oct 2017 23:27:01 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 4498C160BEA for ; Thu, 19 Oct 2017 01:27:00 +0200 (CEST) Received: (qmail 18066 invoked by uid 500); 18 Oct 2017 23:26:59 -0000 Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ignite.apache.org Delivered-To: mailing list dev@ignite.apache.org Received: (qmail 18053 invoked by uid 99); 18 Oct 2017 23:26:58 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 18 Oct 2017 23:26:58 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id D8BEE18056F for ; Wed, 18 Oct 2017 23:26:57 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 4.481 X-Spam-Level: **** X-Spam-Status: No, score=4.481 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=2, KAM_BADIPHTTP=2, KAM_SHORT=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RCVD_IN_SORBS_SPAM=0.5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=disabled Authentication-Results: spamd3-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=gridgain-com.20150623.gappssmtp.com Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id O0LvBx4y7EhY for ; Wed, 18 Oct 2017 23:26:48 +0000 (UTC) Received: from mail-oi0-f52.google.com (mail-oi0-f52.google.com [209.85.218.52]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTPS id DB8535F5CC for ; Wed, 18 Oct 2017 23:26:47 +0000 (UTC) Received: by mail-oi0-f52.google.com with SMTP id n82so11808893oig.3 for ; Wed, 18 Oct 2017 16:26:47 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gridgain-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to; bh=rKnaxoGonPCMUy91OkyY5HBJCpN5Kq1SLBmSjVbZ7O4=; b=kxys5ZnLCFlyayMyOLJB7mAUzHQdnhvkNr/x2JMMnO3vUgD/gUKPtd0nZcddVvIkHn x75VBfRtQRVblBBKOAXCDUnaFAGVgBbROCicDB7huIdmzf+BKU/g6uGDk1jLlCgzsKzv QMMtaM3h4edldGdi09RCNgVG9yvvJKzxdl3H+rWhnrraIPeWUrJPFGpEiDvkIng0Z/O2 Ra2i4A/7CQ9Gsz/ioXj/NljsfSTwNoXxHSmv6zoXOnpppzywnzqOFDuiw1cX23+xr4Am Hhz2VxJXk2l7LX7d947O6r/wKncmI7uCdDJxqp6hIScgouZ2dr7zkf5u0NCuz/2NO8YF 6uhQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to; bh=rKnaxoGonPCMUy91OkyY5HBJCpN5Kq1SLBmSjVbZ7O4=; b=Ne9lWJ2AxZWrz5nw4EOhdxrULeZiN74J6g+n4xhlFdXNNcjj71AJ1YQrbpEujI6HGX HRpF2GMbY7hfdLwUAQEhzHP+ZRbKeDu0VG285teLmjIOJjxRb5/321JL9b3M6x5pQDXs RYzQxwtxpWVPFYiQP6kAcAXN3ZSkfQAzRytqX7ZVbRgg6aL+AummFJ+i6nR+pbNZdWVT WgdjLG+tqDFaif6KdsXwH/JN2sgqiHFX9kbf/PN+oSebCa7n+iCeMiVn7jt4GrPPzuj2 DKs9+66vj74/Y89KNnwG9ihxW7DqejqzxQmpOEA3bgIFdMXvJeHLHIdW6gl7jm0cT6ai meww== X-Gm-Message-State: AMCzsaXkHJRBaWM+YmrSsCJmicrJzJOwP2acb9sOHjhIqp7sKmdEZjIK iofs86E35duBYWle9kIxBl1DE4YPnOF+QR+5VQePNNbN X-Google-Smtp-Source: ABhQp+R/RAW4CNSOfSrikl8hIWU9m35iJ/hw/sDARGCXy9QAR+b8Gsr8C/QJ+/gUNNvGO43lwUjrzG5JTP346bvVvyQ= X-Received: by 10.157.48.124 with SMTP id w57mr5392099otd.440.1508369206215; Wed, 18 Oct 2017 16:26:46 -0700 (PDT) MIME-Version: 1.0 Received: by 10.157.59.183 with HTTP; Wed, 18 Oct 2017 16:26:15 -0700 (PDT) In-Reply-To: References: <19BBC834-B22D-4CBF-B6B4-8A1405227959@apache.org> <0CAB33E3-1E6C-48A2-9131-7D9BE3E1FDAB@apache.org> <3ad7522f-60df-4ea6-bafd-c0e336ad702b@apache.org> <8C8201C8-3815-4183-A499-7EBD9ED2B408@apache.org> <2094EBF7-236C-446F-A595-122D164F1B68@apache.org> From: Prachi Garg Date: Wed, 18 Oct 2017 16:26:15 -0700 Message-ID: Subject: Re: Adding sqlline tool to Apache Ignite project To: dev@ignite.apache.org Content-Type: multipart/alternative; boundary="001a113b1bbcba2258055bda92e5" archived-at: Wed, 18 Oct 2017 23:27:02 -0000 --001a113b1bbcba2258055bda92e5 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Here is the link for SQLLine documentation - https://apacheignite-sql.readme.io/v2.1/docs/sqlline -P On Tue, Oct 17, 2017 at 11:54 AM, Denis Magda wrote: > Looks good to me. Prachi will help us documenting the tool usage: > https://issues.apache.org/jira/browse/IGNITE-6656 < > https://issues.apache.org/jira/browse/IGNITE-6656> > > However, I can=E2=80=99t conceive how to see a table structure (columns a= nd their > types, indexes with names and types) using SQLLine. I=E2=80=99ve tried !m= etadata > with a variety of parameters but no luck. As for !indexes and !tables > commands they just print out table names and secondary indexes omitting > columns, indexes types and *primary indexes*. Considering that Ignite > doesn=E2=80=99t support standard *describe* command I assumed SQLLine wou= ld help us > out. But how do I do this with SQLLine? > > =E2=80=94 > Denis > > > On Oct 17, 2017, at 4:33 AM, Oleg Ostanin wrote= : > > > > New example build with sqlline: > > > > https://ci.ignite.apache.org/viewLog.html?buildId=3D894407& > tab=3Dartifacts&buildTypeId=3DIgniteRelease_XxxFromMirrorIgniteRelease3Pr= e > pareVote#!1rrb2,1esn4zrslm4po,-h8h0hn9vvvxp > > > > > > On Wed, Oct 11, 2017 at 1:00 AM, Denis Magda wrote: > > > >> Oleg, > >> > >> Looks good to me. Please consider the notes left in the ticket. I want > us > >> to prepare a script for Windows, review the language for help notice a= nd > >> errors, put together documentation. Prachi will be able to help with t= he > >> editing and documentation. > >> > >> =E2=80=94 > >> Denis > >> > >>> On Oct 9, 2017, at 10:13 AM, Oleg Ostanin > wrote: > >>> > >>> New build with fixed argument parsing: > >>> https://ci.ignite.apache.org/viewLog.html?buildId=3D882282& > >> tab=3Dartifacts&buildTypeId=3DIgniteRelease_XxxFromMirrorIgniteRelease= 3Pre > >> pareVote#!1rrb2,1esn4zrslm4po,-h8h0hn9vvvxp > >>> > >>> On Mon, Oct 9, 2017 at 5:38 PM, Denis Magda > wrote: > >>> > >>>> I think it=E2=80=99s a must have for the ticket resolution. > >>>> > >>>> Denis > >>>> > >>>> On Monday, October 9, 2017, Anton Vinogradov < > avinogradov@gridgain.com> > >>>> wrote: > >>>> > >>>>> Any plans to have ignitesql.bat? > >>>>> > >>>>> On Mon, Oct 9, 2017 at 5:29 PM, Oleg Ostanin >>>>> > wrote: > >>>>> > >>>>>> Another build with sqlline included: > >>>>>> https://ci.ignite.apache.org/viewLog.html?buildId=3D881120& > >>>>>> tab=3Dartifacts&buildTypeId=3DIgniteRelease_ > >> XxxFromMirrorIgniteRelease3Pre > >>>>>> pareVote#!1rrb2,-wpvx2aopzexz,1esn4zrslm4po,-h8h0hn9vvvxp > >>>>>> > >>>>>> On Sun, Oct 8, 2017 at 5:11 PM, Denis Magda >>>>> > wrote: > >>>>>> > >>>>>>> No more doubts on my side. +1 for Vladimir=E2=80=99s suggestion. > >>>>>>> > >>>>>>> Denis > >>>>>>> > >>>>>>> On Saturday, October 7, 2017, Dmitriy Setrakyan < > >>>> dsetrakyan@apache.org > >>>>> > > >>>>>>> wrote: > >>>>>>> > >>>>>>>> I now tend to agree with Vladimir. We should always require that > >>>> some > >>>>>>>> address is specified. The help menu should clearly state how to > >>>>> connect > >>>>>>> to > >>>>>>>> a localhost. > >>>>>>>> > >>>>>>>> D. > >>>>>>>> > >>>>>>>> On Sat, Oct 7, 2017 at 12:44 AM, Vladimir Ozerov < > >>>>> vozerov@gridgain.com > >>>>>>>> > > >>>>>>>> wrote: > >>>>>>>> > >>>>>>>>> Denis, > >>>>>>>>> > >>>>>>>>> Default Ignite configuration uses multicast, this is why you do > >>>> not > >>>>>>> need > >>>>>>>> to > >>>>>>>>> change anything. Ignite node is always both a server (listens) > >>>> and > >>>>> a > >>>>>>>> client > >>>>>>>>> (connects). > >>>>>>>>> > >>>>>>>>> This will not work for ignitesql, as this is a client. And in > >>>> real > >>>>>>>>> deployments it will connect to remote nodes, not local. So the > >>>>>> earlier > >>>>>>> we > >>>>>>>>> explain user how to do this, the better. This is why it should > >>>> not > >>>>>> work > >>>>>>>> out > >>>>>>>>> of the box connecting to 127.0.0.1. No magic for users please. > >>>>>>>>> > >>>>>>>>> This is what user will see (draft): > >>>>>>>>>> ./ignitesql.sh > >>>>>>>>>> Please specify the host: ignitesql.sh [host]; type --help for > >>>>> more > >>>>>>>>> information. > >>>>>>>>>> ./ignitesql.sh 192.168.12.55 > >>>>>>>>>> Connected successfully. > >>>>>>>>> > >>>>>>>>> Again, specifying parameters manually is not poor UX. This is > >>>>>> excellent > >>>>>>>> UX, > >>>>>>>>> as user learns on his own how to connect to a node in 1 minute. > >>>>> Most > >>>>>>>>> command line tools work this way. > >>>>>>>>> > >>>>>>>>> =D1=81=D0=B1, 7 =D0=BE=D0=BA=D1=82. 2017 =D0=B3. =D0=B2 7:12, D= mitriy Setrakyan < > >>>>> dsetrakyan@apache.org > >>>>>>>> >: > >>>>>>>>> > >>>>>>>>>> How does the binding happen? Can we bind to everything, like w= e > >>>>> do > >>>>>> in > >>>>>>>>>> Ignite? > >>>>>>>>>> > >>>>>>>>>> On Fri, Oct 6, 2017 at 2:51 PM, Denis Magda >>>>> > >>>>>>>> > 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=E2=80=99s > >>>>>>>>>>> suggestion forcing to set the host name/IP (port is optional) > >>>>> for > >>>>>>> the > >>>>>>>>>> sake > >>>>>>>>>>> of usability or leaver 127.0.0.1 as default? > >>>>>>>>>>> > >>>>>>>>>>> =E2=80=94 > >>>>>>>>>>> Denis > >>>>>>>>>>> > >>>>>>>>>>>> On Oct 6, 2017, at 12:21 PM, Denis Magda < > >>>> dmagda@apache.org > >>>>> > >>>>>>>> > wrote: > >>>>>>>>>>>> > >>>>>>>>>>>>> But, we need to support =E2=80=9Chelp=E2=80=9D (-h, -help) = argument > >>>> listing > >>>>>> all > >>>>>>>> the > >>>>>>>>>>> parameters accepted by the tools. > >>>>>>>>>>>> > >>>>>>>>>>>> Meant accepted by the ignitesql script only such as host > >>>>> name. > >>>>>>>>>>>> > >>>>>>>>>>>> =E2=80=94 > >>>>>>>>>>>> Denis > >>>>>>>>>>>> > >>>>>>>>>>>>> On Oct 6, 2017, at 12:20 PM, Denis Magda < > >>>> dmagda@apache.org > >>>>> > >>>>>>>> > wrote: > >>>>>>>>>>>>> > >>>>>>>>>>>>> Really nice, could click through the getting started [1] > >>>> in > >>>>> a > >>>>>>>>> minute! > >>>>>>>>>>>>> > >>>>>>>>>>>>> +1 to rename the script to =E2=80=9Cignitesql=E2=80=9D. Vla= dimir=E2=80=99s 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=E2=80= =99s > >>>>>> excellent > >>>>>>>> for > >>>>>>>>>> dev > >>>>>>>>>>> time. > >>>>>>>>>>>>> > >>>>>>>>>>>>> But, we need to support =E2=80=9Chelp=E2=80=9D (-h, -help) = argument > >>>> listing > >>>>>> all > >>>>>>>> the > >>>>>>>>>>> parameters accepted by the tools. > >>>>>>>>>>>>> > >>>>>>>>>>>>> Please consider our feedback and share the next build once > >>>>>> it=E2=80=99s > >>>>>>>>> ready. > >>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>>> [1] https://apacheignite-sql.readme.io/v2.1/docs/getting- > >>>>>>> started > >>>>>>>> < > >>>>>>>>>>> https://apacheignite-sql.readme.io/v2.1/docs/getting-started > >>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>>> =E2=80=94 > >>>>>>>>>>>>> Denis > >>>>>>>>>>>>> > >>>>>>>>>>>>>> On Oct 6, 2017, at 9:04 AM, Anton Vinogradov < > >>>>>>>>>> avinogradov@gridgain.com > > >>>>>>>>>>> wrote: > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> How about sqlconsole.sh or sqlcmd.sh ? > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> On Fri, Oct 6, 2017 at 6:04 PM, >>>>> > >>>>>>>> > wrote: > >>>>>>>>>>>>>> > >>>>>>>>>>>>>>> I like ignitesql. > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> =E2=81=A3D.=E2=80=8B > >>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> On Oct 6, 2017, 4:49 PM, at 4:49 PM, Vladimir Ozerov < > >>>>>>>>>>> vozerov@gridgain.com > > >>>>>>>>>>>>>>> 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 < > >>>>>>>>>> skozlov@gridgain.com > > >>>>>>>>>>>>>>>> wrote: > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> Denis > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> The link below has included sqlline. Please take a > >>>> look: > >>>>>>>>>>>>>>>>> https://ci.ignite.apache.org/ > >>>>> viewLog.html?buildId=3D875441& > >>>>>>>>>>>>>>>>> buildTypeId=3DIgniteRelease_ > >>>>> XxxFromMirrorIgniteRelease3Pre > >>>>>>>>>>>>>>>>> pareVote&tab=3Dartifacts#!1rrb2,-wpvx2aopzexz > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>> On Thu, Oct 5, 2017 at 7:48 PM, Denis Magda < > >>>>>>>> dmagda@apache.org > > >>>>>>>>>>>>>>>> wrote: > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> Here is the original ticket [1]. Ilya, closed the one > >>>>>>> created > >>>>>>>>> by > >>>>>>>>>>>>>>>> you as a > >>>>>>>>>>>>>>>>>> duplicate. > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> In addition to the tool=E2=80=99s jar inclusion in Ign= ite=E2=80=99s > >>>>>> binary > >>>>>>>>>> releases > >>>>>>>>>>>>>>>> let=E2=80=99s > >>>>>>>>>>>>>>>>>> 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=3Dtrue > >>>>>>>>>>>>>>>> --verbose=3Dtrue > >>>>>>>>>>>>>>>>>> --showWarnings=3Dtrue --showNestedErrs=3Dtrue -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 > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>> >>>>>>>>>>> jdbc-thin-driver > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> [1] https://issues.apache.org/ > >>>> jira/browse/IGNITE-5608 > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> =E2=80=94 > >>>>>>>>>>>>>>>>>> Denis > >>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>> On Oct 5, 2017, at 9:02 AM, Sergey Kozlov < > >>>>>>>>> skozlov@gridgain.com > > >>>>>>>>>>>>>>>> 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 > >>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>> 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 > >>>>>>>>>>>>>>>> > > >>>>>>>>>>>>>>>>>> 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 < > >>>>>> jhyde@apache.org > >>>>>>>> >: > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>> Denis, > >>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>> I=E2=80=99m glad you=E2=80=99re thinking of using SQ= LLine. Under > >>>> the > >>>>>> BSD > >>>>>>>>>>>>>>>> license, you > >>>>>>>>>>>>>>>>>>>> don=E2=80=99t 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=E2=80=99s unlikely that you=E2=80=99ll n= eed > >>>>> Ignite-specific > >>>>>>>>>>>>>>>> extensions =E2=80=94 > >>>>>>>>>>>>>>>>>>>> SQLLine just exposes what comes through the JDBC > >>>>>> driver =E2=80=94 > >>>>>>>> but > >>>>>>>>>>>>>>>> we can > >>>>>>>>>>>>>>>>>>>> discuss if the need arises. The Hive project forked > >>>>>>> SQLLine > >>>>>>>>>>>>>>>> into its > >>>>>>>>>>>>>>>>>> own > >>>>>>>>>>>>>>>>>>>> Beeline module and I=E2=80=99d like to avoid a repea= t of > >>>>> that. > >>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>> Julian > >>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>> On Aug 29, 2017, at 6:35 PM, Denis Magda < > >>>>>>>> dmagda@apache.org > >>>>>>>>>> > >>>>>>>>>>>>>>>>> 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=E2=80=99s > >>>>>>>>>>>>>>>>>> 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=E2=80=99s > >>>>>>>>>>>>>>>>>>>> technical documentation. > >>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>> [1] https://people.apache.org/~jhyde/ < > >>>>>>>>>>>>>>>>> https://people.apache.org/~jh > >>>>>>>>>>>>>>>>>>> yde/ > >>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>> [2] https://github.com/julianhyde/sqlline > >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> julianhyde/sqlline> > >>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>> =E2=80=94 > >>>>>>>>>>>>>>>>>>>>> Denis > >>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>> On Aug 25, 2017, at 9:17 AM, Denis Magda < > >>>>>>>>> dmagda@apache.org > >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> dmagda@apache.org >> > >>>>> wrote: > >>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>> Hi Ilya, > >>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>> Thanks for the clarification! Referring to the > >>>> page > >>>>>>>> shared > >>>>>>>>>>>>>>>> by you > >>>>>>>>>>>>>>>>>> [1] > >>>>>>>>>>>>>>>>>>>> if we need to get author=E2=80=99s consent in a writ= ten > >>>> 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=E2=80=99ll reach out the author requesting the p= ermit. > >>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>> =E2=80=94 > >>>>>>>>>>>>>>>>>>>>>> Denis > >>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>>> On Aug 25, 2017, at 9:12 AM, Ilya Kasnacheev < > >>>>>>>>>>>>>>>>>>>> ilya.kasnacheev@gmail.com > >>>>> >>>>>>>> ilya.kasnacheev@gmail.com > >>>>>>>>>>> > >>>>>>>>>>>>>>>> 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 < > >>>>>>>> dmagda@apache.org > >>>>>>>>>> : > >>>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>>>> Hi Ilya, > >>>>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>>>> It will be a useful addition to Ignite. I would > >>>>>>> include > >>>>>>>>>>>>>>>> the tool > >>>>>>>>>>>>>>>>>> in > >>>>>>>>>>>>>>>>>>>> one of > >>>>>>>>>>>>>>>>>>>>>>>> the nearest releases. It=E2=80=99s license is BS= D-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> > >>>>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>>>> =E2=80=94 > >>>>>>>>>>>>>>>>>>>>>>>> Denis > >>>>>>>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>>>>>>> On Aug 25, 2017, at 8:32 AM, Ilya Suntsov < > >>>>>>>>>>>>>>>>> isuntsov@gridgain.com > >>>>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>>>>>>> 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 > >>>>>>>>>>>>>>>>> > >>>>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>> > >>>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>> > >>>>> > >>>> > >> > >> > > --001a113b1bbcba2258055bda92e5--