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:
> >
> >
>
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
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/795
+1 It looks great. Thanks!
---
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/800
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/804
+1 by inspection, great job here.
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/810
+1 by inspection
---
Github user cestella closed the pull request at:
https://github.com/apache/metron/pull/797
---
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/795
---
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
---
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
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
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
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 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 the issue:
https://github.com/apache/metron/pull/808
+1 by inspection. Thanks for fixing this. It can be a little annoying.
---
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 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 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 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_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_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 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 justinleet commented on the issue:
https://github.com/apache/metron/pull/806
+1 by inspection. Thank for contributing!
---
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_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 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 cestella commented on the issue:
https://github.com/apache/metron/pull/795
Alright, changes included. Did I miss anything @nickwallen ?
---
29 matches
Mail list logo