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

ASF GitHub Bot commented on KYLIN-3697:
---------------------------------------

codecov-io commented on issue #353: KYLIN-3697 Check port availability when 
starts kylin instance
URL: https://github.com/apache/kylin/pull/353#issuecomment-440281351
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/353?src=pr&el=h1) Report
   > Merging [#353](https://codecov.io/gh/apache/kylin/pull/353?src=pr&el=desc) 
into 
[master](https://codecov.io/gh/apache/kylin/commit/4e0340737fc39364c23c6b7d6d5f604a8e2eca7f?src=pr&el=desc)
 will **increase** coverage by `<.01%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/353/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/353?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff              @@
   ##             master     #353      +/-   ##
   ============================================
   + Coverage      23.6%   23.61%   +<.01%     
   - Complexity     4824     4825       +1     
   ============================================
     Files          1133     1133              
     Lines         68102    68102              
     Branches       9659     9659              
   ============================================
   + Hits          16077    16080       +3     
   + Misses        50458    50457       -1     
   + Partials       1567     1565       -2
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/353?src=pr&el=tree) | Coverage Δ 
| Complexity Δ | |
   |---|---|---|---|
   | 
[...ain/java/org/apache/kylin/cube/model/CubeDesc.java](https://codecov.io/gh/apache/kylin/pull/353/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL21vZGVsL0N1YmVEZXNjLmphdmE=)
 | `64.28% <0%> (ø)` | `179% <0%> (ø)` | :arrow_down: |
   | 
[...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/353/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh)
 | `70.21% <0%> (+0.6%)` | `7% <0%> (ø)` | :arrow_down: |
   | 
[...a/org/apache/kylin/dict/Number2BytesConverter.java](https://codecov.io/gh/apache/kylin/pull/353/diff?src=pr&el=tree#diff-Y29yZS1kaWN0aW9uYXJ5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9kaWN0L051bWJlcjJCeXRlc0NvbnZlcnRlci5qYXZh)
 | `82.53% <0%> (+0.79%)` | `18% <0%> (+1%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/353?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/kylin/pull/353?src=pr&el=footer). Last 
update 
[4e03407...28d1427](https://codecov.io/gh/apache/kylin/pull/353?src=pr&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   

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


> check port availability when starts kylin instance
> --------------------------------------------------
>
>                 Key: KYLIN-3697
>                 URL: https://issues.apache.org/jira/browse/KYLIN-3697
>             Project: Kylin
>          Issue Type: Bug
>    Affects Versions: v2.5.1
>            Reporter: May Zhou
>            Priority: Minor
>
> As I ran the script and it turned out with no error message, and I ran the 
> kylin.sh start script, the console output indicated that a kylin instance had 
> successfully started, but when I visit the <host>:<port>/kylin, I found 
> another service was running rather than kylin.
>   
>   I can only find the error message in the kylin.out, which is not so 
> user-friendly.
> > Caused by: java.net.BindException: Address already in use (Bind failed)
>       at java.net.PlainSocketImpl.socketBind(Native Method)
>       at 
> java.net.AbstractPlainSocketImpl.bind(AbstractPlainSocketImpl.java:387)
>       at java.net.ServerSocket.bind(ServerSocket.java:375)
> I am wondering why check-env does not check the availability of port, and the 
> error message could be threw out more user-friendly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to