[GitHub] metron issue #933: METRON-1452 Rebase Dev Environment on Latest CentOS 6

2018-02-21 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/933 @mmiklavc - the special cut has a larger swap volume than the standard image. It also is customized to what Metron required at the time to run. That's pretty much it. If that's still desirable

[GitHub] metron issue #784: METRON-1229: Management UI type is part of the declaratio...

2017-10-05 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/784 +1 by inspection. ---

[GitHub] metron issue #680: METRON-1078: Metron Indexing fails to stop during "Stop A...

2017-08-14 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/680 @justinleet (or anyone really), I'd like to push this today, still 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

[GitHub] metron issue #680: METRON-1078: Metron Indexing fails to stop during "Stop A...

2017-08-07 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/680 @justinleet - this seems better. The change is to not error when the rest service is unavailable but use that as an indication that the topologies are stopped. The rest is code formatting because

[GitHub] metron issue #680: DO NOT MERGE: METRON-1078: Metron Indexing fails to stop ...

2017-08-04 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/680 Oh yeah, thanks for clarifying, @anandsubbu - I'm not going to change the 'happy path' logic, so when the UI is up, it will work the way it did prior. --- If your project is set up for it, you

[GitHub] metron issue #680: DO NOT MERGE: METRON-1078: Metron Indexing fails to stop ...

2017-08-04 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/680 I'm starting to think the best approach would to assume the service is stopped if we can't get to the ui (which we kind of do anyway because of how status works). Anybody absolutely hate

[GitHub] metron issue #680: METRON-1078: Metron Indexing fails to stop during "Stop A...

2017-08-04 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/680 Thanks @justinleet - I'm going to hold off on this, @anandsubbu did some additional testing and it didn't actually solve the problem. :( --- If your project is set up for it, you can reply

[GitHub] metron issue #678: METRON-1076: Management UI status check always reports th...

2017-08-03 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/678 +1 by inspection. Good stuff, much improved, thanks. --- 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

[GitHub] metron issue #481: METRON-322 Global Batching and Flushing

2017-07-20 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/481 My test environment is being funky, I'm +1 by inspection- thanks for the work, please don't let me hold you up any more. --- If your project is set up for it, you can reply to this email

[GitHub] metron issue #481: METRON-322 Global Batching and Flushing

2017-07-18 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/481 @mattf-horton - sorry about the hurry up and wait. I'll run this up tomorrow. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] metron issue #481: METRON-322 Global Batching and Flushing

2017-07-07 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/481 Awesome! I can run it up on a live system and see what's what. Thanks! --- 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

[GitHub] metron issue #481: METRON-322 Global Batching and Flushing

2017-07-06 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/481 @mattf-horton - Can I offer some help getting back to this? Is a pretty important fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] metron issue #627: METRON-1007 ambari integration for metron-management

2017-07-01 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/627 +1 (inspection). Nice one, Otto, thanks. Just bumped into this yesterday. --- 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

[GitHub] metron issue #614: METRON-992: Create performance tuning guide

2017-06-29 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/metron/pull/614 +1. Used this with a largish instance, worked well, thanks! --- 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

[GitHub] metron pull request #581: METRON-844: Install Metron Management UI with Amba...

2017-05-26 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/metron/pull/581#discussion_r118778981 --- Diff: metron-interface/metron-config/scripts/package.json --- @@ -9,7 +9,8 @@ "http-proxy-middleware": "0.17.4",

[GitHub] incubator-metron pull request #576: METRON-945: Resolve merge conflict in me...

2017-05-09 Thread dlyle65535
GitHub user dlyle65535 opened a pull request: https://github.com/apache/incubator-metron/pull/576 METRON-945: Resolve merge conflict in metron.spec ## Contributor Comments This changeset will allow the rpm build to complete. Currently, master will fail trying to build the rpms

[GitHub] incubator-metron pull request #574: METRON-934: Component and task id are mi...

2017-05-09 Thread dlyle65535
GitHub user dlyle65535 opened a pull request: https://github.com/apache/incubator-metron/pull/574 METRON-934: Component and task id are missing in the indexing topology Hdfs file names. ## Contributor Comments Prior to this changeset, all files written to hdfs will have

[GitHub] incubator-metron pull request #563: METRON-840: All "ambari_*" hosts need to...

2017-05-03 Thread dlyle65535
GitHub user dlyle65535 opened a pull request: https://github.com/apache/incubator-metron/pull/563 METRON-840: All "ambari_*" hosts need to have a /localrepo folder ## Contributor Comments Fixes broken EC2 deployment. With the default config, each host running needs

[GitHub] incubator-metron pull request #561: METRON-913: Create IP Report in Zeppelin

2017-05-02 Thread dlyle65535
GitHub user dlyle65535 opened a pull request: https://github.com/apache/incubator-metron/pull/561 METRON-913: Create IP Report in Zeppelin ## Contributor Comments For a given IP address, produces a report of: * Most Frequent Connections (Yaf, defaults to 24 hours

[GitHub] incubator-metron issue #552: METRON-857 FIX: Full_Dev regression: use the pr...

2017-04-26 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/552 Ditto. +1, thanks Otto! --- 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

[GitHub] incubator-metron issue #459: METRON-726: Clean up mvn site generation

2017-04-26 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/459 I think you're suffering from the same intermittent issues we have on master. Concur that you didn't add anything that obviously would create a problem and you may have helped. Maybe

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/538 +1 - pulled in the dependency changes, deleted my .m2 cache and the build ran as expected. Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-metron pull request #544: METRON-866 Quickdev and Fulldev readme m...

2017-04-24 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/544#discussion_r112950226 --- Diff: metron-deployment/vagrant/full-dev-platform/README.md --- @@ -24,35 +26,38 @@ Any platform that supports these tools is suitable

[GitHub] incubator-metron pull request #543: METRON-857 Ability to completely build p...

2017-04-22 Thread dlyle65535
Github user dlyle65535 commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/543#discussion_r112808661 --- Diff: metron-deployment/packaging/docker/rpm-docker/pom.xml --- @@ -227,4 +183,97

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-21 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/538 Also @mmiklavc, please remember if the intended effect was to make the rpm build happen after everything else completed then install + pom dependencies didn't work. If you take a close

[GitHub] incubator-metron issue #538: METRON-868 Fix documentation on building RPMs

2017-04-20 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/538 In fact, having given it additional thought, the dependency mechanism is a way of saying "hey Maven, make sure you pull those artifacts into our .m2 cache" which is

[GitHub] incubator-metron issue #540: METRON-869 Include build instructions for full-...

2017-04-20 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/540 You shouldn't have to build Metron prior to running 'vagrant up', the Ansible provisioner does that. Following these instructions, you'll actually build Metron twice for each build