Re: Review Request 51307: Catch IOError.

2016-08-29 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review147179 --- Ship it! Ship It! - Maxim Khutornenko On Aug. 23, 2016, 9:12

Re: Review Request 51307: Catch IOError.

2016-08-29 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review147178 --- Ship it! Master (56906a7) is green with this patch. ./build-s

Re: Review Request 51307: Catch IOError.

2016-08-29 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review147172 --- @ReviewBot retry - Stephan Erb On Aug. 23, 2016, 11:12 p.m., Da

Re: Review Request 51307: Catch IOError.

2016-08-23 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review146579 --- Master (c115ac6) is red with this patch. ./build-support/jenkins

Re: Review Request 51307: Catch IOError.

2016-08-23 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review146573 --- Ship it! Ship It! - Joshua Cohen On Aug. 23, 2016, 9:12 p.m.

Re: Review Request 51307: Catch IOError.

2016-08-23 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/ --- (Updated Aug. 23, 2016, 9:12 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 51307: Catch IOError.

2016-08-23 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review146572 --- Master (c115ac6) is red with this patch. ./build-support/jenkins

Re: Review Request 51307: Catch IOError.

2016-08-23 Thread David Robinson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/ --- (Updated Aug. 23, 2016, 9:03 p.m.) Review request for Aurora, Joshua Cohen and

Re: Review Request 51307: Catch IOError.

2016-08-22 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review146461 --- Master (9b34a40) is green with this patch. ./build-support/jenki

Re: Review Request 51307: Catch IOError.

2016-08-22 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51307/#review146459 --- Thanks for fixing this! A very minor unit test would be great here