[GitHub] eagle issue #965: EAGLE-1059: fix a bug in PolicyResource.java

2017-08-04 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/965
  
@haoch No. I put this feature in v0.5.1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #965: EAGLE-1059: fix a bug in PolicyResource.java

2017-07-31 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/965

EAGLE-1059: fix a bug in PolicyResource.java

https://issues.apache.org/jira/browse/EAGLE-1059

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1059

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/965.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #965


commit e40da89cc4f040b78b96534927d3297507342224
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-07-31T09:22:18Z

fix a bug in PolicyResource.java




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #963: Eagle-1046 Eagle supports policies import to a new site fr...

2017-07-11 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/963
  
+1. A very useful feature for policy management on multiple sites. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #960: EAGLE-1056: fix a link bug in the email template

2017-06-23 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/960

EAGLE-1056: fix a link bug in the email template

https://issues.apache.org/jira/browse/EAGLE-1056

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1056

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/960.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #960


commit 5069cdae34722eeb3d8d51787d62ceb4ebffabca
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-06-23T08:30:04Z

fix alert link bug




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #959: EAGLE-1051: update removePolicy

2017-06-21 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/959

EAGLE-1051: update removePolicy

https://issues.apache.org/jira/browse/EAGLE-1051

There is no need to do this check. As eagle has a table 
`policy_publishment` which has defined the relationship between policies and 
publishments. 

{code}
CREATE TABLE IF NOT EXISTS policy_publishment (
  policyId VARCHAR(50),
  publishmentName VARCHAR(50),
  PRIMARY KEY(policyId, publishmentName),
  CONSTRAINT `policy_id_fk` FOREIGN KEY (`policyId`) REFERENCES 
`policy_definition` (`id`) ON DELETE CASCADE ON UPDATE CASCADE,
  CONSTRAINT `publishment_id_fk` FOREIGN KEY (`publishmentName`) REFERENCES 
`publishment` (`id`) ON DELETE CASCADE ON UPDATE CASCADE
);
{code}



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1051

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/959.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #959


commit 118228d83fba85b467c73b30f5f1dc3dac7f2514
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-06-20T12:58:06Z

update removePolicy




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #955: EAGLE-1048 Delete an alert publisher on Eagle UI

2017-06-20 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/955
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #954: EAGLE-1047 View all alert publishers on Eagle UI

2017-06-20 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/954
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #953: EAGLE-1049 Support metric filter in the metric preview pag...

2017-06-20 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/953
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #952: EAGLE-1044: Support policy Import using a policy pr...

2017-06-19 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/952

EAGLE-1044: Support policy Import using a policy prototype

https://issues.apache.org/jira/browse/EAGLE-1044'

Provided APIs:
*  load policies to new site "sandbox" from policy prototypes by `POST 
/rest/policyProto/loadToSite/sandbox`
* create a new policy prototype with an existing policy by `POST 
/rest/policyProto/saveAsProto`
* update or create a policy prototype by by `POST /rest/policyProto`
* get all policy prototypes by `GET /rest/policyProto`
* delete a policy prototype by `DELETE /rest/policyProto/{uuid}`


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1044

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/952.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #952


commit b7ff02e14b1de8ca9a92324e08307c3823a28d8d
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-06-20T05:36:30Z

add policyProto resource




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #950: EAGLE-1043 display date when series cross the date

2017-06-14 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/950
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #949: EAGLE-1039 recheck when policy init and keep origin settin...

2017-06-14 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/949
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #948: EAGLE-1042 support preview chart title

2017-06-14 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/948
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #947: EAGLE-1041: Support policy processing pipeline

2017-06-13 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/947

EAGLE-1041: Support policy processing pipeline

https://issues.apache.org/jira/browse/EAGLE-1041

Two updates:
* if an inputStream is an intermediate stream (defined by select clause 
statement), then remove it from inputStream list and outputStream list
* if an inputStream is an intermediate stream, remove its PartitionSpec 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1041

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/947.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #947


commit ea47b849700a89e15bc6a2e6b28317ee7a50b315
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-06-13T12:22:15Z

update alertEditCtrl.js




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #946: [MINOR]: update eagle-assembly/src/main/doc/README....

2017-06-08 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/946

[MINOR]: update eagle-assembly/src/main/doc/README.md



Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the PR title is formatted like:
   `[EAGLE-] Description of pull request`
 - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle minor

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/946.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #946


commit 0272613bbb270aa9b5dc964b1f82818097db93d2
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-06-08T06:33:24Z

update eagle-assembly/src/main/doc/README.md




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #945: EAGLE-1037 Add alertDeduplication configurations on Eagle ...

2017-06-07 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/945
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #944: EAGLE-1038: Support alertDuplication customization ...

2017-06-07 Thread qingwen220
Github user qingwen220 commented on a diff in the pull request:

https://github.com/apache/eagle/pull/944#discussion_r120550375
  
--- Diff: 
eagle-core/eagle-alert-parent/eagle-alert/alert-common/src/main/java/org/apache/eagle/alert/engine/model/AlertStreamEvent.java
 ---
@@ -39,6 +39,7 @@
 private long createdTime;
 private String category;
 private AlertSeverity severity = AlertSeverity.WARNING;
+private boolean duplicationChecked = false;
--- End diff --

what if the default value changes in different JDK versions? 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #943: [MINOR] Fxi sprk/egle issues in JPM module

2017-06-07 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/943
  
@asdf2014  I do not think these are big issues. 

On the other hand, this changes will have a big impact for users who have 
already installed these features. (old data are missing). 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #944: EAGLE-1038: Support alertDuplication customization ...

2017-06-07 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/944

EAGLE-1038: Support alertDuplication customization for each policy

https://issues.apache.org/jira/browse/EAGLE-1038

* support duplication check for each outputStream of a policy
* compatible with duplication check in old versions (publisher check)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle minor

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/944.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #944


commit 85a9a9257d6a14d782b2fd5e615c92c44116ce0d
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-06-07T06:51:22Z

Support alertDuplication customization for each policy




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #942: Eagle-1032 Add policy duplication settings in the policy d...

2017-06-06 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/942
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #939: EAGLE-1029: fix bug in generateGroupbyMonitorMetadata

2017-05-27 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/939
  
@RalphSu  I have added a unit test. This test verifies the correctness of 
assigning of a policy to a new alert engine topology. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #940: MINOR: enrich MAP_REDUCE_JOB_STREAM

2017-05-26 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/940

MINOR: enrich MAP_REDUCE_JOB_STREAM

Add 5 fields in MAP_REDUCE_JOB_STREAM
* numTotalMaps
* numTotalReduces
* duration
* avgMapTime
* avgReduceTime

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle minor

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/940.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #940


commit 986262d218726aa25854df5bb9ab5f7737b067d4
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-26T13:11:18Z

enrich MAP_REDUCE_JOB_STREAM

commit 023fc8738980db35c73c3dda19387507372b0043
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-26T13:14:03Z

update




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #939: EAGLE-1029: fix bug in generateGroupbyMonitorMetadata

2017-05-26 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/939
  
This bug can be produced as follows

1. create policies with stream1 to use up all the working slot of alert 
engine topology1
2. setup another alert engine topology topology2
3. create a policy with stream1 and trigger the scheduler to schedule
4. check schedule result by GET rest/metadata/schedulestates, and compare 
groupSpecs & alertSpecs



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #939: EAGLE-1029: fix bug in generateGroupbyMonitorMetada...

2017-05-26 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/939

EAGLE-1029: fix bug in generateGroupbyMonitorMetadata

https://issues.apache.org/jira/browse/EAGLE-1029

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1029

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/939.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #939


commit 1596192b901ec141a538b66b6aaa1520455ef9a4
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-26T11:12:17Z

fix bug in generateSpoutMonitorMetadata




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #938: EAGLE-1024: Monitor jobs with high RPC throughput

2017-05-18 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/938

EAGLE-1024: Monitor jobs with high RPC throughput

https://issues.apache.org/jira/browse/EAGLE-1024

* add job RPC data in MAP_REDUCE_JOB_STREAM 
* refactor org.apache.eagle.jpm.analyzer.publisher.EmailPublisher 
* support new config 'application.analyzerReport.alertLevel' to define 
alert level

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1024

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/938.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #938


commit 5beddd6c55512570f20532f15d42d961b442cecf
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-18T15:02:57Z

add JobRpcAnalysisEvaluator

commit 9011e49fbd7d82b83e455dfdceebf1b4d1359eef
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-19T05:29:20Z

refactor EmailPublisher.java




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #937: [MINOR] Sync 0.5 branch with master branch changes

2017-05-18 Thread qingwen220
Github user qingwen220 closed the pull request at:

https://github.com/apache/eagle/pull/937


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #935: EAGLE-1023: Update jmx metric collector scripts

2017-05-18 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/935
  
@jhsenjaliya  updated. Thanks


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #937: [MINOR] Sync 0.5 branch with master branch changes

2017-05-17 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/937

[MINOR] Sync 0.5 branch with master branch changes



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/apache/eagle master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/937.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #937


commit 845f891517a2befccb73267c54fc08f8dc2c811a
Author: zombieJ <smith3...@gmail.com>
Date:   2017-05-12T08:41:44Z

[EAGLE-1020] JPM UI overview page OPS unit error

https://issues.apache.org/jira/browse/EAGLE-1020

Author: zombieJ <smith3...@gmail.com>

Closes #931 from zombieJ/EAGLE-1020.

commit 342df9974c91f845608c9ff3c65b8c1818d56884
Author: zombieJ <smith3...@gmail.com>
Date:   2017-05-15T07:22:21Z

[EAGLE-1021] JPM Overview OPS trend use max agg

https://issues.apache.org/jira/browse/EAGLE-1021

Author: zombieJ <smith3...@gmail.com>

Closes #932 from zombieJ/EAGLE-1021.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #935: EAGLE-1023: Update jmx metric collector scripts

2017-05-17 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/935

EAGLE-1023: Update jmx metric collector scripts

https://issues.apache.org/jira/browse/EAGLE-1023

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1023

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/935.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #935


commit 47e899121bb8fdbbc33b497e92c871ecd5394ffc
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-17T07:36:38Z

update jmx metric collector scripts




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #934: EAGLE-1022: Fix runtime exception when parsing spar...

2017-05-16 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/934

EAGLE-1022: Fix runtime exception when parsing 
spark.yarn.executor.memoryOverhead

https://issues.apache.org/jira/browse/EAGLE-1022

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1022

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/934.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #934


commit 20b16bb23245a792065f3ccd981ef5d14792e2b4
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-17T03:13:09Z

fix runtime exception when parse spark.yarn.executor.memoryOverhead




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #933: EAGLE-1016: fix keyword "group" conflict in JDBC st...

2017-05-15 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/933

EAGLE-1016: fix keyword "group" conflict in JDBC storage

https://issues.apache.org/jira/browse/EAGLE-1016

If you meet the same issue after applying this patch, please drop the old 
table first. 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1016

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/933.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #933


commit f46e217401aae25c77ded605c54a79e96a85afbe
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-15T11:34:50Z

fix keyword group conflict in mysql

commit 252513d6f93faf8896457db4317b9d0f74b66cab
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-15T11:40:37Z

update eagle.conf

commit 27d827d2e967f5cd09837f5a213f360cfcf3cf6e
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-05-15T11:49:01Z

add tags in SystemServiceTopologyAPIEntity.java




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #930: [EAGLE-1017] New further: support for query alert of a spe...

2017-05-14 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/930
  
@6591812  Could you copy your jira ticket link to your pull request 
description, which helps to link the pr to the ticket?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #928: EAGLE-1015: add an interface to add storm configura...

2017-04-27 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/928

EAGLE-1015: add an interface to add storm configuration in an application

https://issues.apache.org/jira/browse/EAGLE-1015

Support to add storm config value of type number or string in an 
application. However, to make storm overrides these custom values,  one rule is 
the configuration must start with 'application.storm.'. For example: 

`application.workers` to override `workers`
`application.nimbus.host` to override `nimbus.host`


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1015

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/928.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #928


commit 58834f640fe0f48f542e985f9411425639dbe47b
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-28T03:21:08Z

Add an interface to add storm configuration in an application




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #926: EAGLE-993: add duplicate removal settings in policy...

2017-04-27 Thread qingwen220
Github user qingwen220 commented on a diff in the pull request:

https://github.com/apache/eagle/pull/926#discussion_r113704536
  
--- Diff: 
eagle-core/eagle-alert-parent/eagle-alert/alert-engine/src/main/java/org/apache/eagle/alert/engine/runner/AlertPublisherBolt.java
 ---
@@ -87,6 +90,12 @@ public void execute(Tuple input) {
 if (logEventEnabled) {
 LOG.info("Alert publish bolt {}/{} with partition {} 
received event: {}", this.getBoltId(), this.context.getThisTaskId(), partition, 
event);
 }
+if (deduplicatorMap != null && 
deduplicatorMap.containsKey(event.getPolicyId())) {
+List eventList = 
deduplicatorMap.get(event.getPolicyId()).dedup(event);
+if (eventList == null || eventList.isEmpty()) {
+return;
--- End diff --

You are right. Thanks


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #927: EAGLE-1014: add exception handling in CorrelationSp...

2017-04-27 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/927

EAGLE-1014: add exception handling in CorrelationSpout.java

https://issues.apache.org/jira/browse/EAGLE-1014

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-1014

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/927.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #927


commit 066efad85599561922aa58435f5c156bae6f3b0c
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-27T12:12:36Z

add exception handling in CorrelationSpout.java

commit 13578251ad907c0f5edf9452a96b5d57b1971028
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-27T12:20:33Z

add a log in CorrelationSpout.java




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #926: EAGLE-993: add duplicate removal settings in policy...

2017-04-20 Thread qingwen220
Github user qingwen220 commented on a diff in the pull request:

https://github.com/apache/eagle/pull/926#discussion_r112595515
  
--- Diff: 
eagle-core/eagle-alert-parent/eagle-alert/alert-engine/src/main/java/org/apache/eagle/alert/engine/publisher/impl/AlertEmailPublisher.java
 ---
@@ -215,7 +214,6 @@ public PublishmentType getPluginType() {
 .name("Email")
 .type(AlertEmailPublisher.class)
 .description("Email alert publisher")
-.field("subject")
--- End diff --

Yes. 

As we already define the email subject in the policy definition. it's a 
little duplicated. How do you think?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #926: EAGLE-993: add duplicate removal settings in policy...

2017-04-20 Thread qingwen220
Github user qingwen220 commented on a diff in the pull request:

https://github.com/apache/eagle/pull/926#discussion_r112595389
  
--- Diff: 
eagle-jpm/eagle-jpm-mr-history/src/test/resources/job_1479206441898_508949_conf.xml
 ---
@@ -1,18 +1,164 @@
-
--- End diff --

My mistake. I will add it back.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #926: EAGLE-993: add duplicate removal settings in policy...

2017-04-20 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/926

EAGLE-993: add duplicate removal settings in policy definition

https://issues.apache.org/jira/browse/EAGLE-993

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-993

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/926.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #926


commit 13dc0ff75ae812aef1e599fca36809db704ee840
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-06T11:34:09Z

refine publishmentType

commit 5953465b12c87f0d0f98a5220da87e35d78a9c2e
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-06T11:35:23Z

update policy deletion

commit 7faf383445626ccc7650cf925f8637c0dbc204ac
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-07T13:50:01Z

add AlertDeduplication

commit 92165c143386274616c8fcb5483c7b1b46fdb42b
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-06T11:34:09Z

refine publishmentType

commit 6db2bdfe49ae63fb85b849198fa38b825b373450
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-06T11:35:23Z

update policy deletion

commit b447b4dde577ca5f2e9df7e41380d270a40008e6
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-07T13:50:01Z

add AlertDeduplication

commit be54624b74eb65f91e9cbd8ebd9bccd19ac30874
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-11T01:28:37Z

Merge branch 'EAGLE-993' of https://github.com/qingwen220/incubator-eagle 
into EAGLE-993




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #923: MINOR: rename 'HbaseServiceInstance' as 'HBaseServi...

2017-04-19 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/923

MINOR:  rename 'HbaseServiceInstance' as 'HBaseServiceInstance'



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle minor

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/923.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #923


commit 20d7a1555f269c1e1728cd843e1369ae453deaa0
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-12T06:46:53Z

refine app config

commit 9a2871a7d2424e790a41833ad5b3eb541452b61c
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-17T09:38:43Z

Merge branch 'master' of https://github.com/qingwen220/incubator-eagle

commit 7e1732e610e66f84199f15d2ae37bd7cb2d510dc
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-18T08:55:16Z

Merge branch 'master' of https://github.com/qingwen220/incubator-eagle

commit 475f6be932b17278560b9a58599ea15e2e8c864e
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-04-19T06:05:03Z

change 'HbaseServiceInstance' to 'HBaseServiceInstance'

commit 2f83af03255c41f0d2de243d46b41ecd94bef920
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-04-19T06:17:01Z

update




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #920: [EAGLE-1009] Fix losing exception issues

2017-04-18 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/920
  
I don't think there are exceptions lost in the code you modified.  All 
these try statements are followed by catch statement which doesn't throw a new 
exception anymore. 

I agree with your idea that DON'T return in finally block





---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #919: EAGLE-1008: java.lang.NullPointerException in JHFEv...

2017-04-18 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/919

EAGLE-1008: java.lang.NullPointerException in JHFEventReaderBase.close

https://issues.apache.org/jira/browse/EAGLE-1008

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle minor

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/919.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #919


commit 20d7a1555f269c1e1728cd843e1369ae453deaa0
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-12T06:46:53Z

refine app config

commit 9a2871a7d2424e790a41833ad5b3eb541452b61c
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-17T09:38:43Z

Merge branch 'master' of https://github.com/qingwen220/incubator-eagle

commit 3099b71e204d8f3c115b82b4e21495e84e932fb0
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-18T06:52:34Z

fix null pointer exception in JobCounters.java




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #910: MINOR: refine app config

2017-04-12 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/910

MINOR: refine app config



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/910.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #910


commit 20d7a1555f269c1e1728cd843e1369ae453deaa0
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-12T06:46:53Z

refine app config




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #902: [EAGLE-985] Upgrade hbase dependency to 1.2.5

2017-04-05 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/902
  
Yes. We expect after 0.5, Eagle can support Kafka 0.10 & storm 0.10. 

There is a concern. We have a coprocessor module in Eagle for HBase 0.98. I 
am not sure if we could support this function after upgrade


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #902: [EAGLE-985] Upgrade hbase dependency to 1.2.5

2017-04-05 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/902
  
@jhsenjaliya @tedyu  After this upgrade, does we still support HBase 0.98?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #903: [EAGLE-986] HBaseStorage unable to deserialize Deserialize...

2017-04-04 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/903
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #900: EAGLE-989: Fix a bug in Resource Manager HA checker

2017-04-01 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/900

EAGLE-989: Fix a bug in Resource Manager HA checker

https://issues.apache.org/jira/browse/EAGLE-989

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-989

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/900.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #900


commit f0591dd6d1060c24be144ca90e4ec7afd8da2e85
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-04-01T09:03:37Z

fix a bug in resource manager ha checker

commit 5b470d586e3141e56e8446565f4dca1a377bb53b
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-04-01T09:30:35Z

fix failed unittest




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #895: EAGLE-971: fix a bug that duplicated queues are gen...

2017-03-28 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/895

EAGLE-971: fix a bug that duplicated queues are generated under a monitored 
stream

https://issues.apache.org/jira/browse/EAGLE-971

New policies for alert spec generation
1. each alert bolt has no more than 'coordinator.policiesPerBolt' policies.
2. each alert bolt has no more than 'coordinator.streamsPerBolt' queues if 
'reuseBoltInStreams' is true
3. NO queues on one alert bolt have the same StreamGroup.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/eagle EAGLE-971

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/895.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #895


commit c4950daa2bd6f1805664fab8593e95d5baaf2531
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-03-28T12:27:06Z

fix a bug that duplicated queues are generated under a monitored stream




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #892: [MINOR] Fix Topology Check system config thrown NullPointe...

2017-03-27 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/892
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #869: EAGLE-946: Refactor MRRunningJobApp & HadoopQueueApp

2017-03-14 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/869
  
endpointConfig.requestsNum: 1
endpointConfig.requestLimit: null
endpointConfig.requestTimeInHour: 6

By default, our application queries all running applications in one 
request, which keeps the same behavior as before. If users want to distribute 
the request load on resource manager, they can increase 
`endpointConfig.requestsNum`, and set a proper value to 
`endpointConfig.requestLimit` if limited number is required. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #872: Eagle-949 Support job suggestion

2017-03-14 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/872
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #869: EAGLE-946: Refactor MRRunningJobApp & HadoopQueueAp...

2017-03-13 Thread qingwen220
Github user qingwen220 commented on a diff in the pull request:

https://github.com/apache/eagle/pull/869#discussion_r105818056
  
--- Diff: 
eagle-jpm/eagle-jpm-util/src/main/java/org/apache/eagle/jpm/util/resourcefetch/RMResourceFetcher.java
 ---
@@ -112,40 +110,77 @@ private String getMRFinishedJobURL(String 
lastFinishedTime) {
 + lastFinishedTime + "&" + Constants.ANONYMOUS_PARAMETER;
 }
 
-private List doFetchRunningApplicationsList(String urlString, 
Constants.CompressionType compressionType) throws Exception {
-List result = new ArrayList<>(0);
-InputStream is = null;
+private String getAccepedAppURL() {
+String baseUrl = 
URLUtil.removeTrailingSlash(selector.getSelectedUrl());
+return String.format("%s/%s?state=ACCEPTED&%s", baseUrl, 
Constants.V2_APPS_URL, Constants.ANONYMOUS_PARAMETER);
+}
+
+private List doFetchRunningApplicationsList(Constants.JobType 
jobType,
+ 
Constants.CompressionType compressionType,
+ Object... 
parameter) throws Exception {
+Map<String, AppInfo> result = new HashMap();
+List apps = new ArrayList<>();
 try {
-checkUrl();
-LOG.info("Going to call yarn api to fetch running application 
list: " + urlString);
-is = InputStreamUtils.getInputStream(urlString, null, 
compressionType);
-final AppsWrapper appWrapper = OBJ_MAPPER.readValue(is, 
AppsWrapper.class);
-if (appWrapper != null && appWrapper.getApps() != null && 
appWrapper.getApps().getApp() != null) {
-result = appWrapper.getApps().getApp();
+selector.checkUrl();
+String urlString = getRunningJobURL(jobType, null, null);
+if (parameter.length == 0) {
+return doFetchApplicationsList(urlString, compressionType);
 }
-return result;
-} finally {
-if (is != null) {
-try {
-is.close();
-} catch (Exception e) {
-LOG.warn("{}", e);
-}
+
+int requests = (int) parameter[0];
+if (requests <= 1) {
+return doFetchApplicationsList(urlString, compressionType);
 }
+
+long interval = DateTimeUtil.ONEHOUR * 6 / (requests - 1);
+long currentTime = System.currentTimeMillis();
+long earliestTime = currentTime - DateTimeUtil.ONEHOUR * 6;
--- End diff --

Yes. thanks for reminding me. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #869: EAGLE-946: Refactor MRRunningJobApp & HadoopQueueAp...

2017-03-13 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/869

EAGLE-946: Refactor MRRunningJobApp & HadoopQueueApp

https://issues.apache.org/jira/browse/EAGLE-946

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle EAGLE-946

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/869.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #869


commit 0da3a34aafe8eb780843974fd0dab311449f901c
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-03-10T09:10:50Z

update the funtion of fetching running jobs




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #861: EAGLE-945 support queue in JPM overview

2017-03-09 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/861
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #862: EAGLE-940: HDFS traffic monitor by the namenode jmx...

2017-03-09 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/862

EAGLE-940: HDFS traffic monitor by the namenode jmx data

https://issues.apache.org/jira/browse/EAGLE-940

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle EAGLE-940

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/862.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #862






---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #843: EAGLE-929 UI Policy definition stream group use group name

2017-02-24 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/843
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #833: EAGLE-919: Fix null pointer exception in JobSuggest...

2017-02-21 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/833

EAGLE-919: Fix null pointer exception in JobSuggestionListener

https://issues.apache.org/jira/browse/EAGLE-919

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle EAGLE-919

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/833.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #833


commit 32dfd910269bc2d3c907472d056963cf4f3ef033
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-02-22T03:23:34Z

fix null pointer exception

commit 2eaf5a6c99a6bcdda702760c8da533f88b756403
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-02-22T03:27:18Z

enable jobSuggestion listener




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #819: [EAGLE-911]: Have mongo metadata dbname configurable

2017-02-19 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/819
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #794: Eagle-884 JPM support queue trend view

2017-02-07 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/794
  
look good to me


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #793: EAGLE-882 Stream leaf RunningQueueAPIEntity into Ka...

2017-01-24 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/793

EAGLE-882 Stream leaf RunningQueueAPIEntity into Kafka for queue monitoring

https://issues.apache.org/jira/browse/EAGLE-882

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle EAGLE-882

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/793.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #793


commit 5b56533ce685f03bb2aab7f9f081bb6db224dca0
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-24T10:13:18Z

integrate hadoop queue app with the alert engine

commit 0af737d450042c6b5cf6a2b93db1a043c9d1c275
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-24T10:26:02Z

add extra config

commit 26b198700c75f5b37b8978e4d36e97cd7c54385e
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-24T11:31:30Z

fix RunningQueueAPIEntity persist exception

commit d4130dd53f1449f7effbe6442cfbf224b37a242a
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-25T02:51:34Z

add cleanup

commit 7cf355725fc2a8acac7be8d99fc13c0528238e0c
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-25T03:20:30Z

add userLimitFactor in RunningQueueAPIEntity

commit 6a4b9db815f31e4621c584358159a37368ea70e4
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-25T03:48:20Z

add timestamp




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #790: EAGLE-883 fix RunningQueueAPIEntity.java misconfigu...

2017-01-22 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/790

EAGLE-883 fix RunningQueueAPIEntity.java misconfiguration



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle EAGLE-883

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/790.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #790


commit 646c39bdd5c29dbbcac8eb13ca4e95cdfce1de8b
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-23T07:09:21Z

fix RunningQueueAPIEntity.java misconfiguration




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #789: [MINOR] fix small bugs in MR job performance monito...

2017-01-22 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/789

[MINOR] fix small bugs in MR job performance monitoring



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle minor

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/789.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #789


commit 46f830122aaa4d1f153d2ae5ef66602d81f518cd
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-22T04:16:15Z

fix message format

commit bbd33bf28713c5bc853c4ca76ed1dadaab10b057
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-22T06:08:45Z

disable job performance email publisher




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #787: [MINOR] Change default-rack to /default-rack

2017-01-19 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/787
  
At the initial design, we prefer to extract the 'exact rack info', for 
example,  '/rack-45' -> 'rack-45',  '/' will be removed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #784: EAGLE-859 MapReduce job performance suggestion

2017-01-19 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/784
  
@r7raul1984  Fixed. Thanks


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #785: EAGLE-876 TopologyBaseAPIEntity comparison should c...

2017-01-19 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/785

EAGLE-876 TopologyBaseAPIEntity comparison should consider rack info

https://issues.apache.org/jira/browse/EAGLE-876

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle EAGLE-876

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/785.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #785


commit cb7f41d48081e078b131419a78a3b110856f6867
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-19T09:05:41Z

Fix persisting key

commit 8205d99d075ac1ac90c43072f51f819592cb1d35
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-19T09:11:04Z

fix conflicts

commit 03cfcb139861696160a3c257b090ddb13a2ca89a
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-19T09:13:05Z

Merge branch 'EAGLE-876' of https://github.com/qingwen220/incubator-eagle 
into EAGLE-876




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #784: EAGLE-859 MapReduce job performance suggestion

2017-01-18 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/784

EAGLE-859 MapReduce job performance suggestion

https://issues.apache.org/jira/browse/EAGLE-859

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle EAGLE-859

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/784.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #784


commit 89ba5befd38615c1b35b15f75b610fa0a1e978f0
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-12T03:00:12Z

rename method analysis to analyze

commit f085fce32f4c42b7031bf455e42fdaa7604b6bf8
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-12T06:41:27Z

fix a typo

commit 0d95e1f02b09e18c16795681742fbad72430a826
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-12T07:47:27Z

add job suggestion

commit 69b9646fbdd9e7c4bacf461e9a72305a391e375d
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-13T13:08:12Z

add JobSuggestionListner

commit 5e9c2e0a22e3ff2fabcf28580391064b4b7170b5
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-17T06:34:02Z

add job suggestion processors

commit 46be1b385bb2595edc4e6f2154989261db7cd8d1
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-12T03:00:12Z

rename method analysis to analyze

commit f2f516be66333a3c77d22d731144997081cb3cf8
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-12T06:41:27Z

fix a typo

commit a8a3db5f5cdb3991ec821663b1a9c6f82330c8f9
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-12T07:47:27Z

add job suggestion

commit 6da62e20b8b1fcc30784e1ccbefba0cb9f0c50c7
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-13T13:08:12Z

add JobSuggestionListner

commit 2e756c88579155ea1b8597fcfe2a9b4756427f75
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-17T06:34:02Z

add job suggestion processors

commit d6e3780f07024d2ba84bbb7566970db083d5e233
Author: Qingwen Zhao <qingwen...@gmail.com>
Date:   2017-01-17T06:35:21Z

Merge remote-tracking branch 'refs/remotes/origin/EAGLE-859' into EAGLE-859

commit 634bd709bf91b9182384f3a3cc9cad0ae0d68274
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-17T08:14:49Z

update

commit d0a61b1b4ee08cad8623996ecfac06bb40a28e46
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T02:13:36Z

update JHFEventReaderBase.java

commit 7651da89104d14bdd4ad873521411f13b7d0dd6f
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T03:06:55Z

Fix code checkstyle

commit a902b5af435dc40569a76ba0b3cc870b18607e56
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T07:22:28Z

rename class name

commit 5529e9ab3a06ee043613b66c1434e4eeedf1e613
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T09:20:14Z

refactor code

commit 475411808d472b749f4b50d7bcbae3cf6321772b
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T09:54:28Z

update EmailPublisher.java

commit 81ed7fd024f5a05ff64e61cde44a64521d40c87e
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T11:01:47Z

change level from warning to none

commit e17657ce7848d9ab7ceb0a68a780f78731f9d775
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T13:19:40Z

add a new column in the alert

commit 13054d2adb2671c4a61d0608cef9b4f0493661fd
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-18T13:35:48Z

update AnalyzerReportTemplate.vm

commit 19befb11e3b09de72b45d41df88861fde37834a1
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-19T02:17:19Z

refactor code

commit 218c1e229810fd411a0a2302f985410f70af6ed6
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2017-01-19T06:34:28Z

update




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #769: [MINOR] Change AlertEagleStorePlugin in sql file

2017-01-11 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/769
  
I think the change you suggested it OK


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle issue #769: [MINOR] Change AlertEagleStorePlugin in sql file

2017-01-09 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/eagle/pull/769
  
@r7raul1984  We have two implementations of the AlertEagleStorePlugin. 
{code}
org.apache.eagle.alert.app.AlertEagleStorePlugin
org.apache.eagle.alert.engine.publisher.impl.AlertEagleStorePlugin
{code}

The first one is based on HBase, and the second one is on JDBC. As the 
frontend codes have bound to the second. There maybe some problem if we make 
the change


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] eagle pull request #761: [Minor] update SparkHistoryJobApp & TopologyCheckAp...

2016-12-28 Thread qingwen220
GitHub user qingwen220 opened a pull request:

https://github.com/apache/eagle/pull/761

[Minor] update SparkHistoryJobApp & TopologyCheckApp configs

1. add `service.flushLimit` in SparkHistoryJobApp config
2. update TopologyCheckAppConfig.java

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/qingwen220/incubator-eagle minor

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/eagle/pull/761.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #761


commit 738965a01fe18a11873002566250f5778d1c299a
Author: Zhao, Qingwen <qingwz...@apache.org>
Date:   2016-12-29T06:58:25Z

update SparkHistoryJobApp & TopologyCheckApp configs




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-eagle issue #756: EAGLE-846: HDFS audit log traffic monitoring

2016-12-21 Thread qingwen220
Github user qingwen220 commented on the issue:

https://github.com/apache/incubator-eagle/pull/756
  
sample metric emitted by a bolt
`{
prefix: "GENERIC_METRIC_PREFIX_PLACEHODLER",
timestamp: 148238502,
tags: {
site: "sandbox",
appId: "HDFS_AUDIT_LOG_MONITOR_APP_SANDBOX",
taskId: "9"
},
exp: null,
encodedRowkey: "sZpubX___qba96ufy37xgDkANd9Hby--xwWLfxx0983P",
serializeAlias: null,
serializeVerbose: true,
value: [
12
]
}`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---