[GitHub] coveralls commented on issue #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache

coveralls commented on issue #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache URL: https://github.com/apache/kylin/pull/290#issuecomment-429717932 ## Pull Request Test Coverage Report for [Build 3769](https://coveralls.io/builds/19518586) * **0** of

[GitHub] codecov-io commented on issue #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache

codecov-io commented on issue #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache URL: https://github.com/apache/kylin/pull/290#issuecomment-429717664 # [Codecov](https://codecov.io/gh/apache/kylin/pull/290?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache

asfgit commented on issue #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache URL: https://github.com/apache/kylin/pull/290#issuecomment-429714335 Can one of the admins verify this patch? This

[GitHub] allenma opened a new pull request #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache

allenma opened a new pull request #290: KYLIN-3632 Add configuration that can switch on/off preparedStatement cache URL: https://github.com/apache/kylin/pull/290 Add configuration that can switch on/off preparedStatement cache in kylin server

[GitHub] shaofengshi closed pull request #286: KYLIN-3617 Use job's cache in job scheduler

shaofengshi closed pull request #286: KYLIN-3617 Use job's cache in job scheduler URL: https://github.com/apache/kylin/pull/286 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] shaofengshi closed pull request #289: KYLIN-3630, remove unused fields in the implementations of MeasureType

shaofengshi closed pull request #289: KYLIN-3630, remove unused fields in the implementations of MeasureType URL: https://github.com/apache/kylin/pull/289 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] shaofengshi closed pull request #288: KYLIN-3597 Improve code smell

shaofengshi closed pull request #288: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/288 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] codecov-io commented on issue #289: KYLIN-3630, remove unused fields in the implementations of MeasureType

codecov-io commented on issue #289: KYLIN-3630, remove unused fields in the implementations of MeasureType URL: https://github.com/apache/kylin/pull/289#issuecomment-429619135 # [Codecov](https://codecov.io/gh/apache/kylin/pull/289?src=pr=h1) Report > :exclamation: No coverage uploaded

[GitHub] asfgit commented on issue #289: KYLIN-3630, remove unused fields in the implementations of MeasureType

asfgit commented on issue #289: KYLIN-3630, remove unused fields in the implementations of MeasureType URL: https://github.com/apache/kylin/pull/289#issuecomment-429617849 Can one of the admins verify this patch? This is an

[GitHub] tttMelody opened a new pull request #289: KYLIN-3630, remove unused fields in the implementations of MeasureType

tttMelody opened a new pull request #289: KYLIN-3630, remove unused fields in the implementations of MeasureType URL: https://github.com/apache/kylin/pull/289 This is an automated message from the Apache Git Service. To

[GitHub] codecov-io commented on issue #288: KYLIN-3597 Improve code smell

codecov-io commented on issue #288: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/288#issuecomment-429588453 # [Codecov](https://codecov.io/gh/apache/kylin/pull/288?src=pr=h1) Report > Merging [#288](https://codecov.io/gh/apache/kylin/pull/288?src=pr=desc) into

[GitHub] coveralls edited a comment on issue #288: KYLIN-3597 Improve code smell

coveralls edited a comment on issue #288: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/288#issuecomment-429504585 ## Pull Request Test Coverage Report for [Build 3765](https://coveralls.io/builds/19508188) * **3** of **92** **(3.26%)** changed or added

[GitHub] shaofengshi commented on a change in pull request #286: KYLIN-3617 Use job's cache in job scheduler

shaofengshi commented on a change in pull request #286: KYLIN-3617 Use job's cache in job scheduler URL: https://github.com/apache/kylin/pull/286#discussion_r224976002 ## File path: core-job/src/main/java/org/apache/kylin/job/dao/ExecutableDao.java ## @@ -391,4 +405,13 @@

[GitHub] shaofengshi commented on a change in pull request #288: KYLIN-3597 Improve code smell

shaofengshi commented on a change in pull request #288: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/288#discussion_r224960913 ## File path: core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java ## @@ -100,6 +100,9 @@ public static

[GitHub] coveralls commented on issue #288: KYLIN-3597 Improve code smell

coveralls commented on issue #288: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/288#issuecomment-429504585 ## Pull Request Test Coverage Report for [Build 3763](https://coveralls.io/builds/19501016) * **3** of **78** **(3.85%)** changed or added

[GitHub] asfgit commented on issue #288: KYLIN-3597 Improve code smell

asfgit commented on issue #288: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/288#issuecomment-429493904 Can one of the admins verify this patch? This is an automated message from the Apache Git

[GitHub] caolijun1166 opened a new pull request #288: KYLIN-3597 Improve code smell

caolijun1166 opened a new pull request #288: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/288 KYLIN-3597 Improve code smell This is an automated message from the Apache Git Service. To respond to

[GitHub] caolijun1166 closed pull request #287: KYLIN-3597 Improve code smell

caolijun1166 closed pull request #287: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/287 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] coveralls commented on issue #287: KYLIN-3597 Improve code smell

coveralls commented on issue #287: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/287#issuecomment-429199241 ## Pull Request Test Coverage Report for [Build 3761](https://coveralls.io/builds/19483387) * **3** of **78** **(3.85%)** changed or added

[GitHub] coveralls commented on issue #286: KYLIN-3617 Use job's cache in job scheduler

coveralls commented on issue #286: KYLIN-3617 Use job's cache in job scheduler URL: https://github.com/apache/kylin/pull/286#issuecomment-429196953 ## Pull Request Test Coverage Report for [Build 3759](https://coveralls.io/builds/19483249) * **16** of **32** **(50.0%)** changed

[GitHub] caolijun1166 opened a new pull request #287: KYLIN-3597 Improve code smell

caolijun1166 opened a new pull request #287: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/287 KYLIN-3597 Improve code smell This is an automated message from the Apache Git Service. To respond to

[GitHub] asfgit commented on issue #287: KYLIN-3597 Improve code smell

asfgit commented on issue #287: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/287#issuecomment-429194473 Can one of the admins verify this patch? This is an automated message from the Apache Git

[GitHub] shaofengshi commented on issue #254: KYLIN-3562, refine the logic that updating the user when user logged in.

shaofengshi commented on issue #254: KYLIN-3562, refine the logic that updating the user when user logged in. URL: https://github.com/apache/kylin/pull/254#issuecomment-429194244 Hi jiatao, as the root cause is KYLIN-3499, this change is not needed I think. Let's see how it goes on. Thank

[GitHub] shaofengshi closed pull request #254: KYLIN-3562, refine the logic that updating the user when user logged in.

shaofengshi closed pull request #254: KYLIN-3562, refine the logic that updating the user when user logged in. URL: https://github.com/apache/kylin/pull/254 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] asfgit commented on issue #286: KYLIN-3617 Use job's cache in job scheduler

asfgit commented on issue #286: KYLIN-3617 Use job's cache in job scheduler URL: https://github.com/apache/kylin/pull/286#issuecomment-429193899 Can one of the admins verify this patch? This is an automated message from the

[GitHub] nichunen opened a new pull request #286: KYLIN-3617 Use job's cache in job scheduler

nichunen opened a new pull request #286: KYLIN-3617 Use job's cache in job scheduler URL: https://github.com/apache/kylin/pull/286 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] shaofengshi closed pull request #198: KYLIN-3499 Update HBaseResourceStore.java

shaofengshi closed pull request #198: KYLIN-3499 Update HBaseResourceStore.java URL: https://github.com/apache/kylin/pull/198 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] shaofengshi commented on issue #198: KYLIN-3499 Update HBaseResourceStore.java

shaofengshi commented on issue #198: KYLIN-3499 Update HBaseResourceStore.java URL: https://github.com/apache/kylin/pull/198#issuecomment-429178067 This change adds one more try when got "RetriesExhaustedException", it couldn't avoid such issue completely, but it can reduce the possibility

[GitHub] shaofengshi closed pull request #255: KYLIN-3496 Make calcite extras props available in JDBC Driver

shaofengshi closed pull request #255: KYLIN-3496 Make calcite extras props available in JDBC Driver URL: https://github.com/apache/kylin/pull/255 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] shaofengshi closed pull request #285: KYLIN-3597 fix sonar reported issues

shaofengshi closed pull request #285: KYLIN-3597 fix sonar reported issues URL: https://github.com/apache/kylin/pull/285 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] shaofengshi commented on issue #285: KYLIN-3597 fix sonar reported issues

shaofengshi commented on issue #285: KYLIN-3597 fix sonar reported issues URL: https://github.com/apache/kylin/pull/285#issuecomment-428812522 Integration test failed, I'm checking. This is an automated message from the

[GitHub] codecov-io commented on issue #285: KYLIN-3597 fix sonar reported issues

codecov-io commented on issue #285: KYLIN-3597 fix sonar reported issues URL: https://github.com/apache/kylin/pull/285#issuecomment-428542798 # [Codecov](https://codecov.io/gh/apache/kylin/pull/285?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] shaofengshi opened a new pull request #285: KYLIN-3597 fix sonar reported issues

shaofengshi opened a new pull request #285: KYLIN-3597 fix sonar reported issues URL: https://github.com/apache/kylin/pull/285 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] asfgit commented on issue #285: KYLIN-3597 fix sonar reported issues

asfgit commented on issue #285: KYLIN-3597 fix sonar reported issues URL: https://github.com/apache/kylin/pull/285#issuecomment-428535706 Can one of the admins verify this patch? This is an automated message from the Apache

[GitHub] shaofengshi closed pull request #284: KYLIN-3272 Upgrade Spark to 2.3.2

shaofengshi closed pull request #284: KYLIN-3272 Upgrade Spark to 2.3.2 URL: https://github.com/apache/kylin/pull/284 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] codecov-io commented on issue #284: KYLIN-3272 Upgrade Spark to 2.3.2

codecov-io commented on issue #284: KYLIN-3272 Upgrade Spark to 2.3.2 URL: https://github.com/apache/kylin/pull/284#issuecomment-428428327 # [Codecov](https://codecov.io/gh/apache/kylin/pull/284?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] coveralls commented on issue #284: KYLIN-3272 Upgrade Spark to 2.3.2

coveralls commented on issue #284: KYLIN-3272 Upgrade Spark to 2.3.2 URL: https://github.com/apache/kylin/pull/284#issuecomment-428427992 ## Pull Request Test Coverage Report for [Build 3750](https://coveralls.io/builds/19439178) * **0** of **2** **(0.0%)** changed or added

[GitHub] caolijun1166 opened a new pull request #284: KYLIN-3272 Upgrade Spark to 2.3.2

caolijun1166 opened a new pull request #284: KYLIN-3272 Upgrade Spark to 2.3.2 URL: https://github.com/apache/kylin/pull/284 KYLIN-3272 Upgrade Spark to 2.3.2 This is an automated message from the Apache Git Service. To

[GitHub] shaofengshi closed pull request #281: KYLIN-2924 disable google error-prone for travis-ci

shaofengshi closed pull request #281: KYLIN-2924 disable google error-prone for travis-ci URL: https://github.com/apache/kylin/pull/281 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] shaofengshi closed pull request #282: KYLIN-3597 Close resources after they are used.

shaofengshi closed pull request #282: KYLIN-3597 Close resources after they are used. URL: https://github.com/apache/kylin/pull/282 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] shaofengshi closed pull request #283: KYLIN-3597 fix code smells

shaofengshi closed pull request #283: KYLIN-3597 fix code smells URL: https://github.com/apache/kylin/pull/283 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] codecov-io edited a comment on issue #283: KYLIN-3597 fix code smells

codecov-io edited a comment on issue #283: KYLIN-3597 fix code smells URL: https://github.com/apache/kylin/pull/283#issuecomment-427846885 # [Codecov](https://codecov.io/gh/apache/kylin/pull/283?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] coveralls edited a comment on issue #283: KYLIN-3597 fix code smells

coveralls edited a comment on issue #283: KYLIN-3597 fix code smells URL: https://github.com/apache/kylin/pull/283#issuecomment-427846521 ## Pull Request Test Coverage Report for [Build 3746](https://coveralls.io/builds/19415872) * **6** of **31** **(19.35%)** changed or added

[GitHub] codecov-io commented on issue #283: KYLIN-3597 fix code smells

codecov-io commented on issue #283: KYLIN-3597 fix code smells URL: https://github.com/apache/kylin/pull/283#issuecomment-427846885 # [Codecov](https://codecov.io/gh/apache/kylin/pull/283?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base (`master@09c5668`).

[GitHub] coveralls commented on issue #283: KYLIN-3597 fix code smells

coveralls commented on issue #283: KYLIN-3597 fix code smells URL: https://github.com/apache/kylin/pull/283#issuecomment-427846521 ## Pull Request Test Coverage Report for [Build 3745](https://coveralls.io/builds/19403977) * **5** of **32** **(15.63%)** changed or added relevant

[GitHub] coveralls commented on issue #282: KYLIN-3597 Close resources after they are used.

coveralls commented on issue #282: KYLIN-3597 Close resources after they are used. URL: https://github.com/apache/kylin/pull/282#issuecomment-427801287 ## Pull Request Test Coverage Report for [Build 3743](https://coveralls.io/builds/19401685) * **0** of **28** **(0.0%)**

[GitHub] codecov-io commented on issue #282: KYLIN-3597 Close resources after they are used.

codecov-io commented on issue #282: KYLIN-3597 Close resources after they are used. URL: https://github.com/apache/kylin/pull/282#issuecomment-427800871 # [Codecov](https://codecov.io/gh/apache/kylin/pull/282?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] shaofengshi opened a new pull request #283: KYLIN-3597 fix code smells

shaofengshi opened a new pull request #283: KYLIN-3597 fix code smells URL: https://github.com/apache/kylin/pull/283 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] asfgit commented on issue #283: KYLIN-3597 fix code smells

asfgit commented on issue #283: KYLIN-3597 fix code smells URL: https://github.com/apache/kylin/pull/283#issuecomment-427800168 Can one of the admins verify this patch? This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #282: KYLIN-3597 Close resources after they are used.

asfgit commented on issue #282: KYLIN-3597 Close resources after they are used. URL: https://github.com/apache/kylin/pull/282#issuecomment-427795874 Can one of the admins verify this patch? This is an automated message from

[GitHub] caolijun1166 opened a new pull request #282: KYLIN-3597 Close resources after they are used.

caolijun1166 opened a new pull request #282: KYLIN-3597 Close resources after they are used. URL: https://github.com/apache/kylin/pull/282 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] codecov-io commented on issue #281: KYLIN-2924 disable google error-prone for travis-ci

codecov-io commented on issue #281: KYLIN-2924 disable google error-prone for travis-ci URL: https://github.com/apache/kylin/pull/281#issuecomment-427762253 # [Codecov](https://codecov.io/gh/apache/kylin/pull/281?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request

[GitHub] shaofengshi commented on issue #281: KYLIN-2924 disable google error-prone for travis-ci

shaofengshi commented on issue #281: KYLIN-2924 disable google error-prone for travis-ci URL: https://github.com/apache/kylin/pull/281#issuecomment-427756893 Just try if it can make travis-ci happy This is an automated

[GitHub] shaofengshi opened a new pull request #281: KYLIN-2924 disable google error-prone for travis-ci

shaofengshi opened a new pull request #281: KYLIN-2924 disable google error-prone for travis-ci URL: https://github.com/apache/kylin/pull/281 Travis-ci failed with message: " The job exceeded the maximum log length, and has been terminated." I try to disable errorprone check

[GitHub] asfgit commented on issue #281: KYLIN-2924 disable google error-prone for travis-ci

asfgit commented on issue #281: KYLIN-2924 disable google error-prone for travis-ci URL: https://github.com/apache/kylin/pull/281#issuecomment-427755080 Can one of the admins verify this patch? This is an automated message

[GitHub] shaofengshi closed pull request #279: KYLIN-3594 remove unnecessary sql files in integration test

shaofengshi closed pull request #279: KYLIN-3594 remove unnecessary sql files in integration test URL: https://github.com/apache/kylin/pull/279 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] coveralls edited a comment on issue #279: KYLIN-3594 remove unnecessary sql files in integration test

coveralls edited a comment on issue #279: KYLIN-3594 remove unnecessary sql files in integration test URL: https://github.com/apache/kylin/pull/279#issuecomment-427654298 ## Pull Request Test Coverage Report for [Build 3735](https://coveralls.io/builds/19392205) * **0** of **0**

[GitHub] coveralls commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem

coveralls commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem URL: https://github.com/apache/kylin/pull/280#issuecomment-427655794 ## Pull Request Test Coverage Report for [Build 3734](https://coveralls.io/builds/19392189) * **1** of **7** **(14.29%)**

[GitHub] codecov-io commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem

codecov-io commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem URL: https://github.com/apache/kylin/pull/280#issuecomment-427655739 # [Codecov](https://codecov.io/gh/apache/kylin/pull/280?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] asfgit commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem

asfgit commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem URL: https://github.com/apache/kylin/pull/280#issuecomment-427654284 Can one of the admins verify this patch? This is an automated message from

[GitHub] coveralls commented on issue #279: KYLIN-3594 remove unnecessary sql files in integration test

coveralls commented on issue #279: KYLIN-3594 remove unnecessary sql files in integration test URL: https://github.com/apache/kylin/pull/279#issuecomment-427654298 ## Pull Request Test Coverage Report for [Build 3733](https://coveralls.io/builds/19392081) * **0** of **0** changed

[GitHub] asfgit commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem

asfgit commented on issue #280: KYLIN-3586 Fix remaining boxing/unboxing problem URL: https://github.com/apache/kylin/pull/280#issuecomment-427654285 Can one of the admins verify this patch? This is an automated message from

[GitHub] codecov-io commented on issue #279: KYLIN-3594 remove unnecessary sql files in integration test

codecov-io commented on issue #279: KYLIN-3594 remove unnecessary sql files in integration test URL: https://github.com/apache/kylin/pull/279#issuecomment-427654295 # [Codecov](https://codecov.io/gh/apache/kylin/pull/279?src=pr=h1) Report > :exclamation: No coverage uploaded for pull

[GitHub] caolijun1166 opened a new pull request #280: KYLIN-3586 Fix remaining boxing/unboxing problem

caolijun1166 opened a new pull request #280: KYLIN-3586 Fix remaining boxing/unboxing problem URL: https://github.com/apache/kylin/pull/280 This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit commented on issue #279: remove unnecessary sql files in integration test

asfgit commented on issue #279: remove unnecessary sql files in integration test URL: https://github.com/apache/kylin/pull/279#issuecomment-427652789 Can one of the admins verify this patch? This is an automated message from

[GitHub] hit-lacus opened a new pull request #279: remove unnecessary sql files in integration test

hit-lacus opened a new pull request #279: remove unnecessary sql files in integration test URL: https://github.com/apache/kylin/pull/279 KYLIN-3594 This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit commented on issue #279: remove unnecessary sql files in integration test

asfgit commented on issue #279: remove unnecessary sql files in integration test URL: https://github.com/apache/kylin/pull/279#issuecomment-427652788 Can one of the admins verify this patch? This is an automated message from

[GitHub] shaofengshi closed pull request #268: KYLIN-3594 Obey JDBC standard

shaofengshi closed pull request #268: KYLIN-3594 Obey JDBC standard URL: https://github.com/apache/kylin/pull/268 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] codecov-io edited a comment on issue #268: KYLIN-3594 Obey JDBC standard

codecov-io edited a comment on issue #268: KYLIN-3594 Obey JDBC standard URL: https://github.com/apache/kylin/pull/268#issuecomment-425625134 # [Codecov](https://codecov.io/gh/apache/kylin/pull/268?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] hit-lacus opened a new pull request #268: KYLIN-3594 Obey JDBC standard

hit-lacus opened a new pull request #268: KYLIN-3594 Obey JDBC standard URL: https://github.com/apache/kylin/pull/268 https://issues.apache.org/jira/projects/KYLIN/issues/KYLIN-3594 This is an automated message from the

[GitHub] hit-lacus closed pull request #268: KYLIN-3594 Obey JDBC standard

hit-lacus closed pull request #268: KYLIN-3594 Obey JDBC standard URL: https://github.com/apache/kylin/pull/268 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] coveralls commented on issue #278: KYLIN-3597 Close JavaSparkContext after used.

coveralls commented on issue #278: KYLIN-3597 Close JavaSparkContext after used. URL: https://github.com/apache/kylin/pull/278#issuecomment-425706651 ## Pull Request Test Coverage Report for [Build 3728](https://coveralls.io/builds/19267812) * **0** of **184** **(0.0%)** changed

[GitHub] codecov-io commented on issue #278: KYLIN-3597 Close JavaSparkContext after used.

codecov-io commented on issue #278: KYLIN-3597 Close JavaSparkContext after used. URL: https://github.com/apache/kylin/pull/278#issuecomment-425706587 # [Codecov](https://codecov.io/gh/apache/kylin/pull/278?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] asfgit commented on issue #278: KYLIN-3597 Close JavaSparkContext after used.

asfgit commented on issue #278: KYLIN-3597 Close JavaSparkContext after used. URL: https://github.com/apache/kylin/pull/278#issuecomment-425705185 Can one of the admins verify this patch? This is an automated message from the

[GitHub] caolijun1166 opened a new pull request #278: KYLIN-3597 Close JavaSparkContext after used.

caolijun1166 opened a new pull request #278: KYLIN-3597 Close JavaSparkContext after used. URL: https://github.com/apache/kylin/pull/278 KYLIN-3597 Use try-with-resources to close JavaSparkContext after used. This is an

[GitHub] asfgit commented on issue #278: KYLIN-3597 Close JavaSparkContext after used.

asfgit commented on issue #278: KYLIN-3597 Close JavaSparkContext after used. URL: https://github.com/apache/kylin/pull/278#issuecomment-425705184 Can one of the admins verify this patch? This is an automated message from the

[GitHub] shaofengshi commented on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit…

shaofengshi commented on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit… URL: https://github.com/apache/kylin/pull/276#issuecomment-425696064 Merged. This is an automated message from the

[GitHub] shaofengshi closed pull request #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit…

shaofengshi closed pull request #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit… URL: https://github.com/apache/kylin/pull/276 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] codecov-io edited a comment on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit…

codecov-io edited a comment on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit… URL: https://github.com/apache/kylin/pull/276#issuecomment-425686030 # [Codecov](https://codecov.io/gh/apache/kylin/pull/276?src=pr=h1) Report > Merging

[GitHub] shaofengshi closed pull request #248: KYLIN-3588, fix potentially duplicate putting in RemoveBlackoutRealiz…

shaofengshi closed pull request #248: KYLIN-3588, fix potentially duplicate putting in RemoveBlackoutRealiz… URL: https://github.com/apache/kylin/pull/248 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] shaofengshi closed pull request #249: KYLIN-3581 Obey the contract of 'x.equals(y) should be consistent with x.compareTo(y)'

shaofengshi closed pull request #249: KYLIN-3581 Obey the contract of 'x.equals(y) should be consistent with x.compareTo(y)' URL: https://github.com/apache/kylin/pull/249 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] shaofengshi closed pull request #251: KYLIN-3583, KYLIN-3579

shaofengshi closed pull request #251: KYLIN-3583, KYLIN-3579 URL: https://github.com/apache/kylin/pull/251 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[GitHub] codecov-io commented on issue #251: KYLIN-3583, KYLIN-3579

codecov-io commented on issue #251: KYLIN-3583, KYLIN-3579 URL: https://github.com/apache/kylin/pull/251#issuecomment-425690030 # [Codecov](https://codecov.io/gh/apache/kylin/pull/251?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base (`master@f42e937`). [Click

[GitHub] codecov-io commented on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit…

codecov-io commented on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit… URL: https://github.com/apache/kylin/pull/276#issuecomment-425686030 # [Codecov](https://codecov.io/gh/apache/kylin/pull/276?src=pr=h1) Report > :exclamation: No coverage uploaded

[GitHub] shaofengshi closed pull request #277: KYLIN-3602 Enable more checkstyle rules

shaofengshi closed pull request #277: KYLIN-3602 Enable more checkstyle rules URL: https://github.com/apache/kylin/pull/277 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] codecov-io commented on issue #277: KYLIN-3602 Enable more checkstyle rules

codecov-io commented on issue #277: KYLIN-3602 Enable more checkstyle rules URL: https://github.com/apache/kylin/pull/277#issuecomment-425644108 # [Codecov](https://codecov.io/gh/apache/kylin/pull/277?src=pr=h1) Report > Merging

[GitHub] coveralls commented on issue #277: KYLIN-3602 Enable more checkstyle rules

coveralls commented on issue #277: KYLIN-3602 Enable more checkstyle rules URL: https://github.com/apache/kylin/pull/277#issuecomment-425644083 ## Pull Request Test Coverage Report for [Build 3719](https://coveralls.io/builds/19261940) * **0** of **0** changed or added relevant

[GitHub] asfgit commented on issue #277: KYLIN-3602 Enable more checkstyle rules

asfgit commented on issue #277: KYLIN-3602 Enable more checkstyle rules URL: https://github.com/apache/kylin/pull/277#issuecomment-425642269 Can one of the admins verify this patch? This is an automated message from the

[GitHub] ZhouYC627 opened a new pull request #277: KYLIN-3602 Enable more checkstyle rules

ZhouYC627 opened a new pull request #277: KYLIN-3602 Enable more checkstyle rules URL: https://github.com/apache/kylin/pull/277 Enable more checkstyle rules and suppress the checkstyle errors of old files in _dev-support/checkstyle-suppressions.xml_

[GitHub] asfgit commented on issue #277: KYLIN-3602 Enable more checkstyle rules

asfgit commented on issue #277: KYLIN-3602 Enable more checkstyle rules URL: https://github.com/apache/kylin/pull/277#issuecomment-425642270 Can one of the admins verify this patch? This is an automated message from the

[GitHub] shaofengshi closed pull request #270: KYLIN-2924 enable google error-prone in compile phase

shaofengshi closed pull request #270: KYLIN-2924 enable google error-prone in compile phase URL: https://github.com/apache/kylin/pull/270 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit…

asfgit commented on issue #276: KYLIN-3605 upgrade hadoop-common and zookeeper version to fix securit… URL: https://github.com/apache/kylin/pull/276#issuecomment-425635060 Can one of the admins verify this patch? This is an

[GitHub] shaofengshi closed pull request #272: KYLIN-3232 Add document for ops tools

shaofengshi closed pull request #272: KYLIN-3232 Add document for ops tools URL: https://github.com/apache/kylin/pull/272 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] asfgit commented on issue #275: Fix pictures' URL invalid

asfgit commented on issue #275: Fix pictures' URL invalid URL: https://github.com/apache/kylin/pull/275#issuecomment-425629403 Can one of the admins verify this patch? This is an automated message from the Apache Git Service.

[GitHub] asfgit commented on issue #275: Fix pictures' URL invalid

asfgit commented on issue #275: Fix pictures' URL invalid URL: https://github.com/apache/kylin/pull/275#issuecomment-425629402 Can one of the admins verify this patch? This is an automated message from the Apache Git Service.

[GitHub] coveralls commented on issue #268: KYLIN-3594 Obey JDBC standard

coveralls commented on issue #268: KYLIN-3594 Obey JDBC standard URL: https://github.com/apache/kylin/pull/268#issuecomment-425625274 ## Pull Request Test Coverage Report for [Build 3714](https://coveralls.io/builds/19260478) * **4** of **7** **(57.14%)** changed or added

[GitHub] codecov-io commented on issue #268: KYLIN-3594 Obey JDBC standard

codecov-io commented on issue #268: KYLIN-3594 Obey JDBC standard URL: https://github.com/apache/kylin/pull/268#issuecomment-425625134 # [Codecov](https://codecov.io/gh/apache/kylin/pull/268?src=pr=h1) Report > Merging [#268](https://codecov.io/gh/apache/kylin/pull/268?src=pr=desc) into

[GitHub] coveralls commented on issue #274: KYLIN-3597 Improve code smell

coveralls commented on issue #274: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/274#issuecomment-425620750 ## Pull Request Test Coverage Report for [Build 3713](https://coveralls.io/builds/19260169) * **21** of **38** **(55.26%)** changed or added

[GitHub] codecov-io commented on issue #274: KYLIN-3597 Improve code smell

codecov-io commented on issue #274: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/274#issuecomment-425620454 # [Codecov](https://codecov.io/gh/apache/kylin/pull/274?src=pr=h1) Report > Merging [#274](https://codecov.io/gh/apache/kylin/pull/274?src=pr=desc) into

[GitHub] hit-lacus opened a new pull request #274: KYLIN-3597 Improve code smell

hit-lacus opened a new pull request #274: KYLIN-3597 Improve code smell URL: https://github.com/apache/kylin/pull/274 https://issues.apache.org/jira/projects/KYLIN/issues/KYLIN-3597 This is an automated message from the

  1   2   3   4   5   6   7   8   9   10   >