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
