date:20180228

Re: Questions about query: cache, jdbc driver.

2018-02-28 Thread Xixin

Hi Yang,
I met the similar problem
sql:"select distinct uid from table where daytime=? limit 70"
ps.setString(1,"2018-01-03")
store the result to ArrayList, and print the size of ArrayList.
when I set the daytime to different dates, the same result I got.

I changed the sql, just change limit value for example
sql:"select distinct uid from table where daytime=? limit *60*"
then I got the right value the same like query from GUI:count (distinct uid)

is it a bug? or something I query in a wrong way?
Thanks in advance!

--
Sent from: http://apache-kylin.74782.x6.nabble.com/

[jira] [Created] (KYLIN-3269) 把project的权限授权给不存在用户时，出现错误。

2018-02-28 Thread RenZhiMin (JIRA)

RenZhiMin created KYLIN-3269:


 Summary: 把project的权限授权给不存在用户时，出现错误。
 Key: KYLIN-3269
 URL: https://issues.apache.org/jira/browse/KYLIN-3269
 Project: Kylin
  Issue Type: Bug
  Components: Web 
Affects Versions: v2.0.0
Reporter: RenZhiMin


把project的权限授权给不存在用户时，会提示用户不存在，并且project上之前的权限都丢失了！



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

Re: [VOTE] Release apache-kylin-2.3.0 (RC3)

2018-02-28 Thread Jianhua Peng

+1

On 2018/03/01 03:35:31, Billy Liu  wrote: 
> Hi all,
> 
> Now the known issue has been fixed, and all unit tests and integration
> tests passed. I have created a new build for Apache Kylin 2.3.0,
> release candidate 3.
> 
> Changes highlights:
>[KYLIN-2727] - Introduce cube planner able to select cost-effective
> cuboids to be built by cost-based algorithms
>[KYLIN-2726] - Introduce a dashboard for showing kylin service
> related metrics, like query count, query latency, job count, etc
>[KYLIN-3125] - Support SparkSql in Cube building step âCreate
> Intermediate Flat Hive Tableâ
>[KYLIN-3044] - Support SQL Server as data source
>[KYLIN-2960] - Support user/group and role authentication for LDAP
> 
> And more than 240 issues fixed.
> 
> Thanks to everyone who has contributed to this release.
> Here's release notes:
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12341685=12316121
> 
> The commit to be voted upon:
> https://github.com/apache/kylin/commit/1179ee72d0a2a2c629a649751ef43ae9e82dc87a
> 
> Its hash is 1179ee72d0a2a2c629a649751ef43ae9e82dc87a
> 
> The artifacts to be voted on are located here:
> https://dist.apache.org/repos/dist/dev/kylin/apache-kylin-2.3.0-rc3/
> 
> The hashes of the artifacts are as follows:
> apache-kylin-2.3.0-src.tar.gz.md5 138a96260dc2c8b8f9947b0bcbb11152
> apache-kylin-2.3.0-src.tar.gz.sha1 b305bb91144d2d1470bd1a8578a111732d04c6f8
> 
> A staged Maven repository is available for review at:
> https://repository.apache.org/content/repositories/orgapachekylin-1048/
> 
> Release artifacts are signed with the following key:
> https://people.apache.org/keys/committer/billyliu.asc
> 
> Please vote on releasing this package as Apache Kylin 2.3.0
> 
> The vote is open for the next 72 hours and passes if a majority of at
> least three +1 PPMC votes are cast.
> 
> [ ] +1 Release this package as Apache Kylin 2.3.0
> [ ]  0  I don't feel strongly about it, but I'm okay with the release
> [ ] -1  Do not release this package because...
> 
> Here is my vote:
> 
> +1 (binding)
> 
> With Warm regards
> 
> Billy Liu
>

[VOTE] Release apache-kylin-2.3.0 (RC3)

2018-02-28 Thread Billy Liu

Hi all,

Now the known issue has been fixed, and all unit tests and integration
tests passed. I have created a new build for Apache Kylin 2.3.0,
release candidate 3.

Changes highlights:
   [KYLIN-2727] - Introduce cube planner able to select cost-effective
cuboids to be built by cost-based algorithms
   [KYLIN-2726] - Introduce a dashboard for showing kylin service
related metrics, like query count, query latency, job count, etc
   [KYLIN-3125] - Support SparkSql in Cube building step “Create
Intermediate Flat Hive Table”
   [KYLIN-3044] - Support SQL Server as data source
   [KYLIN-2960] - Support user/group and role authentication for LDAP

And more than 240 issues fixed.

Thanks to everyone who has contributed to this release.
Here's release notes:
https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12341685=12316121

The commit to be voted upon:
https://github.com/apache/kylin/commit/1179ee72d0a2a2c629a649751ef43ae9e82dc87a

Its hash is 1179ee72d0a2a2c629a649751ef43ae9e82dc87a

The artifacts to be voted on are located here:
https://dist.apache.org/repos/dist/dev/kylin/apache-kylin-2.3.0-rc3/

The hashes of the artifacts are as follows:
apache-kylin-2.3.0-src.tar.gz.md5 138a96260dc2c8b8f9947b0bcbb11152
apache-kylin-2.3.0-src.tar.gz.sha1 b305bb91144d2d1470bd1a8578a111732d04c6f8

A staged Maven repository is available for review at:
https://repository.apache.org/content/repositories/orgapachekylin-1048/

Release artifacts are signed with the following key:
https://people.apache.org/keys/committer/billyliu.asc

Please vote on releasing this package as Apache Kylin 2.3.0

The vote is open for the next 72 hours and passes if a majority of at
least three +1 PPMC votes are cast.

[ ] +1 Release this package as Apache Kylin 2.3.0
[ ]  0  I don't feel strongly about it, but I'm okay with the release
[ ] -1  Do not release this package because...

Here is my vote:

+1 (binding)

With Warm regards

Billy Liu

[GitHub] coveralls commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox

coveralls commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter 
condition for jobs UI part
URL: https://github.com/apache/kylin/pull/109#issuecomment-369454934
 
 
   ## Pull Request Test Coverage Report for [Build 
3009](https://coveralls.io/builds/15751349)
   
   * **0** of **0**  **(NaN%)** changed or added relevant lines in **0** files 
are covered.
   * **15** unchanged lines in **4** files lost coverage.
   * Overall coverage decreased (**-0.03%**) to **23.429%**
   
   ---
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/15751349/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L553)
 | 1 | 78.12% |
   | 
[core-common/src/main/java/org/apache/kylin/common/KylinConfigCannotInitException.java](https://coveralls.io/builds/15751349/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2FKylinConfigCannotInitException.java#L24)
 | 2 | 0.0% |
   | 
[core-common/src/main/java/org/apache/kylin/common/KylinConfig.java](https://coveralls.io/builds/15751349/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2FKylinConfig.java#L278)
 | 4 | 33.19% |
   | 
[core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java](https://coveralls.io/builds/15751349/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2FKylinConfigBase.java#L67)
 | 8 | 16.22% |
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/15751349/badge)](https://coveralls.io/builds/15751349)
 |
   | :-- | --: |
   | Change from base [Build 3006](https://coveralls.io/builds/15741228): |  
-0.03% |
   | Covered Lines: | 13997 |
   | Relevant Lines: | 59742 |
   
   ---
   # ?  - [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] codecov-io commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox

codecov-io commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter 
condition for jobs UI part
URL: https://github.com/apache/kylin/pull/109#issuecomment-369454646
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/109?src=pr=h1) Report
   > Merging [#109](https://codecov.io/gh/apache/kylin/pull/109?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/kylin/commit/76934db8fcddabf437f490e77c8f06c9a4a3db86?src=pr=desc)
 will **decrease** coverage by `0.02%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/109/graphs/tree.svg?token=JawVgbgsVo=pr=650=150)](https://codecov.io/gh/apache/kylin/pull/109?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #109  +/-   ##
   
   - Coverage 21.47%   21.45%   -0.03% 
   + Complexity 3822 3817   -5 
   
 Files   992  992  
 Lines 5974259742  
 Branches   8639 8639  
   
   - Hits  1283012815  -15 
   - Misses4573245747  +15 
 Partials   1180 1180
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/109?src=pr=tree) | Coverage ? 
| Complexity ? | |
   |---|---|---|---|
   | 
[...e/kylin/common/KylinConfigCannotInitException.java](https://codecov.io/gh/apache/kylin/pull/109/diff?src=pr=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZ0Nhbm5vdEluaXRFeGNlcHRpb24uamF2YQ==)
 | `0% <0%> (-100%)` | `0% <0%> (-1%)` | |
   | 
[...main/java/org/apache/kylin/common/KylinConfig.java](https://codecov.io/gh/apache/kylin/pull/109/diff?src=pr=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZy5qYXZh)
 | `30.67% <0%> (-1.69%)` | `20% <0%> (-1%)` | |
   | 
[.../java/org/apache/kylin/common/KylinConfigBase.java](https://codecov.io/gh/apache/kylin/pull/109/diff?src=pr=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZ0Jhc2UuamF2YQ==)
 | `14.98% <0%> (-1.24%)` | `35% <0%> (-2%)` | |
   | 
[...a/org/apache/kylin/dict/Number2BytesConverter.java](https://codecov.io/gh/apache/kylin/pull/109/diff?src=pr=tree#diff-Y29yZS1kaWN0aW9uYXJ5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9kaWN0L051bWJlcjJCeXRlc0NvbnZlcnRlci5qYXZh)
 | `81.74% <0%> (-0.8%)` | `17% <0%> (-1%)` | |
   | 
[...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/109/diff?src=pr=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh)
 | `69.6% <0%> (-0.61%)` | `7% <0%> (?)` | |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/109?src=pr=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/109?src=pr=footer). Last 
update 
[76934db...31cc006](https://codecov.io/gh/apache/kylin/pull/109?src=pr=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

[GitHub] coveralls commented on issue #107: kylin-3119

2018-02-28 Thread GitBox

coveralls commented on issue #107: kylin-3119
URL: https://github.com/apache/kylin/pull/107#issuecomment-368738219
 
 
   ## Pull Request Test Coverage Report for [Build 
3008](https://coveralls.io/builds/15751320)
   
   * **16** of **16**  **(100.0%)** changed or added relevant lines in **1** 
file are covered.
   * **22** unchanged lines in **6** files lost coverage.
   * Overall coverage decreased (**-0.02%**) to **23.438%**
   
   ---
   
   
   |  Files with Coverage Reduction | New Missed Lines | % |
   | :-|--|--: |
   | 
[core-metadata/src/main/java/org/apache/kylin/source/datagen/ColumnGenerator.java](https://coveralls.io/builds/15751320/source?filename=core-metadata%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fsource%2Fdatagen%2FColumnGenerator.java#L317)
 | 1 | 81.08% |
   | 
[core-common/src/main/java/org/apache/kylin/common/KylinConfigCannotInitException.java](https://coveralls.io/builds/15751320/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2FKylinConfigCannotInitException.java#L24)
 | 2 | 0.0% |
   | 
[core-metadata/src/main/java/org/apache/kylin/metadata/cachesync/Broadcaster.java](https://coveralls.io/builds/15751320/source?filename=core-metadata%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fmetadata%2Fcachesync%2FBroadcaster.java#L145)
 | 2 | 60.2% |
   | 
[core-common/src/main/java/org/apache/kylin/common/KylinConfig.java](https://coveralls.io/builds/15751320/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2FKylinConfig.java#L278)
 | 4 | 33.19% |
   | 
[core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://coveralls.io/builds/15751320/source?filename=core-cube%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcube%2Finmemcubing%2FMemDiskStore.java#L439)
 | 5 | 77.81% |
   | 
[core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java](https://coveralls.io/builds/15751320/source?filename=core-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fkylin%2Fcommon%2FKylinConfigBase.java#L67)
 | 8 | 16.22% |
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/15751320/badge)](https://coveralls.io/builds/15751320)
 |
   | :-- | --: |
   | Change from base [Build 3006](https://coveralls.io/builds/15741228): |  
-0.02% |
   | Covered Lines: | 14004 |
   | Relevant Lines: | 59749 |
   
   ---
   # ?  - [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] codecov-io commented on issue #107: kylin-3119

2018-02-28 Thread GitBox

codecov-io commented on issue #107: kylin-3119
URL: https://github.com/apache/kylin/pull/107#issuecomment-368738780
 
 
   # [Codecov](https://codecov.io/gh/apache/kylin/pull/107?src=pr=h1) Report
   > Merging [#107](https://codecov.io/gh/apache/kylin/pull/107?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/kylin/commit/76934db8fcddabf437f490e77c8f06c9a4a3db86?src=pr=desc)
 will **decrease** coverage by `0.01%`.
   > The diff coverage is `75%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/kylin/pull/107/graphs/tree.svg?src=pr=JawVgbgsVo=650=150)](https://codecov.io/gh/apache/kylin/pull/107?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #107  +/-   ##
   
   - Coverage 21.47%   21.46%   -0.02% 
 Complexity 3822 3822  
   
 Files   992  992  
 Lines 5974259749   +7 
 Branches   8639 8640   +1 
   
   - Hits  1283012823   -7 
   - Misses4573245747  +15 
   + Partials   1180 1179   -1
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/kylin/pull/107?src=pr=tree) | Coverage ? 
| Complexity ? | |
   |---|---|---|---|
   | 
[...in/java/org/apache/kylin/query/util/QueryUtil.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-cXVlcnkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL3F1ZXJ5L3V0aWwvUXVlcnlVdGlsLmphdmE=)
 | `55.88% <75%> (+6.7%)` | `15 <0> (+2)` | :arrow_up: |
   | 
[...e/kylin/common/KylinConfigCannotInitException.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZ0Nhbm5vdEluaXRFeGNlcHRpb24uamF2YQ==)
 | `0% <0%> (-100%)` | `0% <0%> (-1%)` | |
   | 
[...main/java/org/apache/kylin/common/KylinConfig.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZy5qYXZh)
 | `30.67% <0%> (-1.69%)` | `20% <0%> (-1%)` | |
   | 
[...g/apache/kylin/source/datagen/ColumnGenerator.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1tZXRhZGF0YS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vc291cmNlL2RhdGFnZW4vQ29sdW1uR2VuZXJhdG9yLmphdmE=)
 | `70.94% <0%> (-1.36%)` | `8% <0%> (?)` | |
   | 
[.../java/org/apache/kylin/common/KylinConfigBase.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2NvbW1vbi9LeWxpbkNvbmZpZ0Jhc2UuamF2YQ==)
 | `14.98% <0%> (-1.24%)` | `35% <0%> (-2%)` | |
   | 
[...g/apache/kylin/metadata/cachesync/Broadcaster.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1tZXRhZGF0YS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3lsaW4vbWV0YWRhdGEvY2FjaGVzeW5jL0Jyb2FkY2FzdGVyLmphdmE=)
 | `54.72% <0%> (-1%)` | `21% <0%> (?)` | |
   | 
[...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2lubWVtY3ViaW5nL01lbURpc2tTdG9yZS5qYXZh)
 | `69.3% <0%> (-0.92%)` | `7% <0%> (?)` | |
   | 
[.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1jdWJlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9reWxpbi9jdWJlL2N1Ym9pZC9UcmVlQ3Vib2lkU2NoZWR1bGVyLmphdmE=)
 | `65.35% <0%> (+0.78%)` | `0% <0%> (?)` | :arrow_down: |
   | 
[...apache/kylin/job/execution/AbstractExecutable.java](https://codecov.io/gh/apache/kylin/pull/107/diff?src=pr=tree#diff-Y29yZS1qb2Ivc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2t5bGluL2pvYi9leGVjdXRpb24vQWJzdHJhY3RFeGVjdXRhYmxlLmphdmE=)
 | `53.14% <0%> (+1.44%)` | `48% <0%> (+2%)` | :arrow_up: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/kylin/pull/107?src=pr=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/107?src=pr=footer). Last 
update 
[76934db...e01648b](https://codecov.io/gh/apache/kylin/pull/107?src=pr=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

[GitHub] chenzhx closed pull request #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox

chenzhx closed pull request #109: KYLIN-3265 Add jobSearchMode as a filter 
condition for jobs UI part
URL: https://github.com/apache/kylin/pull/109
 
 
   

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/webapp/app/js/controllers/job.js b/webapp/app/js/controllers/job.js
index 51de7874e0..130f14dffc 100644
--- a/webapp/app/js/controllers/job.js
+++ b/webapp/app/js/controllers/job.js
@@ -28,6 +28,7 @@ KylinApp
 //$scope.projects = [];
 $scope.action = {};
 $scope.timeFilter = 
jobConfig.timeFilter[JobList.jobFilter.timeFilterId];
+$scope.searchMode = 
jobConfig.searchMode[JobList.jobFilter.searchModeId];
 if ($routeParams.jobTimeFilter) {
 $scope.timeFilter = 
jobConfig.timeFilter[$routeParams.jobTimeFilter];
 }
@@ -82,6 +83,7 @@ KylinApp
 $scope.cubeName=$scope.cubeName == ""?null:$scope.cubeName;
 JobList.jobFilter.cubeName = $scope.cubeName;
 JobList.jobFilter.timeFilterId = $scope.timeFilter.value;
+JobList.jobFilter.searchModeId = _.indexOf(jobConfig.searchMode, 
$scope.searchMode);
 JobList.jobFilter.statusIds = statusIds;
 
 var jobRequest = {
@@ -90,7 +92,8 @@ KylinApp
 status: statusIds,
 offset: offset,
 limit: limit,
-timeFilter: $scope.timeFilter.value
+timeFilter: $scope.timeFilter.value,
+jobSearchMode: $scope.searchMode.value
 };
 $scope.state.loading = true;
 
diff --git a/webapp/app/js/model/jobConfig.js b/webapp/app/js/model/jobConfig.js
index 4548366579..19ae882c98 100644
--- a/webapp/app/js/model/jobConfig.js
+++ b/webapp/app/js/model/jobConfig.js
@@ -40,6 +40,11 @@ KylinApp.constant('jobConfig', {
 {attr: 'last_modified', name: 'Last Modified Time'},
 {attr: 'duration', name: 'Duration'}
   ],
+  searchMode: [
+{name: 'CUBING', value: 'CUBING_ONLY'},
+{name: 'CHECK POINT', value: 'CHECKPOINT_ONLY'},
+{name: 'ALL', value: 'ALL'}
+  ],
   queryitems: [
   {attr: 'server', name: 'Server'},
   {attr: 'user', name: 'User'},
diff --git a/webapp/app/js/model/jobListModel.js 
b/webapp/app/js/model/jobListModel.js
index 45948d63e3..197532c076 100755
--- a/webapp/app/js/model/jobListModel.js
+++ b/webapp/app/js/model/jobListModel.js
@@ -26,6 +26,7 @@ KylinApp.service('JobList',function(JobService,$q){
 this.jobFilter = {
 cubeName : null,
 timeFilterId : 1,
+searchModeId: 2,
 statusIds: []
 };
 
@@ -33,6 +34,7 @@ KylinApp.service('JobList',function(JobService,$q){
 this.jobFilter = {
   cubeName : null,
   timeFilterId : 1,
+  searchModeId: 2,
   statusIds: []
 };
 };
diff --git a/webapp/app/partials/jobs/jobList.html 
b/webapp/app/partials/jobs/jobList.html
index 1ff6ea4f2f..ae05590669 100644
--- a/webapp/app/partials/jobs/jobList.html
+++ b/webapp/app/partials/jobs/jobList.html
@@ -45,6 +45,9 @@
   
   
+  
+

[GitHub] asfgit commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox

asfgit commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter 
condition for jobs UI part
URL: https://github.com/apache/kylin/pull/109#issuecomment-369451560
 
 
   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

Re: Questions about query: cache, jdbc driver.

[jira] [Created] (KYLIN-3269) 把project的权限授权给不存在用户时，出现错误。

Re: [VOTE] Release apache-kylin-2.3.0 (RC3)

[VOTE] Release apache-kylin-2.3.0 (RC3)

[GitHub] coveralls commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

[GitHub] codecov-io commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

[GitHub] coveralls commented on issue #107: kylin-3119

[GitHub] codecov-io commented on issue #107: kylin-3119

[GitHub] chenzhx closed pull request #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

[GitHub] asfgit commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

10 matches

Site Navigation

Mail list logo

Footer information