Jira Rights

2016-10-28 Thread Otto Fowler
How can I get rights in jira to assign issues to myself etc? Otto

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/318 @ottobackwards - concur. If it exceeds the start/stop timeout (defaults to 30 seconds), Monit will terminate the start/stop process and try again. So, 60 worked on my larger machines

[GitHub] incubator-metron issue #327: METRON-521: Stellar function documentation need...

2016-10-28 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/327 +1 pending the mentioned corrections. --- 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

[GitHub] incubator-metron pull request #327: METRON-521: Stellar function documentati...

2016-10-28 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/incubator-metron/pull/327#discussion_r85597596 --- Diff: metron-platform/metron-common/README.md --- @@ -332,24 +346,24 @@ The following functions are supported: * Description: Transforms

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/318 I think this monit timeout issue is part of the problem on low resource machines, and with 'zombie' storm threads being left behind --- If your project is set up for it, you can

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/318 @mmiklavc PR sent. 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 this feature

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/318 A note for the community - the /tmp file problem **did** reoccur for us. As it turns out, the timeout default for starting up topologies in Monit was set too low. Normally, Storm cleans

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/318 @dlyle65535 I don't have a strong opinion on it. I'm giving attribution to @justinleet on this PR, since he laid the foundation of pretty much everything here. If you file separately we

Re: [DISCUSS] Build Log Usability

2016-10-28 Thread David Lyle
For clarity, I meant to say "any of the exhaust logging from the integration testing framework". The logging from the tests are the good stuff. :) On Fri, Oct 28, 2016 at 3:13 PM, David Lyle wrote: > I think you noticed the main problem with turning logging off entirely.

Re: [DISCUSS] Build Log Usability

2016-10-28 Thread David Lyle
I think you noticed the main problem with turning logging off entirely. I'd be inclined to have two files: one which defaults to INFO and another that defaults to ERROR for Travis. We can give a -Dlog4j.configuration=file:log4j.config.set.to.ERROR.only for travis, which I think Otto suggested.

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/318 @mmiklavc - I had to make a small tweak to the Quick Dev Vagrantfile for the new image. It's backwardly compatible, fwiw. Just added ambari-slave to the default tags. Do you

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/318 Yes. The Packer stuff is part of Metron and those instructions are in a

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/318 Are there instructions for doing either of those things? --- 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] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/318 This ran well on EC2- deployment was good, expected data flow was good, Kafka offset tracking worked as expected. I'm +1, but there's some things to do prior to pulling this in

[GitHub] incubator-metron issue #330: Metron 506 - Add Otto Fowler to committers

2016-10-28 Thread ottobackwards
Github user ottobackwards commented on the issue: https://github.com/apache/incubator-metron/pull/330 Kicking travis --- 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

[GitHub] incubator-metron pull request #330: Metron 506 - Add Otto Fowler to committe...

2016-10-28 Thread ottobackwards
GitHub user ottobackwards reopened a pull request: https://github.com/apache/incubator-metron/pull/330 Metron 506 - Add Otto Fowler to committers Add Otto Fowler to committers list on main site You can merge this pull request into a Git repository by running: $ git pull

Re: [DISCUSS] Build Log Usability

2016-10-28 Thread Otto Fowler
What if we had a set of log4j.properties just for travis? I think we can script travis and deploy them then. We could have full logging for local builds, and travis specific high level logging at the same time. On October 27, 2016 at 20:52:46, David Lyle (dlyle65...@gmail.com) wrote: I'd

[GitHub] incubator-metron pull request #325: METRON-512 up default junit to 4.12

2016-10-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/325 --- 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 if the

[GitHub] incubator-metron issue #318: METRON-495: Upgrade Storm to 1.0.x

2016-10-28 Thread mmiklavc
Github user mmiklavc commented on the issue: https://github.com/apache/incubator-metron/pull/318 Before we accept this, I want to point out that I've changed the dependencies_with_url.csv file and that it's probably worth a look. --- If your project is set up for it, you can reply

[GitHub] incubator-metron pull request #330: Metron 506 - Add Otto Fowler to committe...

2016-10-28 Thread ottobackwards
GitHub user ottobackwards opened a pull request: https://github.com/apache/incubator-metron/pull/330 Metron 506 - Add Otto Fowler to committers Add Otto Fowler to committers list on main site You can merge this pull request into a Git repository by running: $ git pull

METRON-378 - variables that don't exist with STELLAR built in arithmetic functions

2016-10-28 Thread Otto Fowler
I was looking at METRON-378, and was wondering if we could discuss a little bit. This issue, as stated is that if your stellar expression references variables that are not passed in at runtime, the call still succeeds and passes back a value. >From the issue: @Test(expected =

[GitHub] incubator-metron pull request #315: METRON-464: Force co-location of all Met...

2016-10-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-metron/pull/315 --- 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 if the