[GitHub] incubator-metron issue #233: METRON-396: Make Stellar function resolution ha...

2016-09-08 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/233 I had to address this issue as part of some other work. It is included in #242. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-metron issue #233: METRON-396: Make Stellar function resolution ha...

2016-09-06 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/233 Yeah, I see what you're saying; that's a good catch. You are doing some testing for initialization and you want to uninitialize. One thing we could do is provide a function in the

[GitHub] incubator-metron issue #233: METRON-396: Make Stellar function resolution ha...

2016-09-06 Thread nickwallen
Github user nickwallen commented on the issue: https://github.com/apache/incubator-metron/pull/233 I have been tracing an issue caused by static initialization. I have a Stellar function that uses values within the Context to perform initialization. I have multiple tests in the

[GitHub] incubator-metron issue #233: METRON-396: Make Stellar function resolution ha...

2016-09-04 Thread dlyle65535
Github user dlyle65535 commented on the issue: https://github.com/apache/incubator-metron/pull/233 +1, good deal. --- 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 issue #233: METRON-396: Make Stellar function resolution ha...

2016-09-02 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/233 Ok, merged master in. --- 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

[GitHub] incubator-metron issue #233: METRON-396: Make Stellar function resolution ha...

2016-09-02 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/233 Yeah getting to that; in a bit of a Internet connection blackhole On Fri, Sep 2, 2016 at 07:43 Nick Allen wrote: > FYI - Your branch needs merged

[GitHub] incubator-metron issue #233: METRON-396: Make Stellar function resolution ha...

2016-08-26 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/incubator-metron/pull/233 Ok, this should be a transparent change to the topology. Things should continue to function as expected. I also verified that this did not break pcap's `query` filter. As such,