Github user sardell commented on the issue:
https://github.com/apache/metron/pull/1156
+1. Tested on full dev.
---
Github user ruffle1986 commented on the issue:
https://github.com/apache/metron/pull/1156
@merrimanr Looks good to me.
---
Github user sardell commented on the issue:
https://github.com/apache/metron/pull/1156
@merrimanr I reviewed and tested on full dev. I have the same thought as
@ruffle1986 regarding the cancelling of the subscription. We should add that
line inside the ngOnDestroy lifecycle hook. Besi
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/1156
Thanks @tiborm and @ruffle1986. I merged in the latest feature branch
which included the onDestroy code. I also fixed a regression introduced by
https://github.com/apache/metron/pull/1142 and ad