IMO the hive-only version is still useful, so +1 to submitting + then create a JIRA to get the other options done later.
Colm. On Wed, Jul 12, 2017 at 3:32 PM, Alexander Kolbasov <[email protected]> wrote: > I should be able to provide the Hive version of CLI for 1.8, but adding > non-hive interfaces would require more work. Do you think that it would > still be useful to have hive-only version (which can be later extended for > other components) or it is better to wait until we have everything ready? > > - Alex > > On Wed, Jul 12, 2017 at 11:41 AM, Colm O hEigeartaigh <[email protected] > > wrote: > >> I updated the JIRA to include 1.8.0 as it would be good to get it in for >> this release, if you have time to fix the issues raised above. >> >> Colm. >> >> On Wed, Jun 28, 2017 at 11:54 AM, Colm O hEigeartaigh < >> [email protected]> >> wrote: >> >> > >> > On Tue, Jun 27, 2017 at 11:39 PM, Alexander Kolbasov < >> [email protected]> >> > wrote: >> > >> >> a) I think it supports command history with arrow keys. >> >> >> > >> > It would be good to integrate this if it is supported by cliche - >> > currently it doesn't work. >> > >> > >> >> d) slf4j jars - isn't it handled already - most of other things need it >> >> >> > >> > No, not on the sentry-ha branch anyway - there are no slf4j jars in >> > "sentry-dist/target/apache-sentry-1.8.0-SNAPSHOT-bin/ >> > apache-sentry-1.8.0-SNAPSHOT-bin/lib/". >> > >> > >> >> e) I plan to merge it to master - there is no reason it shouldn't work >> on >> >> master as well. >> >> >> > >> > Great! Another question for you...is there any way to get it to work >> with >> > Kafka (or Solr?). If I try to add a Kafka privilege it errors >> complaining >> > that a "Server" is required (as per Hive): >> > >> > Caused by: org.apache.thrift.protocol.TProtocolException: Required >> field >> > 'serverName' is unset! Struct:TSentryPrivilege(privilegeScope:SERVER, >> > serverName:null, action:*, createTime:1498646259888, grantOption:FALSE) >> > >> > Colm. >> > >> > >> >> >> >> On Mon, Jun 26, 2017 at 4:50 AM, Colm O hEigeartaigh < >> [email protected] >> >> > >> >> wrote: >> >> >> >> > Thanks, it looks pretty cool. A few comments/questions: >> >> > >> >> > a) Is there a way to configure it to list again the previous command >> by >> >> > pressing the up arrow key? >> >> > b) Is this meant to deprecate the Sentry Shell? Or compliment it? >> >> > c) Will you add a script to the distribution bin directory to make it >> >> easy >> >> > to run it? >> >> > d) The distribution lib directory needs the slf4j jars in it as well >> to >> >> get >> >> > it to run there. >> >> > e) Will you merge this to master as well so that will be in 1.8.0? >> >> > >> >> > Colm. >> >> > >> >> > On Fri, Jun 23, 2017 at 6:27 PM, Alexander Kolbasov < >> [email protected] >> >> > >> >> > wrote: >> >> > >> >> > > Here is another doc about Cliche from the original developer: >> >> > > >> >> > > https://www.codeproject.com/Articles/55088/Create-Minimal- >> >> > > UI-in-No-Time-at-all-Using-Cliche-S >> >> > > >> >> > > On Fri, Jun 23, 2017 at 1:22 AM, Colm O hEigeartaigh < >> >> > [email protected]> >> >> > > wrote: >> >> > > >> >> > > > Is there a tutorial on any documentation on how it works? MIT >> >> license >> >> > is >> >> > > > fine, just make sure that we update our NOTICE and LICENSE file >> >> > > > accordingly, and make sure the MIT license is shipped with the >> >> > > > distribution. >> >> > > > >> >> > > > Colm. >> >> > > > >> >> > > > On Fri, Jun 23, 2017 at 12:27 AM, Alexander Kolbasov < >> >> > [email protected] >> >> > > > >> >> > > > wrote: >> >> > > > >> >> > > > > We had this discussion a while ago, I'd like to being it back >> on >> >> the >> >> > > > table. >> >> > > > > >> >> > > > > I implemented a simple Sentry interactive CLI that is based on >> >> Cliche >> >> > > > > library (https://github.com/budhash/cliche). >> >> > > > > >> >> > > > > The library is available under MIT license - I guess it >> shouldn't >> >> be >> >> > a >> >> > > > > problem. >> >> > > > > >> >> > > > > I opened SENTRY-1812 to track this issue and posted the initial >> >> patch >> >> > > > > there. >> >> > > > > >> >> > > > > Please let me know what do you think about it. >> >> > > > > >> >> > > > > - Alex >> >> > > > > >> >> > > > >> >> > > > >> >> > > > >> >> > > > -- >> >> > > > Colm O hEigeartaigh >> >> > > > >> >> > > > Talend Community Coder >> >> > > > http://coders.talend.com >> >> > > > >> >> > > >> >> > >> >> > >> >> > >> >> > -- >> >> > Colm O hEigeartaigh >> >> > >> >> > Talend Community Coder >> >> > http://coders.talend.com >> >> > >> >> >> > >> > >> > >> > -- >> > Colm O hEigeartaigh >> > >> > Talend Community Coder >> > http://coders.talend.com >> > >> >> >> >> -- >> Colm O hEigeartaigh >> >> Talend Community Coder >> http://coders.talend.com >> > > -- Colm O hEigeartaigh Talend Community Coder http://coders.talend.com
