[GitHub] incubator-eagle pull request: EAGLE-280 Update logstash-kafka-conf...

2016-04-26 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/166 EAGLE-280 Update logstash-kafka-conf.md added a logstash sample file for logstash2.x version You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-eagle pull request: EAGLE-285 Connector to be used by ot...

2016-04-29 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/168 EAGLE-285 Connector to be used by other program to interact with Eagle - Created a module eagle-connector for external service to use - Created java classes for external service to

[GitHub] incubator-eagle pull request: EAGLE-285 Connector to be used by ot...

2016-04-29 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/169 EAGLE-285 Connector to be used by other program to interact with Eagle - Created a module eagle-connector for external service to use - Created java classes for external service to

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

2016-05-03 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/175 This pull request is just for MapR integration code review - This is my local branch for MapR integration Solved: - EAGLE-282:Auditlogparser for MapR's audi

[GitHub] incubator-eagle pull request: EAGLE-280 EAGLE-284 Added a profile ...

2016-05-06 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/179 EAGLE-280 EAGLE-284 Added a profile in pom.xml for MapR dependency and updated doc for logstash EAGLE-280:Update logstash-kafka-conf.md - Updated logstash-kafka-conf.md and included

[GitHub] incubator-eagle pull request: EAGLE-280 EAGLE-284 Added a profile ...

2016-05-06 Thread DadanielZ
Github user DadanielZ commented on the pull request: https://github.com/apache/incubator-eagle/pull/179#issuecomment-217587085 please 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

[GitHub] incubator-eagle pull request: EAGLE-285 Connector to be used by ot...

2016-05-11 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/168 --- 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-285 Connector to be used by ot...

2016-05-11 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/169 --- 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-280 EAGLE-284 Added a profile ...

2016-05-11 Thread DadanielZ
Github user DadanielZ commented on the pull request: https://github.com/apache/incubator-eagle/pull/179#issuecomment-218620439 Can someone review it? 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

[GitHub] incubator-eagle pull request: EAGLE-280 EAGLE-284 Added a profile ...

2016-05-16 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/179 --- 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-280 EAGLE-284 Added a profile ...

2016-05-17 Thread DadanielZ
GitHub user DadanielZ reopened a pull request: https://github.com/apache/incubator-eagle/pull/179 EAGLE-280 EAGLE-284 Added a profile in pom.xml for MapR dependency and updated doc for logstash EAGLE-280:Update logstash-kafka-conf.md - Updated logstash-kafka-conf.md and

[GitHub] incubator-eagle pull request: EAGLE-280 EAGLE-284 Added a profile ...

2016-05-25 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/179 --- 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-280 EAGLE-282 EAGLE-284 EAGLE-...

2016-05-25 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/200 EAGLE-280 EAGLE-282 EAGLE-284 EAGLE-307 EAGLE-280:Update logstash-kafka-conf.md - Updated logstash-kafka-conf.md and included a sample of logstash config file for logstash version 2

[GitHub] incubator-eagle pull request: EAGLE-280 EAGLE-282 EAGLE-284 EAGLE-...

2016-05-25 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/200 --- 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-280 EAGLE-282 EAGLE-284 EAGLE-...

2016-05-25 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/201 EAGLE-280 EAGLE-282 EAGLE-284 EAGLE-307: Mapr integration EAGLE-280:Update logstash-kafka-conf.md Updated logstash-kafka-conf.md and included a sample of logstash config file

[GitHub] incubator-eagle pull request: EAGLE-280 EAGLE-282 EAGLE-284 EAGLE-...

2016-05-26 Thread DadanielZ
Github user DadanielZ commented on the pull request: https://github.com/apache/incubator-eagle/pull/201#issuecomment-221936747 commits merged --- 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-280 EAGLE-282 EAGLE-284 EAGLE-...

2016-05-26 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/201 --- 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-315: Add tutorial for mapr aud...

2016-05-26 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/202 EAGLE-315: Add tutorial for mapr audit log monitoring - Added docs for users to enable auditing in MapR to get FSaudit logs You can merge this pull request into a Git repository by

[GitHub] incubator-eagle pull request: EAGLE-315: Add tutorial for mapr aud...

2016-05-27 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/202 --- 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 #166: EAGLE-280 Update logstash-kafka-conf.md

2016-06-02 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/166 --- 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 issue #219: EAGLE-325 Combine multiple web resource into cen...

2016-06-06 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/219 Hi Hao, I think you forgot to add ".github/" in eagle-parent pom for ".github/PULL_REQUEST_TEMPLATE.md". It didn't pass my local license check. ---

[GitHub] incubator-eagle pull request #234: Update "logstash-kafka-conf.md"

2016-06-09 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/234 Update "logstash-kafka-conf.md" in logstash conf file, "sincedb_path" should be full path of a file, not a dir. You can merge this pull request into a Git r

[GitHub] incubator-eagle pull request #234: Update "logstash-kafka-conf.md"

2016-06-16 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/234 --- 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 #300: EAGLE-283 Convert fid,uid in MapR's audit...

2016-08-04 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/300 EAGLE-283 Convert fid,uid in MapR's audit log to FIle/folder name, user name 1. Added resolver for user's input of "volume", "file/folder", "Status&quo

[GitHub] incubator-eagle issue #300: EAGLE-283 Convert fid,uid in MapR's audit log to...

2016-08-04 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/300 As I mentioned, user id conversion is a little bit tricky, it will be great if someone can give me a hint on user id conversion issue. Thanks and regards, Da --- If your

[GitHub] incubator-eagle issue #300: EAGLE-283 Convert fid,uid in MapR's audit log to...

2016-08-04 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/300 By the way, "id converter" will make http request to mapr host. As mapr use https connection, " javax.net.ssl.SSLHandshakeException Error" will occur if

[GitHub] incubator-eagle pull request #300: EAGLE-283 Convert fid,uid in MapR's audit...

2016-08-04 Thread DadanielZ
Github user DadanielZ commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/300#discussion_r73637287 --- Diff: eagle-webservice/src/main/resources/application.conf --- @@ -16,13 +16,24 @@ { eagle { service

[GitHub] incubator-eagle pull request #300: EAGLE-283 Convert fid,uid in MapR's audit...

2016-08-04 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/300 --- 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 #300: EAGLE-283 Convert fid,uid in MapR's audit...

2016-08-04 Thread DadanielZ
GitHub user DadanielZ reopened a pull request: https://github.com/apache/incubator-eagle/pull/300 EAGLE-283 Convert fid,uid in MapR's audit log to FIle/folder name, user name 1. Added resolver for user's input of "volume", "file/folder", "Statu

[GitHub] incubator-eagle issue #300: EAGLE-283 Convert fid,uid in MapR's audit log to...

2016-08-04 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/300 @haoch thanks for your advice! XD Working on a feature branch will be a good practice. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-eagle pull request #300: EAGLE-283 Convert fid,uid in MapR's audit...

2016-08-05 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/300 --- 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 #307: EAGLE-283: Convert fid,uid in MapR's audi...

2016-08-05 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/307 EAGLE-283: Convert fid,uid in MapR's audit log to FIle/folder name, user name 1. Added resolver for user's input of "volume", "file/folder", "Statu

[GitHub] incubator-eagle issue #307: EAGLE-283: Convert fid,uid in MapR's audit log t...

2016-08-07 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/307 @yonzhang to make it clean, I will close this pr and create a new one. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-eagle pull request #307: EAGLE-283: Convert fid,uid in MapR's audi...

2016-08-07 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/307 --- 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 #321: EAGLE-283: Convert fid,uid in MapR's audi...

2016-08-09 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/321 EAGLE-283: Convert fid,uid in MapR's audit log to FIle/folder name, user name 1. Added resolver for user's input of "volume", "file/folder", "Statu

[GitHub] incubator-eagle issue #321: EAGLE-283: Convert fid,uid in MapR's audit log t...

2016-08-10 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/321 Thank you Edward. Closing this pr as it has been merged. --- 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

[GitHub] incubator-eagle pull request #321: EAGLE-283: Convert fid,uid in MapR's audi...

2016-08-10 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/321 --- 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 issue #365: Test email alert publisher: configuration and co...

2016-08-24 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/365 merged, please close 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

[GitHub] incubator-eagle issue #375: [EAGLE-492] Fix negative total execution time an...

2016-08-24 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/375 Patch apply failed, please fix it first. error: patch failed: eagle-jpm/eagle-jpm-entity/src/main/java/org/apache/eagle/jpm/spark/crawl/JHFSparkEventReader.java:21 error

[GitHub] incubator-eagle issue #375: [EAGLE-492] Fix negative total execution time an...

2016-08-24 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/375 merged, plz close this PR. --- 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 issue #375: [EAGLE-492] Fix negative total execution time an...

2016-08-25 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/375 @haoch Thank you, this is very useful 👍 --- 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 issue #385: [EAGLE-499] Fix flushing entities for spark hist...

2016-08-25 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/385 @pkuwm merged, plz close this PR. --- 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 #402: Eagle-447 convert eagle-maprfs app to use...

2016-08-29 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/402 Eagle-447 convert eagle-maprfs app to use new app framework eagle-maprfs app should use new app framework to manage its lifecycle You can merge this pull request into a Git repository

[GitHub] incubator-eagle issue #402: Eagle-447 convert eagle-maprfs app to use new ap...

2016-08-30 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/402 @yonzhang Their code are almost the same except that: "AbstractHdfsAuditLogApplication" use "HdfsAuditLogParserBolt", "AbstractMapRFSAuditL

[GitHub] incubator-eagle issue #402: Eagle-447 convert eagle-maprfs app to use new ap...

2016-08-30 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/402 @haoch thanks for the review, will add them later. --- 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 issue #402: Eagle-447 convert eagle-maprfs app to use new ap...

2016-08-30 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/402 @yonzhang, I see now. In “ AbstractHdfsAuditLogApplication.java”, there is an overrided function which use "HdfsAuditLogParserBolt" directly: ``` @Override

[GitHub] incubator-eagle pull request #402: Eagle-447 convert eagle-maprfs app to use...

2016-08-30 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/402 --- 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 #402: Eagle-447 convert eagle-maprfs app to use...

2016-08-30 Thread DadanielZ
GitHub user DadanielZ reopened a pull request: https://github.com/apache/incubator-eagle/pull/402 Eagle-447 convert eagle-maprfs app to use new app framework eagle-maprfs app should use new app framework to manage its lifecycle You can merge this pull request into a Git repository

[GitHub] incubator-eagle pull request #402: Eagle-447 convert eagle-maprfs app to use...

2016-08-30 Thread DadanielZ
Github user DadanielZ closed the pull request at: https://github.com/apache/incubator-eagle/pull/402 --- 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 #410: Eagle-447 convert eagle-maprfs app to use...

2016-08-30 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/410 Eagle-447 convert eagle-maprfs app to use new app framework eagle-maprfs app should use new app framework to manage its lifecycle - in AbstractHdfsAuditLogApplication, instead

[GitHub] incubator-eagle pull request #412: This is for code review only: EAGLE-432 A...

2016-08-31 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/412 This is for code review only: EAGLE-432 Application status monitoring App framework needs to probe application status periodically or on demand - this is based on dropwizard+guava

[GitHub] incubator-eagle issue #412: This is for code review only: EAGLE-432 Applicat...

2016-08-31 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/412 hi Team, I am working on application status part and here is an initial code review, just want make sure all corner cases are covered and the work flow is correct. My idea is

[GitHub] incubator-eagle pull request #412: This is for code review only: EAGLE-432 A...

2016-09-07 Thread DadanielZ
Github user DadanielZ commented on a diff in the pull request: https://github.com/apache/incubator-eagle/pull/412#discussion_r77945175 --- Diff: eagle-core/eagle-app/eagle-app-base/src/main/java/org/apache/eagle/app/service/impl/ApplicationManagementServiceImpl.java

[GitHub] incubator-eagle issue #412: This is for code review only: EAGLE-432 Applicat...

2016-09-07 Thread DadanielZ
Github user DadanielZ commented on the issue: https://github.com/apache/incubator-eagle/pull/412 @haoch Thank you for your 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 #461: [EAGLE-432] Application status monitoring

2016-09-27 Thread DadanielZ
GitHub user DadanielZ opened a pull request: https://github.com/apache/incubator-eagle/pull/461 [EAGLE-432] Application status monitoring Be sure to do all of the following to help us incorporate your contribution quickly and easily: - [ ] Make sure the PR