Re: Review Request 62229: SENTRY-1940 Sentry should time out threads waiting for notifications

2017-09-11 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62229/#review185121 --- Fix it, then Ship it! The patch looks good overall. I just left

Re: Review Request 62229: SENTRY-1940 Sentry should time out threads waiting for notifications

2017-09-11 Thread Vamsee Yarlagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62229/#review185120 --- Ship it! LGTM. - Vamsee Yarlagadda On Sept. 11, 2017, 9:35 p

Review Request 62229: SENTRY-1940 Sentry should time out threads waiting for notifications

2017-09-11 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62229/ --- Review request for sentry, Arjun Mishra, Brian Towles, Na Li, Sergio Pena, and V