Re: Review Request 53872: SENTRY-1525: Provide script to run Sentry directly from the repo

2016-12-02 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53872/ --- (Updated Dec. 3, 2016, 5 a.m.) Review request for sentry, Colm O hEigeartaigh,

Re: Review Request 53872: SENTRY-1525: Provide script to run Sentry directly from the repo

2016-12-02 Thread Alexander Kolbasov
> On Nov. 28, 2016, 6:53 p.m., Hao Hao wrote: > > bin/run_sentry.sh, line 24 > > > > > > Should it be configurable? Configurable in what way? I don't want this script to have any argument since all of them are going

Review Request 54338: SENTRY-1515: Cleanup exception handling in SentryStore

2016-12-02 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54338/ --- Review request for sentry, Colin Ma, Hao Hao, kalyan kumar kalvagadda, Vamsee Ya