[GitHub] incubator-eagle pull request #656: [MINOR] fix metadata store config read is...

2016-11-15 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/656 --- 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

[GitHub] incubator-eagle pull request #656: [MINOR] fix metadata store config read is...

2016-11-15 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/656 [MINOR] fix metadata store config read issue Fix the "No configuration setting" exception & logic error. You can merge this pull request into a Git repository by runni

[GitHub] incubator-eagle pull request #638: [EAGLE-750] Improve coordinator schedule ...

2016-11-09 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/638 [EAGLE-750] Improve coordinator schedule strategy to reuse alert work slot 1. Add switch coordinator.reuseBoltInStreams to enable/disable reuse bolt in multiple streams 2. Add

[GitHub] incubator-eagle issue #631: EAGLE-758: Add tuple log for spout & alert bolt

2016-11-09 Thread garrettlish
Github user garrettlish commented on the issue: https://github.com/apache/incubator-eagle/pull/631 @RalphSu @haoch I add switch for per-event log, could u please review - https://github.com/apache/incubator-eagle/pull/633? --- If your project is set up for it, you can reply

[GitHub] incubator-eagle pull request #633: [EAGLE-758] add switch to enable and disa...

2016-11-09 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/633 [EAGLE-758] add switch to enable and disable per event log You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator

[GitHub] incubator-eagle issue #631: EAGLE-758: Add tuple log for spout & alert bolt

2016-11-09 Thread garrettlish
Github user garrettlish commented on the issue: https://github.com/apache/incubator-eagle/pull/631 storm metrics cannot provide metrics base on group by, etc. You are correct, we cannot enable the per-event log in everywhere, I will add a config to enable/disable the per-event log

[GitHub] incubator-eagle pull request #631: EAGLE-758: Add tuple log for spout & aler...

2016-11-09 Thread garrettlish
Github user garrettlish commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/631#discussion_r87147672 --- Diff: eagle-core/eagle-alert-parent/eagle-alert/alert-engine/src/main/java/org/apache/eagle/alert/engine/spout/SpoutOutputCollectorWrapper.java

[GitHub] incubator-eagle pull request #631: EAGLE-758: Add tuple log for spout & aler...

2016-11-08 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/631 EAGLE-758: Add tuple log for spout & alert bolt Without tuple log for spout & alert bolt, it is hard to troubleshoot event missing issue, we don't know whether the spout/alert

[GitHub] incubator-eagle pull request #629: EAGLE-755: Publishment without streamIds ...

2016-11-08 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/629 EAGLE-755: Publishment without streamIds update encountered NPE fix NPE issue by use google Objects.equals method. You can merge this pull request into a Git repository by running

[GitHub] incubator-eagle pull request #613: EAGLE-741: Make publishment settings both...

2016-11-06 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/613 EAGLE-741: Make publishment settings both policy & stream awareness Currently our publishment is defined policy specific, we cannot publish the alert into different places for

[GitHub] incubator-eagle issue #556: EAGLE-670: make kafka publisher configurable and...

2016-10-24 Thread garrettlish
Github user garrettlish commented on the issue: https://github.com/apache/incubator-eagle/pull/556 Thx @haoch @RalphSu. I thought we set async for kafka producer and what we need to change is using callback rather than future wait, it is wrong, thanks for pointing out. I

[GitHub] incubator-eagle pull request #556: EAGLE-670: make kafka publisher configura...

2016-10-24 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/556 EAGLE-670: make kafka publisher configurable and default is async Kafka send alert in sync would limit the throughput. Make this configurable, and use async by default You can merge

[GitHub] incubator-eagle pull request #553: EAGLE-672: remove mongo state store for d...

2016-10-23 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/553 EAGLE-672: remove mongo state store for dedup Currently the MongoStateStore is not a common use case for eagle, we need to remove the specific use case. You can merge this pull

[GitHub] incubator-eagle pull request #547: EAGLE-660: Stream delete mongo implementa...

2016-10-21 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/547 --- 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

[GitHub] incubator-eagle pull request #547: EAGLE-660: Stream delete mongo implementa...

2016-10-21 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/547 EAGLE-660: Stream delete mongo implementation is not working we are using name rather than streamId to delete stream, it is incorrect, change it to use streamId You can merge

[GitHub] incubator-eagle pull request #541: EAGLE-648: Stream router does not work pr...

2016-10-20 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/541 EAGLE-648: Stream router does not work properly sometimes Stream router does not work properly when multiple policies use the same input stream with the same partition definition

[GitHub] incubator-eagle pull request #534: EAGLE-641: Publishment reload may introdu...

2016-10-19 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/534 EAGLE-641: Publishment reload may introduce NPE The publishment initialization may have problem which will cause reload NPE. You can merge this pull request into a Git repository

[GitHub] incubator-eagle pull request #478: EAGLE-594: Remove raw alert from kafka pu...

2016-10-08 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/478 EAGLE-594: Remove raw alert from kafka publisher We are leveraging configured deduplicator to dedup the duplicated alerts before publish to kafka, email, slack, etc. However, sometimes

[GitHub] incubator-eagle pull request #466: EAGLE-579: Cloned stream alert event miss...

2016-09-29 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/466 EAGLE-579: Cloned stream alert event missed stream name We are trying to clone stream alert event if there is a state transition (we need to update last state dedup count and first

[GitHub] incubator-eagle pull request #465: EAGLE-578: Make dedup state value case in...

2016-09-28 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/465 EAGLE-578: Make dedup state value case insensitive Currently our dedup feature supports both stateful and stateless dedup. For stateful dedup, we are relying the state transition to do

[GitHub] incubator-eagle issue #452: update joiner from storm to google library

2016-09-28 Thread garrettlish
Github user garrettlish commented on the issue: https://github.com/apache/incubator-eagle/pull/452 @yonzhang add description, thanks for reminder --- 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

[GitHub] incubator-eagle pull request #463: EAGLE-576: change dedup db store to per p...

2016-09-28 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/463 EAGLE-576: change dedup db store to per publishment rather than global You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish

[GitHub] incubator-eagle pull request #462: EAGLE-576: dedup enhancements

2016-09-28 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/462 EAGLE-576: dedup enhancements 1. per publishment per cache, without global cache 2. support dedup without state field 3. emit raw alert into configured namespace

[GitHub] incubator-eagle pull request #454: [FIX] add dedup fields into publishment c...

2016-09-26 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/454 [FIX] add dedup fields into publishment compare consideration You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish

[GitHub] incubator-eagle pull request #453: [fix] add dedup fields into publishment c...

2016-09-26 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/453 --- 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

[GitHub] incubator-eagle pull request #453: [fix] add dedup fields into publishment c...

2016-09-26 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/453 [fix] add dedup fields into publishment compare consideration You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish

[GitHub] incubator-eagle pull request #452: update joiner from storm to google librar...

2016-09-26 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/452 --- 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

[GitHub] incubator-eagle pull request #452: update joiner from storm to google librar...

2016-09-26 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/452 update joiner from storm to google library You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator-eagle master

[GitHub] incubator-eagle pull request #442: EAGLE-550: periodically clean up dedup ca...

2016-09-19 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/442 EAGLE-550: periodically clean up dedup cache You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator-eagle master

[GitHub] incubator-eagle pull request #437: EAGLE-544: Enhance dedup to support exten...

2016-09-17 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/437 EAGLE-544: Enhance dedup to support extended deduplicator You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator

[GitHub] incubator-eagle pull request #428: EAGLE-531: Dedup alerts according to stat...

2016-09-08 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/428 EAGLE-531: Dedup alerts according to state change You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator-eagle

[GitHub] incubator-eagle pull request #423: EAGLE-531: Dedup alerts according to stat...

2016-09-08 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/423 --- 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

[GitHub] incubator-eagle pull request #423: EAGLE-531: Dedup alerts according to stat...

2016-09-08 Thread garrettlish
Github user garrettlish commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/423#discussion_r77951655 --- Diff: eagle-core/eagle-alert-parent/eagle-alert/pom.xml --- @@ -291,6 +295,32 @@ commons-cli

[GitHub] incubator-eagle pull request #423: EAGLE-531: Dedup alerts according to stat...

2016-09-07 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/423 EAGLE-531: Dedup alerts according to state change Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [ ] Make sure

[GitHub] incubator-eagle pull request #413: EAGLE-519: no data alert enhancement

2016-09-02 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/413 --- 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

[GitHub] incubator-eagle pull request #322: [EAGLE-435] make coordiantor schedule ope...

2016-08-14 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/322 --- 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

[GitHub] incubator-eagle pull request #322: [EAGLE-435] make coordiantor schedule ope...

2016-08-10 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/322 [EAGLE-435] make coordiantor schedule operation exclusive in distributed deployment You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-eagle pull request #317: [EAGLE-435] make coordiantor schedule ope...

2016-08-10 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/317 --- 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

[GitHub] incubator-eagle pull request #319: [EAGLE-439] Introduce policy enable/disab...

2016-08-09 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/319 [EAGLE-439] Introduce policy enable/disable support You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator-eagle

[GitHub] incubator-eagle pull request #318: [EAGLE-439] Introduce policy enable/disab...

2016-08-09 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/318 --- 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

[GitHub] incubator-eagle pull request #313: [EAGLE-434] fix policy change listener da...

2016-08-09 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/313 --- 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

[GitHub] incubator-eagle pull request #318: [EAGLE-439] Introduce policy enable/disab...

2016-08-09 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/318 [EAGLE-439] Introduce policy enable/disable support You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator-eagle

[GitHub] incubator-eagle pull request #313: [EAGLE-434] fix policy change listener da...

2016-08-09 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/313 [EAGLE-434] fix policy change listener daemon not working issue You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish

[GitHub] incubator-eagle pull request #299: fix policy change listener daemon not wor...

2016-08-04 Thread garrettlish
Github user garrettlish closed the pull request at: https://github.com/apache/incubator-eagle/pull/299 --- 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

[GitHub] incubator-eagle pull request #299: fix policy change listener daemon not wor...

2016-08-04 Thread garrettlish
GitHub user garrettlish opened a pull request: https://github.com/apache/incubator-eagle/pull/299 fix policy change listener daemon not working issue You can merge this pull request into a Git repository by running: $ git pull https://github.com/garrettlish/incubator-eagle