[jira] [Closed] (KYLIN-2014) Support multiple languages

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI closed KYLIN-2014.
---
Resolution: Duplicate

> Support multiple languages
> --
>
> Key: KYLIN-2014
> URL: https://issues.apache.org/jira/browse/KYLIN-2014
> Project: Kylin
>  Issue Type: New Feature
>  Components: Web 
>Reporter: Shaofeng SHI
>
> Till today English is the only language on Kylin's Web GUI; Need to support 
> multiple languages. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-2014) Support multiple languages

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI commented on KYLIN-2014:
-

Thanks Yongjie, let's use KYLIN-1000 to track and then close this one. I have 
change the assignee of KYLIN-1000 to you, your contribution is welcomed.

> Support multiple languages
> --
>
> Key: KYLIN-2014
> URL: https://issues.apache.org/jira/browse/KYLIN-2014
> Project: Kylin
>  Issue Type: New Feature
>  Components: Web 
>Reporter: Shaofeng SHI
>
> Till today English is the only language on Kylin's Web GUI; Need to support 
> multiple languages. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Assigned] (KYLIN-1000) i18n and localization support

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI reassigned KYLIN-1000:
---

  Assignee: yongjie zhao  (was: Zhong,Jason)
Issue Type: Wish  (was: New Feature)

Yong jie, please go ahead. Thanks!

> i18n and localization support
> -
>
> Key: KYLIN-1000
> URL: https://issues.apache.org/jira/browse/KYLIN-1000
> Project: Kylin
>  Issue Type: Wish
>  Components: Web 
>Affects Versions: v1.0, v1.4.0
>Reporter: Luke Han
>Assignee: yongjie zhao
>
> There are many use cases of Apache Kylin are requiring to support other 
> language rather than English, like Chinese, Japanese, EU side and others.
> Open this JIRA to tracking internationalization and localization of Kylin. 
> The first phase is to support multi-language of web UI, like Chinese.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Comment Edited] (KYLIN-3059) Query got IllegalArgumentException

2017-11-30 Thread Xingxing Di (JIRA)

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

Xingxing Di edited comment on KYLIN-3059 at 12/1/17 7:04 AM:
-

[~liyang.g...@gmail.com] Happy to hear that . 
I saw this in kylin2.2  :
{code:java}
TblColRef(TableRef table, ColumnDesc column) {

checkArgument(table.getTableDesc().getIdentity().equals(column.getTable().getIdentity()));
this.table = table;
this.column = column;
}
{code}

I had merge this  into our kylin (v1.6)  , I'm not sure if that can resolve the 
problem. 


was (Author: dixingx...@yeah.net):
[~liyang.g...@gmail.com] Happy to hear that . 
I saw this in kylin2.2  :
{code:java}
TblColRef(TableRef table, ColumnDesc column) {

checkArgument(table.getTableDesc().getIdentity().equals(column.getTable().getIdentity()));
this.table = table;
this.column = column;
}
{code}

I had merge this  into our kylin(v 1.6)  , I'm not sure if that can resolve the 
problem. 

> Query got IllegalArgumentException
> --
>
> Key: KYLIN-3059
> URL: https://issues.apache.org/jira/browse/KYLIN-3059
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.6.0
> Environment: apache-kylin-1.6.0-hbase1.x
> apache hadoop 2.7.2
> hbase-1.2.4
>Reporter: Xingxing Di
>Assignee: liyang
> Attachments: query.log
>
>
> A query got an exception , the error trace was :
> {code:java}
> Caused by: java.lang.IllegalArgumentException
> at 
> com.google.common.base.Preconditions.checkArgument(Preconditions.java:76)
> at 
> org.apache.kylin.metadata.model.TblColRef.(TblColRef.java:105)
> at 
> org.apache.kylin.metadata.model.TblColRef.columnForUnknownModel(TblColRef.java:74)
> at 
> org.apache.kylin.query.relnode.OLAPTableScan.buildColumnRowType(OLAPTableScan.java:231)
> {code}
> the code is : 
> {code:java}
> TblColRef(TableRef table, ColumnDesc column) {
> checkArgument(table.getTableDesc() == column.getTable()); // this 
> line cause exception
> this.table = table;
> this.column = column;
> } 
> {code}
> and after we did "Reload metadata", query executed successful.
> I dont know what will cause this exception,  i think it may related to 
> ProjectL2Cache or ProjectManager.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3059) Query got IllegalArgumentException

2017-11-30 Thread Xingxing Di (JIRA)

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

Xingxing Di commented on KYLIN-3059:


[~liyang.g...@gmail.com] Happy to hear that . 
I saw this in kylin2.2  :
{code:java}
TblColRef(TableRef table, ColumnDesc column) {

checkArgument(table.getTableDesc().getIdentity().equals(column.getTable().getIdentity()));
this.table = table;
this.column = column;
}
{code}

I had merge this  into our kylin(v 1.6)  , I'm not sure if that can resolve the 
problem. 

> Query got IllegalArgumentException
> --
>
> Key: KYLIN-3059
> URL: https://issues.apache.org/jira/browse/KYLIN-3059
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.6.0
> Environment: apache-kylin-1.6.0-hbase1.x
> apache hadoop 2.7.2
> hbase-1.2.4
>Reporter: Xingxing Di
>Assignee: liyang
> Attachments: query.log
>
>
> A query got an exception , the error trace was :
> {code:java}
> Caused by: java.lang.IllegalArgumentException
> at 
> com.google.common.base.Preconditions.checkArgument(Preconditions.java:76)
> at 
> org.apache.kylin.metadata.model.TblColRef.(TblColRef.java:105)
> at 
> org.apache.kylin.metadata.model.TblColRef.columnForUnknownModel(TblColRef.java:74)
> at 
> org.apache.kylin.query.relnode.OLAPTableScan.buildColumnRowType(OLAPTableScan.java:231)
> {code}
> the code is : 
> {code:java}
> TblColRef(TableRef table, ColumnDesc column) {
> checkArgument(table.getTableDesc() == column.getTable()); // this 
> line cause exception
> this.table = table;
> this.column = column;
> } 
> {code}
> and after we did "Reload metadata", query executed successful.
> I dont know what will cause this exception,  i think it may related to 
> ProjectL2Cache or ProjectManager.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3059) Query got IllegalArgumentException

2017-11-30 Thread liyang (JIRA)

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

liyang commented on KYLIN-3059:
---

Thanks for the good report. From this and a few related reports, we have 
identified that the metadata cache mechanism can corrupt in certain concurrency 
scenarios. Will attempt a fix soon.

> Query got IllegalArgumentException
> --
>
> Key: KYLIN-3059
> URL: https://issues.apache.org/jira/browse/KYLIN-3059
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Affects Versions: v1.6.0
> Environment: apache-kylin-1.6.0-hbase1.x
> apache hadoop 2.7.2
> hbase-1.2.4
>Reporter: Xingxing Di
>Assignee: liyang
> Attachments: query.log
>
>
> A query got an exception , the error trace was :
> {code:java}
> Caused by: java.lang.IllegalArgumentException
> at 
> com.google.common.base.Preconditions.checkArgument(Preconditions.java:76)
> at 
> org.apache.kylin.metadata.model.TblColRef.(TblColRef.java:105)
> at 
> org.apache.kylin.metadata.model.TblColRef.columnForUnknownModel(TblColRef.java:74)
> at 
> org.apache.kylin.query.relnode.OLAPTableScan.buildColumnRowType(OLAPTableScan.java:231)
> {code}
> the code is : 
> {code:java}
> TblColRef(TableRef table, ColumnDesc column) {
> checkArgument(table.getTableDesc() == column.getTable()); // this 
> line cause exception
> this.table = table;
> this.column = column;
> } 
> {code}
> and after we did "Reload metadata", query executed successful.
> I dont know what will cause this exception,  i think it may related to 
> ProjectL2Cache or ProjectManager.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Resolved] (KYLIN-3031) KeywordDefaultDirtyHack should ignore case of default like other database does

2017-11-30 Thread liyang (JIRA)

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

liyang resolved KYLIN-3031.
---
   Resolution: Fixed
Fix Version/s: v2.3.0

> KeywordDefaultDirtyHack should ignore case of default like other database does
> --
>
> Key: KYLIN-3031
> URL: https://issues.apache.org/jira/browse/KYLIN-3031
> Project: Kylin
>  Issue Type: Bug
>Reporter: Zhong Yanghong
>Assignee: liyang
> Fix For: v2.3.0
>
> Attachments: APACHE-KYLIN-3031.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3031) KeywordDefaultDirtyHack should ignore case of default like other database does

2017-11-30 Thread liyang (JIRA)

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

liyang commented on KYLIN-3031:
---

Commit merged.
https://github.com/apache/kylin/commit/a94b4796a6c4732af7b138e46f844d89bcdac8d9


> KeywordDefaultDirtyHack should ignore case of default like other database does
> --
>
> Key: KYLIN-3031
> URL: https://issues.apache.org/jira/browse/KYLIN-3031
> Project: Kylin
>  Issue Type: Bug
>Reporter: Zhong Yanghong
>Assignee: liyang
> Attachments: APACHE-KYLIN-3031.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Comment Edited] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua edited comment on KYLIN-3072 at 12/1/17 6:23 AM:
--

Hi [~Shaofengshi], I'm sorry. I did not describe this issue clearly. If you 
still think there is no problem, I will close the issue according to your 
review. The essence of this problem is that the logic of parsing is flawed. The 
current logic is as following:
1. Firstly,  the "--" was parsed.
2. Secondly, the "/* */" was parsed.
Based on the above parsing logic, once the "/\* */" contains the "\-\-" and the 
"\*/" and the "\-\-" are in the same line , the parsing will be wrong. Because 
the "\*/" will be removed when the program parses the "\--". When parsing the 
"/\* */" again, the program parses fail because the "\*/" was removed.

What we modify is the wrong logic of the program. Based on the above logic 
modification, all possible combinations will not be a problem, do you think?


was (Author: peng.jianhua):
Hi [~Shaofengshi], I'm sorry. I did not describe this issue clearly. If you 
still think there is no problem, I will close the issue according to your 
review. The essence of this problem is that the logic of parsing is flawed. The 
current logic is as following:
1. Firstly,  the "--" was parsed.
2. Secondly, the "/* */" was parsed.
Based on the above parsing logic, once the "/\* */" contains the "\--" and the 
"\*/" and the "\--" are in the same line , the parsing will be wrong. Because 
the "\*/" will be removed when the program parses the "\--". When parsing the 
"/\* */" again, the program parses fail because the "\*/" was removed.

What we modify is the wrong logic of the program. Based on the above logic 
modification, all possible combinations will not be a problem, do you think?

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /\*comments--comments*/
> we should remove comments like '/\*comments*/' first,then remove comments 
> like '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Comment Edited] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua edited comment on KYLIN-3072 at 12/1/17 6:22 AM:
--

Hi [~Shaofengshi], I'm sorry. I did not describe this issue clearly. If you 
still think there is no problem, I will close the issue according to your 
review. The essence of this problem is that the logic of parsing is flawed. The 
current logic is as following:
1. Firstly,  the "--" was parsed.
2. Secondly, the "/* */" was parsed.
Based on the above parsing logic, once the "/\* */" contains the "\--" and the 
"\*/" and the "\--" are in the same line , the parsing will be wrong. Because 
the "\*/" will be removed when the program parses the "\--". When parsing the 
"/\* */" again, the program parses fail because the "\*/" was removed.

What we modify is the wrong logic of the program. Based on the above logic 
modification, all possible combinations will not be a problem, do you think?


was (Author: peng.jianhua):
Hi [~Shaofengshi], I'm sorry. I did not describe this issue clearly. If you 
still think there is no problem, I will close the issue according to your 
review. The essence of this problem is that the logic of parsing is flawed. The 
current logic is as following:
1. Firstly,  the "--" was parsed.
2. Secondly, the "/* */" was parsed.
Based on the above parsing logic, once the "/\* */" contains the "--" and the 
"\*/" and the "--" are in the same line , the parsing will be wrong. Because 
the "\*/" will be removed when the program parses the "--". When parsing the 
"/\* */" again, the program parses fail because the "\*/" was removed.

What we modify is the wrong logic of the program. Based on the above logic 
modification, all possible combinations will not be a problem, do you think?

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /\*comments--comments*/
> we should remove comments like '/\*comments*/' first,then remove comments 
> like '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Comment Edited] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua edited comment on KYLIN-3072 at 12/1/17 6:21 AM:
--

Hi [~Shaofengshi], I'm sorry. I did not describe this issue clearly. If you 
still think there is no problem, I will close the issue according to your 
review. The essence of this problem is that the logic of parsing is flawed. The 
current logic is as following:
1. Firstly,  the "--" was parsed.
2. Secondly, the "/* */" was parsed.
Based on the above parsing logic, once the "/\* */" contains the "--" and the 
"\*/" and the "--" are in the same line , the parsing will be wrong. Because 
the "\*/" will be removed when the program parses the "--". When parsing the 
"/\* */" again, the program parses fail because the "\*/" was removed.

What we modify is the wrong logic of the program. Based on the above logic 
modification, all possible combinations will not be a problem, do you think?


was (Author: peng.jianhua):
Hi [~Shaofengshi], I'm sorry. I did not describe this issue clearly. If you 
still think there is no problem, I will close the issue according to your 
review. The essence of this problem is that the logic of parsing is flawed. The 
current logic is as following:
1. Firstly,  the "--" was parsed.
2. Secondly, the "/* */" was parsed.
Based on the above parsing logic, once the "/* */" contains the "--" and the 
"*/" and the "--" are in the same line , the parsing will be wrong. Because the 
"*/" will be removed when the program parses the "--". When parsing the "/* */" 
again, the program parses fail because the "*/" was removed.

What we modify is the wrong logic of the program. Based on the above logic 
modification, all possible combinations will not be a problem, do you think?

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /\*comments--comments*/
> we should remove comments like '/\*comments*/' first,then remove comments 
> like '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua commented on KYLIN-3072:
-

Hi [~Shaofengshi], I'm sorry. I did not describe this issue clearly. If you 
still think there is no problem, I will close the issue according to your 
review. The essence of this problem is that the logic of parsing is flawed. The 
current logic is as following:
1. Firstly,  the "--" was parsed.
2. Secondly, the "/* */" was parsed.
Based on the above parsing logic, once the "/* */" contains the "--" and the 
"*/" and the "--" are in the same line , the parsing will be wrong. Because the 
"*/" will be removed when the program parses the "--". When parsing the "/* */" 
again, the program parses fail because the "*/" was removed.

What we modify is the wrong logic of the program. Based on the above logic 
modification, all possible combinations will not be a problem, do you think?

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /\*comments--comments*/
> we should remove comments like '/\*comments*/' first,then remove comments 
> like '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-2014) Support multiple languages

2017-11-30 Thread yongjie zhao (JIRA)

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

yongjie zhao commented on KYLIN-2014:
-

this issue looks like #KYLIN-1000, please merge both issue. I can do this 
feature, could you assign it to me?

> Support multiple languages
> --
>
> Key: KYLIN-2014
> URL: https://issues.apache.org/jira/browse/KYLIN-2014
> Project: Kylin
>  Issue Type: New Feature
>  Components: Web 
>Reporter: Shaofeng SHI
>
> Till today English is the only language on Kylin's Web GUI; Need to support 
> multiple languages. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3070) Add a config property for flat table storage format

2017-11-30 Thread RongH (JIRA)

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

RongH commented on KYLIN-3070:
--

I want to do it. Could you assign this issue to me?

> Add a config property for flat table storage format
> ---
>
> Key: KYLIN-3070
> URL: https://issues.apache.org/jira/browse/KYLIN-3070
> Project: Kylin
>  Issue Type: Improvement
>  Components: Job Engine
>Affects Versions: v2.2.0
> Environment: HDP 2.5.6, Kylin 2.2.0
>Reporter: Vsevolod Ostapenko
>Priority: Minor
>  Labels: newbie
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Flat table storage format is currently hard-coded as SEQUENCEFILE in the 
> core-job/src/main/java/org/apache/kylin/job/JoinedFlatTable.java
> That prevents using Impala as a SQL engine while using beeline CLI (via 
> custom JDBC URL), as Impala cannot write sequence files.
> Adding a parameter to kylin.properties to override the default setting would 
> address the issue.
> Removing a hard-coded value for storage format might be good idea in and on 
> itself.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-2986) Segments overlap issue may happen when concurrent merge or build

2017-11-30 Thread liyang (JIRA)

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

liyang commented on KYLIN-2986:
---

Why discarding a job didn't delete the related segment, have any one trace down 
the root cause?

> Segments overlap issue may happen when concurrent merge or build
> 
>
> Key: KYLIN-2986
> URL: https://issues.apache.org/jira/browse/KYLIN-2986
> Project: Kylin
>  Issue Type: Bug
>Reporter: Zhong Yanghong
>Assignee: Zhong Yanghong
>Priority: Trivial
>  Labels: scope
> Attachments: screenshot-1.png
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3069) Add proper time zone support to the WebUI instead of GMT/PST kludge

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI commented on KYLIN-3069:
-

No problem, thank you!

> Add proper time zone support to the WebUI instead of GMT/PST kludge
> ---
>
> Key: KYLIN-3069
> URL: https://issues.apache.org/jira/browse/KYLIN-3069
> Project: Kylin
>  Issue Type: Bug
>  Components: Web 
>Affects Versions: v2.2.0
> Environment: HDP 2.5.3, Kylin 2.2.0
>Reporter: Vsevolod Ostapenko
>Assignee: Zhixiong Chen
>Priority: Minor
>   Original Estimate: 168h
>  Remaining Estimate: 168h
>
> Time zone handling logic in the WebUI is a kludge, coded to parse only 
> "GMT-N" time zone specifications and defaulting to PST, if parsing is not 
> successful (kylin/webapp/app/js/filters/filter.js)
> Integrating moment and moment time zone (http://momentjs.com/timezone/docs/) 
> into the product, would allow correct time zone handling.
> For the users who happen to reside in the geographical locations that do 
> observe day light savings time, usage of GMT-N format is very inconvenient 
> and info reported by the UI in various places is perplexing.
> Needless to say that the GMT moniker itself is long deprecated.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3075) Improvement kylin.sh, remove duplicate dependency check, let kylin start faster.

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua commented on KYLIN-3075:
-

Hi [~Shaofengshi], Base on the following reasons, I think we should Improvement 
it.
1. Its logic is not very clear, there are repeated load dependent components, 
the result is too much use of computer resources;
2. And lots of the value for parameters was repeatedly loaded.
3. Based on the above 2 points, the start speed of the kylin is slow.

The value of this issue is following:
1. The start logic for kylin is clearer. So we are more easy to maintain and 
expand.
2. When the kylin starts, the kylin saves more system computer resources.
3. When the kylin starts, the issue speed up the kylin startup speed.

> Improvement kylin.sh, remove duplicate dependency check, let kylin start 
> faster.
> 
>
> Key: KYLIN-3075
> URL: https://issues.apache.org/jira/browse/KYLIN-3075
> Project: Kylin
>  Issue Type: Improvement
>Reporter: peng.jianhua
>Assignee: peng.jianhua
>Priority: Minor
> Attachments: 
> 0001-KYLIN-3075-Improvement-kylin.sh-remove-duplicate-dep.patch
>
>
> Improvement kylin.sh, remove duplicate dependency check, let kylin start 
> faster.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI commented on KYLIN-3072:
-

I know it; My question is, do you see a case that such comment style generated 
from a BI tool like Tableau, Cognos or MSTR? or it is made by manual? 

If by manual, there are many possible combinations, can this patch solve all of 
them? Besides, Calcite can recognize comments, Kylin doesn't need take over 
that too much.

I don't think this is a solid enhancement that worth a JIRA to discuss on it. 
If a strange comment couldn't be recognized and got error, then please correct 
it, that's it.

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /\*comments--comments*/
> we should remove comments like '/\*comments*/' first,then remove comments 
> like '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3072:

Description: 
select * from kylin_sales /\*comments--comments*/

we should remove comments like '/*comments*/' first,then remove comments like 
'--comments' , or the sql above will have an exception.
please refer to 01.png.

if we exchange the sequence of this two comment patterns,we will get 02.png.

  was:
select * from kylin_sales /*comments--comments*/

we should remove comments like '/*comments*/' first,then remove comments like 
'--comments' , or the sql above will have an exception.
please refer to 01.png.

if we exchange the sequence of this two comment patterns,we will get 02.png.


> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /\*comments--comments*/
> we should remove comments like '/*comments*/' first,then remove comments like 
> '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3072:

Description: 
select * from kylin_sales /\*comments--comments*/

we should remove comments like '/\*comments*/' first,then remove comments like 
'--comments' , or the sql above will have an exception.
please refer to 01.png.

if we exchange the sequence of this two comment patterns,we will get 02.png.

  was:
select * from kylin_sales /\*comments--comments*/

we should remove comments like '/*comments*/' first,then remove comments like 
'--comments' , or the sql above will have an exception.
please refer to 01.png.

if we exchange the sequence of this two comment patterns,we will get 02.png.


> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /\*comments--comments*/
> we should remove comments like '/\*comments*/' first,then remove comments 
> like '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3075) Improvement kylin.sh, remove duplicate dependency check, let kylin start faster.

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI commented on KYLIN-3075:
-

What's the value here? Can you elaborate on it?

> Improvement kylin.sh, remove duplicate dependency check, let kylin start 
> faster.
> 
>
> Key: KYLIN-3075
> URL: https://issues.apache.org/jira/browse/KYLIN-3075
> Project: Kylin
>  Issue Type: Improvement
>Reporter: peng.jianhua
>Assignee: peng.jianhua
>Priority: Minor
> Attachments: 
> 0001-KYLIN-3075-Improvement-kylin.sh-remove-duplicate-dep.patch
>
>
> Improvement kylin.sh, remove duplicate dependency check, let kylin start 
> faster.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Resolved] (KYLIN-3019) The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should have the same hint

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI resolved KYLIN-3019.
-
Resolution: Fixed

> The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should 
> have the same hint
> 
>
> Key: KYLIN-3019
> URL: https://issues.apache.org/jira/browse/KYLIN-3019
> Project: Kylin
>  Issue Type: Improvement
>Reporter: peng.jianhua
>Assignee: peng.jianhua
>Priority: Minor
> Fix For: v2.3.0
>
> Attachments: 
> 0001-KYLIN-3019-The-pop-up-window-of-Calculate-Cardinalit.patch, 01.PNG, 
> 02.PNG
>
>
> 01.png is when we click the 'Calculate Cardinality' button,it shows nothing.
> 02.png is when we click the 'Load Hive Table' button,it shows the hint of 
> that we will choose the 'Default' database when there is no database input.
> In my opinion,they should keep the same.
> And we also need the hint when we input in the pop-up window of 'Calculate 
> Cardinality' .



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3070) Add a config property for flat table storage format

2017-11-30 Thread Vsevolod Ostapenko (JIRA)

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

Vsevolod Ostapenko commented on KYLIN-3070:
---

I don't mind to work on this issue, but I'm new to the process and existing 
instructions on setting up dev environment seem a bit outdated.
If there are updated version for a non-sandbox HDP 2.5.x install and Kylin 
2.2.x, I'd like to have that in order to have my environment setup correctly.

> Add a config property for flat table storage format
> ---
>
> Key: KYLIN-3070
> URL: https://issues.apache.org/jira/browse/KYLIN-3070
> Project: Kylin
>  Issue Type: Bug
>  Components: Job Engine
>Affects Versions: v2.2.0
> Environment: HDP 2.5.6, Kylin 2.2.0
>Reporter: Vsevolod Ostapenko
>Assignee: Dong Li
>Priority: Minor
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> Flat table storage format is currently hard-coded as SEQUENCEFILE in the 
> core-job/src/main/java/org/apache/kylin/job/JoinedFlatTable.java
> That prevents using Impala as a SQL engine while using beeline CLI (via 
> custom JDBC URL), as Impala cannot write sequence files.
> Adding a parameter to kylin.properties to override the default setting would 
> address the issue.
> Removing a hard-coded value for storage format might be good idea in and on 
> itself.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3068) HiveColumnCardinalityJob.java is using deprecated parameter name for HDFS block size

2017-11-30 Thread Vsevolod Ostapenko (JIRA)

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

Vsevolod Ostapenko commented on KYLIN-3068:
---

I don't mind trying to contribute back to the project, but I find existing 
instructions on setting up dev environment somewhat outdated.
Since Kylin 2.2.x now pulls and packages Spark 2.1.1 as part of the 
distribution tar, do I really need to have spark 1.6.x configured to run the 
tests?
Is there an updated/yet unpublished dev env setup instruction?


> HiveColumnCardinalityJob.java is using deprecated parameter name for HDFS 
> block size
> 
>
> Key: KYLIN-3068
> URL: https://issues.apache.org/jira/browse/KYLIN-3068
> Project: Kylin
>  Issue Type: Bug
>  Components: Job Engine
>Affects Versions: v2.2.0
> Environment: HDP 2.5.3, Kylin 2.2.0
>Reporter: Vsevolod Ostapenko
>Assignee: Dong Li
>Priority: Minor
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> While setting MR job configuration HiveColumnCardinalityJob.java uses 
> deprecated parameter name for HDFS block size.
> Since Hadoop 2.5 (at least) recommended parameter name is dfs.blocksize, 
> while the existing code (2.2.0 and earlier) uses dfs.block.size (please refer 
> to the list of Hadoop deprecated parameters - 
> http://hadoop.apache.org/docs/r2.7.3/hadoop-project-dist/hadoop-common/DeprecatedProperties.html)



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Assigned] (KYLIN-3067) Provide web interface for dimension capping feature

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI reassigned KYLIN-3067:
---

Assignee: Zhixiong Chen  (was: Zhong,Jason)

> Provide web interface for dimension capping feature
> ---
>
> Key: KYLIN-3067
> URL: https://issues.apache.org/jira/browse/KYLIN-3067
> Project: Kylin
>  Issue Type: Improvement
>  Components: Web 
>Reporter: Roger Shi
>Assignee: Zhixiong Chen
>
> As [KYLIN-2363] has provided the dimension capping feature in the backend, we 
> need a proper web interface.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3019) The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should have the same hint

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3019:

Attachment: (was: 
0001-KYLIN-3019-The-pop-up-window-of-Calculate-Cardinalit.patch)

> The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should 
> have the same hint
> 
>
> Key: KYLIN-3019
> URL: https://issues.apache.org/jira/browse/KYLIN-3019
> Project: Kylin
>  Issue Type: Improvement
>Reporter: peng.jianhua
>Assignee: peng.jianhua
>Priority: Minor
> Fix For: v2.3.0
>
> Attachments: 
> 0001-KYLIN-3019-The-pop-up-window-of-Calculate-Cardinalit.patch, 01.PNG, 
> 02.PNG
>
>
> 01.png is when we click the 'Calculate Cardinality' button,it shows nothing.
> 02.png is when we click the 'Load Hive Table' button,it shows the hint of 
> that we will choose the 'Default' database when there is no database input.
> In my opinion,they should keep the same.
> And we also need the hint when we input in the pop-up window of 'Calculate 
> Cardinality' .



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (KYLIN-3069) Add proper time zone support to the WebUI instead of GMT/PST kludge

2017-11-30 Thread Shaofeng SHI (JIRA)

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

Shaofeng SHI commented on KYLIN-3069:
-

Vsevolod, this is a good suggestion; Would you like to contribute a patch to 
Kylin, as you already made the investigation? Thanks!

> Add proper time zone support to the WebUI instead of GMT/PST kludge
> ---
>
> Key: KYLIN-3069
> URL: https://issues.apache.org/jira/browse/KYLIN-3069
> Project: Kylin
>  Issue Type: Bug
>  Components: Web 
>Affects Versions: v2.2.0
> Environment: HDP 2.5.3, Kylin 2.2.0
>Reporter: Vsevolod Ostapenko
>Assignee: Zhixiong Chen
>Priority: Minor
>   Original Estimate: 168h
>  Remaining Estimate: 168h
>
> Time zone handling logic in the WebUI is a kludge, coded to parse only 
> "GMT-N" time zone specifications and defaulting to PST, if parsing is not 
> successful (kylin/webapp/app/js/filters/filter.js)
> Integrating moment and moment time zone (http://momentjs.com/timezone/docs/) 
> into the product, would allow correct time zone handling.
> For the users who happen to reside in the geographical locations that do 
> observe day light savings time, usage of GMT-N format is very inconvenient 
> and info reported by the UI in various places is perplexing.
> Needless to say that the GMT moniker itself is long deprecated.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3072:

Description: 
select * from kylin_sales /*comments--comments*/

we should remove comments like '/*comments*/' first,then remove comments like 
'--comments' , or the sql above will have an exception.
please refer to 01.png.

if we exchange the sequence of this two comment patterns,we will get 02.png.

  was:
select * from kylin_sales /*comments--comments*/

we should remove comments like /*comments*/ first,or the sql above will have an 
error.


> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /*comments--comments*/
> we should remove comments like '/*comments*/' first,then remove comments like 
> '--comments' , or the sql above will have an exception.
> please refer to 01.png.
> if we exchange the sequence of this two comment patterns,we will get 02.png.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3072:

Attachment: 01.png
02.PNG

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch, 01.png, 
> 02.PNG
>
>
> select * from kylin_sales /*comments--comments*/
> we should remove comments like /*comments*/ first,or the sql above will have 
> an error.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3072:

Description: 
select * from kylin_sales /*comments--comments*/

we should remove comments like /*comments*/ first,or the sql above will have an 
error.

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch
>
>
> select * from kylin_sales /*comments--comments*/
> we should remove comments like /*comments*/ first,or the sql above will have 
> an error.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3072:

Attachment: 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch

> Fix the regular expression in function 'removeCommentInSql'
> ---
>
> Key: KYLIN-3072
> URL: https://issues.apache.org/jira/browse/KYLIN-3072
> Project: Kylin
>  Issue Type: Bug
>  Components: Query Engine
>Reporter: peng.jianhua
>Assignee: peng.jianhua
> Attachments: 
> 0001-KYLIN-3072-Fix-the-regular-expression-in-function-re.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3019) The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should have the same hint

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3019:

Attachment: (was: 
0001-KYLIN-3019-The-pop-up-window-of-Calculate-Cardinalit.patch)

> The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should 
> have the same hint
> 
>
> Key: KYLIN-3019
> URL: https://issues.apache.org/jira/browse/KYLIN-3019
> Project: Kylin
>  Issue Type: Improvement
>Reporter: peng.jianhua
>Assignee: peng.jianhua
>Priority: Minor
> Fix For: v2.3.0
>
> Attachments: 
> 0001-KYLIN-3019-The-pop-up-window-of-Calculate-Cardinalit.patch, 01.PNG, 
> 02.PNG
>
>
> 01.png is when we click the 'Calculate Cardinality' button,it shows nothing.
> 02.png is when we click the 'Load Hive Table' button,it shows the hint of 
> that we will choose the 'Default' database when there is no database input.
> In my opinion,they should keep the same.
> And we also need the hint when we input in the pop-up window of 'Calculate 
> Cardinality' .



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3019) The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should have the same hint

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3019:

Attachment: 0001-KYLIN-3019-The-pop-up-window-of-Calculate-Cardinalit.patch

> The pop-up window of 'Calculate Cardinality' and 'Load Hive Table' should 
> have the same hint
> 
>
> Key: KYLIN-3019
> URL: https://issues.apache.org/jira/browse/KYLIN-3019
> Project: Kylin
>  Issue Type: Improvement
>Reporter: peng.jianhua
>Assignee: peng.jianhua
>Priority: Minor
> Fix For: v2.3.0
>
> Attachments: 
> 0001-KYLIN-3019-The-pop-up-window-of-Calculate-Cardinalit.patch, 01.PNG, 
> 02.PNG
>
>
> 01.png is when we click the 'Calculate Cardinality' button,it shows nothing.
> 02.png is when we click the 'Load Hive Table' button,it shows the hint of 
> that we will choose the 'Default' database when there is no database input.
> In my opinion,they should keep the same.
> And we also need the hint when we input in the pop-up window of 'Calculate 
> Cardinality' .



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Updated] (KYLIN-3073) When you save a new query condition, the 'Saved Queries' tab page will not be automatically refreshed.

2017-11-30 Thread peng.jianhua (JIRA)

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

peng.jianhua updated KYLIN-3073:

Attachment: 0001-KYLIN-3073-When-you-save-a-new-query-condition-the-S.patch

> When you save a new query condition, the 'Saved Queries' tab page will not be 
> automatically refreshed.
> --
>
> Key: KYLIN-3073
> URL: https://issues.apache.org/jira/browse/KYLIN-3073
> Project: Kylin
>  Issue Type: Bug
>  Components: Web 
>Affects Versions: v2.3.0
>Reporter: peng.jianhua
>Assignee: peng.jianhua
>  Labels: patch
> Attachments: 
> 0001-KYLIN-3073-When-you-save-a-new-query-condition-the-S.patch, 
> add_a_new_record.png, init_status.png, manual_refresh.png, 
> saved_queries_not_refresh.png
>
>
> When you save a new query condition, the 'Saved Queries' tab will not be 
> automatically refreshed.
> 1. Open the 'Insight' page;
> 2. Choose the 'Saved Queries' tab page, you will see the current page is like 
> [^init_status.png];
> 3. Resubmit anyone query, then save it, just like [^add_a_new_record.png];
> 4. When you saved it, you will find the 'Saved Queries' tab page will not be 
> automatically refreshed, refer to [^saved_queries_not_refresh.png];
> 5. You should manually refresh it by clicking the 'Saved Queries' tab, refer 
> to [^manual_refresh.png].
> I check the code and find the reason, when closing the 'Save Query' dialog, 
> the code doesn't trigger refreshing 'Saved Queries' tab.
> I have repaired this code, please check the patch, thanks!



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Created] (KYLIN-3074) Change cube access to project access in ExternalAclProvider.java

2017-11-30 Thread peng.jianhua (JIRA)
peng.jianhua created KYLIN-3074:
---

 Summary: Change cube access to project access in 
ExternalAclProvider.java
 Key: KYLIN-3074
 URL: https://issues.apache.org/jira/browse/KYLIN-3074
 Project: Kylin
  Issue Type: Bug
Reporter: peng.jianhua
Assignee: peng.jianhua
Priority: Minor






--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Created] (KYLIN-3073) When you save a new query condition, the 'Saved Queries' tab page will not be automatically refreshed.

2017-11-30 Thread peng.jianhua (JIRA)
peng.jianhua created KYLIN-3073:
---

 Summary: When you save a new query condition, the 'Saved Queries' 
tab page will not be automatically refreshed.
 Key: KYLIN-3073
 URL: https://issues.apache.org/jira/browse/KYLIN-3073
 Project: Kylin
  Issue Type: Bug
  Components: Web 
Affects Versions: v2.3.0
Reporter: peng.jianhua
Assignee: peng.jianhua
 Attachments: add_a_new_record.png, init_status.png, 
manual_refresh.png, saved_queries_not_refresh.png

When you save a new query condition, the 'Saved Queries' tab will not be 
automatically refreshed.
1. Open the 'Insight' page;
2. Choose the 'Saved Queries' tab page, you will see the current page is like 
[^init_status.png];
3. Resubmit anyone query, then save it, just like [^add_a_new_record.png];
4. When you saved it, you will find the 'Saved Queries' tab page will not be 
automatically refreshed, refer to [^saved_queries_not_refresh.png];
5. You should manually refresh it by clicking the 'Saved Queries' tab, refer to 
[^manual_refresh.png].

I check the code and find the reason, when closing the 'Save Query' dialog, the 
code doesn't trigger refreshing 'Saved Queries' tab.
I have repaired this code, please check the patch, thanks!




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Created] (KYLIN-3072) Fix the regular expression in function 'removeCommentInSql'

2017-11-30 Thread peng.jianhua (JIRA)
peng.jianhua created KYLIN-3072:
---

 Summary: Fix the regular expression in function 
'removeCommentInSql'
 Key: KYLIN-3072
 URL: https://issues.apache.org/jira/browse/KYLIN-3072
 Project: Kylin
  Issue Type: Bug
  Components: Query Engine
Reporter: peng.jianhua
Assignee: peng.jianhua






--
This message was sent by Atlassian JIRA
(v6.4.14#64029)