[GitHub] incubator-eagle pull request: Fix compile errors after changing pa...

2015-11-19 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/2 Fix compile errors after changing package name and remove useless code You can merge this pull request into a Git repository by running: $ git pull https://github.com/qingwen220

[GitHub] incubator-eagle pull request: Fix compile errors after changing pa...

2015-11-20 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/2#issuecomment-158332460 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

[GitHub] incubator-eagle pull request: Fix compile errors after changing pa...

2015-11-20 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/2 --- 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: [Eagle-7] Expose storm config as ind...

2015-11-25 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/5 [Eagle-7] Expose storm config as individual eagle topology config envContextConfig.stormConfigFile = You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-eagle pull request: [EAGLE-7] Expose storm config as ind...

2015-11-25 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/5 --- 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: [EAGLE-7] Expose storm config as ind...

2015-11-25 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/6 [EAGLE-7] Expose storm config as individual eagle topology config envContextConfig.stormConfigFile = *absolute pathname* https://issues.apache.org/jira/browse/EAGLE-7 You can

[GitHub] incubator-eagle pull request: [EAGLE-55] Add a missing anonymous p...

2015-11-27 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/9 [EAGLE-55] Add a missing anonymous parameter in hive running spout JIRA: https://issues.apache.org/jira/browse/EAGLE-55 You can merge this pull request into a Git repository by running

[GitHub] incubator-eagle pull request: [EAGLE-58] Enhance hive query log pa...

2015-11-30 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/12 [EAGLE-58] Enhance hive query log parsing New features: 1. support create/drop/alter 2. support select expression parsing You can merge this pull request into a Git repository by

[GitHub] incubator-eagle pull request: [EAGLE-74] Delete unused doc

2015-12-08 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/20 [EAGLE-74] Delete unused doc https://issues.apache.org/jira/browse/EAGLE-74 You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-eagle pull request: [EAGLE-74] Change package name from ...

2015-12-14 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/30 [EAGLE-74] Change package name from eagle. to org.apache.eagle https://issues.apache.org/jira/browse/EAGLE-74 You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-eagle pull request: [EAGLE-74] Change package name from ...

2015-12-14 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/30 --- 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: [EAGLE-74] Change package name from ...

2015-12-14 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/30#issuecomment-164632662 Sorry, duplicate pull request --- 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

[GitHub] incubator-eagle pull request: [EAGLE-61] Supports HA and secured c...

2015-12-28 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/40 [EAGLE-61] Supports HA and secured cluster connection on Eagle Web https://issues.apache.org/jira/browse/EAGLE-61 You can merge this pull request into a Git repository by running

[GitHub] incubator-eagle pull request: [EAGLE-100] Add hadoop jmx collector...

2015-12-28 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/41 [EAGLE-100] Add hadoop jmx collector scripts https://issues.apache.org/jira/browse/EAGLE-100 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-eagle pull request: [EAGLE-100] Add hadoop jmx collector...

2015-12-28 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/41 --- 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: EAGLE-100 Provide tools to collect h...

2015-12-28 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/46 EAGLE-100 Provide tools to collect hadoop jmx metrics https://issues.apache.org/jira/browse/EAGLE-100 You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-eagle pull request: EAGLE-100 Provide tools to collect h...

2015-12-29 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/46#issuecomment-167922198 Yes, I agree with your comments. Thanks. I will update the scripts. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-eagle pull request: EAGLE-100 Provide tools to collect h...

2015-12-30 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/46 --- 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: EAGLE-100 Add hadoop jmx collector s...

2015-12-30 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/49 EAGLE-100 Add hadoop jmx collector scripts https://issues.apache.org/jira/browse/EAGLE-100 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-eagle pull request: EAGLE-106 Use nvd3 replace flot for ...

2015-12-31 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/44#issuecomment-168153027 :+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

[GitHub] incubator-eagle pull request: EAGLE-100 Add hadoop jmx collector s...

2016-01-22 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/49 --- 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: EAGLE-81 Notification plugin

2016-02-17 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/90#issuecomment-185118602 +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

[GitHub] incubator-eagle pull request: EAGLE-164 add a missing license in a...

2016-02-21 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/96 EAGLE-164 add a missing license in a markdown file https://issues.apache.org/jira/browse/EAGLE-164 You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-eagle pull request: [EAGLE-61] Supports HA and secured c...

2016-02-21 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/40 --- 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: Eagle-61 classification supports sec...

2016-02-22 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/99 Eagle-61 classification supports secured cluster connection & enable Ldap authentication https://issues.apache.org/jira/browse/EAGLE-61 You can merge this pull request into a

[GitHub] incubator-eagle pull request: Eagle-61 classification supports sec...

2016-02-22 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/99 --- 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: Eagle-61 classification supports sec...

2016-02-22 Thread qingwen220
GitHub user qingwen220 reopened a pull request: https://github.com/apache/incubator-eagle/pull/99 Eagle-61 classification supports secured cluster connection & enable Ldap authentication https://issues.apache.org/jira/browse/EAGLE-61 You can merge this pull request into a

[GitHub] incubator-eagle pull request: EAGLE-164 add a missing license in a...

2016-02-22 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/96#issuecomment-187481896 Have been fixed in another pr, and this one will be closed --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-eagle pull request: EAGLE-164 add a missing license in a...

2016-02-22 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/96 --- 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: EAGLE-139 EAGLE-163 Eagle UI Modular...

2016-02-24 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/103#issuecomment-188232425 @haoch Good catch, 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

[GitHub] incubator-eagle pull request: EAGLE-162 revise main class's packag...

2016-02-25 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/104 EAGLE-162 revise main class's package name (Ambari Eagle Service) https://issues.apache.org/jira/browse/EAGLE-162 You can merge this pull request into a Git repository by ru

[GitHub] incubator-eagle pull request: EAGLE-31 General monitoring UI suppo...

2016-02-28 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/105#issuecomment-190010699 LGTM +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

[GitHub] incubator-eagle pull request: EAGLE-176 Metric dashboard UI keep a...

2016-02-28 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/106#issuecomment-190065684 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

[GitHub] incubator-eagle pull request: EAGLE-173 auto refresh when sensitiv...

2016-03-01 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/107#issuecomment-190655730 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

[GitHub] incubator-eagle pull request: EAGLE-177 Replace Legacy "dataSource...

2016-03-03 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/108 EAGLE-177 Replace Legacy "dataSource" field with "application" in all metadata https://issues.apache.org/jira/browse/EAGLE-177 You can merge this pull request into

[GitHub] incubator-eagle pull request: Eagle-61 classification supports sec...

2016-03-03 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/99 --- 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: Eagle 157 policy metric should be re...

2016-03-03 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/110#issuecomment-192071973 LGTM +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

[GitHub] incubator-eagle pull request: EAGLE-162 revise main class's packag...

2016-03-03 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/104 --- 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: EAGLE-61 classification supports sec...

2016-03-03 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/111 EAGLE-61 classification supports secured cluster connection & enable Ldap authentication https://issues.apache.org/jira/browse/EAGLE-61 You can merge this pull request into a

[GitHub] incubator-eagle pull request: Added RM Policy and GC Policies in R...

2016-03-04 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/109#issuecomment-192207524 @senthilec566 I saw the jenkins build failed. Could you please check the details and make sure the merge won't break down the code compilation -

[GitHub] incubator-eagle pull request: Added RM Policy and GC Policies in R...

2016-03-04 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/109#issuecomment-192211050 LGTM +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

[GitHub] incubator-eagle pull request: EAGLE-185 support build cache

2016-03-04 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/112#issuecomment-192275332 LGTM +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

[GitHub] incubator-eagle pull request: Added RM Policy and GC Policies in R...

2016-03-06 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/109#issuecomment-193063332 I found it and run it in my local environment, and it's OK! Could you check it as well? If this is caused by jenkins, we will report this bug to Mi

[GitHub] incubator-eagle pull request: Added RM Policy and GC Policies in R...

2016-03-06 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/109#issuecomment-193085957 Hi @senthilec566 Michael have checked with it, and this problem may be caused by Jenkins's instability. You can do a little bit of modificatio

[GitHub] incubator-eagle pull request: EAGLE-193 UI metric dashboard suppor...

2016-03-09 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/119#issuecomment-194198982 LGTM +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

[GitHub] incubator-eagle pull request: EAGLE-194 UI show exception alert if...

2016-03-09 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/120#issuecomment-194694310 +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

[GitHub] incubator-eagle pull request: EAGLE-205 Metric dashboard support m...

2016-03-23 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/129#issuecomment-200697351 LGTM +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

[GitHub] incubator-eagle pull request: UI metric dashboard should support o...

2016-03-25 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/132#issuecomment-201204024 +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

[GitHub] incubator-eagle pull request: EAGLE-210 UI application group not d...

2016-03-28 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/133#issuecomment-202705659 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

[GitHub] incubator-eagle pull request: EAGLE-212 Fix AlertDataSourceEntity ...

2016-03-29 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/134 EAGLE-212 Fix AlertDataSourceEntity Bug in Hive web https://issues.apache.org/jira/browse/EAGLE-212 You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-eagle pull request: EAGLE-212 Fix AlertDataSourceEntity ...

2016-03-29 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/134#issuecomment-202781143 About Jenkins failure, I check it in my local environment. All unit tests are passed. For this pr, I will go ahead to merge --- If your project

[GitHub] incubator-eagle pull request: EAGLE-212: Fix AlertDataSourceEntity...

2016-04-07 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/138 EAGLE-212: Fix AlertDataSourceEntity Bug in Hive web This pr fixes JSON format check bug introduced before. https://issues.apache.org/jira/browse/EAGLE-212 You can merge this

[GitHub] incubator-eagle pull request: EAGLE-212: Fix AlertDataSourceEntity...

2016-04-07 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/138 --- 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: EAGLE-212: Fix AlertDataSourceEntity...

2016-04-07 Thread qingwen220
GitHub user qingwen220 reopened a pull request: https://github.com/apache/incubator-eagle/pull/138 EAGLE-212: Fix AlertDataSourceEntity Bug in Hive web This pr fixes JSON format check bug introduced before. https://issues.apache.org/jira/browse/EAGLE-212 You can merge

[GitHub] incubator-eagle pull request: EAGLE-214 Policy edit page need auto...

2016-04-11 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/141#issuecomment-208686575 👍 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

[GitHub] incubator-eagle pull request: EAGLE-

2016-04-13 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/146 EAGLE- https://issues.apache.org/jira/browse/EAGLE-239 https://issues.apache.org/jira/browse/EAGLE-221 https://issues.apache.org/jira/browse/EAGLE-223 Bug fix

[GitHub] incubator-eagle pull request: EAGLE-

2016-04-13 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/146 --- 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: EAGLE-

2016-04-13 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/146#issuecomment-209340462 All unit tests are passed in my local environment. Please help to review this pull request. Thanks! --- If your project is set up for it, you can reply to

[GitHub] incubator-eagle pull request: EAGLE-

2016-04-13 Thread qingwen220
GitHub user qingwen220 reopened a pull request: https://github.com/apache/incubator-eagle/pull/146 EAGLE- https://issues.apache.org/jira/browse/EAGLE-239 https://issues.apache.org/jira/browse/EAGLE-221 https://issues.apache.org/jira/browse/EAGLE-223 Bug fix

[GitHub] incubator-eagle pull request: EAGLE-196 eagle-topology.sh supports...

2016-04-13 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/147 EAGLE-196 eagle-topology.sh supports jar file path as parameter https://issues.apache.org/jira/browse/EAGLE-196 Add optional parameter --jar in EAGLE-HOME/bin/eagle-topology.sh

[GitHub] incubator-eagle pull request: EAGLE-196 eagle-topology.sh supports...

2016-04-13 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/147#issuecomment-209361587 Only update a script bin/eagle-topology.sh, please help to review it. Thanks --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-eagle pull request: EAGLE-239 UI Fix alert detail contex...

2016-04-13 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/145#issuecomment-209711703 This pr may conflict with the entity model, as I have recovered the AlertAPIEntity. alertContext.properties remain here. You can check the newest

[GitHub] incubator-eagle pull request: EAGLE-239 UI Fix alert detail contex...

2016-04-13 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/145#issuecomment-209733102 **Previous** ``` { "timestamp":1460542328725, "tags":{ "site":"sa

[GitHub] incubator-eagle pull request: EAGLE-239 UI Fix alert detail contex...

2016-04-13 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/145#issuecomment-209734147 **Before** ``` { "timestamp":1460542328725, "tags":{ "site":"sa

[GitHub] incubator-eagle pull request: EAGLE-271 Topology management in rem...

2016-04-21 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/160 EAGLE-271 Topology management in remote/local mode including start/stop operations **Related tasks** https://issues.apache.org/jira/browse/EAGLE-272 https://issues.apache.org

[GitHub] incubator-eagle pull request: EAGLE-271 Topology management in rem...

2016-04-21 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/160#issuecomment-213232761 The application manager tutorial is under eagle-docs/tutorial. Hope it can help. --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-eagle pull request: EAGLE-271 Topology management in rem...

2016-04-22 Thread qingwen220
Github user qingwen220 commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/160#discussion_r60705328 --- Diff: eagle-assembly/src/main/conf/eagle-scheduler.conf --- @@ -0,0 +1,41 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] incubator-eagle pull request: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-22 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/161 EAGLE-274 Fix java.lang.ClassNotFoundException: org.slf4j.impl.Log4jLoggerAdapter https://issues.apache.org/jira/browse/EAGLE-274 slf4j-log4j12-*.jar under lib/share conflicts

[GitHub] incubator-eagle pull request: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-22 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/161#issuecomment-213363441 This exception will occur if we storm topology by bin/kafka-stream-monitor.sh, as the classpath contains slf4j-log4j12-*.jar --- If your project is set up

[GitHub] incubator-eagle pull request: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-24 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/161 --- 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: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-24 Thread qingwen220
GitHub user qingwen220 reopened a pull request: https://github.com/apache/incubator-eagle/pull/161 EAGLE-274 Fix java.lang.ClassNotFoundException: org.slf4j.impl.Log4jLoggerAdapter https://issues.apache.org/jira/browse/EAGLE-274 slf4j-log4j12-*.jar under lib/share

[GitHub] incubator-eagle pull request: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-24 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/161#issuecomment-214088831 Close this one, a --- 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

[GitHub] incubator-eagle pull request: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-24 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/161 --- 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: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-24 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/161#issuecomment-214089141 Close this one, and will open a new pull request to master branch --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-eagle pull request: EAGLE-274 Fix java.lang.ClassNotFoun...

2016-04-24 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/162 EAGLE-274 Fix java.lang.ClassNotFoundException: org.slf4j.impl.Log4jLoggerAdapter https://issues.apache.org/jira/browse/EAGLE-274 slf4j-log4j12-*.jar under lib/share conflicts

[GitHub] incubator-eagle pull request: EAGLE-271 Topology management in rem...

2016-04-24 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/160 --- 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: EAGLE-222 Documentation for eagle al...

2016-04-24 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/163 EAGLE-222 Documentation for eagle alert plugin mechnism https://issues.apache.org/jira/browse/EAGLE-222 You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-eagle pull request: EAGLE-275 Eagle email alert bug: $el...

2016-04-25 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/164 EAGLE-275 Eagle email alert bug: $elem["dataSource"] Alert Detected https://issues.apache.org/jira/browse/EAGLE-275 1. Two versions of email template are found. Need

[GitHub] incubator-eagle pull request: EAGLE-275 Eagle email alert bug: $el...

2016-04-25 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/164 --- 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: EAGLE-275 Eagle email alert bug: $el...

2016-04-25 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/164#issuecomment-214220424 Closed this one, and will open a new one to dev branch --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-eagle pull request: EAGLE-275 Eagle email alert bug: $el...

2016-04-25 Thread qingwen220
GitHub user qingwen220 reopened a pull request: https://github.com/apache/incubator-eagle/pull/164 EAGLE-275 Eagle email alert bug: $elem["dataSource"] Alert Detected https://issues.apache.org/jira/browse/EAGLE-275 1. Two versions of email template are found. Need

[GitHub] incubator-eagle pull request: EAGLE-275 Eagle email alert bug: $el...

2016-04-25 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/164 --- 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: EAGLE-275 Eagle email alert bug: $el...

2016-04-25 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/165 EAGLE-275 Eagle email alert bug: $elem["dataSource"] Alert Detected https://issues.apache.org/jira/browse/EAGLE-275 Two versions of email template are found. Need to

[GitHub] incubator-eagle pull request: EAGLE-275 Eagle email alert bug: $el...

2016-04-25 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/165#issuecomment-214583528 But we don't know what the eagle service host is, which depends on the deployment. --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-eagle pull request: EAGLE-222 Documentation for eagle al...

2016-04-25 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/163#issuecomment-214585589 How about this document? Can anyone help to review it 😄 --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-eagle pull request: EAGLE-254 HdfsAuditLog topology keep...

2016-04-27 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/167 EAGLE-254 HdfsAuditLog topology keeps alerting for one piece of log https://issues.apache.org/jira/browse/EAGLE-254 **Fixed two bugs** - HdfsAuditLog topology keeps

[GitHub] incubator-eagle pull request: EAGLE-254 HdfsAuditLog topology keep...

2016-04-27 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/167#issuecomment-215030776 All unit tests are passed in my local environment --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-eagle pull request: EAGLE-289: Fix $elem["dataSource"] i...

2016-05-02 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/170#issuecomment-216417423 It's not a good practice to send the two pull requests (for both EAGLE-287 and EAGLE-289) in the same pull request --- If your project is set up f

[GitHub] incubator-eagle pull request: EAGLE-289: Fix $elem["dataSource"] i...

2016-05-02 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/170#issuecomment-216418981 For pull request #170, I don't think it's a good fix (the code needs testing) Here is my fix in the dev branch, and tested it.

[GitHub] incubator-eagle pull request: EAGLE-222 Documentation for eagle al...

2016-05-02 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/163#issuecomment-216421394 merged into dev branch --- 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: EAGLE-222 Documentation for eagle al...

2016-05-02 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/163 --- 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: EAGLE-254 HdfsAuditLog topology keep...

2016-05-02 Thread qingwen220
Github user qingwen220 closed the pull request at: https://github.com/apache/incubator-eagle/pull/167 --- 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: EAGLE-287. Make EagleStore as the de...

2016-05-05 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/173#issuecomment-217118027 Basically, Adding notification plugin is a issue related to policy definition, but not to a specific topology. I have another idea to handle this

[GitHub] incubator-eagle pull request: This pull request is just for MapR i...

2016-05-05 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/175#issuecomment-217165646 That's really a big pull request! I don't have too much background on this new feature, and just check the code that i am fam

[GitHub] incubator-eagle pull request: This pull request is just for MapR i...

2016-05-05 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/175#issuecomment-217170959 Because i have a concern. As the command set are from the hdfs audit log, but not original hdfs commands. If you change the command set here, they may

[GitHub] incubator-eagle pull request: EAGLE-295 Add configuration value to...

2016-05-05 Thread qingwen220
GitHub user qingwen220 opened a pull request: https://github.com/apache/incubator-eagle/pull/176 EAGLE-295 Add configuration value to enable application Manager https://issues.apache.org/jira/browse/EAGLE-295 You can merge this pull request into a Git repository by running

[GitHub] incubator-eagle pull request: Eagle 135: Run eagle in Spark Stream...

2016-05-12 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/185#issuecomment-218759757 Please send this pull request to dev branch. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-eagle pull request: Eagle 135 : Run eagle in Spark Strea...

2016-05-16 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/186#issuecomment-219598407 This is because your code have conflicts with the code from dev. You need to rebase the code from dev branch first. --- If your project is set up for it

[GitHub] incubator-eagle pull request: EAGLE-273 EAGLE-269 Fix exceeding th...

2016-05-16 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/158#issuecomment-219629301 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

[GitHub] incubator-eagle pull request: Eagle 135 : Run Eagle on Spark Strea...

2016-05-17 Thread qingwen220
Github user qingwen220 commented on the pull request: https://github.com/apache/incubator-eagle/pull/189#issuecomment-219639303 Found the tests failed. A bug was found. Seems recently merges break the code. That's bad. Could you help to fix it 😄 --- If your proje

  1   2   3   >