[ 
https://issues.apache.org/jira/browse/SENTRY-1157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15206146#comment-15206146
 ] 

Dapeng Sun edited comment on SENTRY-1157 at 3/22/16 10:43 AM:
--------------------------------------------------------------

Since this jira broken jenkins build, I committed immediately, Thank Colin for 
your review


commit 4643f988a5e0ce2b9749e6365edea3a16482de86
Author: Sun Dapeng <[email protected]>
Date:   Tue Mar 22 18:40:15 2016 +0800

    SENTRY-1157: Fix Unit Tests TestAclsCrud&TestAuthorize failed (Dapeng Sun, 
reviewed by Colin Ma)



was (Author: dapengsun):
Since this jira broken jenkins build, I committed immediately, Thank Colin for 
your review

> Fix Unit Tests TestAclsCrud&TestAuthorize failed
> ------------------------------------------------
>
>                 Key: SENTRY-1157
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1157
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.7.0
>            Reporter: Dapeng Sun
>            Assignee: Dapeng Sun
>         Attachments: SENTRY-1157.001.patch
>
>
> 1. TestUtils.getFreePort() should close the ServerSocket;
> 2. Since there are two services, after we got the first free port, we should 
> start the service immediately, then find another free, start service...



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to