[GitHub] [kylin] coveralls edited a comment on pull request #1596: [KYLIN-4780] Support dingtalk notify

2021-04-13 Thread GitBox


coveralls edited a comment on pull request #1596:
URL: https://github.com/apache/kylin/pull/1596#issuecomment-791204373


   ## Pull Request Test Coverage Report for [Build 
7065](https://coveralls.io/builds/38770165)
   
   * **57** of **333**   **(17.12%)**  changed or added relevant lines in 
**20** files are covered.
   * **35** unchanged lines in **10** files lost coverage.
   * Overall coverage decreased (**-0.02%**) to **28.073%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/NotificationConstants.java](https://coveralls.io/builds/38770165/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FNotificationConstants.java#L21)
 | 0 | 1 | 0.0%
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38770165/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L69)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/constant/NotifyType.java](https://coveralls.io/builds/38770165/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fconstant%2FNotifyType.java#L21)
 | 0 | 2 | 0.0%
   | 
[core-cube/src/main/java/org/apache/kylin/cube/model/CubeDesc.java](https://coveralls.io/builds/38770165/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Fmodel%2FCubeDesc.java#L537)
 | 10 | 13 | 76.92%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/SecretKeyUtil.java](https://coveralls.io/builds/38770165/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FSecretKeyUtil.java#L30)
 | 8 | 12 | 66.67%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/NotifyServiceBase.java](https://coveralls.io/builds/38770165/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2FNotifyServiceBase.java#L31)
 | 0 | 5 | 0.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkNotificationUtil.java](https://coveralls.io/builds/38770165/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkNotificationUtil.java#L26)
 | 12 | 17 | 70.59%
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38770165/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L140)
 | 0 | 5 | 0.0%
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38770165/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L84)
 | 0 | 6 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/CubeService.java](https://coveralls.io/builds/38770165/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FCubeService.java#L582)
 | 0 | 6 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationService.java](https://coveralls.io/builds/38770165/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationService.java#L236)
 | 1 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/CubingJob.java](https://coveralls.io/builds/38770165/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FCubingJob.java#L167)
 | 1 | 1.48% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38770165/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L71)
 | 1 | 0% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38770165/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L143)
 | 1 | 0% |
   | 
[stream-core/src/main/java/org/apache/kylin/stream/core/storage/columnar/GeneralColumnDataReader.java](https://coveralls.io/builds/38770165/source?filename=stream-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fstream%2Fcore%2Fstorage%2Fcolumnar%2FGeneralColumnDataReader.java#L43)
 | 1 | 92.11% |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38770165/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L216)
 | 2 | 0% |
   | 

[GitHub] [kylin] coveralls edited a comment on pull request #1596: [KYLIN-4780] support dingtalk notify

2021-04-11 Thread GitBox


coveralls edited a comment on pull request #1596:
URL: https://github.com/apache/kylin/pull/1596#issuecomment-791204373


   ## Pull Request Test Coverage Report for [Build 
7049](https://coveralls.io/builds/38707212)
   
   * **57** of **347**   **(16.43%)**  changed or added relevant lines in 
**20** files are covered.
   * **14** unchanged lines in **8** files lost coverage.
   * Overall coverage decreased (**-0.02%**) to **28.073%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/NotificationConstants.java](https://coveralls.io/builds/38707212/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FNotificationConstants.java#L21)
 | 0 | 1 | 0.0%
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38707212/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L69)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/constant/NotifyType.java](https://coveralls.io/builds/38707212/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fconstant%2FNotifyType.java#L21)
 | 0 | 2 | 0.0%
   | 
[core-cube/src/main/java/org/apache/kylin/cube/model/CubeDesc.java](https://coveralls.io/builds/38707212/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Fmodel%2FCubeDesc.java#L537)
 | 10 | 13 | 76.92%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/SecretKeyUtil.java](https://coveralls.io/builds/38707212/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FSecretKeyUtil.java#L30)
 | 8 | 12 | 66.67%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/NotifyServiceBase.java](https://coveralls.io/builds/38707212/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2FNotifyServiceBase.java#L31)
 | 0 | 5 | 0.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkNotificationUtil.java](https://coveralls.io/builds/38707212/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkNotificationUtil.java#L26)
 | 12 | 17 | 70.59%
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38707212/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L140)
 | 0 | 5 | 0.0%
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38707212/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L84)
 | 0 | 6 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/CubeService.java](https://coveralls.io/builds/38707212/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FCubeService.java#L582)
 | 0 | 6 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationService.java](https://coveralls.io/builds/38707212/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationService.java#L236)
 | 1 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/CubingJob.java](https://coveralls.io/builds/38707212/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FCubingJob.java#L167)
 | 1 | 1.48% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38707212/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L71)
 | 1 | 0% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38707212/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L143)
 | 1 | 0% |
   | 
[stream-core/src/main/java/org/apache/kylin/stream/core/storage/columnar/GeneralColumnDataReader.java](https://coveralls.io/builds/38707212/source?filename=stream-core%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fstream%2Fcore%2Fstorage%2Fcolumnar%2FGeneralColumnDataReader.java#L43)
 | 1 | 92.11% |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38707212/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L216)
 | 2 | 0% |
   | 

[GitHub] [kylin] coveralls edited a comment on pull request #1596: [KYLIN-4780] support dingtalk notify

2021-03-28 Thread GitBox


coveralls edited a comment on pull request #1596:
URL: https://github.com/apache/kylin/pull/1596#issuecomment-791204373


   ## Pull Request Test Coverage Report for [Build 
7008](https://coveralls.io/builds/38321943)
   
   * **55** of **322**   **(17.08%)**  changed or added relevant lines in 
**21** files are covered.
   * **1106** unchanged lines in **27** files lost coverage.
   * Overall coverage increased (+**0.2%**) to **28.108%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkNotificationUtil.java](https://coveralls.io/builds/38321943/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkNotificationUtil.java#L25)
 | 3 | 4 | 75.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/NotificationConstants.java](https://coveralls.io/builds/38321943/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FNotificationConstants.java#L21)
 | 0 | 1 | 0.0%
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38321943/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L69)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/constant/NotifyType.java](https://coveralls.io/builds/38321943/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fconstant%2FNotifyType.java#L21)
 | 0 | 2 | 0.0%
   | 
[core-cube/src/main/java/org/apache/kylin/cube/model/CubeDesc.java](https://coveralls.io/builds/38321943/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Fmodel%2FCubeDesc.java#L536)
 | 10 | 13 | 76.92%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkTemplateProvider.java](https://coveralls.io/builds/38321943/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkTemplateProvider.java#L50)
 | 12 | 16 | 75.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/SecretKeyUtil.java](https://coveralls.io/builds/38321943/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FSecretKeyUtil.java#L30)
 | 8 | 12 | 66.67%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/NotifyServiceBase.java](https://coveralls.io/builds/38321943/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2FNotifyServiceBase.java#L31)
 | 0 | 5 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38321943/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L140)
 | 0 | 5 | 0.0%
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38321943/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L84)
 | 0 | 6 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationService.java](https://coveralls.io/builds/38321943/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationService.java#L236)
 | 1 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/CubingJob.java](https://coveralls.io/builds/38321943/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FCubingJob.java#L167)
 | 1 | 1.48% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38321943/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L71)
 | 1 | 0% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38321943/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L143)
 | 1 | 0% |
   | 
[core-job/src/main/java/org/apache/kylin/job/impl/threadpool/DefaultScheduler.java](https://coveralls.io/builds/38321943/source?filename=core-job%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fjob%2Fimpl%2Fthreadpool%2FDefaultScheduler.java#L193)
 | 2 | 79.07% |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38321943/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L216)
 | 2 | 0% |
   | 

[GitHub] [kylin] coveralls edited a comment on pull request #1596: [KYLIN-4780] support dingtalk notify

2021-03-19 Thread GitBox


coveralls edited a comment on pull request #1596:
URL: https://github.com/apache/kylin/pull/1596#issuecomment-791204373


   ## Pull Request Test Coverage Report for [Build 
6985](https://coveralls.io/builds/38076867)
   
   * **62** of **335**   **(18.51%)**  changed or added relevant lines in 
**21** files are covered.
   * **856** unchanged lines in **22** files lost coverage.
   * Overall coverage increased (+**0.2%**) to **28.147%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkNotificationUtil.java](https://coveralls.io/builds/38076867/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkNotificationUtil.java#L25)
 | 3 | 4 | 75.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/NotificationConstants.java](https://coveralls.io/builds/38076867/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FNotificationConstants.java#L21)
 | 0 | 1 | 0.0%
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38076867/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L69)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/constant/NotifyType.java](https://coveralls.io/builds/38076867/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fconstant%2FNotifyType.java#L21)
 | 0 | 2 | 0.0%
   | 
[core-cube/src/main/java/org/apache/kylin/cube/model/CubeDesc.java](https://coveralls.io/builds/38076867/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Fmodel%2FCubeDesc.java#L536)
 | 10 | 13 | 76.92%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkTemplateProvider.java](https://coveralls.io/builds/38076867/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkTemplateProvider.java#L55)
 | 19 | 23 | 82.61%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/SecretKeyUtil.java](https://coveralls.io/builds/38076867/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FSecretKeyUtil.java#L30)
 | 8 | 12 | 66.67%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/NotifyServiceBase.java](https://coveralls.io/builds/38076867/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2FNotifyServiceBase.java#L31)
 | 0 | 5 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38076867/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L140)
 | 0 | 5 | 0.0%
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38076867/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L84)
 | 0 | 6 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationService.java](https://coveralls.io/builds/38076867/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationService.java#L235)
 | 1 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/CubingJob.java](https://coveralls.io/builds/38076867/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FCubingJob.java#L166)
 | 1 | 1.47% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/38076867/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L71)
 | 1 | 0% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/38076867/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L143)
 | 1 | 0% |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/38076867/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L216)
 | 2 | 0% |
   | 
[tool/src/main/java/org/apache/kylin/tool/query/ProbabilityGenerator.java](https://coveralls.io/builds/38076867/source?filename=tool%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Ftool%2Fquery%2FProbabilityGenerator.java#L42)
 | 3 | 76.32% |
   | 

[GitHub] [kylin] coveralls edited a comment on pull request #1596: [KYLIN-4780] support dingtalk notify

2021-03-15 Thread GitBox


coveralls edited a comment on pull request #1596:
URL: https://github.com/apache/kylin/pull/1596#issuecomment-791204373


   ## Pull Request Test Coverage Report for [Build 
6972](https://coveralls.io/builds/37949501)
   
   * **62** of **342**   **(18.13%)**  changed or added relevant lines in 
**22** files are covered.
   * **258** unchanged lines in **13** files lost coverage.
   * Overall coverage increased (+**0.2%**) to **28.165%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkNotificationUtil.java](https://coveralls.io/builds/37949501/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkNotificationUtil.java#L25)
 | 3 | 4 | 75.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/NotificationConstants.java](https://coveralls.io/builds/37949501/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FNotificationConstants.java#L21)
 | 0 | 1 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/controller/StreamingV2Controller.java](https://coveralls.io/builds/37949501/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fcontroller%2FStreamingV2Controller.java#L520)
 | 0 | 1 | 0.0%
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/37949501/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L69)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/constant/NotifyType.java](https://coveralls.io/builds/37949501/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fconstant%2FNotifyType.java#L21)
 | 0 | 2 | 0.0%
   | 
[core-cube/src/main/java/org/apache/kylin/cube/model/CubeDesc.java](https://coveralls.io/builds/37949501/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Fmodel%2FCubeDesc.java#L536)
 | 10 | 13 | 76.92%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkTemplateProvider.java](https://coveralls.io/builds/37949501/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkTemplateProvider.java#L55)
 | 19 | 23 | 82.61%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/SecretKeyUtil.java](https://coveralls.io/builds/37949501/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FSecretKeyUtil.java#L30)
 | 8 | 12 | 66.67%
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/37949501/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L84)
 | 0 | 6 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/37949501/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L141)
 | 0 | 6 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationService.java](https://coveralls.io/builds/37949501/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationService.java#L235)
 | 1 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/CubingJob.java](https://coveralls.io/builds/37949501/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FCubingJob.java#L166)
 | 1 | 1.47% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/37949501/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L71)
 | 1 | 0% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/37949501/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L145)
 | 1 | 0% |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/37949501/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L216)
 | 2 | 0% |
   | 
[core-job/src/main/java/org/apache/kylin/job/execution/AbstractExecutable.java](https://coveralls.io/builds/37949501/source?filename=core-job%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fjob%2Fexecution%2FAbstractExecutable.java#L235)
 | 5 | 58.3% |
   | 

[GitHub] [kylin] coveralls edited a comment on pull request #1596: [KYLIN-4780] support dingtalk notify

2021-03-09 Thread GitBox


coveralls edited a comment on pull request #1596:
URL: https://github.com/apache/kylin/pull/1596#issuecomment-791204373


   ## Pull Request Test Coverage Report for [Build 
6954](https://coveralls.io/builds/37804993)
   
   * **62** of **281**   **(22.06%)**  changed or added relevant lines in 
**21** files are covered.
   * **174** unchanged lines in **11** files lost coverage.
   * Overall coverage increased (+**0.005%**) to **27.961%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkNotificationUtil.java](https://coveralls.io/builds/37804993/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkNotificationUtil.java#L25)
 | 3 | 4 | 75.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/NotificationConstant.java](https://coveralls.io/builds/37804993/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FNotificationConstant.java#L21)
 | 0 | 1 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/controller/StreamingV2Controller.java](https://coveralls.io/builds/37804993/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fcontroller%2FStreamingV2Controller.java#L520)
 | 0 | 1 | 0.0%
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/37804993/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L69)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/constant/NotifyType.java](https://coveralls.io/builds/37804993/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fconstant%2FNotifyType.java#L21)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/37804993/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L269)
 | 0 | 2 | 0.0%
   | 
[core-cube/src/main/java/org/apache/kylin/cube/model/CubeDesc.java](https://coveralls.io/builds/37804993/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Fmodel%2FCubeDesc.java#L536)
 | 10 | 13 | 76.92%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkTemplateProvider.java](https://coveralls.io/builds/37804993/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkTemplateProvider.java#L55)
 | 19 | 23 | 82.61%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/SecretKeyUtil.java](https://coveralls.io/builds/37804993/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FSecretKeyUtil.java#L30)
 | 8 | 12 | 66.67%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/NotifyServiceBase.java](https://coveralls.io/builds/37804993/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2FNotifyServiceBase.java#L27)
 | 0 | 6 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationService.java](https://coveralls.io/builds/37804993/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationService.java#L234)
 | 1 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/CubingJob.java](https://coveralls.io/builds/37804993/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FCubingJob.java#L166)
 | 1 | 1.47% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/37804993/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L71)
 | 1 | 0% |
   | 
[tool/src/main/java/org/apache/kylin/tool/query/ProbabilityGenerator.java](https://coveralls.io/builds/37804993/source?filename=tool%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Ftool%2Fquery%2FProbabilityGenerator.java#L42)
 | 1 | 81.58% |
   | 
[core-job/src/main/java/org/apache/kylin/job/impl/threadpool/DefaultScheduler.java](https://coveralls.io/builds/37804993/source?filename=core-job%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fjob%2Fimpl%2Fthreadpool%2FDefaultScheduler.java#L193)
 | 2 | 79.07% |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/37804993/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L216)
 | 2 | 0% |
   | 

[GitHub] [kylin] coveralls edited a comment on pull request #1596: [KYLIN-4780] support dingtalk notify

2021-03-09 Thread GitBox


coveralls edited a comment on pull request #1596:
URL: https://github.com/apache/kylin/pull/1596#issuecomment-791204373


   ## Pull Request Test Coverage Report for [Build 
6951](https://coveralls.io/builds/37772030)
   
   * **62** of **281**   **(22.06%)**  changed or added relevant lines in 
**21** files are covered.
   * **174** unchanged lines in **10** files lost coverage.
   * Overall coverage increased (+**0.004%**) to **27.96%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkNotificationUtil.java](https://coveralls.io/builds/37772030/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkNotificationUtil.java#L25)
 | 3 | 4 | 75.0%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/NotificationConstant.java](https://coveralls.io/builds/37772030/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FNotificationConstant.java#L21)
 | 0 | 1 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/controller/StreamingV2Controller.java](https://coveralls.io/builds/37772030/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fcontroller%2FStreamingV2Controller.java#L520)
 | 0 | 1 | 0.0%
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/37772030/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L69)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/constant/NotifyType.java](https://coveralls.io/builds/37772030/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fconstant%2FNotifyType.java#L21)
 | 0 | 2 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java](https://coveralls.io/builds/37772030/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fjob%2FKylinHealthCheckJob.java#L269)
 | 0 | 2 | 0.0%
   | 
[core-cube/src/main/java/org/apache/kylin/cube/model/CubeDesc.java](https://coveralls.io/builds/37772030/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Fmodel%2FCubeDesc.java#L536)
 | 10 | 13 | 76.92%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/DingTalkTemplateProvider.java](https://coveralls.io/builds/37772030/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FDingTalkTemplateProvider.java#L55)
 | 19 | 23 | 82.61%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/util/SecretKeyUtil.java](https://coveralls.io/builds/37772030/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2Futil%2FSecretKeyUtil.java#L30)
 | 8 | 12 | 66.67%
   | 
[core-common/src/main/java/org/apache/kylin/common/notify/NotifyServiceBase.java](https://coveralls.io/builds/37772030/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fnotify%2FNotifyServiceBase.java#L27)
 | 0 | 6 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationService.java](https://coveralls.io/builds/37772030/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationService.java#L234)
 | 1 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/LookupSnapshotBuildJob.java](https://coveralls.io/builds/37772030/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FLookupSnapshotBuildJob.java#L71)
 | 1 | 0% |
   | 
[core-job/src/main/java/org/apache/kylin/job/impl/threadpool/DefaultScheduler.java](https://coveralls.io/builds/37772030/source?filename=core-job%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fjob%2Fimpl%2Fthreadpool%2FDefaultScheduler.java#L193)
 | 2 | 79.07% |
   | 
[cube-migration/src/main/java/org/apache/kylin/rest/service/MigrationRuleSet.java](https://coveralls.io/builds/37772030/source?filename=cube-migration%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FMigrationRuleSet.java#L216)
 | 2 | 0% |
   | 
[engine-mr/src/main/java/org/apache/kylin/engine/mr/CubingJob.java](https://coveralls.io/builds/37772030/source?filename=engine-mr%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fengine%2Fmr%2FCubingJob.java#L166)
 | 2 | 1.47% |
   | 
[core-job/src/main/java/org/apache/kylin/job/execution/AbstractExecutable.java](https://coveralls.io/builds/37772030/source?filename=core-job%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fjob%2Fexecution%2FAbstractExecutable.java#L320)
 | 4 | 58.78% |
   |