[GitHub] metron issue #888: METRON-1389: Zeppelin notebook import does not work with ...

2018-01-24 Thread ottobackwards 
Github user ottobackwards commented on the issue:

https://github.com/apache/metron/pull/888
  
+1 by inspection


---

[GitHub] metron issue #888: METRON-1389: Zeppelin notebook import does not work with ...

2018-01-24 Thread nickwallen 
Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/888
  
+1 by inspection.  Nice work @anandsubbu .

We'll need to figure out this intermittent test failure (impacting all PRs, 
not just yours) before we merge.





---

[GitHub] metron issue #888: METRON-1389: Zeppelin notebook import does not work with ...

2018-01-24 Thread anandsubbu 
Github user anandsubbu commented on the issue:

https://github.com/apache/metron/pull/888
  
@ottobackwards @cestella and @nickwallen - any other feedback on this PR? 

The travis failures seems to be in an unrelated piece of code.


---