From reviews-return-1755-archive-asf-public=cust-asf.ponee.io@livy.incubator.apache.org Mon Sep 2 07:28:30 2019 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id D14C7180658 for ; Mon, 2 Sep 2019 09:28:29 +0200 (CEST) Received: (qmail 75816 invoked by uid 500); 2 Sep 2019 07:44:30 -0000 Mailing-List: contact reviews-help@livy.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@livy.incubator.apache.org Delivered-To: mailing list reviews@livy.incubator.apache.org Received: (qmail 75805 invoked by uid 99); 2 Sep 2019 07:44:30 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 02 Sep 2019 07:44:30 +0000 From: GitBox To: reviews@livy.apache.org Subject: [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 Message-ID: <156740930914.25227.1984520741519172718.gitbox@gitbox.apache.org> Date: Mon, 02 Sep 2019 07:28:29 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit 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