Sentry-jdk-1.7 - Build # 648 - Fixed

2016-05-02 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #648) Status: Fixed Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/648/ to view the results.

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Ashish Singh
> On May 2, 2016, 9:45 p.m., Gregory Chanan wrote: > > Why put the caching at the client layer instead of the provider backend > > layer? It seems unlikely to me that the average client actually wants > > caching or need caching, whereas an arbitrary external service probably > > does. > > A

Re: Sentry jira dashboard

2016-05-02 Thread Hao Hao
Thanks a lot Sravya! This is really pretty useful. One comment for the dashboard: similar to code review queues, can we have a label or status for “Design Review” to track the new features Jira that have designs need to be reviewed? Best, Hao On Mon, May 2, 2016 at 6:35 PM, Sravya Tirukkovalur w

Sentry jira dashboard

2016-05-02 Thread Sravya Tirukkovalur
Hi all, Thought it might be useful to have a dashboard that we all share for the project to understand what is on the review and triage queues. We can also use it to facilitate our monthly hangouts. Here is a link. Please let me know what else you would like to see. https://issues.apache.org/jira/

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Gregory Chanan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/#review131419 --- Note: these comments are based on the assumption you are implement

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Gregory Chanan
> On May 2, 2016, 9:45 p.m., Gregory Chanan wrote: > > Why put the caching at the client layer instead of the provider backend > > layer? It seems unlikely to me that the average client actually wants > > caching or need caching, whereas an arbitrary external service probably > > does. > > A

Sentry-jdk-1.7 - Build # 647 - Failure

2016-05-02 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7 (build #647) Status: Failure Check console output at https://builds.apache.org/job/Sentry-jdk-1.7/647/ to view the results.

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Gregory Chanan
> On May 2, 2016, 9:45 p.m., Gregory Chanan wrote: > > Why put the caching at the client layer instead of the provider backend > > layer? It seems unlikely to me that the average client actually wants > > caching or need caching, whereas an arbitrary external service probably > > does. > > A

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Ashish Singh
> On May 2, 2016, 9:45 p.m., Gregory Chanan wrote: > > Why put the caching at the client layer instead of the provider backend > > layer? It seems unlikely to me that the average client actually wants > > caching or need caching, whereas an arbitrary external service probably > > does. > > A

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Ashish Singh
> On May 2, 2016, 9:45 p.m., Gregory Chanan wrote: > > Why put the caching at the client layer instead of the provider backend > > layer? It seems unlikely to me that the average client actually wants > > caching or need caching, whereas an arbitrary external service probably > > does. > > A

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Ashish Singh
> On May 2, 2016, 9:45 p.m., Gregory Chanan wrote: > > Why put the caching at the client layer instead of the provider backend > > layer? It seems unlikely to me that the average client actually wants > > caching or need caching, whereas an arbitrary external service probably > > does. Any a

Re: Review Request 46912: SENTRY-1228: SimpleFileProviderBackend error message missing spaces

2016-05-02 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46912/#review131389 --- Ship it! Ship It! - Vamsee Yarlagadda On May 2, 2016, 9:57 p

Review Request 46912: SENTRY-1228: SimpleFileProviderBackend error message missing spaces

2016-05-02 Thread Gregory Chanan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46912/ --- Review request for sentry and Vamsee Yarlagadda. Repository: sentry Descripti

Re: Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Gregory Chanan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/#review131381 --- Why put the caching at the client layer instead of the provider ba

Review Request 46909: SENTRY-1229: Add caching to SentryGenericServiceClientDefaultImpl.

2016-05-02 Thread Ashish Singh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46909/ --- Review request for sentry, Dapeng Sun, Gregory Chanan, Hao Hao, and Sravya Tiruk

Review Request 46886: SENTRY-1223: Add validator for authorization type hierarchy

2016-05-02 Thread Colin Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46886/ --- Review request for sentry and Dapeng Sun. Repository: sentry Description