@echauchot 
I updated the changes
a. added constants for test cases
b. removed fail(), could not remove try-catch block as i need to assert against 
cause of exception (which is IOException), so keeping it unchanged.

If there is a way i can start IT on jenkins ?
I ran IT locally on my laptop sharing the screen shot here. 
<img width="1680" alt="screen shot 2018-08-22 at 12 17 06" 
src="https://user-images.githubusercontent.com/1793534/44458189-b9f9c000-a605-11e8-85ed-f45a0d6c73ee.png";>
<img width="1680" alt="screen shot 2018-08-22 at 12 11 49" 
src="https://user-images.githubusercontent.com/1793534/44458190-b9f9c000-a605-11e8-8043-41b2b3f9a084.png";>


[ Full content available at: https://github.com/apache/beam/pull/6146 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to