Re: Review Request 62659: RANGER-1814 : Move the reader into a local variable in LocalFileLogBuffer

2017-10-02 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62659/#review186832 --- Ship it! Ship It! - Colm O hEigeartaigh On Oct. 2, 2017,

Re: Review Request 62659: RANGER-1814 : Move the reader into a local variable in LocalFileLogBuffer

2017-10-02 Thread Zsombor Gegesy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62659/ --- (Updated Oct. 2, 2017, 9:40 a.m.) Review request for ranger. Changes ---

Re: Review Request 62659: RANGER-1814 : Move the reader into a local variable in LocalFileLogBuffer

2017-09-29 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62659/#review186667 --- Can you fix the whitespace errors in the patch? - Colm O

Re: Review Request 62659: RANGER-1814 : Move the reader into a local variable in LocalFileLogBuffer

2017-09-28 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62659/#review186584 --- Ship it! Ship It! - Alejandro Fernandez On Sept. 28, 2017,

Review Request 62659: RANGER-1814 : Move the reader into a local variable in LocalFileLogBuffer

2017-09-28 Thread Zsombor Gegesy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62659/ --- Review request for ranger. Bugs: RANGER-1814