jianzhenwu commented on PR #416: URL: https://github.com/apache/incubator-livy/pull/416#issuecomment-1704563058
> Looks good, only a question and a minor style change My point is that this test is used to cover the case where Furetur#isCompleted is true.  This may be a bit redundant. If you suggest removing it, I can add a commit to this PR to remove 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
