Congratulations!
On October 20, 2017 at 12:10:42, James Sirota (jsir...@apache.org) wrote:
The Project Management Committee (PMC) for Apache Metron
has invited Raghu Mitra to become a committer and we are pleased
to announce that he has accepted.
Being a committer enables easier
Congratulations, Raghu!
Jon
On Fri, Oct 20, 2017, 12:11 Simon Elliston Ball
wrote:
> Congratulations Raghu. Well deserved with all that awesome UI work that’s
> coming in.
>
> Simon
>
> > On 20 Oct 2017, at 17:10, James Sirota wrote:
> >
> >
>
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/810
+1 by inspection
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/804
+1 by inspection, great job here.
---
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/800
---
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/795
---
Github user cestella closed the pull request at:
https://github.com/apache/metron/pull/797
---
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/795
+1 It looks great. Thanks!
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/795
Alright, changes included. Did I miss anything @nickwallen ?
---
GitHub user dbist opened a pull request:
https://github.com/apache/metron/pull/810
METRON-1270 fix for warnings missing @return tag argument in
metron-analytics/metron-profiler-common and metron-profiler-client
## Contributor Comments
[WARNING]
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/808
---
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146031435
--- Diff:
metron-platform/metron-zookeeper/src/main/java/org/apache/metron/zookeeper/ZKCache.java
---
@@ -0,0 +1,141 @@
+/**
+ * Licensed to the
GitHub user nickwallen opened a pull request:
https://github.com/apache/metron/pull/809
METRON-1266 Profiler - SASL Authentication Failed
When running the Profiler on a cluster that has multiple nodes and is
secured by Kerberos, it was observed that the HBaseBolt was unable to
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/806
+1 by inspection. Thank for contributing!
---
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146021528
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/zookeeper/configurations/ProfilerUpdater.java
---
@@ -0,0 +1,114 @@
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146021013
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/zookeeper/configurations/ProfilerUpdater.java
---
@@ -0,0 +1,114 @@
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146020850
--- Diff:
metron-platform/metron-zookeeper/src/main/java/org/apache/metron/zookeeper/ZKCache.java
---
@@ -0,0 +1,141 @@
+/**
+ * Licensed to the
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146020712
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/zookeeper/configurations/IndexingUpdater.java
---
@@ -0,0 +1,88 @@
+/**
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146020428
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/zookeeper/configurations/ProfilerUpdater.java
---
@@ -0,0 +1,114 @@
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/808
+1 by inspection. Thanks for fixing this. It can be a little annoying.
---
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146019541
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/zookeeper/configurations/ProfilerUpdater.java
---
@@ -0,0 +1,114 @@
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146018459
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/zookeeper/configurations/ProfilerUpdater.java
---
@@ -0,0 +1,114 @@
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/795
Alright, great reviews. I think I have reacted to the comments; let me
know if I missed anything.
---
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146012465
--- Diff:
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java
---
@@ -179,6 +181,8 @@ public
Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r146010894
--- Diff:
metron-platform/metron-zookeeper/src/main/java/org/apache/metron/zookeeper/ZKCache.java
---
@@ -0,0 +1,141 @@
+/**
+ * Licensed to the
Congratulations Raghu. Well deserved with all that awesome UI work that’s
coming in.
Simon
> On 20 Oct 2017, at 17:10, James Sirota wrote:
>
>
>
> The Project Management Committee (PMC) for Apache Metron
> has invited Raghu Mitra to become a committer and we are pleased
The Project Management Committee (PMC) for Apache Metron
has invited Raghu Mitra to become a committer and we are pleased
to announce that he has accepted.
Being a committer enables easier contribution to the
project since there is no need to go via the patch
submission process. This should
Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/metron/pull/795#discussion_r145992573
--- Diff:
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/StormControllerIntegrationTest.java
---
@@ -179,6 +181,8 @@ public
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/795
I tested this thoroughly and everything works as expected. I made one
small comment that I feel is optional. This is an awesome PR. +1
---
29 matches
Mail list logo