kylin-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (KYLIN-3250) Upgrade jetty version to 9.2.24.v20180105
Date Tue, 29 May 2018 02:49:00 GMT

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

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

codecov-io commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030
URL: https://github.com/apache/kylin/pull/143#issuecomment-392638206
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/143?src=pr&el=h1) Report
   > Merging [#143](https://codecov.io/gh/apache/kylin/pull/143?src=pr&el=desc) into
[master](https://codecov.io/gh/apache/kylin/commit/2b1e9f4b965a47dde999eca073bdbf49eecdb63d?src=pr&el=desc)
will **increase** coverage by `0.02%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/kylin/pull/143/graphs/tree.svg?height=150&width=650&token=JawVgbgsVo&src=pr)](https://codecov.io/gh/apache/kylin/pull/143?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff              @@
   ##             master     #143      +/-   ##
   ============================================
   + Coverage     21.83%   21.86%   +0.02%     
     Complexity     3888     3888              
   ============================================
     Files           985      985              
     Lines         59716    59716              
     Branches       8623     8623              
   ============================================
   + Hits          13039    13056      +17     
   + Misses        45459    45449      -10     
   + Partials       1218     1211       -7
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/kylin/pull/143?src=pr&el=tree) | Coverage
Δ | Complexity Δ | |
   |---|---|---|---|
   | [...g/apache/kylin/source/datagen/ColumnGenerator.java](https://codecov.io/gh/apache/kylin/pull/143/diff?src=pr&el=tree#diff-Y29yZS1tZXRhZGF0YS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vc291cmNlL2RhdGFnZW4vQ29sdW1uR2VuZXJhdG9yLmphdmE=)
| `72.29% <0%> (+1.35%)` | `8% <0%> (ø)` | :arrow_down: |
   | [.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java](https://codecov.io/gh/apache/kylin/pull/143/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2N1Ym9pZC9UcmVlQ3Vib2lkU2NoZWR1bGVyLmphdmE=)
| `66.15% <0%> (+2.3%)` | `0% <0%> (ø)` | :arrow_down: |
   | [...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/143/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh)
| `70.21% <0%> (+3.64%)` | `7% <0%> (ø)` | :arrow_down: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/kylin/pull/143?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/143?src=pr&el=footer).
Last update [2b1e9f4...4326c39](https://codecov.io/gh/apache/kylin/pull/143?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:
users@infra.apache.org


> Upgrade jetty version to 9.2.24.v20180105
> -----------------------------------------
>
>                 Key: KYLIN-3250
>                 URL: https://issues.apache.org/jira/browse/KYLIN-3250
>             Project: Kylin
>          Issue Type: Improvement
>          Components: Tools, Build and Test
>            Reporter: Ted Yu
>            Priority: Minor
>             Fix For: v2.4.0
>
>
> Currently Kylin depends on 9.2.20.v20161216 which is quite old.
> Memory leak was fixed in Jetty 9.4 and later backported however the version of Jetty
9.2.20 came before this fix.  See [https://github.com/eclipse/jetty.project/issues/1804] and
[https://github.com/apache/hbase/blob/master/pom.xml#L1416.]
> This issue is to upgrade to [9.3.22.v20171030|https://mvnrepository.com/artifact/org.eclipse.jetty/jetty-server/9.3.22.v20171030]
which is the latest version of 9.3



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

Mime
View raw message