[GitHub] incubator-metron issue #159: METRON-237 Remove metron-ui from the code base

2016-06-17 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/159 +1, looks good --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishe

[GitHub] incubator-metron issue #159: METRON-237 Remove metron-ui from the code base

2016-06-17 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/159 +1! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or i

[GitHub] incubator-metron issue #159: METRON-237 Remove metron-ui from the code base

2016-06-17 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/159 @dlyle65535 Ran on Vagrant quick dev - verified Kibana UI comes up and Storm topologies are running. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-metron issue #159: METRON-237 Remove metron-ui from the code base

2016-06-17 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/159 Good deal @mmiklavc - always happy to see dead code go away. Did you run this on Vagrant and/or EC2? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] incubator-metron issue #159: METRON-237 Remove metron-ui from the code base

2016-06-17 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/159 If we're going to remove metron-ui, which I completely support btw, we are going to need to trim down the sections in the LICENSE around metron-ui components. --- If your project is set