Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/946
thanks @mmiklavc
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/964
Yes, lets get it in there
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/964
Should we think about exposing them as separate things in ambari over all?
Go all the way with this?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/946
@cestella " Yeah, I think that's the approach, however, there's a snag.
Storm requires us to create uber jars, so probably what we want to do is have
users actually put the
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/946
would this have any effect on people using x-pack alternatives?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/946
wait, does this PR mean we *require* x-pack from now on?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/951
Why are there so many changes to the package locks vs. to the package files
themselves?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/943
What about upgrading.md?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/942
Maybe making this function so generic is going to necessitate it being so
complicated that it is harder to maintain etc.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/940
Maybe the issue has to do with our keys, and their distribution as the size
get's larger? Maybe when we get larger sizes we get more collisions and end up
calling equals() more or something.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/940
This should have the equiv. diagram and documentation ( i believe as shown
above ) to the original split join strategy.
---
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron-bro-plugin-kafka/pull/6#discussion_r172243410
--- Diff: src/KafkaWriter.cc ---
@@ -54,20 +66,49 @@ KafkaWriter::KafkaWriter(WriterFrontend* frontend):
WriterBackend(frontend), for
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/940
have we thought to send a mail to the storm dev list and ask if anyone has
done this? potential issues?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/940
If we integrated storm with yarn this would also be a problem, as our
resource management may be at odds with yarn's. I think?
What would be nice is if storm could manage the poo
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/853
+1 by inspection
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/853
Sorry, I'll try to get back to this today
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/943
I think I mentioned contrib. They don't have to go in contrib, I think at
the time someone mentioned not wanting to maintain them.. If we don't then I
thought contrib would make sen
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/853#discussion_r170125479
--- Diff:
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/AlertServiceImpl.java
---
@@ -37,15 +47,21 @@
@Service
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/940#discussion_r170056794
--- Diff:
metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/bolt/UnifiedEnrichmentBolt.java
---
@@ -0,0 +1,323
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/853#discussion_r169515112
--- Diff:
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/AlertServiceImpl.java
---
@@ -37,15 +47,21 @@
@Service
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/853
This looks great, one small comment from review
---
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/853#discussion_r169441455
--- Diff:
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/AlertServiceImpl.java
---
@@ -37,15 +47,21 @@
@Service
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/939
It is almost like hw has a different issue tracking system
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/939
BUG- is not a valid title for this project.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/939
Can you put some kind of description in here? What does that BUG refer to?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/579
+1 one. Thank you @ctramnitz. I know how it can be to have a long review
process. This is really good work and I look forward to seeing what you do
next ;)
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/853
I am not an expert in hbase, I cannot say how i'd implement it.
The version thing we can leave out, it will be just as good or bad as what
we have in zookeeper right?
I thin
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/579#discussion_r168779259
--- Diff: Upgrading.md ---
@@ -19,6 +19,24 @@ limitations under the License.
This document constitutes a per-version listing of changes of
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/579
maybe regression isn't the right word -
"Until we have something generic to (pre-)parse syslog before it reaches
the message parser I assumed the messages will be stripped off
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/853
Of course re-reading your comment, I see you specifically didn't account
for these, but I think there are some basic tweaks to allow for this that don't
presuppose far flung 20% use cases.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/853
First, nice work.
Second, I have this question:
Since we will have some unknown number of rest clients, that may want to
save 'configuration' associated with a user, an
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/579
after that I'll merge
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/579
@ctramnitz one thing, since this is a regression, technically, we need to
update the release notes / upgrade guide.
Can you add a note to the Upgrading.md about the removal of the Syslog
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/579
Im +1 on this. I would like to get comment from @simonellistonball et al
on the change for syslog
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/934
I would just like it documented as to why we are not using the HDP solr
mpack.
Although, I would think we would be using that mpack as the example for
ours?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/934
Can I ask a question : why aren't we using
https://docs.hortonworks.com/HDPDocuments/HDP2/HDP-2.6.4/bk_solr-search-installation/content/ch_hdp-search-install-ambari.html?
Is there a r
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/934
Ran through test, everything worked fine. +1
On the questions:
- What Solr parameters do we expose in Ambari? All of them?
People are going to want to tune indexing
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
+1
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
@merrimanr did you push the configs to zookeeper?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
Everything worked, just wondering about the questions above before I check
off.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
don't we have to push the new config to zookeeper?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
Why is it "solr.zookeeper" : "localhost:9983" and not solr.ip?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/914
Um, it is updated.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/914
Just a note on parser docs, part of the mythical 777 is support for per
parser readme, that would be nice here.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
Cool, I'll run the test plan
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/914
yeah, you said in the docs, will do
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/914
http://goessner.net/articles/JsonPath/
https://github.com/json-path/JsonPath
http://jsonpath.herokuapp.com
https://docs.spring.io/spring-boot/docs/current/reference/html/boot
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
See, we explored it. Thanks!!
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
Sorry, maybe I misunderstood what I have read here:
https://lucidworks.com/2013/08/23/understanding-transaction-logs-softcommit-and-commit-in-sorlcloud/
I thought that you could
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/929
Should we explore auto-commit?
Also, maybe we should rename all this stuff to SolrCloud, since Solr and
SolrCloud *are* different in some use of terms and concepts?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/928
I'll add 2 smileys next time, to help get my point across. I would have
expected a response more along the lines of:
"Upgrading Ambari to the 2.6 release line shows little r
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/928
Shouldn't the title of this PR be "Update ambari from 2.5.x to 2.6.x"?
That seems the bigger thing :)
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/922
So, the idea is that we have a 'core' for each parser? and for errors?
---
GitHub user ottobackwards opened a pull request:
https://github.com/apache/metron/pull/924
METRON-1299 In MetronError tests, don't test for HostName if getHostName
wouldn't work
MetronError ignores exceptions from
InetAddress.getLocalHost().getHostName() and leaves the f
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/865
Hey @JonZeolla how is it going?
---
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/922#discussion_r165662614
--- Diff:
metron-platform/metron-solr/src/test/java/org/apache/metron/solr/schema/SchemaTranslatorTest.java
---
@@ -0,0 +1,188
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/920#discussion_r165662017
--- Diff:
metron-stellar/stellar-common/src/test/java/org/apache/metron/stellar/dsl/functions/ShellFunctionsTest.java
---
@@ -40,8 +45,8
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/920#discussion_r165661676
--- Diff:
metron-stellar/stellar-common/src/test/java/org/apache/metron/stellar/dsl/functions/ShellFunctionsTest.java
---
@@ -40,8 +45,8
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/920#discussion_r165659854
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/shell/cli/PausableInput.java
---
@@ -36,8 +37,8
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/922#discussion_r165548937
--- Diff:
metron-platform/metron-solr/src/test/java/org/apache/metron/solr/schema/SchemaTranslatorTest.java
---
@@ -0,0 +1,188
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/922#discussion_r165548986
--- Diff:
metron-platform/metron-solr/src/test/java/org/apache/metron/solr/schema/SchemaTranslatorTest.java
---
@@ -0,0 +1,188
GitHub user ottobackwards opened a pull request:
https://github.com/apache/metron/pull/920
METRON-1438 Move SHELL functions from metron-management to stellar-common
Part of making stellar more modular and stand alone is consolidation of
stellar functionality scattered through
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/919
No, I am sorry, I thought that was understood. I have not reviewed, but I
would go ahead based on @cestella
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/919
this is a user utility though, not a dev one. I would create a new
support-scripts folder
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/579
@ctramnitz thank you! Let us know where you are at and if we can help
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/857
What is the status of this pr? it is 29 day without comment, and
conflicted, literally, and perhaps figuratively
---
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/856#discussion_r165048687
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/utils/validation/StellarZookeeperBasedValidator.java
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/873
bump
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/690
Done, let me know if the changes are sufficient @JonZeolla
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/830
Can you take master again? This pr still builds metron twice when doing
vagrant up.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/684
We should make this a feature branch
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/775
@DimDroll any reproduction? Is this still happening for you?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/670
Deconflict and bump?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/890
Thanks! please take care of the jira
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/858
Re-upping what I said before
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/579
bump?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/789
This needs deconflict, and a relevant check
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/912
please remember to take care of your jira
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/912
+1 by inspection. Thank you for the contribution
---
GitHub user ottobackwards opened a pull request:
https://github.com/apache/metron/pull/914
METRON-1397 Support for JSON Path and complex documents in JSONMapParser
It would be useful for implementors to have the ability to ingest more
complex documents without having to stand up a
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/912#discussion_r164303375
--- Diff:
metron-stellar/stellar-common/src/test/java/org/apache/metron/stellar/dsl/functions/RegExFunctionsTest.java
---
@@ -68,4 +68,19 @@ public
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/912#discussion_r164303393
--- Diff:
metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/RegExFunctions.java
---
@@ -100,4 +100,39 @@ public
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/903
+1 still stands
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/903
+1 -> ran up both images, everything checked out
Ship it
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/901
I'm going to wait for @lvets a chance to try his scenario
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/903
Sure.
Question. Do we expect there to be issues with 2.6? Is this PR and
Casey's 2.6 pr going to conflict or have issues? How will we know to retest
this after that one lands if
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/902
+1
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/910
@mmiklavc I think that is a great idea for the project. And the
appropriately factored facade classes as well.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/910
This looks awesome at first look. I'm a big fan of doing this regardless
of the shading issue. The only thought that comes to mind is that there is a
tipping point where a *Utils class
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/903
@nickwallen let me know when you feel ok about it, I'll run it through
again.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/903
Failure during vagrant up for metron-on-ubuntu
```
2018-01-25 10:41:49,302 p=37541 u=ottofowler | [0;31mfatal: [node1]:
FAILED! => {"changed": true,
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/903
Trying this now.
Only comment on the content here is there is a _lot_ going on in this pr.
A lot of while I'm here I might as well work.
It might have been better to have kept
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/901#discussion_r163868868
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/enrichment_commands.py
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/902
I have another script I want to add for tracking master in feature branches
after this as well
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/902
I can almost always be bought off with a jira number
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/888
+1 by inspection
---
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/metron/pull/901#discussion_r163616902
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/enrichment_commands.py
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/873
ok, it is MAGIC_TIME! @simonellistonball
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/873
Plus it lets me get away with
```java
public static final String MAGIC_TIME = "%time";
```
so bonus
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/873
well, %timing seem to make sense to me, as it is the timing of the last
statement. %time didn't occur to me. I don't use Jupyter, and don't time
things in the shell that ofte
1 - 100 of 1314 matches
Mail list logo