[GitHub] incubator-eagle pull request #630: EAGLE-756: make sure publish change is se...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/630 EAGLE-756: make sure publish change is sent to runtime bolts During test with Alert Engine publisher, it looks like when we have slack publishment, the metadata reload become slow refreshed when use change the metadata and manually trigger the coordinator schedule. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/630.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 #630 commit e514041d33868971bc9cc91b1a6ffa53cf7028bd Author: mizeng Date: 2016-11-08T23:37:39Z EAGLE-756: make sure publish change is sent to runtime bolts --- 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 pull request #566: EAGLE-683: Improve metadata store perform...
Github user mizeng commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/566#discussion_r85482632 --- Diff: eagle-core/eagle-alert-parent/eagle-alert/alert-metadata-parent/alert-metadata/src/main/java/org/apache/eagle/alert/metadata/impl/MongoMetadataDaoImpl.java --- @@ -127,41 +157,41 @@ private void init() { BsonDocument doc1 = new BsonDocument(); IndexOptions io1 = new IndexOptions().background(true).name("versionIndex"); doc1.append("version", new BsonInt32(1)); -scheduleStates = db.getCollection("schedule_specs"); +scheduleStates = getCollection("schedule_specs"); scheduleStates.createIndex(doc1, io1); -spoutSpecs = db.getCollection("spoutSpecs"); +spoutSpecs = getCollection("spoutSpecs"); { IndexOptions ioInternal = new IndexOptions().background(true).name("topologyIdIndex"); BsonDocument docInternal = new BsonDocument(); docInternal.append("topologyId", new BsonInt32(1)); spoutSpecs.createIndex(docInternal, ioInternal); } -alertSpecs = db.getCollection("alertSpecs"); +alertSpecs = getCollection("alertSpecs"); { IndexOptions ioInternal = new IndexOptions().background(true).name("topologyNameIndex"); BsonDocument docInternal = new BsonDocument(); docInternal.append("topologyName", new BsonInt32(1)); alertSpecs.createIndex(docInternal, ioInternal); } -groupSpecs = db.getCollection("groupSpecs"); +groupSpecs = getCollection("groupSpecs"); --- End diff -- groupSpecs is part of schedule specs.. --- 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 #566: EAGLE-683: Improve metadata store performance
Github user mizeng commented on the issue: https://github.com/apache/incubator-eagle/pull/566 @RalphSu plz help to review. --- 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 pull request #566: EAGLE-683: Improve metadata store perform...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/566 EAGLE-683: Improve metadata store performance Currently, we enable periodically schedule in coordinator service itself. If this is enabled, the schedule spec will finally used up metadata storage. We should 1. Disable periodically schedule by default (storage like mysql doesn't have capped feature) 2. For mongodb storage, use capped collection for schedule_specs, policy_snapshots, streamSnapshots, groupSpecs, alertSpecs, monitoredStreams, spoutSpecs (all schedule spec related collection). Let use default config of 20,000 as capped collection size. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/566.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 #566 commit 802570a19e9ca4a7b5f4f05e2b01a40cbe2d2d15 Author: mizeng Date: 2016-10-26T06:22:33Z EAGLE-683: Improve metadata store performance Currently, we enable periodically schedule in coordinator service itself. If this is enabled, the schedule spec will finally used up metadata storage. So we need to improve metadata store performance by: 1. Disable periodically schedule by default (storage like mysql doesn't have capped feature). 2. For mongodb storage, use capped collection for schedule_specs, policy_snapshots, streamSnapshots, groupSpecs, alertSpecs, monitoredStreams, spoutSpecs (all schedule spec related collection). --- 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 pull request #554: EAGLE-671: incorrect metric system use : ...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/554 EAGLE-671: incorrect metric system use : too many metric object creat⦠Currently there're too many metric objects create and registered in AlertBolt. This will impact the performance. In AlertBolt: we should register one metric to metric system, then use the metric object to send metric. Not every-time register the metric to metric system. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/554.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 #554 commit 5fd881a2eb7cd5750e1b1c52b12657c24491e77e Author: mizeng Date: 2016-10-24T02:29:03Z EAGLE-671: incorrect metric system use : too many metric object create and registered --- 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 pull request #501: [EAGLE-616] AlertEngine: Reduce the conne...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/501 [EAGLE-616] AlertEngine: Reduce the connection from alert-service to ZK In QA, it used to show too much connection from alert-service to zookeeper. We only use zk for exclusive lock and spec version notify, should not have so much connection. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle EAGLE-616 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/501.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 #501 commit 30f881d3cb24d167d521d6477e6b728b38c923dd Author: mizeng Date: 2016-10-13T09:28:06Z [EAGLE-616] AlertEngine: Reduce the connection from alert-service to ZK --- 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 #501: [EAGLE-616] AlertEngine: Reduce the connection f...
Github user mizeng commented on the issue: https://github.com/apache/incubator-eagle/pull/501 @RalphSu could you plz help to review? --- 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 #498: EAGLE-614: NPE in DynamicPolicyLoader
Github user mizeng commented on the issue: https://github.com/apache/incubator-eagle/pull/498 @RalphSu could you plz help to review? --- 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 pull request #498: EAGLE-614: NPE in DynamicPolicyLoader
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/498 EAGLE-614: NPE in DynamicPolicyLoader Find exception as below: INFO [2016-10-13 01:15:02,965] org.apache.eagle.alert.coordinator.trigger.DynamicPolicyLoader: policies loader start. INFO [2016-10-13 01:15:02,966] org.apache.eagle.alert.service.MetadataServiceClientImpl: query URL http://localhost:8080/rest/metadata/policies ERROR [2016-10-13 01:15:02,972] org.apache.eagle.alert.coordinator.trigger.DynamicPolicyLoader: error loading policy, but continue to run ! java.lang.NullPointerException: null ! at org.apache.eagle.alert.engine.coordinator.PolicyDefinition.equals(PolicyDefinition.java:150) ~[alert-service-0.5.0-incubating-SNAPSHOT-shaded.jar :0.5.0-incubating-SNAPSHOT] ! at org.apache.eagle.alert.coordinator.trigger.DynamicPolicyLoader.run(DynamicPolicyLoader.java:69) ~[alert-service-0.5.0-incubating-SNAPSHOT-shaded. jar:0.5.0-incubating-SNAPSHOT] ! at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [na:1.8.0_102] ! at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308) [na:1.8.0_102] ! at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180) [na:1.8.0_102] ! at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294) [na:1.8.0_102] ! at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [na:1.8.0_102] ! at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [na:1.8.0_102] ! at java.lang.Thread.run(Thread.java:745) [na:1.8.0_102] we have compare null with other object which should be avoided. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle EAGLE-614 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/498.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 #498 commit abe54782c2dbd840864f02bc57ea1659ec6672bc Author: mizeng Date: 2016-10-13T01:41:15Z EAGLE-614: NPE in DynamicPolicyLoader --- 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 #486: EAGLE-602: Exception that Spec Version [xxx] of ...
Github user mizeng commented on the issue: https://github.com/apache/incubator-eagle/pull/486 @RalphSu could you help to review? --- 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 pull request #486: EAGLE-602: Exception that Spec Version [x...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/486 EAGLE-602: Exception that Spec Version [xxx] of AlertBolt is newer th⦠Just find exception as below: 2016-10-06 22:51:41 org.apache.eagle.alert.engine.runner.AlertBolt [WARN] Spec Version [spec_version_1475817732537] of AlertBolt is newer than Stream Event Version [null]! Seems there's an error when get meta version from stream event or initialising failed when stream event version is null. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/486.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 #486 --- 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 pull request #479: EAGLE-570: Refactor to make sure alert en...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/479 EAGLE-570: Refactor to make sure alert engine publisher only have gen⦠Eagle-567 intend to provide better formatting of slack channel message. But the implementation introduce too much specific logic (like defined field names) that beyond a common alerting engine should have. So need to refactor to move the specific logic to either vendor specific extension or have configurable formatting. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/479.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 #479 commit 83595c67848c4f43496251a6094705b427043198 Author: mizeng Date: 2016-10-09T03:09:37Z EAGLE-570: Refactor to make sure alert engine publisher only have general feature --- 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 pull request #467: fix spoutSpecs load met invalid FieldName...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/467 fix spoutSpecs load met invalid FieldName(Dot) exception 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/mizeng/incubator-eagle master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/467.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 #467 commit 64efd4297210f33300de755513f6d5f34dc1de32 Author: mizeng Date: 2016-09-29T12:38:22Z fix spoutSpecs load met invalid FieldName(Dot) exception --- 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 pull request #456: EAGLE-567: Enhance Slack publish format
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/456 EAGLE-567: Enhance Slack publish format 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/mizeng/incubator-eagle master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/456.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 #456 commit 4a5078bdf9e75bc8e7e69029026b15af3a0de0af Author: mizeng Date: 2016-09-27T07:15:16Z EAGLE-567: Enhance Slack publish format --- 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 #442: EAGLE-550: periodically clean up dedup cache
Github user mizeng commented on the issue: https://github.com/apache/incubator-eagle/pull/442 +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] incubator-eagle pull request #441: EAGLE-550: Add Alert publish implementati...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/441 EAGLE-550: Add Alert publish implementation for Slack 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/mizeng/incubator-eagle UMP-158 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/441.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 #441 commit 19d6113a0419531652b1e71bed25560e4ed5bf95 Author: mizeng Date: 2016-09-14T08:30:29Z EAGLE-550: Add Alert publish implementation for Slack --- 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 pull request #376: EAGLE-491: AlertPublisherImpl dedup throw...
Github user mizeng closed the pull request at: https://github.com/apache/incubator-eagle/pull/376 --- 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 pull request #376: EAGLE-491: AlertPublisherImpl dedup throw...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/376 EAGLE-491: AlertPublisherImpl dedup throws exception when dedup colum⦠â¦n is not configured You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle EAGLE-491 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/376.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 #376 commit 065f1c230249555f9eb4dbdf595200da4e863831 Author: mizeng Date: 2016-08-23T01:52:24Z EAGLE-491: AlertPublisherImpl dedup throws exception when dedup column is not configured --- 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 pull request #360: add consumer shell and fix some wrong par...
Github user mizeng closed the pull request at: https://github.com/apache/incubator-eagle/pull/360 --- 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 pull request #360: add consumer shell and fix some wrong par...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/360 add consumer shell and fix some wrong parameter names in alert-engine You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle develop Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/360.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 #360 commit 863273e70a79dadb9ecf9a7bf4c8a9ef01283545 Author: mizeng Date: 2016-08-18T09:11:20Z add consumer shell and fix some wrong parameter names in alert-engine --- 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 pull request #352: EAGLE-462: Alert Dudup
Github user mizeng closed the pull request at: https://github.com/apache/incubator-eagle/pull/352 --- 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 pull request #355: EAGLE-471: fix AlertBolt introduce un-ser...
Github user mizeng closed the pull request at: https://github.com/apache/incubator-eagle/pull/355 --- 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 pull request #355: EAGLE-471: fix AlertBolt introduce un-ser...
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/355 EAGLE-471: fix AlertBolt introduce un-serializble member You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle EAGLE-471 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/355.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 #355 commit 476b9930c6619a797af7b14b7480bf935b6d01f8 Author: mizeng Date: 2016-08-17T05:51:08Z EAGLE-471: fix AlertBolt introduce un-serializble member --- 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 pull request #352: EAGLE-462: Alert Dudup
Github user mizeng commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/352#discussion_r74903923 --- Diff: eagle-core/eagle-alert-parent/eagle-alert/alert-common/src/main/java/org/apache/eagle/alert/engine/model/AlertStreamEvent.java --- @@ -35,6 +35,7 @@ private StreamDefinition schema; private String createdBy; private long createdTime; +private boolean isDuplicate = false; --- End diff -- currently it is only for test. But should not "duplicate" be one property for StreamEvent? It sounds more naturally. --- 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 pull request #352: EAGLE-462: Alert Dudup
Github user mizeng commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/352#discussion_r74899457 --- Diff: eagle-core/eagle-alert-parent/eagle-alert/alert-engine/src/test/resources/router/publishments.json --- @@ -0,0 +1,23 @@ +[ +{ + "name":"test-stream-output", + "type":"org.apache.eagle.alert.engine.publisher.impl.AlertEmailPublisher", + "policyIds": [ + "perfmon_cpu_host_check", "perfmon_cpu_pool_check" + ], + "properties": { + "subject":"UMP Test Alert", --- End diff -- got it. --- 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 pull request #352: EAGLE-462: Alert Dudup
Github user mizeng commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/352#discussion_r74899429 --- Diff: eagle-core/eagle-alert-parent/eagle-alert/alert-engine/src/main/java/org/apache/eagle/alert/engine/publisher/impl/AlertEmailPublisher.java --- @@ -69,6 +68,10 @@ public void onAlert(AlertStreamEvent event) throws Exception { if(event == null) { return; } + +if (emailConfig.get(PublishConstants.SENDER).equals("sen...@corp.com")){ --- End diff -- mostly Yes. --- 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 pull request #352: EAGLE-462: Alert Dudup
Github user mizeng commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/352#discussion_r74899209 --- Diff: eagle-core/eagle-alert-parent/eagle-alert/alert-common/src/main/java/org/apache/eagle/alert/engine/coordinator/Publishment.java --- @@ -32,6 +32,7 @@ private String type; private List policyIds; private String dedupIntervalMin; +private List customDedupFields; --- End diff -- sounds better. --- 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 pull request #352: EAGLE-462: Alert Dudup
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/352 EAGLE-462: Alert Dudup 1. same metric generate alert should be deduped 2. user can define tags which is considered by dedup. by default will only consider about policyId and streamId. You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle EAGLE-462 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/352.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 #352 commit c8aae06e15e86c64b8deedc7eab1c3139f0b0b4e Author: mizeng Date: 2016-08-16T07:48:14Z EAGLE-462: Alert Dudup 1. same metric generate alert should be deduped 2. user can define tags which is considered by dedup. by default will only consider about policyId and streamId. --- 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 pull request #330: EAGLE-450: Alert check meta confliction
Github user mizeng closed the pull request at: https://github.com/apache/incubator-eagle/pull/330 --- 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 pull request #330: EAGLE-450: Alert check meta confliction
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/330 EAGLE-450: Alert check meta confliction You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle EAGLE-450 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/330.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 #330 commit c3ad49b3ee84121d93082f65de4962f820b44621 Author: mizeng Date: 2016-08-11T10:16:28Z EAGLE-450: Alert check meta confliction --- 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 pull request #328: EAGLE-440: Alert mongodb storage refine
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/328 EAGLE-440: Alert mongodb storage refine You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle develop Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/328.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 #328 commit 678437aadb5ca6e5c7256fabd40b4966e4227911 Author: mizeng Date: 2016-08-10T08:04:16Z EAGLE-440: Alert mongodb storage refine --- 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 pull request #324: EAGLE-440: Alert mongodb storage refine
Github user mizeng closed the pull request at: https://github.com/apache/incubator-eagle/pull/324 --- 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 pull request #324: EAGLE-440: Alert mongodb storage refine
GitHub user mizeng opened a pull request: https://github.com/apache/incubator-eagle/pull/324 EAGLE-440: Alert mongodb storage refine You can merge this pull request into a Git repository by running: $ git pull https://github.com/mizeng/incubator-eagle develop Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-eagle/pull/324.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 #324 commit 1fe32803f5c8c93ef9c7f29fe68b5fdc194d5368 Author: mizeng Date: 2016-08-10T08:04:16Z EAGLE-440: Alert mongodb storage refine --- 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. ---