Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/452
@garrettlish please add description to 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
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/442
@garrettlish can you please add more information to this PR so people can
learn what are the changes for?
---
If your project is set up for it, you can reply to this email and have your
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/438
@haoch better please to explain why view path is duplicated.
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/437
@garrettlish can you please add good information to 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
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/433
@wujinhu can you elaborate what are the changes for in this PR? "Fix
conflict" is a very generic title for one PR :-)
---
If your project is set up for it, you can reply to this
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/432
@qingwen220 can you please add good description for this big 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
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/430
@wujinhu can you please add description for people to understand the
changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/429
@wujinhu can you please add sufficient description here for people to
understand what aggregation framework is used for?
---
If your project is set up for it, you can reply to this email
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/428
@garrettlish is this a duplicated PR? or do we need input more valuable
description here?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/426
@pkuwm this is important PR for absence alert, can you please add detailed
description for people to learn what are the changes?
---
If your project is set up for it, you can reply to
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/423
@garrettlish can you please add more description to this PR and
corresponding JIRA ticket EAGLE-531? This PR looks pretty important for people
to learn alert deduplication.
---
If your
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/419
@wujinhu this is pretty important PR, can you add good description so
people who are interested in can understand what are those changes for.
---
If your project is set up for it, you can
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/416
@qingwen220 it is a big PR, can you please add description so later on
people can reference to see what really happened.
---
If your project is set up for it, you can reply to this email
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/413
That is fine. I found more details in JIRA
1. no data for an entire stream won't trigger gap alert (use local time &
batch window)
2. no data policies definition is to
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/413
@garrettlish thanks for the PR. Can you please add more description into
this PR. This PR is pretty important and we believe that people will look at
this PR to see what really happened
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/461
I have reviewed the code, basically it should be working, but there might
be some concurrency issue. But even in this case, I think you can merge this
PR, and refine them afterwards
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/402
@DadanielZ that is why there is already AbstractHdfsAuditLogApplication
there, and MapRApplication can extend this and implement getParserBolt method
---
If your project is set up for it
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/403
sure, we can remove that method
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/403
We may not change the code to use regex matching as it is too slow. We have
achieved more than 10 times performance gain by using string operations than
using regex.
This is critical
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/399
Thanks for the fix. We can remove the internal try/catch and only use outer
try/catch to catch exception and finally it will do ack anyway
---
If your project is set up for it, you can
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/402
Thanks for the improvement. Why we create a new
AbstractMapRFSAuditLogApplication, can't we use
AbstractHdfsAuditLogApplication? They share the same code
---
If your project is s
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/407
good fix, but Senthil can you do PR against develop 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
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/392
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/392
hdfs audit log + alert engine end-2-end
https://issues.apache.org/jira/browse/EAGLE-481
You can merge this pull request into a Git repository by running:
$ git pull https
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/391
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/391
Make alert engine topology to be one application so to be managed by Eagle
API
https://issues.apache.org/jira/browse/EAGLE-481
You can merge this pull request into a Git repository by
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/389
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/389
Create alert metadata based on application stream sink configuration
https://issues.apache.org/jira/browse/EAGLE-493
https://issues.apache.org/jira/browse/EAGLE-490
You can merge this
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/363
great fix and 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 this feature
enabled and
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/368
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 this feature
enabled and wishes so, or
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/368
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/368
use guice module better in app framework
https://issues.apache.org/jira/browse/EAGLE-487
You can merge this pull request into a Git repository by running:
$ git pull https
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/367
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/367
use guice module better in app framework
EAGLE-487
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/yonzhang/incubator-eagle develop
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/366
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/366
eagle-data-process project clean up to contain only common processingâ¦
EAGLE-477
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/364
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/364
clean eagle-data-process project
https://issues.apache.org/jira/browse/EAGLE-477
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/357
LGTM and 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 this feature
enabled and
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/362
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/362
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 this feature
enabled and wishes so, or
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/359
LGTM and 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 this feature
enabled and
GitHub user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/362
app framework improvement of appId, jarPath
https://issues.apache.org/jira/browse/EAGLE-474
You can merge this pull request into a Git repository by running:
$ git pull https
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/358
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/358
data enrichment refactor in eagle security monitoring
EAGLE-470, make enrichment easy to use
You can merge this pull request into a Git repository by running:
$ git pull https
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/353
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/353
EAGLE-469 make dataSource configuration extremely simple
https://issues.apache.org/jira/browse/EAGLE-469
You can merge this pull request into a Git repository by running:
$ git pull
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/350
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/350
clean up eagle-application-service
https://issues.apache.org/jira/browse/EAGLE-468
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/344
LGTM. We keep hadoop-jmx project there to put all related policies,
metadata there.
But this application does not need any pre-processing logic
---
If your project is set up for it
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/349
I still have concern with why Hive can't share spout with JPM applications,
but we can refactor after this PR
---
If your project is set up for it, you can reply to this email and
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/343
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/343
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 this feature
enabled and wishes so, or
GitHub user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/343
clean eagle-data-process
clean unused classes in favor of new app framework
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/yonzhang
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/342
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/342
fix build issue because of PR missing file
PR missed some files because of too many deleted files
You can merge this pull request into a Git repository by running:
$ git pull https
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/341
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/341
clean inner process alert engine code
https://issues.apache.org/jira/browse/EAGLE-449
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/340
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 this feature
enabled and wishes so, or
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/340
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/340
gclog and metric collection conversion
https://issues.apache.org/jira/browse/EAGLE-444
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/339
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/339
convert eagle-gc app to use new app framework
https://issues.apache.org/jira/browse/EAGLE-444
You can merge this pull request into a Git repository by running:
$ git pull https
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/338
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/338
hive query log monitoring application using new app framework
https://issues.apache.org/jira/browse/EAGLE-446
You can merge this pull request into a Git repository by running:
$ git
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/335
LGTM. but to complete application conversion, we need complete
configuration in metadata.xml like application.conf. Future, we may be able to
reduce this duplication.
---
If your project
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/337
Good to see that stateCheck can be aligned with policy evaluation, that
would be easy for user to define state rules. LGTM.
---
If your project is set up for it, you can reply to this
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/316
Thanks for the contribution, Peter, it is merged in
https://github.com/apache/incubator-eagle/commit/502c7e37f842eabc4021c47053dd95bdd1c7efde
Please close this PR.
---
If your project
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/332
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/331
@Override
public void register(ModuleRegistry registry) {
registry.register(MemoryMetadataStore.class, new AbstractModule() {
@Override
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/331
-1 for the early abstraction, we should still use existing implementation
and push abstraction after we tested all the applications.
---
If your project is set up for it, you can reply to
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/334
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 this feature
enabled and wishes so, or
GitHub user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/334
hdfs,hbase,mapr app conversion
https://issues.apache.org/jira/browse/EAGLE-445
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/333
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/332
convert eagle-hdfs-auditlog app to use new app framework
https://issues.apache.org/jira/browse/EAGLE-445
You can merge this pull request into a Git repository by running:
$ git pull
GitHub user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/333
convert eagle-hdfs-auditlog app to use new app framework
https://issues.apache.org/jira/browse/EAGLE-445
You can merge this pull request into a Git repository by running:
$ git pull
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/326
reviewed, good to go
---
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
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/321
Thanks for contributing this. I have merged that please 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
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/323
I have concern with module scope, application only needs know what is
current metadata store type, looks there is no need for scope. Do you have
other use cases where application need know
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/327
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/327
Already reviewed by Hao in patch 315
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/327
Eagle assembly for new app framework
https://issues.apache.org/jira/browse/EAGLE-437
https://issues.apache.org/jira/browse/EAGLE-430
You can merge this pull request into a Git
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/315
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/315
explore new packaging method for eagle using new app framework
https://issues.apache.org/jira/browse/EAGLE-430
You can merge this pull request into a Git repository by running:
$ git
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/312
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/314
Good refinement, good to go
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/312
That is obvious. Again this is to make first use case to work, I will leave
this change to app framework refine.
---
If your project is set up for it, you can reply to this email and have
GitHub user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/312
Hbase monitoring metadata browse API in new app framework
https://issues.apache.org/jira/browse/EAGLE-428
You can merge this pull request into a Git repository by running:
$ git pull
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/310
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/310
Load app web classes and app topology classes using configured lib dir
https://issues.apache.org/jira/browse/EAGLE-426
You can merge this pull request into a Git repository by running
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/307
LM-SJN-00876985:incubator-eagle yonzhang$ git apply 307.patch
307.patch:59: trailing whitespace.
/*
307.patch:60: trailing whitespace.
* Licensed to the Apache Software
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/307
PR apply will fail if this PR includes some new files which are already
there.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/308
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 this feature
enabled and wishes so, or
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/308
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/308
That is right, we don't need notify application of the web related
configuration. Web application should do it by itself by querying
ApplicationEntity, I will modify the PR
---
If
GitHub user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/308
HBase audit monitoring with new app framework
https://issues.apache.org/jira/browse/EAGLE-420
You can merge this pull request into a Git repository by running:
$ git pull https
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/304
---
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 user yonzhang opened a pull request:
https://github.com/apache/incubator-eagle/pull/304
Stream tumbling aggregator library
https://issues.apache.org/jira/browse/EAGLE-410
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user yonzhang closed the pull request at:
https://github.com/apache/incubator-eagle/pull/301
---
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 user yonzhang commented on the issue:
https://github.com/apache/incubator-eagle/pull/301
ok, fixed, and will create another 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
1 - 100 of 252 matches
Mail list logo
