GitHub user cestella opened a pull request:
https://github.com/apache/incubator-metron/pull/150
METRON-222: Address ConcurrentModificationException in BulkMessageWriter
You can merge this pull request into a Git repository by running:
$ git pull
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/142#discussion_r66877352
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/aggregator/Aggregators.java
---
@@ -18,18 +18,21 @@
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/142#discussion_r66876932
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/query/generated/PredicateLexer.java
---
@@ -114,59 +118,74 @@
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/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 user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/146
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/149
---
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 asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/141
---
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 nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/142#discussion_r66814113
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/aggregator/Aggregators.java
---
@@ -18,18 +18,21 @@
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/142#discussion_r66812462
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/query/generated/PredicateLexer.java
---
@@ -114,59 +118,74 @@
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/141
Good fixes. ð
---
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 cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/142
So, I think the main disadvantage of using groovy or any of the scripting
engine integrations is speed and expressibility. Having a targeted domain
specific language is going to be
11 matches
Mail list logo
