[GitHub] metron issue #964: METRON-1491: The indexing topology restart logic is wrong

2018-03-15 Thread cestella
Github user cestella commented on the issue:

https://github.com/apache/metron/pull/964
  
Ok, created 
[METRON-1492](https://issues.apache.org/jira/browse/METRON-1492). 


---


[GitHub] metron issue #964: METRON-1491: The indexing topology restart logic is wrong

2018-03-15 Thread ottobackwards
Github user ottobackwards commented on the issue:

https://github.com/apache/metron/pull/964
  
Yes, lets get it in there


---


[GitHub] metron issue #964: METRON-1491: The indexing topology restart logic is wrong

2018-03-15 Thread cestella
Github user cestella commented on the issue:

https://github.com/apache/metron/pull/964
  
@ottobackwards yeah, definitely; I think that's ultimately where we want to 
go.  The first step to that is separating out these functions like I have in 
this PR.  The next is doing the ambari work which will utilize it.  Shall I 
create a follow-on JIRA?


---


[GitHub] metron issue #964: METRON-1491: The indexing topology restart logic is wrong

2018-03-15 Thread ottobackwards
Github user ottobackwards commented on the issue:

https://github.com/apache/metron/pull/964
  
Should we think about exposing them as separate things in ambari over all?  
Go all the way with this?


---