[GitHub] [incubator-livy] runzhiwang opened a new pull request #222: [LIVY-647]Fix travis failed on batch session should not be gc-ed until application is finished

[GitBox <gi...@apache.org>]

runzhiwang opened a new pull request #222: [LIVY-647]Fix travis failed on batch session should not be gc-ed until application is finished
URL: https://github.com/apache/incubator-livy/pull/222
 
 
   
   ## What changes were proposed in this pull request?
   
   Fix travis failed on "batch session should not be gc-ed until application is finished"
   
   The cause of failed is as follows:
   
   1. When create BatchSessionManager, the GarbageCollector thread will be created, which check session.state. However the session was mocked, and the test thread has not execute doReturn(s).when(session).state, so cause exception.
   
   2. So the fix avoid collecting garbage of mocked session.
   
   ## How was this patch tested?
   
   Existed UT and IT.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services