You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kylin.apache.org by GitBox <gi...@apache.org> on 2019/12/22 23:52:44 UTC
[GitHub] [kylin] weibin0516 opened a new pull request #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
weibin0516 opened a new pull request #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
URL: https://github.com/apache/kylin/pull/1013
…ther mailboxes have no chance to send
----------------------------------------------------------------
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
[GitHub] [kylin] codecov-io edited a comment on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
URL: https://github.com/apache/kylin/pull/1013#issuecomment-568488290
# [Codecov](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=h1) Report
> :exclamation: No coverage uploaded for pull request base (`master@7f16793`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).
> The diff coverage is `0%`.
[](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #1013 +/- ##
=========================================
Coverage ? 25.14%
Complexity ? 6180
=========================================
Files ? 1431
Lines ? 87171
Branches ? 12192
=========================================
Hits ? 21920
Misses ? 63109
Partials ? 2142
```
| [Impacted Files](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
|---|---|---|---|
| [...java/org/apache/kylin/common/util/MailService.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi91dGlsL01haWxTZXJ2aWNlLmphdmE=) | `0% <0%> (ø)` | `0 <0> (?)` | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/kylin/pull/1013?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/1013?src=pr&el=footer). Last update [7f16793...ec049a3](https://codecov.io/gh/apache/kylin/pull/1013?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 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
[GitHub] [kylin] nichunen merged pull request #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
Posted by GitBox <gi...@apache.org>.
nichunen merged pull request #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
URL: https://github.com/apache/kylin/pull/1013
----------------------------------------------------------------
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
[GitHub] [kylin] codecov-io edited a comment on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
Posted by GitBox <gi...@apache.org>.
codecov-io edited a comment on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
URL: https://github.com/apache/kylin/pull/1013#issuecomment-568488290
# [Codecov](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=h1) Report
> :exclamation: No coverage uploaded for pull request base (`master@7f16793`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).
> The diff coverage is `0%`.
[](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #1013 +/- ##
=========================================
Coverage ? 25.14%
Complexity ? 6180
=========================================
Files ? 1431
Lines ? 87171
Branches ? 12192
=========================================
Hits ? 21920
Misses ? 63109
Partials ? 2142
```
| [Impacted Files](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
|---|---|---|---|
| [...java/org/apache/kylin/common/util/MailService.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi91dGlsL01haWxTZXJ2aWNlLmphdmE=) | `0% <0%> (ø)` | `0 <0> (?)` | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/kylin/pull/1013?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/1013?src=pr&el=footer). Last update [7f16793...ec049a3](https://codecov.io/gh/apache/kylin/pull/1013?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 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
[GitHub] [kylin] codecov-io commented on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
Posted by GitBox <gi...@apache.org>.
codecov-io commented on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
URL: https://github.com/apache/kylin/pull/1013#issuecomment-568488290
# [Codecov](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=h1) Report
> Merging [#1013](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=desc) into [master](https://codecov.io/gh/apache/kylin/commit/82531818487ff5b2f6a469846d51da6bb50a52a8?src=pr&el=desc) will **decrease** coverage by `0.31%`.
> The diff coverage is `0%`.
[](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #1013 +/- ##
============================================
- Coverage 25.45% 25.14% -0.32%
- Complexity 6164 6180 +16
============================================
Files 1416 1431 +15
Lines 85898 87171 +1273
Branches 12051 12192 +141
============================================
+ Hits 21868 21920 +52
- Misses 61902 63109 +1207
- Partials 2128 2142 +14
```
| [Impacted Files](https://codecov.io/gh/apache/kylin/pull/1013?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
|---|---|---|---|
| [...java/org/apache/kylin/common/util/MailService.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi91dGlsL01haWxTZXJ2aWNlLmphdmE=) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
| [...ylin/source/jdbc/metadata/JdbcMetadataFactory.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL21ldGFkYXRhL0pkYmNNZXRhZGF0YUZhY3RvcnkuamF2YQ==) | `0% <0%> (-25%)` | `0% <0%> (-1%)` | |
| [...ain/java/org/apache/kylin/source/jdbc/SqlUtil.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL1NxbFV0aWwuamF2YQ==) | `43.24% <0%> (-13.52%)` | `20% <0%> (-2%)` | |
| [...ylin/source/jdbc/extensible/JdbcHiveInputBase.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL2V4dGVuc2libGUvSmRiY0hpdmVJbnB1dEJhc2UuamF2YQ==) | `71.42% <0%> (-12.58%)` | `2% <0%> (ø)` | |
| [...rg/apache/kylin/source/jdbc/JdbcHiveInputBase.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-c291cmNlLWpkYmMvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3NvdXJjZS9qZGJjL0pkYmNIaXZlSW5wdXRCYXNlLmphdmE=) | `9.02% <0%> (-12.02%)` | `7% <0%> (-1%)` | |
| [...ylin/job/impl/threadpool/DefaultFetcherRunner.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-Y29yZS1qb2Ivc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2pvYi9pbXBsL3RocmVhZHBvb2wvRGVmYXVsdEZldGNoZXJSdW5uZXIuamF2YQ==) | `54.38% <0%> (-8.12%)` | `6% <0%> (ø)` | |
| [...he/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-Y29yZS1kaWN0aW9uYXJ5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9kaWN0L2xvb2t1cC9jYWNoZS9Sb2Nrc0RCTG9va3VwVGFibGUuamF2YQ==) | `72.97% <0%> (-5.41%)` | `6% <0%> (-1%)` | |
| [...ylin/sdk/datasource/framework/conv/ConvMaster.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-ZGF0YXNvdXJjZS1zZGsvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Nkay9kYXRhc291cmNlL2ZyYW1ld29yay9jb252L0NvbnZNYXN0ZXIuamF2YQ==) | `65.67% <0%> (-2.99%)` | `11% <0%> (ø)` | |
| [...k/datasource/framework/SourceConnectorFactory.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-ZGF0YXNvdXJjZS1zZGsvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Nkay9kYXRhc291cmNlL2ZyYW1ld29yay9Tb3VyY2VDb25uZWN0b3JGYWN0b3J5LmphdmE=) | `55% <0%> (-2.9%)` | `1% <0%> (ø)` | |
| [...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh) | `69.3% <0%> (-1.83%)` | `7% <0%> (ø)` | |
| ... and [49 more](https://codecov.io/gh/apache/kylin/pull/1013/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/kylin/pull/1013?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/1013?src=pr&el=footer). Last update [8253181...ec049a3](https://codecov.io/gh/apache/kylin/pull/1013?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 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
[GitHub] [kylin] weibin0516 removed a comment on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
Posted by GitBox <gi...@apache.org>.
weibin0516 removed a comment on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
URL: https://github.com/apache/kylin/pull/1013#issuecomment-568383762
retest this please
----------------------------------------------------------------
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
[GitHub] [kylin] weibin0516 commented on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
Posted by GitBox <gi...@apache.org>.
weibin0516 commented on issue #1013: KYLIN-4309 Fixed bug that failed to send to one mailbox will caused o…
URL: https://github.com/apache/kylin/pull/1013#issuecomment-568383762
retest this please
----------------------------------------------------------------
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