[jira] [Created] (KYLIN-3643) Derived column from windowSpec not working in where

2018-10-23 Thread May Zhou (JIRA)
May Zhou created KYLIN-3643:
---

 Summary: Derived column from windowSpec not working in where
 Key: KYLIN-3643
 URL: https://issues.apache.org/jira/browse/KYLIN-3643
 Project: Kylin
  Issue Type: Bug
 Environment: Kylin 2.5.0
Reporter: May Zhou


Derived column from windowSpec not working in where with error message of 

> 
-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( PARTITION 
BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID ,PRICE 
,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
taking kylin sample dataset as an example,

 

```

SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES

```

 works good.

 

But 

```

SELECT *
FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
WHERE T.ROW_NUM =1

```

 

throws error messages

 



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


[jira] [Commented] (KYLIN-3638) cube operator doc

2018-10-23 Thread shenqiong (JIRA)


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

shenqiong commented on KYLIN-3638:
--

[~Shaofengshi] I had test company email, 163 email , none is passed.  how could 
this happended!

> cube operator doc
> -
>
> Key: KYLIN-3638
> URL: https://issues.apache.org/jira/browse/KYLIN-3638
> Project: Kylin
>  Issue Type: Wish
>  Components: Documentation
>Affects Versions: v2.4.1
>Reporter: shenqiong
>Priority: Minor
> Attachments: image-2018-10-22-14-30-09-899.png, 
> image-2018-10-22-14-33-42-515.png, 微信截图_20181022143414.png, 
> 微信截图_20181022143516.png
>
>
> I can not find any instruction to delete a cube with status of "DESCBROKEN" 。 
> it can not be deleted by GUI, with error  of null. can you show it in 
> document ?



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


[jira] [Updated] (KYLIN-3643) Derived column from windowSpec not working in where

2018-10-23 Thread May Zhou (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3643?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

May Zhou updated KYLIN-3643:

Description: 
Derived column from windowSpec not working in where with error message of 
{quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID 
,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
 taking kylin sample dataset as an example,
{quote}
 

{{ }}
{{ {{SELECT ROW_NUMBER() OVER (}}
{{ PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE}}
{{ ) AS ROW_NUM}}
{{ ,TRANS_ID}}
{{ ,BUYER_ID}}
{{ ,PRICE}}
{{ ,SELLER_ID}}
{{ ,PART_DT}}
{{ FROM KYLIN_SALES
{{  }}

 works good.

 

But 

 
 {{SELECT *
 FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
 WHERE T.ROW_NUM =1}}
 throws error messages

 

  was:
Derived column from windowSpec not working in where with error message of 
{quote}
 -1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( PARTITION 
BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID ,PRICE 
,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
 taking kylin sample dataset as an example,
{quote}
 

 
{{SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES}}
 

 works good.

 

But 

 
{{SELECT *
 FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
 WHERE T.ROW_NUM =1}}
throws error messages

 


> Derived column from windowSpec not working in where
> ---
>
> Key: KYLIN-3643
> URL: https://issues.apache.org/jira/browse/KYLIN-3643
> Project: Kylin
>  Issue Type: Bug
> Environment: Kylin 2.5.0
>Reporter: May Zhou
>Priority: Critical
>
> Derived column from windowSpec not working in where with error message of 
> {quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
> PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID 
> ,BUYER_ID ,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 
> LIMIT 5"
>  taking kylin sample dataset as an example,
> {quote}
>  
> {{ }}
> {{ {{SELECT ROW_NUMBER() OVER (}}
> {{ PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE}}
> {{ ) AS ROW_NUM}}
> {{ ,TRANS_ID}}
> {{ ,BUYER_ID}}
> {{ ,PRICE}}
> {{ ,SELLER_ID}}
> {{ ,PART_DT}}
> {{ FROM KYLIN_SALES
> {{  }}
>  works good.
>  
> But 
>  
>  {{SELECT *
>  FROM (
>  SELECT ROW_NUMBER() OVER (
>  PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
>  ) AS ROW_NUM
>  ,TRANS_ID
>  ,BUYER_ID
>  ,PRICE
>  ,SELLER_ID
>  ,PART_DT
>  FROM KYLIN_SALES
>  ) T
>  WHERE T.ROW_NUM =1}}
>  throws error messages
>  



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


[jira] [Updated] (KYLIN-3643) Derived column from windowSpec not working in where

2018-10-23 Thread May Zhou (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3643?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

May Zhou updated KYLIN-3643:

Description: 
Derived column from windowSpec not working in where with error message of 
{quote}
 -1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( PARTITION 
BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID ,PRICE 
,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
 taking kylin sample dataset as an example,
{quote}
 

 
{{SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES}}
 

 works good.

 

But 

 
{{SELECT *
 FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
 WHERE T.ROW_NUM =1}}
throws error messages

 

  was:
Derived column from windowSpec not working in where with error message of 

> 
-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( PARTITION 
BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID ,PRICE 
,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
taking kylin sample dataset as an example,

 

```

SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES

```

 works good.

 

But 

```

SELECT *
FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
WHERE T.ROW_NUM =1

```

 

throws error messages

 


> Derived column from windowSpec not working in where
> ---
>
> Key: KYLIN-3643
> URL: https://issues.apache.org/jira/browse/KYLIN-3643
> Project: Kylin
>  Issue Type: Bug
> Environment: Kylin 2.5.0
>Reporter: May Zhou
>Priority: Critical
>
> Derived column from windowSpec not working in where with error message of 
> {quote}
>  -1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
> PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID 
> ,BUYER_ID ,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 
> LIMIT 5"
>  taking kylin sample dataset as an example,
> {quote}
>  
>  
> {{SELECT ROW_NUMBER() OVER (
>  PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
>  ) AS ROW_NUM
>  ,TRANS_ID
>  ,BUYER_ID
>  ,PRICE
>  ,SELLER_ID
>  ,PART_DT
>  FROM KYLIN_SALES}}
>  
>  works good.
>  
> But 
>  
> {{SELECT *
>  FROM (
>  SELECT ROW_NUMBER() OVER (
>  PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
>  ) AS ROW_NUM
>  ,TRANS_ID
>  ,BUYER_ID
>  ,PRICE
>  ,SELLER_ID
>  ,PART_DT
>  FROM KYLIN_SALES
>  ) T
>  WHERE T.ROW_NUM =1}}
> throws error messages
>  



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


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

2018-10-23 Thread GitBox
nichunen commented on issue #304: KYLIN-3617 Use job's cache in job scheduler
URL: https://github.com/apache/kylin/pull/304#issuecomment-432147039
 
 
   It test passed


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-3617) Reduce number of visiting metastore for job scheduler

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

nichunen commented on issue #304: KYLIN-3617 Use job's cache in job scheduler
URL: https://github.com/apache/kylin/pull/304#issuecomment-432147039
 
 
   It test passed


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:
us...@infra.apache.org


> Reduce number of visiting metastore for job scheduler
> -
>
> Key: KYLIN-3617
> URL: https://issues.apache.org/jira/browse/KYLIN-3617
> Project: Kylin
>  Issue Type: Improvement
>  Components: Job Engine
>Affects Versions: v2.4.1
>Reporter: nichunen
>Assignee: nichunen
>Priority: Major
> Fix For: v2.6.0
>
>
> For KYLIN-3470 introduced cache for jobs' metadata, it's also can be used in 
> job scheduler to reduce the pressure on metastore



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


[jira] [Updated] (KYLIN-3643) Derived column from windowSpec not working in where

2018-10-23 Thread May Zhou (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3643?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

May Zhou updated KYLIN-3643:

Description: 
Derived column from windowSpec not working in where with error message of 
{quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID 
,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
 taking kylin sample dataset as an example,
{quote}
 
{quote} 
SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
  
{quote}
 works good.

 

But 

 


{quote}SELECT *
 FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
 WHERE T.ROW_NUM =1
{quote}

 throws error messages

 

  was:
Derived column from windowSpec not working in where with error message of 
{quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID 
,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
 taking kylin sample dataset as an example,
{quote}
 

{{ }}
{{ {{SELECT ROW_NUMBER() OVER (}}
{{ PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE}}
{{ ) AS ROW_NUM}}
{{ ,TRANS_ID}}
{{ ,BUYER_ID}}
{{ ,PRICE}}
{{ ,SELLER_ID}}
{{ ,PART_DT}}
{{ FROM KYLIN_SALES
{{  }}

 works good.

 

But 

 
 {{SELECT *
 FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
 WHERE T.ROW_NUM =1}}
 throws error messages

 


> Derived column from windowSpec not working in where
> ---
>
> Key: KYLIN-3643
> URL: https://issues.apache.org/jira/browse/KYLIN-3643
> Project: Kylin
>  Issue Type: Bug
> Environment: Kylin 2.5.0
>Reporter: May Zhou
>Priority: Critical
>
> Derived column from windowSpec not working in where with error message of 
> {quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
> PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID 
> ,BUYER_ID ,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 
> LIMIT 5"
>  taking kylin sample dataset as an example,
> {quote}
>  
> {quote} 
> SELECT ROW_NUMBER() OVER (
>  PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
>  ) AS ROW_NUM
>  ,TRANS_ID
>  ,BUYER_ID
>  ,PRICE
>  ,SELLER_ID
>  ,PART_DT
>  FROM KYLIN_SALES
>   
> {quote}
>  works good.
>  
> But 
>  
> {quote}SELECT *
>  FROM (
>  SELECT ROW_NUMBER() OVER (
>  PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
>  ) AS ROW_NUM
>  ,TRANS_ID
>  ,BUYER_ID
>  ,PRICE
>  ,SELLER_ID
>  ,PART_DT
>  FROM KYLIN_SALES
>  ) T
>  WHERE T.ROW_NUM =1
> {quote}
>  throws error messages
>  



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


[jira] [Updated] (KYLIN-3643) Derived column from windowSpec not working in where

2018-10-23 Thread May Zhou (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3643?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

May Zhou updated KYLIN-3643:

Description: 
Derived column from windowSpec not working in where with error message of 
{quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID 
,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
 taking kylin sample dataset as an example,
{quote}
 Here I've taken kylin sample dataset as example.
{quote} 
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
  
{quote}
 works good.

 

But 

 
{quote}SELECT *
 FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
 WHERE T.ROW_NUM =1
{quote}
throws error messages

 

  was:
Derived column from windowSpec not working in where with error message of 
{quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID ,BUYER_ID 
,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 LIMIT 5"
 taking kylin sample dataset as an example,
{quote}
 
{quote} 
SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
  
{quote}
 works good.

 

But 

 


{quote}SELECT *
 FROM (
 SELECT ROW_NUMBER() OVER (
 PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
 ) AS ROW_NUM
 ,TRANS_ID
 ,BUYER_ID
 ,PRICE
 ,SELLER_ID
 ,PART_DT
 FROM KYLIN_SALES
 ) T
 WHERE T.ROW_NUM =1
{quote}

 throws error messages

 


> Derived column from windowSpec not working in where
> ---
>
> Key: KYLIN-3643
> URL: https://issues.apache.org/jira/browse/KYLIN-3643
> Project: Kylin
>  Issue Type: Bug
> Environment: Kylin 2.5.0
>Reporter: May Zhou
>Priority: Critical
>
> Derived column from windowSpec not working in where with error message of 
> {quote}-1 while executing SQL: "SELECT * FROM ( SELECT ROW_NUMBER() OVER ( 
> PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE ) AS ROW_NUM ,TRANS_ID 
> ,BUYER_ID ,PRICE ,SELLER_ID ,PART_DT FROM KYLIN_SALES ) T WHERE T.ROW_NUM =1 
> LIMIT 5"
>  taking kylin sample dataset as an example,
> {quote}
>  Here I've taken kylin sample dataset as example.
> {quote} 
>  SELECT ROW_NUMBER() OVER (
>  PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
>  ) AS ROW_NUM
>  ,TRANS_ID
>  ,BUYER_ID
>  ,PRICE
>  ,SELLER_ID
>  ,PART_DT
>  FROM KYLIN_SALES
>   
> {quote}
>  works good.
>  
> But 
>  
> {quote}SELECT *
>  FROM (
>  SELECT ROW_NUMBER() OVER (
>  PARTITION BY BUYER_ID,SELLER_ID ORDER BY PRICE
>  ) AS ROW_NUM
>  ,TRANS_ID
>  ,BUYER_ID
>  ,PRICE
>  ,SELLER_ID
>  ,PART_DT
>  FROM KYLIN_SALES
>  ) T
>  WHERE T.ROW_NUM =1
> {quote}
> throws error messages
>  



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


[jira] [Assigned] (KYLIN-3579) entrySet iterator should be used in BPUSCalculator

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3579?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI reassigned KYLIN-3579:
---

 Assignee: Chao Long
Fix Version/s: v2.6.0

> entrySet iterator should be used in BPUSCalculator 
> ---
>
> Key: KYLIN-3579
> URL: https://issues.apache.org/jira/browse/KYLIN-3579
> Project: Kylin
>  Issue Type: Bug
>Reporter: Ted Yu
>Assignee: Chao Long
>Priority: Minor
> Fix For: v2.6.0
>
>
> BPUSCalculator.calculateBenefitTotal and BPUSCalculator.initCuboidAggCostMap 
> use keySet iterator where entrySet iterator should be used.
> {code}
> for (Long cuboid : cuboidAggCostMapCopy.keySet()) {
> if (cuboidAggCostMapCopy.get(cuboid) < 
> processCuboidAggCostMap.get(cuboid)) {
> {code}
> CubeCodeSystem.newMetricsAggregators has similar issue.



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


[jira] [Resolved] (KYLIN-3579) entrySet iterator should be used in BPUSCalculator

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3579?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI resolved KYLIN-3579.
-
Resolution: Fixed

> entrySet iterator should be used in BPUSCalculator 
> ---
>
> Key: KYLIN-3579
> URL: https://issues.apache.org/jira/browse/KYLIN-3579
> Project: Kylin
>  Issue Type: Bug
>Reporter: Ted Yu
>Assignee: Chao Long
>Priority: Minor
> Fix For: v2.6.0
>
>
> BPUSCalculator.calculateBenefitTotal and BPUSCalculator.initCuboidAggCostMap 
> use keySet iterator where entrySet iterator should be used.
> {code}
> for (Long cuboid : cuboidAggCostMapCopy.keySet()) {
> if (cuboidAggCostMapCopy.get(cuboid) < 
> processCuboidAggCostMap.get(cuboid)) {
> {code}
> CubeCodeSystem.newMetricsAggregators has similar issue.



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


[jira] [Assigned] (KYLIN-3572) Upgrade commons-dbcp to latest commons-dbcp2

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3572?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI reassigned KYLIN-3572:
---

Assignee: zhoujie

> Upgrade commons-dbcp to latest commons-dbcp2
> 
>
> Key: KYLIN-3572
> URL: https://issues.apache.org/jira/browse/KYLIN-3572
> Project: Kylin
>  Issue Type: Task
>Reporter: Ted Yu
>Assignee: zhoujie
>Priority: Major
>
> There have been many improvements and a major release in commons-dbcp. The 
> latest version is 2.5.0.
> This Jira proposes to upgrade the version of commons-dbcp to commons-dbcp2 
> version 2.5.0 in order to benefit from the improvements/additions. This will 
> involve some code changes in addition to the updated dependencies, as the API 
> has changed slightly to use different terminology 



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


[jira] [Assigned] (KYLIN-3608) Move dependency versions to top level pom properties

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3608?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI reassigned KYLIN-3608:
---

Assignee: zhoujie

> Move dependency versions to top level pom properties
> 
>
> Key: KYLIN-3608
> URL: https://issues.apache.org/jira/browse/KYLIN-3608
> Project: Kylin
>  Issue Type: Task
>Reporter: Ted Yu
>Assignee: zhoujie
>Priority: Minor
>
> There are some non-top level pom.xml files where dependency version is 
> referenced directly.
> core-common/pom.xml is an example.
> We should move all dependency versions to top level pom properties



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


[jira] [Assigned] (KYLIN-3611) Upgrade Tomcat to 7.0.91, 8.5.34 or later

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI reassigned KYLIN-3611:
---

Assignee: zhoujie

> Upgrade Tomcat to 7.0.91, 8.5.34 or later
> -
>
> Key: KYLIN-3611
> URL: https://issues.apache.org/jira/browse/KYLIN-3611
> Project: Kylin
>  Issue Type: Improvement
>Reporter: Shaofeng SHI
>Assignee: zhoujie
>Priority: Major
> Fix For: v2.6.0, v2.5.1
>
>
> h2. [SECURITY] CVE-2018-11784 Apache Tomcat - Open Redirect
>  
>  
>  
> CVE-2018-11784 Apache Tomcat - Open Redirect
> Severity: Moderate
> Vendor: The Apache Software Foundation
> Versions Affected:
> Apache Tomcat 9.0.0.M1 to 9.0.11
> Apache Tomcat 8.5.0 to 8.5.33
> Apache Tomcat 7.0.23 to 7.0.90
> The unsupported 8.0.x release line has not been analysed but is likely
> to be affected.
> Description:
> When the default servlet returned a redirect to a directory (e.g.
> redirecting to '/foo/' when the user requested '/foo') a specially
> crafted URL could be used to cause the redirect to be generated to any
> URI of the attackers choice.
> Mitigation:
> Users of the affected versions should apply one of the following
> mitigations:
> - Upgrade to Apache Tomcat 9.0.12 or later.
> - Upgrade to Apache Tomcat 8.5.34 or later.
> - Upgrade to Apache Tomcat 7.0.91 or later.
> - Use mapperDirectoryRedirectEnabled="true" and
>   mapperContextRootRedirectEnabled="true" on the Context to ensure that
>   redirects are issued by the Mapper rather than the default Servlet.
>   See the Context configuration documentation for further important
>   details.
> Credit:
> This vulnerability was found by Sergey Bobrov and reported responsibly
> to the Apache Tomcat Security Team.
> History:
> 2018-10-03 Original advisory
> References:
> [1] [http://tomcat.apache.org/security-9.html]
> [2] [http://tomcat.apache.org/security-8.html]
> [3] [http://tomcat.apache.org/security-7.html]



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


[jira] [Resolved] (KYLIN-3585) Ineffective declaration of volatile field in SparkFactDistinct

2018-10-23 Thread Chao Long (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3585?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chao Long resolved KYLIN-3585.
--
Resolution: Resolved
  Assignee: Chao Long

> Ineffective declaration of volatile field in SparkFactDistinct
> --
>
> Key: KYLIN-3585
> URL: https://issues.apache.org/jira/browse/KYLIN-3585
> Project: Kylin
>  Issue Type: Bug
>Reporter: Ted Yu
>Assignee: Chao Long
>Priority: Minor
>
> In CuboidStatCalculator :
> {code}
> private volatile HLLCounter[] cuboidsHLL;
> {code}
> Though the array is declared volatile, the array elements are not.



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


[jira] [Resolved] (KYLIN-3583) Integer multiplication is performed where Long result is expected

2018-10-23 Thread Chao Long (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3583?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chao Long resolved KYLIN-3583.
--
Resolution: Resolved
  Assignee: Chao Long

> Integer multiplication is performed where Long result is expected
> -
>
> Key: KYLIN-3583
> URL: https://issues.apache.org/jira/browse/KYLIN-3583
> Project: Kylin
>  Issue Type: Bug
>Reporter: Ted Yu
>Assignee: Chao Long
>Priority: Minor
>
> In RecordEventTimeDetail :
> {code}
> long timeStampForWeekBegin = timeStamp;
> timeStampForWeekBegin -= 360 * 24 * 
> (calendar.get(Calendar.DAY_OF_WEEK) - 1);
> {code}
> Casting to long should be done first before multiplication.



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


[GitHub] GinaZhai opened a new pull request #306: Update FAQ and Kylin Configuration

2018-10-23 Thread GitBox
GinaZhai opened a new pull request #306: Update FAQ and Kylin Configuration
URL: https://github.com/apache/kylin/pull/306
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #306: Update FAQ and Kylin Configuration

2018-10-23 Thread GitBox
asfgit commented on issue #306: Update FAQ and Kylin Configuration
URL: https://github.com/apache/kylin/pull/306#issuecomment-432156720
 
 
   Can one of the admins verify this patch?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-3617) Reduce number of visiting metastore for job scheduler

2018-10-23 Thread Zhong Yanghong (JIRA)


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

Zhong Yanghong commented on KYLIN-3617:
---

The commit 52307bab46db6240169492f240a8b69bde5110d8 blocks IT 
*ITDistributedSchedulerTakeOverTest*. Could you help have a check?

> Reduce number of visiting metastore for job scheduler
> -
>
> Key: KYLIN-3617
> URL: https://issues.apache.org/jira/browse/KYLIN-3617
> Project: Kylin
>  Issue Type: Improvement
>  Components: Job Engine
>Affects Versions: v2.4.1
>Reporter: nichunen
>Assignee: nichunen
>Priority: Major
> Fix For: v2.6.0
>
>
> For KYLIN-3470 introduced cache for jobs' metadata, it's also can be used in 
> job scheduler to reduce the pressure on metastore



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


[jira] [Created] (KYLIN-3644) NumberFormatExcetion on null values when building cube with Spark

2018-10-23 Thread Hubert STEFANI (JIRA)
Hubert STEFANI created KYLIN-3644:
-

 Summary: NumberFormatExcetion on null values when building cube 
with Spark
 Key: KYLIN-3644
 URL: https://issues.apache.org/jira/browse/KYLIN-3644
 Project: Kylin
  Issue Type: Bug
  Components: Spark Engine
Affects Versions: v2.5.0
Reporter: Hubert STEFANI
 Attachments: 01_overview_table.jpg, 02_dimension_cube.jpg, 
03_measure_cube.jpg

We encounter an error any time we try to build a cube with the following steps :
 * upload a csv on AWS S3 with following characteristics : the column on which 
the measure will be defined has some null values (Cf. attachment)
 * create a hive table with spark
 * create a model on  top of this table,
 * create a cube with a SUM measure
 * chose Spark as Engine
 * Launch build

Result : The build process fails at '{color:#4383b4}#7 Step Name: {color}Build 
Cube with Spark' with the following error :

 

""

18/10/23 09:25:39 INFO scheduler.DAGScheduler: Job 0 failed: 
saveAsNewAPIHadoopDataset at SparkCubingByLayer.java:253, took 7,277136 s

Exception in thread "main" java.lang.RuntimeException: error execute 
org.apache.kylin.engine.spark.SparkCubingByLayer. Root cause: Job aborted due 
to stage failure: Task 0 in stage 0.0 failed 4 times, most recent failure: Lost 
task 0.3 in stage 0.0 (TID 4, ip-172-31-35-113.eu-west-1.compute.internal, 
executor 4): java.lang.NumberFormatException: For input string: "\N"

    at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:2043)

    at sun.misc.FloatingDecimal.parseDouble(FloatingDecimal.java:110)

    at java.lang.Double.parseDouble(Double.java:538)

    at 
org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:38)

    at 
org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:28)

    at 
org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueOf(BaseCuboidBuilder.java:162)

    at 
org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueObjects(BaseCuboidBuilder.java:127)

    at 
org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:297)

    at 
org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:257)

    at 
org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)

    at 
org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)

"

Note 1: the build  process is OK when run with Map/Reduce Engine.

Note 2: the error doesn't seem to be related to AWS environment.

 

Sample of csv :

ID;CATEGORIE;TEL;MONTANT;MAGASIN;MATRICULE;VILLE;

970;161;6-98-6-6-42;838.47034;Magasin_19;Client_Matricule_28;MARSEILLE;
971;89;62-15-2-64-86;;;Client_Matricule_1;LYON;
972;87;17-64-97-74-42;;;Client_Matricule_105;ORBEC;
973;174;79-33-90-0-55;;Magasin_7;Client_Matricule_55;AJACCIO;
974;172;89-95-71-6-49;141.64174;Magasin_9;Client_Matricule_105;BASTIA;
975;83;7-27-95-28-7;897.28204;;Client_Matricule_199;AJACCIO;
976;170;67-72-18-29-34;164.07967;Magasin_3;Client_Matricule_137;LILLE;
977;130;14-69-4-23-27;1928.9557;Magasin_1;Client_Matricule_17;NOMNOM;
978;43;55-91-84-98-49;891.2691;Magasin_0;Client_Matricule_22;NOMNOM;
979;117;98-96-0-54-39;1636.3994;Magasin_9;Client_Matricule_142;MARSEILLE;
980;163;37-55-76-53-38;;;Client_Matricule_64;NEWYORK;
981;106;32-40-6-46-15;;Magasin_2;Client_Matricule_158;NOMNOM;
982;56;95-60-83-89-90;;;Client_Matricule_102;NOMNOM;
983;168;21-56-62-0-58;;;Client_Matricule_160;NOMNOM;
984;154;92-67-37-94-60;;;Client_Matricule_137;PARIS;

 

 



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


[jira] [Updated] (KYLIN-3644) NumberFormatExcetion on null values when building cube with Spark

2018-10-23 Thread Hubert STEFANI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3644?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Hubert STEFANI updated KYLIN-3644:
--
Attachment: 01_overview_table.jpg
02_dimension_cube.jpg
03_measure_cube.jpg

> NumberFormatExcetion on null values when building cube with Spark
> -
>
> Key: KYLIN-3644
> URL: https://issues.apache.org/jira/browse/KYLIN-3644
> Project: Kylin
>  Issue Type: Bug
>  Components: Spark Engine
>Affects Versions: v2.5.0
>Reporter: Hubert STEFANI
>Priority: Major
> Attachments: 01_overview_table.jpg, 02_dimension_cube.jpg, 
> 03_measure_cube.jpg
>
>
> We encounter an error any time we try to build a cube with the following 
> steps :
>  * upload a csv on AWS S3 with following characteristics : the column on 
> which the measure will be defined has some null values (Cf. attachment)
>  * create a hive table with spark
>  * create a model on  top of this table,
>  * create a cube with a SUM measure
>  * chose Spark as Engine
>  * Launch build
> Result : The build process fails at '{color:#4383b4}#7 Step Name: 
> {color}Build Cube with Spark' with the following error :
>  
> ""
> 18/10/23 09:25:39 INFO scheduler.DAGScheduler: Job 0 failed: 
> saveAsNewAPIHadoopDataset at SparkCubingByLayer.java:253, took 7,277136 s
> Exception in thread "main" java.lang.RuntimeException: error execute 
> org.apache.kylin.engine.spark.SparkCubingByLayer. Root cause: Job aborted due 
> to stage failure: Task 0 in stage 0.0 failed 4 times, most recent failure: 
> Lost task 0.3 in stage 0.0 (TID 4, 
> ip-172-31-35-113.eu-west-1.compute.internal, executor 4): 
> java.lang.NumberFormatException: For input string: "\N"
>     at 
> sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:2043)
>     at sun.misc.FloatingDecimal.parseDouble(FloatingDecimal.java:110)
>     at java.lang.Double.parseDouble(Double.java:538)
>     at 
> org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:38)
>     at 
> org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:28)
>     at 
> org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueOf(BaseCuboidBuilder.java:162)
>     at 
> org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueObjects(BaseCuboidBuilder.java:127)
>     at 
> org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:297)
>     at 
> org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:257)
>     at 
> org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)
>     at 
> org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)
> "
> Note 1: the build  process is OK when run with Map/Reduce Engine.
> Note 2: the error doesn't seem to be related to AWS environment.
>  
> Sample of csv :
> ID;CATEGORIE;TEL;MONTANT;MAGASIN;MATRICULE;VILLE;
> 970;161;6-98-6-6-42;838.47034;Magasin_19;Client_Matricule_28;MARSEILLE;
> 971;89;62-15-2-64-86;;;Client_Matricule_1;LYON;
> 972;87;17-64-97-74-42;;;Client_Matricule_105;ORBEC;
> 973;174;79-33-90-0-55;;Magasin_7;Client_Matricule_55;AJACCIO;
> 974;172;89-95-71-6-49;141.64174;Magasin_9;Client_Matricule_105;BASTIA;
> 975;83;7-27-95-28-7;897.28204;;Client_Matricule_199;AJACCIO;
> 976;170;67-72-18-29-34;164.07967;Magasin_3;Client_Matricule_137;LILLE;
> 977;130;14-69-4-23-27;1928.9557;Magasin_1;Client_Matricule_17;NOMNOM;
> 978;43;55-91-84-98-49;891.2691;Magasin_0;Client_Matricule_22;NOMNOM;
> 979;117;98-96-0-54-39;1636.3994;Magasin_9;Client_Matricule_142;MARSEILLE;
> 980;163;37-55-76-53-38;;;Client_Matricule_64;NEWYORK;
> 981;106;32-40-6-46-15;;Magasin_2;Client_Matricule_158;NOMNOM;
> 982;56;95-60-83-89-90;;;Client_Matricule_102;NOMNOM;
> 983;168;21-56-62-0-58;;;Client_Matricule_160;NOMNOM;
> 984;154;92-67-37-94-60;;;Client_Matricule_137;PARIS;
>  
>  



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


[jira] [Updated] (KYLIN-3644) NumberFormatExcetion on null values when building cube with Spark

2018-10-23 Thread Hubert STEFANI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3644?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Hubert STEFANI updated KYLIN-3644:
--
Attachment: sortieData.csv

> NumberFormatExcetion on null values when building cube with Spark
> -
>
> Key: KYLIN-3644
> URL: https://issues.apache.org/jira/browse/KYLIN-3644
> Project: Kylin
>  Issue Type: Bug
>  Components: Spark Engine
>Affects Versions: v2.5.0
>Reporter: Hubert STEFANI
>Priority: Major
> Attachments: 01_overview_table.jpg, 02_dimension_cube.jpg, 
> 03_measure_cube.jpg, sortieData.csv
>
>
> We encounter an error any time we try to build a cube with the following 
> steps :
>  * upload a csv on AWS S3 with following characteristics : the column on 
> which the measure will be defined has some null values (Cf. attachment)
>  * create a hive table with spark
>  * create a model on  top of this table,
>  * create a cube with a SUM measure
>  * chose Spark as Engine
>  * Launch build
> Result : The build process fails at '{color:#4383b4}#7 Step Name: 
> {color}Build Cube with Spark' with the following error :
>  
> ""
> 18/10/23 09:25:39 INFO scheduler.DAGScheduler: Job 0 failed: 
> saveAsNewAPIHadoopDataset at SparkCubingByLayer.java:253, took 7,277136 s
> Exception in thread "main" java.lang.RuntimeException: error execute 
> org.apache.kylin.engine.spark.SparkCubingByLayer. Root cause: Job aborted due 
> to stage failure: Task 0 in stage 0.0 failed 4 times, most recent failure: 
> Lost task 0.3 in stage 0.0 (TID 4, 
> ip-172-31-35-113.eu-west-1.compute.internal, executor 4): 
> java.lang.NumberFormatException: For input string: "\N"
>     at 
> sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:2043)
>     at sun.misc.FloatingDecimal.parseDouble(FloatingDecimal.java:110)
>     at java.lang.Double.parseDouble(Double.java:538)
>     at 
> org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:38)
>     at 
> org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:28)
>     at 
> org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueOf(BaseCuboidBuilder.java:162)
>     at 
> org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueObjects(BaseCuboidBuilder.java:127)
>     at 
> org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:297)
>     at 
> org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:257)
>     at 
> org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)
>     at 
> org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)
> "
> Note 1: the build  process is OK when run with Map/Reduce Engine.
> Note 2: the error doesn't seem to be related to AWS environment.
>  
> Sample of csv :
> ID;CATEGORIE;TEL;MONTANT;MAGASIN;MATRICULE;VILLE;
> 970;161;6-98-6-6-42;838.47034;Magasin_19;Client_Matricule_28;MARSEILLE;
> 971;89;62-15-2-64-86;;;Client_Matricule_1;LYON;
> 972;87;17-64-97-74-42;;;Client_Matricule_105;ORBEC;
> 973;174;79-33-90-0-55;;Magasin_7;Client_Matricule_55;AJACCIO;
> 974;172;89-95-71-6-49;141.64174;Magasin_9;Client_Matricule_105;BASTIA;
> 975;83;7-27-95-28-7;897.28204;;Client_Matricule_199;AJACCIO;
> 976;170;67-72-18-29-34;164.07967;Magasin_3;Client_Matricule_137;LILLE;
> 977;130;14-69-4-23-27;1928.9557;Magasin_1;Client_Matricule_17;NOMNOM;
> 978;43;55-91-84-98-49;891.2691;Magasin_0;Client_Matricule_22;NOMNOM;
> 979;117;98-96-0-54-39;1636.3994;Magasin_9;Client_Matricule_142;MARSEILLE;
> 980;163;37-55-76-53-38;;;Client_Matricule_64;NEWYORK;
> 981;106;32-40-6-46-15;;Magasin_2;Client_Matricule_158;NOMNOM;
> 982;56;95-60-83-89-90;;;Client_Matricule_102;NOMNOM;
> 983;168;21-56-62-0-58;;;Client_Matricule_160;NOMNOM;
> 984;154;92-67-37-94-60;;;Client_Matricule_137;PARIS;
>  
>  



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


[jira] [Updated] (KYLIN-3644) NumberFormatExcetion on null values when building cube with Spark

2018-10-23 Thread Hubert STEFANI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3644?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Hubert STEFANI updated KYLIN-3644:
--
Attachment: 00_zeppelin_notebook.jpg

> NumberFormatExcetion on null values when building cube with Spark
> -
>
> Key: KYLIN-3644
> URL: https://issues.apache.org/jira/browse/KYLIN-3644
> Project: Kylin
>  Issue Type: Bug
>  Components: Spark Engine
>Affects Versions: v2.5.0
>Reporter: Hubert STEFANI
>Priority: Major
> Attachments: 00_zeppelin_notebook.jpg, 01_overview_table.jpg, 
> 02_dimension_cube.jpg, 03_measure_cube.jpg, sortieData.csv
>
>
> We encounter an error any time we try to build a cube with the following 
> steps :
>  * upload a csv on AWS S3 with following characteristics : the column on 
> which the measure will be defined has some null values (Cf. attachment)
>  * create a hive table with spark
>  * create a model on  top of this table,
>  * create a cube with a SUM measure
>  * chose Spark as Engine
>  * Launch build
> Result : The build process fails at '{color:#4383b4}#7 Step Name: 
> {color}Build Cube with Spark' with the following error :
>  
> ""
> 18/10/23 09:25:39 INFO scheduler.DAGScheduler: Job 0 failed: 
> saveAsNewAPIHadoopDataset at SparkCubingByLayer.java:253, took 7,277136 s
> Exception in thread "main" java.lang.RuntimeException: error execute 
> org.apache.kylin.engine.spark.SparkCubingByLayer. Root cause: Job aborted due 
> to stage failure: Task 0 in stage 0.0 failed 4 times, most recent failure: 
> Lost task 0.3 in stage 0.0 (TID 4, 
> ip-172-31-35-113.eu-west-1.compute.internal, executor 4): 
> java.lang.NumberFormatException: For input string: "\N"
>     at 
> sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:2043)
>     at sun.misc.FloatingDecimal.parseDouble(FloatingDecimal.java:110)
>     at java.lang.Double.parseDouble(Double.java:538)
>     at 
> org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:38)
>     at 
> org.apache.kylin.measure.basic.DoubleIngester.valueOf(DoubleIngester.java:28)
>     at 
> org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueOf(BaseCuboidBuilder.java:162)
>     at 
> org.apache.kylin.engine.mr.common.BaseCuboidBuilder.buildValueObjects(BaseCuboidBuilder.java:127)
>     at 
> org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:297)
>     at 
> org.apache.kylin.engine.spark.SparkCubingByLayer$EncodeBaseCuboid.call(SparkCubingByLayer.java:257)
>     at 
> org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)
>     at 
> org.apache.spark.api.java.JavaPairRDD$$anonfun$pairFunToScalaFun$1.apply(JavaPairRDD.scala:1043)
> "
> Note 1: the build  process is OK when run with Map/Reduce Engine.
> Note 2: the error doesn't seem to be related to AWS environment.
>  
> Sample of csv :
> ID;CATEGORIE;TEL;MONTANT;MAGASIN;MATRICULE;VILLE;
> 970;161;6-98-6-6-42;838.47034;Magasin_19;Client_Matricule_28;MARSEILLE;
> 971;89;62-15-2-64-86;;;Client_Matricule_1;LYON;
> 972;87;17-64-97-74-42;;;Client_Matricule_105;ORBEC;
> 973;174;79-33-90-0-55;;Magasin_7;Client_Matricule_55;AJACCIO;
> 974;172;89-95-71-6-49;141.64174;Magasin_9;Client_Matricule_105;BASTIA;
> 975;83;7-27-95-28-7;897.28204;;Client_Matricule_199;AJACCIO;
> 976;170;67-72-18-29-34;164.07967;Magasin_3;Client_Matricule_137;LILLE;
> 977;130;14-69-4-23-27;1928.9557;Magasin_1;Client_Matricule_17;NOMNOM;
> 978;43;55-91-84-98-49;891.2691;Magasin_0;Client_Matricule_22;NOMNOM;
> 979;117;98-96-0-54-39;1636.3994;Magasin_9;Client_Matricule_142;MARSEILLE;
> 980;163;37-55-76-53-38;;;Client_Matricule_64;NEWYORK;
> 981;106;32-40-6-46-15;;Magasin_2;Client_Matricule_158;NOMNOM;
> 982;56;95-60-83-89-90;;;Client_Matricule_102;NOMNOM;
> 983;168;21-56-62-0-58;;;Client_Matricule_160;NOMNOM;
> 984;154;92-67-37-94-60;;;Client_Matricule_137;PARIS;
>  
>  



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


[jira] [Commented] (KYLIN-2898) Introduce memcached as a distributed cache for queries

2018-10-23 Thread Zhong Yanghong (JIRA)


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

Zhong Yanghong commented on KYLIN-2898:
---

If a distributed cache is adopted, it's better to set the bar 
*kylin.query.cache-threshold-duration* for query cache to 0, which means to put 
query result to cache as much as possible.

> Introduce memcached as a distributed cache for queries
> --
>
> Key: KYLIN-2898
> URL: https://issues.apache.org/jira/browse/KYLIN-2898
> Project: Kylin
>  Issue Type: Sub-task
>  Components: Query Engine
>Affects Versions: v2.1.0
>Reporter: Zhong Yanghong
>Assignee: Wang Ken
>Priority: Major
>




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


[jira] [Issue Comment Deleted] (KYLIN-2898) Introduce memcached as a distributed cache for queries

2018-10-23 Thread Zhong Yanghong (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-2898?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zhong Yanghong updated KYLIN-2898:
--
Comment: was deleted

(was: If a distributed cache is adopted, it's better to set the bar 
*kylin.query.cache-threshold-duration* for query cache to 0, which means to put 
query result to cache as much as possible.)

> Introduce memcached as a distributed cache for queries
> --
>
> Key: KYLIN-2898
> URL: https://issues.apache.org/jira/browse/KYLIN-2898
> Project: Kylin
>  Issue Type: Sub-task
>  Components: Query Engine
>Affects Versions: v2.1.0
>Reporter: Zhong Yanghong
>Assignee: Wang Ken
>Priority: Major
>




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


[jira] [Commented] (KYLIN-2898) Introduce memcached as a distributed cache for queries

2018-10-23 Thread Zhong Yanghong (JIRA)


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

Zhong Yanghong commented on KYLIN-2898:
---

If a distributed cache is adopted, small query results are also better to be 
put into the cache.

> Introduce memcached as a distributed cache for queries
> --
>
> Key: KYLIN-2898
> URL: https://issues.apache.org/jira/browse/KYLIN-2898
> Project: Kylin
>  Issue Type: Sub-task
>  Components: Query Engine
>Affects Versions: v2.1.0
>Reporter: Zhong Yanghong
>Assignee: Wang Ken
>Priority: Major
>




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


[GitHub] coveralls edited a comment on issue #301: KYLIN-3633 Avoid potential dead lock when building global dictionary

2018-10-23 Thread GitBox
coveralls edited a comment on issue #301: KYLIN-3633 Avoid potential dead lock 
when building global dictionary
URL: https://github.com/apache/kylin/pull/301#issuecomment-431662392
 
 
   ## Pull Request Test Coverage Report for [Build 
3797](https://coveralls.io/builds/19669769)
   
   * **6** of **20**   **(30.0%)**  changed or added relevant lines in **3** 
files are covered.
   * **3** unchanged lines in **1** file lost coverage.
   * Overall coverage increased (+**0.004%**) to **23.333%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/global/SegmentAppendTrieDictBuilder.java](https://coveralls.io/builds/19669769/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2Fglobal%2FSegmentAppendTrieDictBuilder.java#L85)
 | 0 | 1 | 0.0%
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java](https://coveralls.io/builds/19669769/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2FGlobalDictionaryBuilder.java#L109)
 | 0 | 3 | 0.0%
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java](https://coveralls.io/builds/19669769/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2FDictionaryGenerator.java#L87)
 | 6 | 16 | 37.5%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/19669769/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L449)
 | 3 | 78.42% |
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/19669769/badge)](https://coveralls.io/builds/19669769)
 |
   | :-- | --: |
   | Change from base [Build 3795](https://coveralls.io/builds/19664534): |  
0.004% |
   | Covered Lines: | 16332 |
   | Relevant Lines: | 69996 |
   
   ---
   # 💛  - [Coveralls](https://coveralls.io)
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-3633) Dead lock may happen in building global dictionary

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

coveralls edited a comment on issue #301: KYLIN-3633 Avoid potential dead lock 
when building global dictionary
URL: https://github.com/apache/kylin/pull/301#issuecomment-431662392
 
 
   ## Pull Request Test Coverage Report for [Build 
3797](https://coveralls.io/builds/19669769)
   
   * **6** of **20**   **(30.0%)**  changed or added relevant lines in **3** 
files are covered.
   * **3** unchanged lines in **1** file lost coverage.
   * Overall coverage increased (+**0.004%**) to **23.333%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/global/SegmentAppendTrieDictBuilder.java](https://coveralls.io/builds/19669769/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2Fglobal%2FSegmentAppendTrieDictBuilder.java#L85)
 | 0 | 1 | 0.0%
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java](https://coveralls.io/builds/19669769/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2FGlobalDictionaryBuilder.java#L109)
 | 0 | 3 | 0.0%
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java](https://coveralls.io/builds/19669769/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2FDictionaryGenerator.java#L87)
 | 6 | 16 | 37.5%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/19669769/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L449)
 | 3 | 78.42% |
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/19669769/badge)](https://coveralls.io/builds/19669769)
 |
   | :-- | --: |
   | Change from base [Build 3795](https://coveralls.io/builds/19664534): |  
0.004% |
   | Covered Lines: | 16332 |
   | Relevant Lines: | 69996 |
   
   ---
   # 💛  - [Coveralls](https://coveralls.io)
   


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:
us...@infra.apache.org


> Dead lock may happen in building global dictionary
> --
>
> Key: KYLIN-3633
> URL: https://issues.apache.org/jira/browse/KYLIN-3633
> Project: Kylin
>  Issue Type: Bug
>  Components: Measure - Count Distinct
>Affects Versions: v2.4.1
>Reporter: nichunen
>Assignee: nichunen
>Priority: Major
> Fix For: v2.5.1
>
>
> During the building of global dict, if an error happens, the builder's lock 
> maybe un-released. This will block all other building jobs with global dict.



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


[GitHub] shaofengshi closed pull request #301: KYLIN-3633 Avoid potential dead lock when building global dictionary

2018-10-23 Thread GitBox
shaofengshi closed pull request #301: KYLIN-3633 Avoid potential dead lock when 
building global dictionary
URL: https://github.com/apache/kylin/pull/301
 
 
   

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 fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git 
a/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java 
b/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java
index db0c302970..7c33b4a1e4 100644
--- 
a/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java
+++ 
b/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java
@@ -75,13 +75,19 @@ public static IDictionaryBuilder 
newDictionaryBuilder(DataType dataType) {
 builder.init(dictInfo, baseId, null);
 
 // add values
-while (valueEnumerator.moveNext()) {
-String value = valueEnumerator.current();
+try {
+while (valueEnumerator.moveNext()) {
+String value = valueEnumerator.current();
 
-boolean accept = builder.addValue(value);
+boolean accept = builder.addValue(value);
 
-if (accept && samples.size() < nSamples && samples.contains(value) 
== false)
-samples.add(value);
+if (accept && samples.size() < nSamples && 
samples.contains(value) == false)
+samples.add(value);
+}
+} catch (IOException e) {
+logger.error("Error during adding dict value.", e);
+builder.clear();
+throw e;
 }
 
 // build
@@ -149,6 +155,12 @@ public boolean addValue(String value) {
 
 return new DateStrDictionary(datePattern, baseId);
 }
+
+
+@Override
+public void clear() {
+// do nothing
+}
 }
 
 private static class TimeDictBuilder implements IDictionaryBuilder {
@@ -171,6 +183,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return new TimeStrDictionary(); // base ID is always 0
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 private static class StringTrieDictBuilder implements IDictionaryBuilder {
@@ -196,6 +213,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build(baseId);
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 private static class StringTrieDictForestBuilder implements 
IDictionaryBuilder {
@@ -219,6 +241,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build();
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 @SuppressWarnings("deprecation")
@@ -245,6 +272,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build(baseId);
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 private static class NumberTrieDictForestBuilder implements 
IDictionaryBuilder {
@@ -268,6 +300,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build();
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 }
diff --git 
a/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
 
b/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
index 9168ca4a1a..d813793ea1 100644
--- 
a/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
+++ 
b/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
@@ -19,8 +19,8 @@
 package org.apache.kylin.dict;
 
 import java.io.IOException;
-
 import java.util.Locale;
+
 import org.apache.kylin.common.KylinConfig;
 import org.apache.kylin.common.lock.DistributedLock;
 import org.apache.kylin.common.util.Dictionary;
@@ -104,6 +104,13 @@ public boolean addValue(String value) {
 return new AppendTrieDictionary<>();
 }
 
+@Override
+public void clear() {
+if (lock.isLocked(getLockPath(sourceColumn))) {
+lock.unlock(getLockPath(sourceColumn));
+}
+}
+
 private String getLockPath(String pathName) {
 return "/dict/" + pathName + "/lock";
 }
diff --git 
a/core-dictionary/src/main/java/org/apache/kylin/dict/IDictionaryBuilder.java 
b/core-dictionary/src/main/java/org/apache/kylin/dict/IDictionaryBuilder.java
index e2a643dbd0..771bfb42c6 100644
--- 
a/core-dictionary/src/main/java/org/apache/kylin/dict/IDictionaryBuilder.java
+++ 
b/core-dictionary/src/main/java/org/a

[jira] [Commented] (KYLIN-3633) Dead lock may happen in building global dictionary

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

shaofengshi closed pull request #301: KYLIN-3633 Avoid potential dead lock when 
building global dictionary
URL: https://github.com/apache/kylin/pull/301
 
 
   

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 fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git 
a/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java 
b/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java
index db0c302970..7c33b4a1e4 100644
--- 
a/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java
+++ 
b/core-dictionary/src/main/java/org/apache/kylin/dict/DictionaryGenerator.java
@@ -75,13 +75,19 @@ public static IDictionaryBuilder 
newDictionaryBuilder(DataType dataType) {
 builder.init(dictInfo, baseId, null);
 
 // add values
-while (valueEnumerator.moveNext()) {
-String value = valueEnumerator.current();
+try {
+while (valueEnumerator.moveNext()) {
+String value = valueEnumerator.current();
 
-boolean accept = builder.addValue(value);
+boolean accept = builder.addValue(value);
 
-if (accept && samples.size() < nSamples && samples.contains(value) 
== false)
-samples.add(value);
+if (accept && samples.size() < nSamples && 
samples.contains(value) == false)
+samples.add(value);
+}
+} catch (IOException e) {
+logger.error("Error during adding dict value.", e);
+builder.clear();
+throw e;
 }
 
 // build
@@ -149,6 +155,12 @@ public boolean addValue(String value) {
 
 return new DateStrDictionary(datePattern, baseId);
 }
+
+
+@Override
+public void clear() {
+// do nothing
+}
 }
 
 private static class TimeDictBuilder implements IDictionaryBuilder {
@@ -171,6 +183,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return new TimeStrDictionary(); // base ID is always 0
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 private static class StringTrieDictBuilder implements IDictionaryBuilder {
@@ -196,6 +213,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build(baseId);
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 private static class StringTrieDictForestBuilder implements 
IDictionaryBuilder {
@@ -219,6 +241,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build();
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 @SuppressWarnings("deprecation")
@@ -245,6 +272,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build(baseId);
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 private static class NumberTrieDictForestBuilder implements 
IDictionaryBuilder {
@@ -268,6 +300,11 @@ public boolean addValue(String value) {
 public Dictionary build() throws IOException {
 return builder.build();
 }
+
+@Override
+public void clear() {
+
+}
 }
 
 }
diff --git 
a/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
 
b/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
index 9168ca4a1a..d813793ea1 100644
--- 
a/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
+++ 
b/core-dictionary/src/main/java/org/apache/kylin/dict/GlobalDictionaryBuilder.java
@@ -19,8 +19,8 @@
 package org.apache.kylin.dict;
 
 import java.io.IOException;
-
 import java.util.Locale;
+
 import org.apache.kylin.common.KylinConfig;
 import org.apache.kylin.common.lock.DistributedLock;
 import org.apache.kylin.common.util.Dictionary;
@@ -104,6 +104,13 @@ public boolean addValue(String value) {
 return new AppendTrieDictionary<>();
 }
 
+@Override
+public void clear() {
+if (lock.isLocked(getLockPath(sourceColumn))) {
+lock.unlock(getLockPath(sourceColumn));
+}
+}
+
 private String getLockPath(String pathName) {
 return "/dict/" + pathName + "/lock";
 }
diff --git 
a/core-dictionary/src/main/java/org/apache/kylin/dict/IDictio

[jira] [Commented] (KYLIN-3633) Dead lock may happen in building global dictionary

2018-10-23 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on KYLIN-3633:


Commit 8b51d4a0cd8c33ded5bb380885a731c381da710d in kylin's branch 
refs/heads/master from [~nichunen]
[ https://gitbox.apache.org/repos/asf?p=kylin.git;h=8b51d4a ]

KYLIN-3633 Avoid potential dead lock when building global dictionary


> Dead lock may happen in building global dictionary
> --
>
> Key: KYLIN-3633
> URL: https://issues.apache.org/jira/browse/KYLIN-3633
> Project: Kylin
>  Issue Type: Bug
>  Components: Measure - Count Distinct
>Affects Versions: v2.4.1
>Reporter: nichunen
>Assignee: nichunen
>Priority: Major
> Fix For: v2.5.1
>
>
> During the building of global dict, if an error happens, the builder's lock 
> maybe un-released. This will block all other building jobs with global dict.



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


[jira] [Assigned] (KYLIN-2894) Change the query cache expiration strategy by signature checking

2018-10-23 Thread Zhong Yanghong (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-2894?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Zhong Yanghong reassigned KYLIN-2894:
-

Assignee: Wang Ken  (was: Zhong Yanghong)

> Change the query cache expiration strategy by signature checking
> 
>
> Key: KYLIN-2894
> URL: https://issues.apache.org/jira/browse/KYLIN-2894
> Project: Kylin
>  Issue Type: Sub-task
>  Components: Query Engine
>Reporter: Zhong Yanghong
>Assignee: Wang Ken
>Priority: Major
>
> Currently to invalid query cache, {{CacheService}} will either invoke 
> {{cleanDataCache}} or {{cleanAllDataCache}}. Both methods will clear all of 
> the query cache, which is very inefficient. In eBay PROD environment, there's 
> around 400 cubing jobs per day, which means the query cache will be cleared 
> very 4 minutes. Then we introduced a signature based cache invalidation 
> strategy. The basic idea is as follows:
> * Add a signature for {{SQLResponse}}, here we choose the cube last build time
> * When fetch {{SQLResponse}} for cache, first check whether the signature is 
> consistent. If not, this cached value is overdue and will be invalidate.



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


[GitHub] codecov-io commented on issue #302: KYLIN-3620 resolve "--" is treated as comment marker between singe qu…

2018-10-23 Thread GitBox
codecov-io commented on issue #302: KYLIN-3620 resolve "--" is treated as 
comment marker between singe qu…
URL: https://github.com/apache/kylin/pull/302#issuecomment-432266197
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=h1) Report
   > :exclamation: No coverage uploaded for pull request base 
(`master@8b51d4a`). [Click here to learn what that 
means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).
   > The diff coverage is `100%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/302/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=tree)
   
   ```diff
   @@Coverage Diff@@
   ## master #302   +/-   ##
   =
 Coverage  ?   21.32%   
 Complexity? 4446   
   =
 Files ? 1089   
 Lines ?69996   
 Branches  ?10098   
   =
 Hits  ?14925   
 Misses?53668   
 Partials  ? 1403
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=tree) | Coverage Δ 
| Complexity Δ | |
   |---|---|---|---|
   | 
[...in/java/org/apache/kylin/query/util/QueryUtil.java](https://codecov.io/gh/apache/kylin/pull/302/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3V0aWwvUXVlcnlVdGlsLmphdmE=)
 | `58.44% <100%> (ø)` | `19 <2> (?)` | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=footer). Last 
update 
[8b51d4a...f5e89c2](https://codecov.io/gh/apache/kylin/pull/302?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:
us...@infra.apache.org


With regards,
Apache Git Services


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

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

codecov-io commented on issue #302: KYLIN-3620 resolve "--" is treated as 
comment marker between singe qu…
URL: https://github.com/apache/kylin/pull/302#issuecomment-432266197
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=h1) Report
   > :exclamation: No coverage uploaded for pull request base 
(`master@8b51d4a`). [Click here to learn what that 
means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).
   > The diff coverage is `100%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/302/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=tree)
   
   ```diff
   @@Coverage Diff@@
   ## master #302   +/-   ##
   =
 Coverage  ?   21.32%   
 Complexity? 4446   
   =
 Files ? 1089   
 Lines ?69996   
 Branches  ?10098   
   =
 Hits  ?14925   
 Misses?53668   
 Partials  ? 1403
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=tree) | Coverage Δ 
| Complexity Δ | |
   |---|---|---|---|
   | 
[...in/java/org/apache/kylin/query/util/QueryUtil.java](https://codecov.io/gh/apache/kylin/pull/302/diff?src=pr&el=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3V0aWwvUXVlcnlVdGlsLmphdmE=)
 | `58.44% <100%> (ø)` | `19 <2> (?)` | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/kylin/pull/302?src=pr&el=footer). Last 
update 
[8b51d4a...f5e89c2](https://codecov.io/gh/apache/kylin/pull/302?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:
us...@infra.apache.org


> "--" 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
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v2.4.1
>Reporter: Scott Fan
>Assignee: Lijun Cao
>Priority: Major
> Fix For: v2.6.0
>
>
> Hi,
> When I exec a SQL like below, there was an error.
> SELECT * FROM TABLEA WHERE NAME='Jason--Arthur'
> Seems like KYLIN use  the – as a comment marker.
> It should not be that between single quotes, I think.
>   
>  Even I send a REST POST contains SQL like that, or do an ODBC query, there 
> was an error too.
> You may tell me that I can use a \ as escape, but I can't change the SQL 
> generated by BI tools like tableau.  
> Thanks
>   



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


[GitHub] GinaZhai opened a new pull request #307: KYLIN-2200 Verify CompileException on UNION ALL query when result onl…

2018-10-23 Thread GitBox
GinaZhai opened a new pull request #307: KYLIN-2200 Verify CompileException on 
UNION ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307
 
 
   …y contains one column and add IT.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #307: KYLIN-2200 Verify CompileException on UNION ALL query when result onl…

2018-10-23 Thread GitBox
asfgit commented on issue #307: KYLIN-2200 Verify CompileException on UNION ALL 
query when result onl…
URL: https://github.com/apache/kylin/pull/307#issuecomment-432473261
 
 
   Can one of the admins verify this patch?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

asfgit commented on issue #307: KYLIN-2200 Verify CompileException on UNION ALL 
query when result onl…
URL: https://github.com/apache/kylin/pull/307#issuecomment-432473261
 
 
   Can one of the admins verify this patch?


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:
us...@infra.apache.org


> CompileException on UNION ALL query when result only contains one column
> 
>
> Key: KYLIN-2200
> URL: https://issues.apache.org/jira/browse/KYLIN-2200
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.5.4.1
>Reporter: Dayue Gao
>Assignee: Dayue Gao
>Priority: Major
> Fix For: v2.5.1
>
> Attachments: KYLIN-2200.patch
>
>
> {code:sql}
> select count(*) from kylin_sales
> union all
> select count(*) from kylin_sales
> {code}
> got following exception
> {noformat}
> Caused by: org.codehaus.commons.compiler.CompileException: Line 82, Column 
> 32: Cannot determine simple type name "Record11_1"
> at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:10092)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5375)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5184)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5165)
> at 
> org.codehaus.janino.UnitCompiler.access$12600(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitReferenceType(UnitCompiler.java:5096)
> at org.codehaus.janino.Java$ReferenceType.accept(Java.java:2880)
> at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5136)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5598)
> at 
> org.codehaus.janino.UnitCompiler.access$13300(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitCast(UnitCompiler.java:5104)
> {noformat}



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


[jira] [Commented] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

GinaZhai opened a new pull request #307: KYLIN-2200 Verify CompileException on 
UNION ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307
 
 
   …y contains one column and add IT.


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:
us...@infra.apache.org


> CompileException on UNION ALL query when result only contains one column
> 
>
> Key: KYLIN-2200
> URL: https://issues.apache.org/jira/browse/KYLIN-2200
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.5.4.1
>Reporter: Dayue Gao
>Assignee: Dayue Gao
>Priority: Major
> Fix For: v2.5.1
>
> Attachments: KYLIN-2200.patch
>
>
> {code:sql}
> select count(*) from kylin_sales
> union all
> select count(*) from kylin_sales
> {code}
> got following exception
> {noformat}
> Caused by: org.codehaus.commons.compiler.CompileException: Line 82, Column 
> 32: Cannot determine simple type name "Record11_1"
> at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:10092)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5375)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5184)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5165)
> at 
> org.codehaus.janino.UnitCompiler.access$12600(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitReferenceType(UnitCompiler.java:5096)
> at org.codehaus.janino.Java$ReferenceType.accept(Java.java:2880)
> at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5136)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5598)
> at 
> org.codehaus.janino.UnitCompiler.access$13300(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitCast(UnitCompiler.java:5104)
> {noformat}



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


[jira] [Created] (KYLIN-3645) Kylin does not clean table metadata when drop project(Kafka Source)

2018-10-23 Thread rongchuan.jin (JIRA)
rongchuan.jin created KYLIN-3645:


 Summary: Kylin does not clean table metadata when drop 
project(Kafka Source)
 Key: KYLIN-3645
 URL: https://issues.apache.org/jira/browse/KYLIN-3645
 Project: Kylin
  Issue Type: Bug
  Components: Metadata, Streaming
Affects Versions: v2.4.1
 Environment: jdk1.8
MacOS
Reporter: rongchuan.jin
 Fix For: v2.5.1


When I use Kafka streaming,there is a bug. My steps:

1.Create a project named p1

2.Import a kafka table named DEFAULT.T1

3.drop project p1

4.create project p2

5.Import a kafka table named DEFAULT.T1

It comes out an error:

The Streaming Config DEFAULT.T1 already exists.



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


[jira] [Assigned] (KYLIN-3645) Kylin does not clean table metadata when drop project(Kafka Source)

2018-10-23 Thread rongchuan.jin (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3645?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

rongchuan.jin reassigned KYLIN-3645:


Assignee: rongchuan.jin

> Kylin does not clean table metadata when drop project(Kafka Source)
> ---
>
> Key: KYLIN-3645
> URL: https://issues.apache.org/jira/browse/KYLIN-3645
> Project: Kylin
>  Issue Type: Bug
>  Components: Metadata, Streaming
>Affects Versions: v2.4.1
> Environment: jdk1.8
> MacOS
>Reporter: rongchuan.jin
>Assignee: rongchuan.jin
>Priority: Minor
> Fix For: v2.5.1
>
>
> When I use Kafka streaming,there is a bug. My steps:
> 1.Create a project named p1
> 2.Import a kafka table named DEFAULT.T1
> 3.drop project p1
> 4.create project p2
> 5.Import a kafka table named DEFAULT.T1
> It comes out an error:
> The Streaming Config DEFAULT.T1 already exists.



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


[GitHub] codecov-io commented on issue #299: Apache kylin 2895 updated

2018-10-23 Thread GitBox
codecov-io commented on issue #299: Apache kylin 2895 updated
URL: https://github.com/apache/kylin/pull/299#issuecomment-432494678
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/299?src=pr&el=h1) Report
   > Merging [#299](https://codecov.io/gh/apache/kylin/pull/299?src=pr&el=desc) 
into 
[master](https://codecov.io/gh/apache/kylin/commit/8b51d4a0cd8c33ded5bb380885a731c381da710d?src=pr&el=desc)
 will **increase** coverage by `0.25%`.
   > The diff coverage is `34.38%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/299/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/299?src=pr&el=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #299  +/-   ##
   
   + Coverage 21.31%   21.57%   +0.25% 
   - Complexity 4448 4580 +132 
   
 Files  1089   +22 
 Lines 6999671151+1155 
 Branches  1009810281 +183 
   
   + Hits  1492215353 +431 
   - Misses5367154326 +655 
   - Partials   1403 1472  +69
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/299?src=pr&el=tree) | Coverage Δ 
| Complexity Δ | |
   |---|---|---|---|
   | 
[...pache/kylin/cache/cachemanager/CacheConstants.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-Y2FjaGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NhY2hlL2NhY2hlbWFuYWdlci9DYWNoZUNvbnN0YW50cy5qYXZh)
 | `0% <0%> (ø)` | `0 <0> (?)` | |
   | 
[.../java/org/apache/kylin/common/KylinConfigBase.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZ0Jhc2UuamF2YQ==)
 | `14.39% <0%> (-0.25%)` | `36 <0> (ø)` | |
   | 
[.../kylin/cache/ehcache/InstrumentedEhCacheCache.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-Y2FjaGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NhY2hlL2VoY2FjaGUvSW5zdHJ1bWVudGVkRWhDYWNoZUNhY2hlLmphdmE=)
 | `0% <0%> (ø)` | `0 <0> (?)` | |
   | 
[...org/apache/kylin/common/debug/BackdoorToggles.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9kZWJ1Zy9CYWNrZG9vclRvZ2dsZXMuamF2YQ==)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...va/org/apache/kylin/rest/service/QueryService.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3Qvc2VydmljZS9RdWVyeVNlcnZpY2UuamF2YQ==)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/kylin/cache/memcached/CacheStats.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-Y2FjaGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NhY2hlL21lbWNhY2hlZC9DYWNoZVN0YXRzLmphdmE=)
 | `0% <0%> (ø)` | `0 <0> (?)` | |
   | 
[...in/storage/hbase/cube/v2/CubeHBaseEndpointRPC.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-c3RvcmFnZS1oYmFzZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vc3RvcmFnZS9oYmFzZS9jdWJlL3YyL0N1YmVIQmFzZUVuZHBvaW50UlBDLmphdmE=)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...apache/kylin/cache/memcached/MemcachedMetrics.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-Y2FjaGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NhY2hlL21lbWNhY2hlZC9NZW1jYWNoZWRNZXRyaWNzLmphdmE=)
 | `0% <0%> (ø)` | `0 <0> (?)` | |
   | 
[...kylin/storage/hbase/cube/v2/SegmentQueryCache.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-c3RvcmFnZS1oYmFzZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vc3RvcmFnZS9oYmFzZS9jdWJlL3YyL1NlZ21lbnRRdWVyeUNhY2hlLmphdmE=)
 | `0% <0%> (ø)` | `0 <0> (?)` | |
   | 
[...ain/java/org/apache/kylin/common/QueryContext.java](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9RdWVyeUNvbnRleHQuamF2YQ==)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | ... and [48 
more](https://codecov.io/gh/apache/kylin/pull/299/diff?src=pr&el=tree-more) | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/299?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/kylin/pull/299?src=pr&el=footer). Last 
update 
[8b51d4a...b2e0f0e](https://codecov.io/gh/apache/kylin/pull/299?src=pr&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


This is an autom

[GitHub] coveralls commented on issue #299: Apache kylin 2895 updated

2018-10-23 Thread GitBox
coveralls commented on issue #299: Apache kylin 2895 updated
URL: https://github.com/apache/kylin/pull/299#issuecomment-432494780
 
 
   ## Pull Request Test Coverage Report for [Build 
3802](https://coveralls.io/builds/19685691)
   
   * **484** of **1201**   **(40.3%)**  changed or added relevant lines in 
**29** files are covered.
   * **9** unchanged lines in **5** files lost coverage.
   * Overall coverage increased (+**0.3%**) to **23.65%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[cache/src/main/java/org/apache/kylin/cache/cachemanager/CacheConstants.java](https://coveralls.io/builds/19685691/source?filename=cache%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcache%2Fcachemanager%2FCacheConstants.java#L21)
 | 0 | 1 | 0.0%
   | 
[cache/src/main/java/org/apache/kylin/cache/cachemanager/RemoteLocalFailOverCacheManager.java](https://coveralls.io/builds/19685691/source?filename=cache%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcache%2Fcachemanager%2FRemoteLocalFailOverCacheManager.java#L49)
 | 12 | 13 | 92.31%
   | 
[core-common/src/main/java/org/apache/kylin/common/debug/BackdoorToggles.java](https://coveralls.io/builds/19685691/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2Fdebug%2FBackdoorToggles.java#L90)
 | 0 | 1 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/response/SQLResponse.java](https://coveralls.io/builds/19685691/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fresponse%2FSQLResponse.java#L227)
 | 7 | 8 | 87.5%
   | 
[server-base/src/main/java/org/apache/kylin/rest/signature/SegmentSignature.java](https://coveralls.io/builds/19685691/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fsignature%2FSegmentSignature.java#L31)
 | 17 | 18 | 94.44%
   | 
[storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/SegmentQueryResult.java](https://coveralls.io/builds/19685691/source?filename=storage-hbase%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fstorage%2Fhbase%2Fcube%2Fv2%2FSegmentQueryResult.java#L48)
 | 28 | 29 | 96.55%
   | 
[cache/src/main/java/org/apache/kylin/cache/memcached/MemcachedConnectionFactoryBuilder.java](https://coveralls.io/builds/19685691/source?filename=cache%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcache%2Fmemcached%2FMemcachedConnectionFactoryBuilder.java#L69)
 | 25 | 28 | 89.29%
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/CacheService.java](https://coveralls.io/builds/19685691/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FCacheService.java#L120)
 | 0 | 4 | 0.0%
   | 
[server-base/src/main/java/org/apache/kylin/rest/signature/RealizationSetCalculator.java](https://coveralls.io/builds/19685691/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fsignature%2FRealizationSetCalculator.java#L46)
 | 28 | 34 | 82.35%
   | 
[cache/src/main/java/org/apache/kylin/cache/memcached/MemcachedCacheConfig.java](https://coveralls.io/builds/19685691/source?filename=cache%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcache%2Fmemcached%2FMemcachedCacheConfig.java#L63)
 | 20 | 28 | 71.43%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[core-common/src/main/java/org/apache/kylin/common/QueryContext.java](https://coveralls.io/builds/19685691/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2FQueryContext.java#L273)
 | 1 | 0.0% |
   | 
[storage-hbase/src/main/java/org/apache/kylin/storage/hbase/cube/v2/CubeHBaseEndpointRPC.java](https://coveralls.io/builds/19685691/source?filename=storage-hbase%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fstorage%2Fhbase%2Fcube%2Fv2%2FCubeHBaseEndpointRPC.java#L247)
 | 1 | 0.0% |
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://coveralls.io/builds/19685691/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2Flookup%2Fcache%2FRocksDBLookupTable.java#L62)
 | 1 | 81.08% |
   | 
[core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/19685691/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L449)
 | 3 | 78.42% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/QueryService.java](https://coveralls.io/builds/19685691/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FQueryService.java#L448)
 | 3 | 0.0% |
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/19685691/badge)](https://coveralls.io/builds/19685691)
 |
   | :-- | --: |
   | Change from base [Build 3801](https://coveralls.io/builds/19685049): |  
0.3% |
   | Covered Lines: | 16827 |
   | Relevant Lines: | 71151 |
   
   ---
   # 💛  - [Coveralls](ht

[GitHub] woyumen4597 opened a new pull request #308: #KYLIN-3645 clean table metadata when drop project

2018-10-23 Thread GitBox
woyumen4597 opened a new pull request #308: #KYLIN-3645 clean table metadata 
when drop project
URL: https://github.com/apache/kylin/pull/308
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] asfgit commented on issue #308: #KYLIN-3645 clean table metadata when drop project

2018-10-23 Thread GitBox
asfgit commented on issue #308: #KYLIN-3645 clean table metadata when drop 
project
URL: https://github.com/apache/kylin/pull/308#issuecomment-432495070
 
 
   Can one of the admins verify this patch?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-3645) Kylin does not clean table metadata when drop project(Kafka Source)

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

woyumen4597 opened a new pull request #308: #KYLIN-3645 clean table metadata 
when drop project
URL: https://github.com/apache/kylin/pull/308
 
 
   


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:
us...@infra.apache.org


> Kylin does not clean table metadata when drop project(Kafka Source)
> ---
>
> Key: KYLIN-3645
> URL: https://issues.apache.org/jira/browse/KYLIN-3645
> Project: Kylin
>  Issue Type: Bug
>  Components: Metadata, Streaming
>Affects Versions: v2.4.1
> Environment: jdk1.8
> MacOS
>Reporter: rongchuan.jin
>Assignee: rongchuan.jin
>Priority: Minor
> Fix For: v2.5.1
>
>
> When I use Kafka streaming,there is a bug. My steps:
> 1.Create a project named p1
> 2.Import a kafka table named DEFAULT.T1
> 3.drop project p1
> 4.create project p2
> 5.Import a kafka table named DEFAULT.T1
> It comes out an error:
> The Streaming Config DEFAULT.T1 already exists.



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


[jira] [Commented] (KYLIN-3645) Kylin does not clean table metadata when drop project(Kafka Source)

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

asfgit commented on issue #308: #KYLIN-3645 clean table metadata when drop 
project
URL: https://github.com/apache/kylin/pull/308#issuecomment-432495070
 
 
   Can one of the admins verify this patch?


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:
us...@infra.apache.org


> Kylin does not clean table metadata when drop project(Kafka Source)
> ---
>
> Key: KYLIN-3645
> URL: https://issues.apache.org/jira/browse/KYLIN-3645
> Project: Kylin
>  Issue Type: Bug
>  Components: Metadata, Streaming
>Affects Versions: v2.4.1
> Environment: jdk1.8
> MacOS
>Reporter: rongchuan.jin
>Assignee: rongchuan.jin
>Priority: Minor
> Fix For: v2.5.1
>
>
> When I use Kafka streaming,there is a bug. My steps:
> 1.Create a project named p1
> 2.Import a kafka table named DEFAULT.T1
> 3.drop project p1
> 4.create project p2
> 5.Import a kafka table named DEFAULT.T1
> It comes out an error:
> The Streaming Config DEFAULT.T1 already exists.



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


[GitHub] codecov-io commented on issue #308: #KYLIN-3645 clean table metadata when drop project

2018-10-23 Thread GitBox
codecov-io commented on issue #308: #KYLIN-3645 clean table metadata when drop 
project
URL: https://github.com/apache/kylin/pull/308#issuecomment-432498572
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=h1) Report
   > Merging [#308](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=desc) 
into 
[master](https://codecov.io/gh/apache/kylin/commit/8b51d4a0cd8c33ded5bb380885a731c381da710d?src=pr&el=desc)
 will **increase** coverage by `<.01%`.
   > The diff coverage is `0%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/308/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #308  +/-   ##
   
   + Coverage 21.31%   21.32%   +<.01% 
   + Complexity 4448 4447   -1 
   
 Files  1089 1089  
 Lines 699967   +4 
 Branches  1009810099   +1 
   
   + Hits  1492214926   +4 
   - Misses5367153672   +1 
   + Partials   1403 1402   -1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=tree) | Coverage Δ 
| Complexity Δ | |
   |---|---|---|---|
   | 
[.../org/apache/kylin/rest/service/ProjectService.java](https://codecov.io/gh/apache/kylin/pull/308/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3Qvc2VydmljZS9Qcm9qZWN0U2VydmljZS5qYXZh)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...he/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://codecov.io/gh/apache/kylin/pull/308/diff?src=pr&el=tree#diff-Y29yZS1kaWN0aW9uYXJ5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9kaWN0L2xvb2t1cC9jYWNoZS9Sb2Nrc0RCTG9va3VwVGFibGUuamF2YQ==)
 | `72.97% <0%> (-5.41%)` | `6% <0%> (-1%)` | |
   | 
[...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/308/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh)
 | `69.6% <0%> (+0.3%)` | `7% <0%> (ø)` | :arrow_down: |
   | 
[...g/apache/kylin/source/datagen/ColumnGenerator.java](https://codecov.io/gh/apache/kylin/pull/308/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/308/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2N1Ym9pZC9UcmVlQ3Vib2lkU2NoZWR1bGVyLmphdmE=)
 | `66.15% <0%> (+2.3%)` | `0% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=footer). Last 
update 
[8b51d4a...a4f0744](https://codecov.io/gh/apache/kylin/pull/308?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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-3645) Kylin does not clean table metadata when drop project(Kafka Source)

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

codecov-io commented on issue #308: #KYLIN-3645 clean table metadata when drop 
project
URL: https://github.com/apache/kylin/pull/308#issuecomment-432498572
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=h1) Report
   > Merging [#308](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=desc) 
into 
[master](https://codecov.io/gh/apache/kylin/commit/8b51d4a0cd8c33ded5bb380885a731c381da710d?src=pr&el=desc)
 will **increase** coverage by `<.01%`.
   > The diff coverage is `0%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/308/graphs/tree.svg?width=650&token=JawVgbgsVo&height=150&src=pr)](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #308  +/-   ##
   
   + Coverage 21.31%   21.32%   +<.01% 
   + Complexity 4448 4447   -1 
   
 Files  1089 1089  
 Lines 699967   +4 
 Branches  1009810099   +1 
   
   + Hits  1492214926   +4 
   - Misses5367153672   +1 
   + Partials   1403 1402   -1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=tree) | Coverage Δ 
| Complexity Δ | |
   |---|---|---|---|
   | 
[.../org/apache/kylin/rest/service/ProjectService.java](https://codecov.io/gh/apache/kylin/pull/308/diff?src=pr&el=tree#diff-c2VydmVyLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3Jlc3Qvc2VydmljZS9Qcm9qZWN0U2VydmljZS5qYXZh)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...he/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://codecov.io/gh/apache/kylin/pull/308/diff?src=pr&el=tree#diff-Y29yZS1kaWN0aW9uYXJ5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9kaWN0L2xvb2t1cC9jYWNoZS9Sb2Nrc0RCTG9va3VwVGFibGUuamF2YQ==)
 | `72.97% <0%> (-5.41%)` | `6% <0%> (-1%)` | |
   | 
[...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/308/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh)
 | `69.6% <0%> (+0.3%)` | `7% <0%> (ø)` | :arrow_down: |
   | 
[...g/apache/kylin/source/datagen/ColumnGenerator.java](https://codecov.io/gh/apache/kylin/pull/308/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/308/diff?src=pr&el=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2N1Ym9pZC9UcmVlQ3Vib2lkU2NoZWR1bGVyLmphdmE=)
 | `66.15% <0%> (+2.3%)` | `0% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/kylin/pull/308?src=pr&el=footer). Last 
update 
[8b51d4a...a4f0744](https://codecov.io/gh/apache/kylin/pull/308?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:
us...@infra.apache.org


> Kylin does not clean table metadata when drop project(Kafka Source)
> ---
>
> Key: KYLIN-3645
> URL: https://issues.apache.org/jira/browse/KYLIN-3645
> Project: Kylin
>  Issue Type: Bug
>  Components: Metadata, Streaming
>Affects Versions: v2.4.1
> Environment: jdk1.8
> MacOS
>Reporter: rongchuan.jin
>Assignee: rongchuan.jin
>Priority: Minor
> Fix For: v2.5.1
>
>
> When I use Kafka streaming,there is a bug. My steps:
> 1.Create a project named p1
> 2.Import a kafka table named DEFAULT.T1
> 3.drop project p1
> 4.create project p2
> 5.Import a kafka table named DEFAULT.T1
> It comes out an error:
> The Streaming Config DEFAULT.T1 already exists.



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

[jira] [Commented] (KYLIN-3645) Kylin does not clean table metadata when drop project(Kafka Source)

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

coveralls commented on issue #308: #KYLIN-3645 clean table metadata when drop 
project
URL: https://github.com/apache/kylin/pull/308#issuecomment-432498959
 
 
   ## Pull Request Test Coverage Report for [Build 
3803](https://coveralls.io/builds/19685945)
   
   * **0** of **4**   **(0.0%)**  changed or added relevant lines in **1** file 
are covered.
   * **7** unchanged lines in **3** files lost coverage.
   * Overall coverage increased (+**0.003%**) to **23.329%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java](https://coveralls.io/builds/19685945/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FProjectService.java#L139)
 | 0 | 4 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://coveralls.io/builds/19685945/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2Flookup%2Fcache%2FRocksDBLookupTable.java#L62)
 | 1 | 81.08% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java](https://coveralls.io/builds/19685945/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FProjectService.java#L143)
 | 2 | 0.0% |
   | 
[core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/19685945/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L449)
 | 4 | 78.12% |
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/19685945/badge)](https://coveralls.io/builds/19685945)
 |
   | :-- | --: |
   | Change from base [Build 3801](https://coveralls.io/builds/19685049): |  
0.003% |
   | Covered Lines: | 16330 |
   | Relevant Lines: | 7 |
   
   ---
   # 💛  - [Coveralls](https://coveralls.io)
   


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:
us...@infra.apache.org


> Kylin does not clean table metadata when drop project(Kafka Source)
> ---
>
> Key: KYLIN-3645
> URL: https://issues.apache.org/jira/browse/KYLIN-3645
> Project: Kylin
>  Issue Type: Bug
>  Components: Metadata, Streaming
>Affects Versions: v2.4.1
> Environment: jdk1.8
> MacOS
>Reporter: rongchuan.jin
>Assignee: rongchuan.jin
>Priority: Minor
> Fix For: v2.5.1
>
>
> When I use Kafka streaming,there is a bug. My steps:
> 1.Create a project named p1
> 2.Import a kafka table named DEFAULT.T1
> 3.drop project p1
> 4.create project p2
> 5.Import a kafka table named DEFAULT.T1
> It comes out an error:
> The Streaming Config DEFAULT.T1 already exists.



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


[GitHub] coveralls commented on issue #308: #KYLIN-3645 clean table metadata when drop project

2018-10-23 Thread GitBox
coveralls commented on issue #308: #KYLIN-3645 clean table metadata when drop 
project
URL: https://github.com/apache/kylin/pull/308#issuecomment-432498959
 
 
   ## Pull Request Test Coverage Report for [Build 
3803](https://coveralls.io/builds/19685945)
   
   * **0** of **4**   **(0.0%)**  changed or added relevant lines in **1** file 
are covered.
   * **7** unchanged lines in **3** files lost coverage.
   * Overall coverage increased (+**0.003%**) to **23.329%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java](https://coveralls.io/builds/19685945/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FProjectService.java#L139)
 | 0 | 4 | 0.0%
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java](https://coveralls.io/builds/19685945/source?filename=core-dictionary%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fdict%2Flookup%2Fcache%2FRocksDBLookupTable.java#L62)
 | 1 | 81.08% |
   | 
[server-base/src/main/java/org/apache/kylin/rest/service/ProjectService.java](https://coveralls.io/builds/19685945/source?filename=server-base%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Frest%2Fservice%2FProjectService.java#L143)
 | 2 | 0.0% |
   | 
[core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/19685945/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L449)
 | 4 | 78.12% |
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/19685945/badge)](https://coveralls.io/builds/19685945)
 |
   | :-- | --: |
   | Change from base [Build 3801](https://coveralls.io/builds/19685049): |  
0.003% |
   | Covered Lines: | 16330 |
   | Relevant Lines: | 7 |
   
   ---
   # 💛  - [Coveralls](https://coveralls.io)
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] GinaZhai commented on issue #307: KYLIN-2200 Verify CompileException on UNION ALL query when result onl…

2018-10-23 Thread GitBox
GinaZhai commented on issue #307: KYLIN-2200 Verify CompileException on UNION 
ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307#issuecomment-432526263
 
 
   Please retest this.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

GinaZhai commented on issue #307: KYLIN-2200 Verify CompileException on UNION 
ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307#issuecomment-432526263
 
 
   Please retest this.


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:
us...@infra.apache.org


> CompileException on UNION ALL query when result only contains one column
> 
>
> Key: KYLIN-2200
> URL: https://issues.apache.org/jira/browse/KYLIN-2200
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.5.4.1
>Reporter: Dayue Gao
>Assignee: Dayue Gao
>Priority: Major
> Fix For: v2.5.1
>
> Attachments: KYLIN-2200.patch
>
>
> {code:sql}
> select count(*) from kylin_sales
> union all
> select count(*) from kylin_sales
> {code}
> got following exception
> {noformat}
> Caused by: org.codehaus.commons.compiler.CompileException: Line 82, Column 
> 32: Cannot determine simple type name "Record11_1"
> at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:10092)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5375)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5184)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5165)
> at 
> org.codehaus.janino.UnitCompiler.access$12600(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitReferenceType(UnitCompiler.java:5096)
> at org.codehaus.janino.Java$ReferenceType.accept(Java.java:2880)
> at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5136)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5598)
> at 
> org.codehaus.janino.UnitCompiler.access$13300(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitCast(UnitCompiler.java:5104)
> {noformat}



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


[GitHub] GinaZhai commented on issue #307: KYLIN-2200 Verify CompileException on UNION ALL query when result onl…

2018-10-23 Thread GitBox
GinaZhai commented on issue #307: KYLIN-2200 Verify CompileException on UNION 
ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307#issuecomment-432527346
 
 
   retest this please


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

GinaZhai commented on issue #307: KYLIN-2200 Verify CompileException on UNION 
ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307#issuecomment-432527346
 
 
   retest this please


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:
us...@infra.apache.org


> CompileException on UNION ALL query when result only contains one column
> 
>
> Key: KYLIN-2200
> URL: https://issues.apache.org/jira/browse/KYLIN-2200
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.5.4.1
>Reporter: Dayue Gao
>Assignee: Dayue Gao
>Priority: Major
> Fix For: v2.5.1
>
> Attachments: KYLIN-2200.patch
>
>
> {code:sql}
> select count(*) from kylin_sales
> union all
> select count(*) from kylin_sales
> {code}
> got following exception
> {noformat}
> Caused by: org.codehaus.commons.compiler.CompileException: Line 82, Column 
> 32: Cannot determine simple type name "Record11_1"
> at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:10092)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5375)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5184)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5165)
> at 
> org.codehaus.janino.UnitCompiler.access$12600(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitReferenceType(UnitCompiler.java:5096)
> at org.codehaus.janino.Java$ReferenceType.accept(Java.java:2880)
> at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5136)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5598)
> at 
> org.codehaus.janino.UnitCompiler.access$13300(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitCast(UnitCompiler.java:5104)
> {noformat}



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


[GitHub] shaofengshi closed pull request #307: KYLIN-2200 Verify CompileException on UNION ALL query when result onl…

2018-10-23 Thread GitBox
shaofengshi closed pull request #307: KYLIN-2200 Verify CompileException on 
UNION ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307
 
 
   

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 fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git 
a/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java 
b/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java
index 59cb4d5048..c6d1f62066 100644
--- a/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java
+++ b/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java
@@ -244,6 +244,11 @@ public void testUnionQuery() throws Exception {
 execAndCompQuery(getQueryFolderPrefix() + 
"src/test/resources/query/sql_union", null, true);
 }
 
+@Test
+public void testUnionallQuery() throws Exception {
+execAndCompQuery(getQueryFolderPrefix() + 
"src/test/resources/query/sql_unionall", null, true);
+}
+
 @Test
 public void testTimeStampAdd() throws Exception {
 execAndCompQuery(getQueryFolderPrefix() + 
"src/test/resources/query/sql_timestamp", null, true);
diff --git a/kylin-it/src/test/resources/query/sql_unionall/query01.sql 
b/kylin-it/src/test/resources/query/sql_unionall/query01.sql
new file mode 100644
index 00..428981153d
--- /dev/null
+++ b/kylin-it/src/test/resources/query/sql_unionall/query01.sql
@@ -0,0 +1,7 @@
+-- unionall
+select count(ORDER_ID) as ORDER_ID
+from TEST_KYLIN_FACT as TEST_A
+where ORDER_ID <> 1
+union all
+select count(ORDER_ID) as ORDER_ID
+from TEST_KYLIN_FACT as TEST_B
diff --git a/kylin-it/src/test/resources/query/sql_unionall/query02.sql 
b/kylin-it/src/test/resources/query/sql_unionall/query02.sql
new file mode 100644
index 00..882828529b
--- /dev/null
+++ b/kylin-it/src/test/resources/query/sql_unionall/query02.sql
@@ -0,0 +1,8 @@
+-- unionall
+select count(*) as count_a
+from TEST_KYLIN_FACT as TEST_A
+where lstg_format_name='FP-GTC'
+union all
+select count(*) as count_a
+from TEST_KYLIN_FACT as TEST_A
+where lstg_format_name='FP-GTC'
\ No newline at end of file
diff --git a/kylin-it/src/test/resources/query/sql_unionall/query03.sql 
b/kylin-it/src/test/resources/query/sql_unionall/query03.sql
new file mode 100644
index 00..b1366f15b2
--- /dev/null
+++ b/kylin-it/src/test/resources/query/sql_unionall/query03.sql
@@ -0,0 +1,10 @@
+-- unionall subquery under join
+select count(*) as cnt
+FROM TEST_KYLIN_FACT as TEST_A
+join (
+select * from TEST_KYLIN_FACT where CAL_DT < DATE '2012-02-01'
+union all
+select * from TEST_KYLIN_FACT where CAL_DT > DATE '2013-12-31'
+) TEST_B
+on TEST_A.ORDER_ID = TEST_B.ORDER_ID
+group by TEST_A.SELLER_ID
diff --git 
a/query/src/main/java/org/apache/kylin/query/relnode/KylinEnumerableUnion.java 
b/query/src/main/java/org/apache/kylin/query/relnode/KylinEnumerableUnion.java
new file mode 100644
index 00..67ed0eeaa9
--- /dev/null
+++ 
b/query/src/main/java/org/apache/kylin/query/relnode/KylinEnumerableUnion.java
@@ -0,0 +1,95 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.kylin.query.relnode;
+
+import org.apache.calcite.adapter.enumerable.EnumerableRel;
+import org.apache.calcite.adapter.enumerable.EnumerableRelImplementor;
+import org.apache.calcite.adapter.enumerable.EnumerableUnion;
+import org.apache.calcite.adapter.enumerable.JavaRowFormat;
+import org.apache.calcite.adapter.enumerable.PhysType;
+import org.apache.calcite.adapter.enumerable.PhysTypeImpl;
+import org.apache.calcite.linq4j.Enumerable;
+import org.apache.calcite.linq4j.ExtendedEnumerable;
+import org.apache.calcite.linq4j.Ord;
+import org.apache.calcite.linq4j.function.EqualityComparer;
+import org.apache.calcite.linq4j.function.Functions;
+import org.apache.calcite.linq4j.tree.BlockBuilder;
+import org.apache.calcite.linq4j.tree.Expression;
+import org.apache.calcite.linq4j.tree.Expressions;
+import org.apache.calcite.linq4j.tree.Types;
+import org.apache.calcite.plan.RelOptCluster;
+import org.apache.calcite.plan.R

[jira] [Commented] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

shaofengshi closed pull request #307: KYLIN-2200 Verify CompileException on 
UNION ALL query when result onl…
URL: https://github.com/apache/kylin/pull/307
 
 
   

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 fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git 
a/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java 
b/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java
index 59cb4d5048..c6d1f62066 100644
--- a/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java
+++ b/kylin-it/src/test/java/org/apache/kylin/query/ITKylinQueryTest.java
@@ -244,6 +244,11 @@ public void testUnionQuery() throws Exception {
 execAndCompQuery(getQueryFolderPrefix() + 
"src/test/resources/query/sql_union", null, true);
 }
 
+@Test
+public void testUnionallQuery() throws Exception {
+execAndCompQuery(getQueryFolderPrefix() + 
"src/test/resources/query/sql_unionall", null, true);
+}
+
 @Test
 public void testTimeStampAdd() throws Exception {
 execAndCompQuery(getQueryFolderPrefix() + 
"src/test/resources/query/sql_timestamp", null, true);
diff --git a/kylin-it/src/test/resources/query/sql_unionall/query01.sql 
b/kylin-it/src/test/resources/query/sql_unionall/query01.sql
new file mode 100644
index 00..428981153d
--- /dev/null
+++ b/kylin-it/src/test/resources/query/sql_unionall/query01.sql
@@ -0,0 +1,7 @@
+-- unionall
+select count(ORDER_ID) as ORDER_ID
+from TEST_KYLIN_FACT as TEST_A
+where ORDER_ID <> 1
+union all
+select count(ORDER_ID) as ORDER_ID
+from TEST_KYLIN_FACT as TEST_B
diff --git a/kylin-it/src/test/resources/query/sql_unionall/query02.sql 
b/kylin-it/src/test/resources/query/sql_unionall/query02.sql
new file mode 100644
index 00..882828529b
--- /dev/null
+++ b/kylin-it/src/test/resources/query/sql_unionall/query02.sql
@@ -0,0 +1,8 @@
+-- unionall
+select count(*) as count_a
+from TEST_KYLIN_FACT as TEST_A
+where lstg_format_name='FP-GTC'
+union all
+select count(*) as count_a
+from TEST_KYLIN_FACT as TEST_A
+where lstg_format_name='FP-GTC'
\ No newline at end of file
diff --git a/kylin-it/src/test/resources/query/sql_unionall/query03.sql 
b/kylin-it/src/test/resources/query/sql_unionall/query03.sql
new file mode 100644
index 00..b1366f15b2
--- /dev/null
+++ b/kylin-it/src/test/resources/query/sql_unionall/query03.sql
@@ -0,0 +1,10 @@
+-- unionall subquery under join
+select count(*) as cnt
+FROM TEST_KYLIN_FACT as TEST_A
+join (
+select * from TEST_KYLIN_FACT where CAL_DT < DATE '2012-02-01'
+union all
+select * from TEST_KYLIN_FACT where CAL_DT > DATE '2013-12-31'
+) TEST_B
+on TEST_A.ORDER_ID = TEST_B.ORDER_ID
+group by TEST_A.SELLER_ID
diff --git 
a/query/src/main/java/org/apache/kylin/query/relnode/KylinEnumerableUnion.java 
b/query/src/main/java/org/apache/kylin/query/relnode/KylinEnumerableUnion.java
new file mode 100644
index 00..67ed0eeaa9
--- /dev/null
+++ 
b/query/src/main/java/org/apache/kylin/query/relnode/KylinEnumerableUnion.java
@@ -0,0 +1,95 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.kylin.query.relnode;
+
+import org.apache.calcite.adapter.enumerable.EnumerableRel;
+import org.apache.calcite.adapter.enumerable.EnumerableRelImplementor;
+import org.apache.calcite.adapter.enumerable.EnumerableUnion;
+import org.apache.calcite.adapter.enumerable.JavaRowFormat;
+import org.apache.calcite.adapter.enumerable.PhysType;
+import org.apache.calcite.adapter.enumerable.PhysTypeImpl;
+import org.apache.calcite.linq4j.Enumerable;
+import org.apache.calcite.linq4j.ExtendedEnumerable;
+import org.apache.calcite.linq4j.Ord;
+import org.apache.calcite.linq4j.function.EqualityComparer;
+import org.apache.calcite.linq4j.function.Functions;
+import org.apache.calcit

[jira] [Commented] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on KYLIN-2200:


Commit 494933e08307d741a99417c6c43a9d1b833d77f4 in kylin's branch 
refs/heads/master from gaodayue
[ https://gitbox.apache.org/repos/asf?p=kylin.git;h=494933e ]

KYLIN-2200 Verify CompileException on UNION ALL query when result only contains 
one column and add IT.


> CompileException on UNION ALL query when result only contains one column
> 
>
> Key: KYLIN-2200
> URL: https://issues.apache.org/jira/browse/KYLIN-2200
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.5.4.1
>Reporter: Dayue Gao
>Assignee: Dayue Gao
>Priority: Major
> Fix For: v2.5.1
>
> Attachments: KYLIN-2200.patch
>
>
> {code:sql}
> select count(*) from kylin_sales
> union all
> select count(*) from kylin_sales
> {code}
> got following exception
> {noformat}
> Caused by: org.codehaus.commons.compiler.CompileException: Line 82, Column 
> 32: Cannot determine simple type name "Record11_1"
> at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:10092)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5375)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5184)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5165)
> at 
> org.codehaus.janino.UnitCompiler.access$12600(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitReferenceType(UnitCompiler.java:5096)
> at org.codehaus.janino.Java$ReferenceType.accept(Java.java:2880)
> at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5136)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5598)
> at 
> org.codehaus.janino.UnitCompiler.access$13300(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitCast(UnitCompiler.java:5104)
> {noformat}



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


[jira] [Resolved] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-2200?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI resolved KYLIN-2200.
-
Resolution: Fixed

> CompileException on UNION ALL query when result only contains one column
> 
>
> Key: KYLIN-2200
> URL: https://issues.apache.org/jira/browse/KYLIN-2200
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.5.4.1
>Reporter: Dayue Gao
>Assignee: Dayue Gao
>Priority: Major
> Fix For: v2.5.1
>
> Attachments: KYLIN-2200.patch
>
>
> {code:sql}
> select count(*) from kylin_sales
> union all
> select count(*) from kylin_sales
> {code}
> got following exception
> {noformat}
> Caused by: org.codehaus.commons.compiler.CompileException: Line 82, Column 
> 32: Cannot determine simple type name "Record11_1"
> at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:10092)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5375)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5184)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5165)
> at 
> org.codehaus.janino.UnitCompiler.access$12600(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitReferenceType(UnitCompiler.java:5096)
> at org.codehaus.janino.Java$ReferenceType.accept(Java.java:2880)
> at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5136)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5598)
> at 
> org.codehaus.janino.UnitCompiler.access$13300(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitCast(UnitCompiler.java:5104)
> {noformat}



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


[jira] [Commented] (KYLIN-2200) CompileException on UNION ALL query when result only contains one column

2018-10-23 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on KYLIN-2200:


Commit 3cf84f08b3876daff0e027dc84d0d38eb1e1 in kylin's branch 
refs/heads/2.5.x from gaodayue
[ https://gitbox.apache.org/repos/asf?p=kylin.git;h=3cf84fe ]

KYLIN-2200 Verify CompileException on UNION ALL query when result only contains 
one column and add IT.


> CompileException on UNION ALL query when result only contains one column
> 
>
> Key: KYLIN-2200
> URL: https://issues.apache.org/jira/browse/KYLIN-2200
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.5.4.1
>Reporter: Dayue Gao
>Assignee: Dayue Gao
>Priority: Major
> Fix For: v2.5.1
>
> Attachments: KYLIN-2200.patch
>
>
> {code:sql}
> select count(*) from kylin_sales
> union all
> select count(*) from kylin_sales
> {code}
> got following exception
> {noformat}
> Caused by: org.codehaus.commons.compiler.CompileException: Line 82, Column 
> 32: Cannot determine simple type name "Record11_1"
> at 
> org.codehaus.janino.UnitCompiler.compileError(UnitCompiler.java:10092)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5375)
> at 
> org.codehaus.janino.UnitCompiler.getReferenceType(UnitCompiler.java:5184)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5165)
> at 
> org.codehaus.janino.UnitCompiler.access$12600(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitReferenceType(UnitCompiler.java:5096)
> at org.codehaus.janino.Java$ReferenceType.accept(Java.java:2880)
> at org.codehaus.janino.UnitCompiler.getType(UnitCompiler.java:5136)
> at org.codehaus.janino.UnitCompiler.getType2(UnitCompiler.java:5598)
> at 
> org.codehaus.janino.UnitCompiler.access$13300(UnitCompiler.java:183)
> at 
> org.codehaus.janino.UnitCompiler$16.visitCast(UnitCompiler.java:5104)
> {noformat}



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


[jira] [Commented] (KYLIN-3633) Dead lock may happen in building global dictionary

2018-10-23 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on KYLIN-3633:


Commit 542720e4f6433ea21495e0256fc04ecfe3b03ba8 in kylin's branch 
refs/heads/2.5.x from [~nichunen]
[ https://gitbox.apache.org/repos/asf?p=kylin.git;h=542720e ]

KYLIN-3633 Avoid potential dead lock when building global dictionary


> Dead lock may happen in building global dictionary
> --
>
> Key: KYLIN-3633
> URL: https://issues.apache.org/jira/browse/KYLIN-3633
> Project: Kylin
>  Issue Type: Bug
>  Components: Measure - Count Distinct
>Affects Versions: v2.4.1
>Reporter: nichunen
>Assignee: nichunen
>Priority: Major
> Fix For: v2.5.1
>
>
> During the building of global dict, if an error happens, the builder's lock 
> maybe un-released. This will block all other building jobs with global dict.



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


[jira] [Resolved] (KYLIN-3633) Dead lock may happen in building global dictionary

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3633?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI resolved KYLIN-3633.
-
Resolution: Fixed

> Dead lock may happen in building global dictionary
> --
>
> Key: KYLIN-3633
> URL: https://issues.apache.org/jira/browse/KYLIN-3633
> Project: Kylin
>  Issue Type: Bug
>  Components: Measure - Count Distinct
>Affects Versions: v2.4.1
>Reporter: nichunen
>Assignee: nichunen
>Priority: Major
> Fix For: v2.5.1
>
>
> During the building of global dict, if an error happens, the builder's lock 
> maybe un-released. This will block all other building jobs with global dict.



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


[GitHub] shaofengshi closed pull request #306: Update FAQ and Kylin Configuration

2018-10-23 Thread GitBox
shaofengshi closed pull request #306: Update FAQ and Kylin Configuration
URL: https://github.com/apache/kylin/pull/306
 
 
   

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 fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/website/_docs/gettingstarted/faq.md 
b/website/_docs/gettingstarted/faq.md
index 793ff6479a..f23ec958e6 100644
--- a/website/_docs/gettingstarted/faq.md
+++ b/website/_docs/gettingstarted/faq.md
@@ -10,10 +10,6 @@ since: v0.6.x
 
   * No, Kylin is an OLAP engine with SQL interface. The SQL queries need be 
matched with the pre-defined OLAP model.
 
- How to compare Kylin with other SQL engines like Hive, Presto, Spark SQL, 
Impala?
-
-  * They answer a query in different ways. Kylin is not a replacement for 
them, but a supplement (query accelerator). Many users run Kylin together with 
other SQL engines. For the high frequent query patterns, building Cubes can 
greatly improve the performance and also offload cluster workloads. For less 
queried patterns or ad-hoc queries, ther MPP engines are more flexible.
-
  What's a typical scenario to use Apache Kylin?
 
   * Kylin can be the best option if you have a huge table (e.g., >100 million 
rows), join with lookup tables, while queries need be finished in the second 
level (dashboards, interactive reports, business intelligence, etc), and the 
concurrent users can be dozens or hundreds.
@@ -30,6 +26,10 @@ since: v0.6.x
 
   * It depends on a couple of factors, for example, dimension/measure number, 
dimension cardinality, cuboid number, compression algorithm, etc. You can 
optimize the cube expansion in many ways to control the size.
 
+ How to compare Kylin with other SQL engines like Hive, Presto, Spark SQL, 
Impala?
+
+  * They answer a query in different ways. Kylin is not a replacement for 
them, but a supplement (query accelerator). Many users run Kylin together with 
other SQL engines. For the high frequent query patterns, building Cubes can 
greatly improve the performance and also offload cluster workloads. For less 
queried patterns or ad-hoc queries, ther MPP engines are more flexible.
+  
  How to compare Kylin with Druid?
 
   * Druid is more suitable for real-time analysis. Kylin is more focus on OLAP 
case. Druid has good integration with Kafka as real-time streaming; Kylin 
fetches data from Hive or Kafka in batches. The real-time capability of Kylin 
is still under development.
@@ -82,10 +82,19 @@ But if you do want, there are some workarounds. 1) Add the 
primary key as a dime
 
   * Yes, but they are private APIs, incline to change over versions (without 
notification). By design, Kylin expects the user to create a new 
project/model/cube in Kylin's web GUI.
 
+ How to define a snowflake model(with two fact tables)?
+
+  * In the snowflake model, there is only one fact table also. But you could 
define lookup table joins with another lookup table.
+  * If the query pattern between your two "fact" tables is fixed, just like 
factA left join with factB. You could define factB as a lookup table and skip 
the snapshot for this huge lookup table.
+
  Where does the cube locate, can I directly read cube from HBase without 
going through Kylin API?
 
   * Cube is stored in HBase. Each cube segment is an HBase table. The 
dimension values will be composed as the row key. The measures will be 
serialized in columns. To improve the storage efficiency, both dimension and 
measure values will be encoded to bytes. Kylin will decode the bytes to origin 
values after fetching from HBase. Without Kylin's metadata, the HBase tables 
are not readable.
 
+ How to design a cube?
+
+  * Please check: 
[https://www.slideshare.net/YangLi43/design-cube-in-apache-kylin]()
+
  How to encrypt cube data?
 
   * You can enable encryption at HBase side. Refer 
https://hbase.apache.org/book.html#hbase.encryption.server for more details.
@@ -94,6 +103,14 @@ But if you do want, there are some workarounds. 1) Add the 
primary key as a dime
 
   * Kylin doesn't have a built-in scheduler for this. You can trigger that 
through Rest API from external scheduler services, like Linux cron job, Apache 
Airflow, etc.
 
+ How to export/import cube/project across different Kylin environments?
+
+  * Please check: [http://kylin.apache.org/docs/howto/howto_use_cli.html]()
+
+ How to view kylin cube's hbase table without encoding?
+
+  * To view the origin data, please use SQL to query Kylin. Kylin will convert 
the SQL query to HBase access and then decode the data. You can use Rest API, 
JDBC, ODBC drivers to connect with Kylin.
+  
  Does Kylin support Hadoop 3 and HBase 2.0?
 
   * From v2.5.0, Kylin will provide a binary package for Hadoop 3 and HBase 2.
@@ -308,13 +325,3 @@ vi ./tomcat/webapps/kylin/WEB-INF/classes/kylinSecurity.xml
 

[GitHub] shaofengshi closed pull request #302: KYLIN-3620 resolve "--" is treated as comment marker between singe qu…

2018-10-23 Thread GitBox
shaofengshi closed pull request #302: KYLIN-3620 resolve "--" is treated as 
comment marker between singe qu…
URL: https://github.com/apache/kylin/pull/302
 
 
   

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 fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java 
b/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java
index 1e7498b511..5aad5e0e26 100644
--- a/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java
+++ b/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java
@@ -184,7 +184,7 @@ public static boolean isSelectStatement(String sql) {
 
 public static String removeCommentInSql(String sql1) {
 // match two patterns, one is "-- comment", the other is "/* comment 
*/"
-final String[] commentPatterns = new String[] { "--[^\r\n]*", 
"/\\*[\\s\\S]*?\\*/" };
+final String[] commentPatterns = new String[]{"--(?!.*\\*/).*?[\r\n]", 
"/\\*(.|\r|\n)*?\\*/"};
 
 for (int i = 0; i < commentPatterns.length; i++) {
 sql1 = sql1.replaceAll(commentPatterns[i], "");
diff --git a/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java 
b/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java
index bf6f8363be..0b04e6d306 100644
--- a/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java
+++ b/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java
@@ -169,10 +169,10 @@ public void testForceLimit() {
 public void testRemoveCommentInSql() {
 
 String originSql = "select count(*) from test_kylin_fact where price > 
10.0";
+String originSql2 = "select count(*) from test_kylin_fact where 
TEST_COLUMN != 'not--a comment'";
 
 {
 String sqlWithComment = "-- comment \n" + originSql;
-
 Assert.assertEquals(originSql, 
QueryUtil.removeCommentInSql(sqlWithComment));
 }
 
@@ -223,9 +223,57 @@ public void testRemoveCommentInSql() {
 }
 
 {
-String sqlWithComment = "/* comment1 * \ncomment2 */ -- comment 
3\n" + originSql + "-- comment 5";
+String sqlWithComment = "/* comment1 * \ncomment2 */ -- comment 
3\n" + originSql + "-- comment 5\n";
 Assert.assertEquals(originSql, 
QueryUtil.removeCommentInSql(sqlWithComment));
 }
+
+{
+String sqlWithComment2 = "/* comment1 * \ncomment2 */ -- comment 
5\n" + originSql2 + "/* comment3 / comment4 */";
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "/* comment1 * comment2 */ /* comment3 / 
comment4 */ -- comment 5\n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "/* comment1 * comment2 */ " + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "/* comment1/comment2 */ " + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "-- \n -- comment \n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "-- comment \n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "-- comment \n -- comment\n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+String content = "--  One-line comment and /**range\n" +
+"/*\n" +
+"Multi-line comment\r\n" +
+"--  Multi-line comment*/\n" +
+"select price as " +
+"/*\n" +
+"Multi-line comment\r\n" +
+"--  Multi-line comment*/\n" +
+"revenue from /*One-line comment-- One-line comment*/ 
v_lineitem;";
+String expectedContent = "select price as revenue from  v_lineitem;";
+String trimmedContent = 
QueryUtil.removeCommentInSql(content).replaceAll("\n", "").trim();
+Assert.assertEquals(trimmedContent, expectedContent);
 }
 
 @Test


 


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:
us...@infra.apache.org


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

2018-10-23 Thread ASF subversion and git services (JIRA)


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

ASF subversion and git services commented on KYLIN-3620:


Commit 9c3165a1e10a830b48463a23aa26419986b5abfa in kylin's branch 
refs/heads/master from [~caolijun1166]
[ https://gitbox.apache.org/repos/asf?p=kylin.git;h=9c3165a ]

KYLIN-3620 resolve "--" is treated as comment marker between singe quotes


> "--" 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
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v2.4.1
>Reporter: Scott Fan
>Assignee: Lijun Cao
>Priority: Major
> Fix For: v2.6.0
>
>
> Hi,
> When I exec a SQL like below, there was an error.
> SELECT * FROM TABLEA WHERE NAME='Jason--Arthur'
> Seems like KYLIN use  the – as a comment marker.
> It should not be that between single quotes, I think.
>   
>  Even I send a REST POST contains SQL like that, or do an ODBC query, there 
> was an error too.
> You may tell me that I can use a \ as escape, but I can't change the SQL 
> generated by BI tools like tableau.  
> Thanks
>   



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


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

2018-10-23 Thread ASF GitHub Bot (JIRA)


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

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

shaofengshi closed pull request #302: KYLIN-3620 resolve "--" is treated as 
comment marker between singe qu…
URL: https://github.com/apache/kylin/pull/302
 
 
   

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 fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java 
b/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java
index 1e7498b511..5aad5e0e26 100644
--- a/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java
+++ b/query/src/main/java/org/apache/kylin/query/util/QueryUtil.java
@@ -184,7 +184,7 @@ public static boolean isSelectStatement(String sql) {
 
 public static String removeCommentInSql(String sql1) {
 // match two patterns, one is "-- comment", the other is "/* comment 
*/"
-final String[] commentPatterns = new String[] { "--[^\r\n]*", 
"/\\*[\\s\\S]*?\\*/" };
+final String[] commentPatterns = new String[]{"--(?!.*\\*/).*?[\r\n]", 
"/\\*(.|\r|\n)*?\\*/"};
 
 for (int i = 0; i < commentPatterns.length; i++) {
 sql1 = sql1.replaceAll(commentPatterns[i], "");
diff --git a/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java 
b/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java
index bf6f8363be..0b04e6d306 100644
--- a/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java
+++ b/query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java
@@ -169,10 +169,10 @@ public void testForceLimit() {
 public void testRemoveCommentInSql() {
 
 String originSql = "select count(*) from test_kylin_fact where price > 
10.0";
+String originSql2 = "select count(*) from test_kylin_fact where 
TEST_COLUMN != 'not--a comment'";
 
 {
 String sqlWithComment = "-- comment \n" + originSql;
-
 Assert.assertEquals(originSql, 
QueryUtil.removeCommentInSql(sqlWithComment));
 }
 
@@ -223,9 +223,57 @@ public void testRemoveCommentInSql() {
 }
 
 {
-String sqlWithComment = "/* comment1 * \ncomment2 */ -- comment 
3\n" + originSql + "-- comment 5";
+String sqlWithComment = "/* comment1 * \ncomment2 */ -- comment 
3\n" + originSql + "-- comment 5\n";
 Assert.assertEquals(originSql, 
QueryUtil.removeCommentInSql(sqlWithComment));
 }
+
+{
+String sqlWithComment2 = "/* comment1 * \ncomment2 */ -- comment 
5\n" + originSql2 + "/* comment3 / comment4 */";
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "/* comment1 * comment2 */ /* comment3 / 
comment4 */ -- comment 5\n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "/* comment1 * comment2 */ " + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "/* comment1/comment2 */ " + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "-- \n -- comment \n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "-- comment \n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+{
+String sqlWithComment2 = "-- comment \n -- comment\n" + originSql2;
+Assert.assertEquals(originSql2, 
QueryUtil.removeCommentInSql(sqlWithComment2));
+}
+
+String content = "--  One-line comment and /**range\n" +
+"/*\n" +
+"Multi-line comment\r\n" +
+"--  Multi-line comment*/\n" +
+"select price as " +
+"/*\n" +
+"Multi-line comment\r\n" +
+"--  Multi-line comment*/\n" +
+"revenue from /*One-line comment-- One-line comment*/ 
v_lineitem;";
+String expectedContent = "select price as revenue from  v_lineitem;";
+String trimmedContent = 
QueryUtil.removeCommentInSql(content).replaceAll("\n", "").trim();
+Assert.assertEquals(trimmedContent, expectedContent);
 }
 
 @Test


 

---

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

2018-10-23 Thread Shaofeng SHI (JIRA)


 [ 
https://issues.apache.org/jira/browse/KYLIN-3620?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Shaofeng SHI resolved KYLIN-3620.
-
Resolution: Fixed

> "--" 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
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v2.4.1
>Reporter: Scott Fan
>Assignee: Lijun Cao
>Priority: Major
> Fix For: v2.6.0
>
>
> Hi,
> When I exec a SQL like below, there was an error.
> SELECT * FROM TABLEA WHERE NAME='Jason--Arthur'
> Seems like KYLIN use  the – as a comment marker.
> It should not be that between single quotes, I think.
>   
>  Even I send a REST POST contains SQL like that, or do an ODBC query, there 
> was an error too.
> You may tell me that I can use a \ as escape, but I can't change the SQL 
> generated by BI tools like tableau.  
> Thanks
>   



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


[jira] [Created] (KYLIN-3646) Add instruction about rowkey sequence in "Advanced setting" page

2018-10-23 Thread Shaofeng SHI (JIRA)
Shaofeng SHI created KYLIN-3646:
---

 Summary: Add instruction about rowkey sequence in "Advanced 
setting" page
 Key: KYLIN-3646
 URL: https://issues.apache.org/jira/browse/KYLIN-3646
 Project: Kylin
  Issue Type: Improvement
  Components: Web 
Reporter: Shaofeng SHI


The position of dimension in rowkey is important for performance (both build 
and query); But this message is not clear for a normal user. We'd better add 
instruction on the web GUI, so that user knows to adjust it by some rule.



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