[GitHub] incubator-eagle pull request #630: EAGLE-756: make sure publish change is se...

2016-11-08 Thread mizeng
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

[GitHub] incubator-eagle pull request #566: EAGLE-683: Improve metadata store perform...

2016-10-28 Thread mizeng
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

[GitHub] incubator-eagle issue #566: EAGLE-683: Improve metadata store performance

2016-10-25 Thread mizeng
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

[GitHub] incubator-eagle pull request #566: EAGLE-683: Improve metadata store perform...

2016-10-25 Thread mizeng
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

[GitHub] incubator-eagle pull request #554: EAGLE-671: incorrect metric system use : ...

2016-10-23 Thread mizeng
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 perfor

[GitHub] incubator-eagle pull request #501: [EAGLE-616] AlertEngine: Reduce the conne...

2016-10-13 Thread mizeng
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

[GitHub] incubator-eagle issue #501: [EAGLE-616] AlertEngine: Reduce the connection f...

2016-10-13 Thread mizeng
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

[GitHub] incubator-eagle issue #498: EAGLE-614: NPE in DynamicPolicyLoader

2016-10-12 Thread mizeng
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

[GitHub] incubator-eagle pull request #498: EAGLE-614: NPE in DynamicPolicyLoader

2016-10-12 Thread mizeng
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

[GitHub] incubator-eagle issue #486: EAGLE-602: Exception that Spec Version [xxx] of ...

2016-10-11 Thread mizeng
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

[GitHub] incubator-eagle pull request #486: EAGLE-602: Exception that Spec Version [x...

2016-10-10 Thread mizeng
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

[GitHub] incubator-eagle pull request #479: EAGLE-570: Refactor to make sure alert en...

2016-10-08 Thread mizeng
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

[GitHub] incubator-eagle pull request #467: fix spoutSpecs load met invalid FieldName...

2016-09-29 Thread mizeng
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

[GitHub] incubator-eagle pull request #456: EAGLE-567: Enhance Slack publish format

2016-09-27 Thread mizeng
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

[GitHub] incubator-eagle issue #442: EAGLE-550: periodically clean up dedup cache

2016-09-21 Thread mizeng
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

[GitHub] incubator-eagle pull request #441: EAGLE-550: Add Alert publish implementati...

2016-09-19 Thread mizeng
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

[GitHub] incubator-eagle pull request #376: EAGLE-491: AlertPublisherImpl dedup throw...

2016-08-22 Thread mizeng
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

[GitHub] incubator-eagle pull request #376: EAGLE-491: AlertPublisherImpl dedup throw...

2016-08-22 Thread mizeng
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] incubator-eagle pull request #360: add consumer shell and fix some wrong par...

2016-08-19 Thread mizeng
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

[GitHub] incubator-eagle pull request #360: add consumer shell and fix some wrong par...

2016-08-18 Thread mizeng
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

[GitHub] incubator-eagle pull request #352: EAGLE-462: Alert Dudup

2016-08-17 Thread mizeng
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

[GitHub] incubator-eagle pull request #355: EAGLE-471: fix AlertBolt introduce un-ser...

2016-08-17 Thread mizeng
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

[GitHub] incubator-eagle pull request #355: EAGLE-471: fix AlertBolt introduce un-ser...

2016-08-16 Thread mizeng
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

[GitHub] incubator-eagle pull request #352: EAGLE-462: Alert Dudup

2016-08-16 Thread mizeng
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

[GitHub] incubator-eagle pull request #352: EAGLE-462: Alert Dudup

2016-08-16 Thread mizeng
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

[GitHub] incubator-eagle pull request #352: EAGLE-462: Alert Dudup

2016-08-16 Thread mizeng
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

[GitHub] incubator-eagle pull request #352: EAGLE-462: Alert Dudup

2016-08-16 Thread mizeng
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

[GitHub] incubator-eagle pull request #352: EAGLE-462: Alert Dudup

2016-08-16 Thread mizeng
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

[GitHub] incubator-eagle pull request #330: EAGLE-450: Alert check meta confliction

2016-08-15 Thread mizeng
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

[GitHub] incubator-eagle pull request #330: EAGLE-450: Alert check meta confliction

2016-08-11 Thread mizeng
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

[GitHub] incubator-eagle pull request #328: EAGLE-440: Alert mongodb storage refine

2016-08-10 Thread mizeng
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

[GitHub] incubator-eagle pull request #324: EAGLE-440: Alert mongodb storage refine

2016-08-10 Thread mizeng
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

[GitHub] incubator-eagle pull request #324: EAGLE-440: Alert mongodb storage refine

2016-08-10 Thread mizeng
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