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

2018-10-11 Thread GitBox
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

2018-10-11 Thread GitBox
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

2018-10-11 Thread GitBox
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

2018-10-11 Thread GitBox
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.

2018-10-11 Thread GitBox
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.

2018-10-11 Thread GitBox
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

2018-10-11 Thread GitBox
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

2018-10-11 Thread GitBox
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

[jira] [Created] (KYLIN-3627) calcite-core-1.13.0-kylin-r4

2018-10-11 Thread liuzhixin (JIRA)
liuzhixin created KYLIN-3627: Summary: calcite-core-1.13.0-kylin-r4 Key: KYLIN-3627 URL: https://issues.apache.org/jira/browse/KYLIN-3627 Project: Kylin Issue Type: Bug Reporter:

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

2018-10-11 Thread GitBox
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

2018-10-11 Thread GitBox
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

Re: [DISCUSS] Columnar storage engine for Apache Kylin

2018-10-11 Thread ShaoFeng Shi
JIRA and sub-tasks are created for this. Welcome to comment there: https://issues.apache.org/jira/browse/KYLIN-3621 ShaoFeng Shi 于2018年10月8日周一 下午2:45写道: > I agree; the new storage should be Hadoop/HDFS compliant, and also need be > cloud storage (like S3, blob storage) friendly, as more and

[jira] [Created] (KYLIN-3626) Allow customization for Parquet path

2018-10-11 Thread Shaofeng SHI (JIRA)
Shaofeng SHI created KYLIN-3626: --- Summary: Allow customization for Parquet path Key: KYLIN-3626 URL: https://issues.apache.org/jira/browse/KYLIN-3626 Project: Kylin Issue Type: Sub-task

[jira] [Created] (KYLIN-3624) Convert cuboid to Parquet in Spark

2018-10-11 Thread Shaofeng SHI (JIRA)
Shaofeng SHI created KYLIN-3624: --- Summary: Convert cuboid to Parquet in Spark Key: KYLIN-3624 URL: https://issues.apache.org/jira/browse/KYLIN-3624 Project: Kylin Issue Type: Sub-task

[jira] [Created] (KYLIN-3625) Query engine for Parquet

2018-10-11 Thread Shaofeng SHI (JIRA)
Shaofeng SHI created KYLIN-3625: --- Summary: Query engine for Parquet Key: KYLIN-3625 URL: https://issues.apache.org/jira/browse/KYLIN-3625 Project: Kylin Issue Type: Sub-task

[jira] [Created] (KYLIN-3623) Convert cuboid to Parquet in MR

2018-10-11 Thread Shaofeng SHI (JIRA)
Shaofeng SHI created KYLIN-3623: --- Summary: Convert cuboid to Parquet in MR Key: KYLIN-3623 URL: https://issues.apache.org/jira/browse/KYLIN-3623 Project: Kylin Issue Type: Sub-task

[jira] [Created] (KYLIN-3622) Cube layout in Parquet

2018-10-11 Thread Shaofeng SHI (JIRA)
Shaofeng SHI created KYLIN-3622: --- Summary: Cube layout in Parquet Key: KYLIN-3622 URL: https://issues.apache.org/jira/browse/KYLIN-3622 Project: Kylin Issue Type: Sub-task

[jira] [Created] (KYLIN-3621) Parquet as Cube storage

2018-10-11 Thread Shaofeng SHI (JIRA)
Shaofeng SHI created KYLIN-3621: --- Summary: Parquet as Cube storage Key: KYLIN-3621 URL: https://issues.apache.org/jira/browse/KYLIN-3621 Project: Kylin Issue Type: New Feature

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

2018-10-11 Thread GitBox
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:

[jira] [Created] (KYLIN-3620) "--" should not be a comment marker use between single quotes in SQL

2018-10-11 Thread Scott Fan (JIRA)
Scott Fan created KYLIN-3620: Summary: "--" should not be a comment marker use between single quotes in SQL Key: KYLIN-3620 URL: https://issues.apache.org/jira/browse/KYLIN-3620 Project: Kylin

Re: [DISCUSS] Not sending Github PR notifications to dev@kylin

2018-10-11 Thread Luke Han
Thanks Shaofeng Best Regards! - Luke Han On Tue, Oct 9, 2018 at 5:14 PM ShaoFeng Shi wrote: > This discussion has lasted for one week, there are 10 +1s and no -1, so I > think this change proposal has been accepted by the community, will tell > the Apache infra team to go

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

2018-10-11 Thread GitBox
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

Re: "--" should not be an annotation use between single quotes in SQL

2018-10-11 Thread ShaoFeng Shi
I remember there was a change, which is to make "--" as the comment marker; the words after "--" in a line will be skipped. Please report a JIRA. Scott Fan 于2018年10月11日周四 下午2:54写道: > Hi, > > When I exec a SQL below, KYLIN occur an error. > > SELECT * FROM TABLEA WHERE NAME='Jason--Arthur' > >

"--" should not be an annotation use between single quotes in SQL

2018-10-11 Thread Scott Fan
Hi, When I exec a SQL below, KYLIN occur an error. SELECT * FROM TABLEA WHERE NAME='Jason--Arthur' Seems like KYLIN use '--' as an annotation. Thanks