Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/432
I am trying to test this out - my vm is not doing well at the moment
however. Hopefully I can get it straight
---
If your project is set up for it, you can reply to this email
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/432
[vagrant@node1 tmp]$ /usr/metron/0.3.0/bin/flatfile_loader.sh -i
http://s3.amazonaws.com/alexa-static/top-1m.csv.zip -t enrichment -c t -e
./extractor.json -p 5 -b 128
Exception
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/444
you bet btw -the build on this pr isn't even going at all
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/444
integration tests took 43 minutes
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/444
#14 passed
Elapsed time 43 min 10 sec
Total time 1 hr 6 min 21 sec
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/444
So to test this - if you have travis set up for your git account, I think
you just push it to a branch in your repo and let travis build it?
---
If your project is set up
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/361
Thanks for the contribution
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/444
#14.1 passed
Elapsed time 23 min 28 sec
On my second try around 5pm
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/461
Does this mean you can get rid of the local geo database in the parsers
resource dir?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/438
That makes sense, thanks for the explanation.
I am +1 for having SOLR specific flattening, although documenting well
needs to be a priority.
Also, if we do this, please log
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/438
let me know which one to comment on!
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/462
+1, build and tests run ( mvn -q clean integration-test install &&
build_utils/verify_licenses.sh ), on machine that was failing every every time.
---
If your project i
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/455
ok - i'm not building this branch, just what i've seen from other builds
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/455
re: "site.xml is now a fully auto-generated file" that would be why it is
showing in git as unstaged but modified?
Is there a way we can stop that from
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/438
I think what we are saying is 1. I think that is the best way to move
forward.
Just make sure that this commit isn't merged into an RC for 0.3.1
---
If your project is set up
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/449
I think routing from stellar within routing in storm is confusing.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/449
I'm sorry, no Nick I don't
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/450
Honestly, I was thinking more about the 'kill everything' part, which seems
to be repeated a lot
---
If your project is set up for it, you can reply to this email and have your
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/457
METRON-724 add [IN] to the documented keywords that require escaping
The keyword 'in' must be escaped to be used in Stellar, but is not
documented as such.
This PR adds
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/461
I am seeing this on my local branch, not the PR
On February 22, 2017 at 09:13:12, Nick Allen (notificati...@github.com)
wrote:
The build error does not seem
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/438
I commented on 735 so it wins. #settled
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/438
Nick - if you look at the JSONMapParser, Casey and I implemented a
flattener that you would use for this. So, all of our json stuff from that
_was_ already flattened. I am more
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/454
@mattf-horton are you ok on this after the revisions?
---
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
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/454
METRON-716 site-book README.md
Initial draft of a readme for site-book documentation.
Questions: should we list the tools used more explicitly?
You can merge this pull
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/452
Please rename the PR to start with "METRON-715". Then we can land this
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/453
Just real quick before I start to review:
- What are the performance implications of this? How can we measure that?
- What is the effect of running this on quick dev
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/453#discussion_r100815725
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/error/MetronError.java
---
@@ -0,0 +1,200
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/453#discussion_r100815478
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/error/MetronError.java
---
@@ -0,0 +1,200
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/453#discussion_r100817317
--- Diff:
metron-platform/metron-elasticsearch/src/main/config/elasticsearch_error.properties
---
@@ -0,0 +1,69 @@
+# Licensed
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/453#discussion_r100820671
--- Diff:
metron-platform/metron-indexing/src/main/config/zookeeper/indexing/error.json
---
@@ -0,0 +1,17 @@
+{
+ "
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/453#discussion_r100813285
--- Diff:
metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-env.xml
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/453
That makes complete sense, we should call that stuff out pre-review. I
think what we are seeing is that folks have some really good ideas and are
willing to contribute
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/453
What comes to mind is the 'source' of an error. Is this error wrong
because METRON thinks it is invalid, or it it wrong because of some other
configuration specific evaluation. I
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/453
@cestella - how can we utilize the error indexing if we were going to say -
output errors or warnings that there were deprecated stellar statements?
---
If your project is set up
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/456
I put a reference to the verification doc at the top ' for smoke test guide
' and then put a checkbox for running in vagrant - how's that?
---
If your project is set up for it, you
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/456
So - why don't we come up with a cut down version of that, maybe restricted
to trying quick or full dev? or reference and link like we do with the full
dev guide?
---
If your
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/456
Thanks for the review!
I am not sure about verifying builds. I don't think we should put
everything that is in the Dev Guide or the Verification in the PR template
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/456#discussion_r101297815
--- Diff: .github/PULL_REQUEST_TEMPLATE.md ---
@@ -0,0 +1,40 @@
+Thank you for submitting a contribution to Apache Metron (Incubating
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/422
There are also problems with shutting down topologies, since we don't pass
in the wait to storm ( at least we didn't. I need to find the pr/jira for
that. All external calls from
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/316
Also, the point that this may be better managed from within ambari as a
view may need consideration.
On January 19, 2017 at 08:58:51, JJ Meyer (notificati
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/220
Can we close this then?
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/467
Thank you for clarifying, that makes sense. Sorry to confuse the issue
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/466
[INFO]
[INFO] Reactor Summary:
[INFO]
[INFO] Metron
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/467
There are two things here I *think*
1. The test assumptions and the failure on some platforms ( if this is
indeed the problem )
2. What this means for the real use
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/467
What are the performance penalties of doing this?
Part of the discussion in
[HADOOP-12009](https://issues.apache.org/jira/browse/HADOOP-12009) that lead
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/466
So - the test is - build without changing anything after clone, and there
should not be shown in git status right?
---
If your project is set up for it, you can reply to this email
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/466
Does building or running the tests have anything to do with it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/468
Run the free up space script
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
ansible-playbook -v playbooks/metron_build.yml
Using
/Users/ottofowler/src/apache/forks/incubator-metron/metron-deployment/ansible.cfg
as config file
[WARNING
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
Do I have to create an inventory with my local machine name?
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
I want to run the playbook to just build the rpm's, not deploy. So just
metron_build
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
How are you running the playbook? I cannot get it to execute
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
can you share your command line? Do you run it from /playbooks? does you
-i an inventory? Did you copy or create an ansible.cfg?
---
If your project is set up for it, you can
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
I need to run the playbook though.
It doesn't match any hosts
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
ansible-playbook -v -i "localhost," -c local playbooks/metron_build.yml
now it is running and i'll see about the errors
---
If your project is set up for it, you
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
Maybe we should add that line to the doc or create a script??
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/436
Well, if we don't want to put deployment things with the src, then ansible
is a more flexible and easier to use tool for certain tasks too. But we will
talk about it when I get
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/436#discussion_r103235131
--- Diff: metron-deployment/roles/kibana/README.md ---
@@ -1,35 +0,0 @@
-Kibana 4
--- End diff --
Is this documented
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/436#discussion_r103238671
--- Diff: metron-deployment/roles/kibana/README.md ---
@@ -1,35 +0,0 @@
-Kibana 4
--- End diff --
The file I commented
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/436#discussion_r103239678
--- Diff: metron-deployment/roles/java_jdk/defaults/main.yml ---
@@ -15,4 +15,4 @@
# limitations under the License
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/271
Never mind - just found the field name converter
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/271
@cestella , from looking at the ES templates etc, it seems that ':' is used
as the name separator and not '.'. Should the the unfolding not use the same?
---
If your project is set
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/270
I don't think I should be carrying the other commits. I didn't rebase.
I'll redo
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user ottobackwards closed the pull request at:
https://github.com/apache/incubator-metron/pull/270
---
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
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/271
METRON-449 JSONMapParser should unfold maps to arbitrary depths
The JSON Parser currently only unfolds maps at the root level. The parser
would support a wider range of sources
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/284
METRON-474 - fix vagrant ansible defaults for quick and full
Added defaults to match the run.sh parameters and removed the run.sh files.
Added a mention in the readmes as well
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/292
METRON-171 add .class to .gitignore
Add .class files to .gitignore
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ottobackwards
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/293
METRON-473 Add LENGTH() To Stellar
Add LENGTH to Stellar string functions and unit tests.
You can merge this pull request into a Git repository by running:
$ git pull https
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/284#discussion_r81776066
--- Diff: metron-deployment/vagrant/quick-dev-platform/README.md ---
@@ -5,7 +5,7 @@ This project fully automates the provisioning
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/293
@mmiklavc I had seen that. The tests in StellarTests test the compiler and
the variable resolution etc as well is my point. It is worth having both. If
both were done from
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/296
OK, if I change my functions to not throw then it works. This doesn't seem
right to me, but it may be something to do with why the tests are run that way.
---
If your project
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/293
OK I have it in. Here is the thing that I found, I know there is a pull
about fixing IS_EMPTY from throwing exceptions - and I had to take that same
return 0 approach instead
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/296
I'm working on making LENGTH work like IS_EMPTY. I can't get either one to
work as both are failing validation. There is no StellarTest for IS_EMPTY ( I
have added one to check
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/293#discussion_r82259618
--- Diff:
metron-platform/metron-common/src/test/java/org/apache/metron/common/stellar/StellarTest.java
---
@@ -30,9 +30,7 @@
import
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/293
I took this jira to get some start of an idea about Stellar, and after
debugging through it to track down to exitVariable to find the argument
resolution and then back up
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/293
It would make a nice little POC -
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/293
ok, as above, I'm not sure about x->null to do that, but I am the noob here
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/293
@cestella I can do that. Should I move LENGTH to DataStructureFunctions
then?
Also - I notice that IS_EMPTY throws IllegalStateExceptions instead of
returning null, is that how
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/299
METRON-425 Stellar transformation fails to handle special characters
The issue is that per our grammar the arithmetic and comparison operators
need to be escaped to be used
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/299
So - based on
`identifier_operand : (TRUE | FALSE) # LogicalConst
| arithmetic_expr #ArithmeticOperands
| STRING_LITERAL
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/293#discussion_r82218396
--- Diff:
metron-platform/metron-common/src/main/java/org/apache/metron/common/dsl/functions/StringFunctions.java
---
@@ -222,4 +222,16
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/293#discussion_r82218813
--- Diff:
metron-platform/metron-common/src/test/java/org/apache/metron/common/stellar/StellarTest.java
---
@@ -285,6 +285,18 @@ public void
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/293#discussion_r82218691
--- Diff:
metron-platform/metron-common/src/test/java/org/apache/metron/common/stellar/StellarTest.java
---
@@ -285,6 +285,18 @@ public void
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/261
Casey, would you feel about adding in recursion, such that nested maps will
be unfolded?
I have that working.
---
If your project is set up for it, you can reply to this email
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/261
No problem - do you require a jira?
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/261
Should I open a new jira and do a PR?
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/261
METRON-449
---
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 user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/261
Is it correct that multiple separate instances of this parser ( or any
parser ) could be configured by adding multiple zookeeper jsons with separate
kafka topics?
---
If your
Github user ottobackwards commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/279#discussion_r80999009
--- Diff: metron-deployment/inventory/quick-dev-platform/group_vars/all ---
@@ -0,0 +1,95 @@
+#
+# Licensed to the Apache Software
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/283
METRON-472: add runAgain.sh to handle a proper vagrant provision
When you fail after running run.sh in quickdev you will be told to run
vagrant provision... but what you really want
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/283
No you are right
I thought I closed the pr, I guess I did not.
Sorry
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user ottobackwards opened a pull request:
https://github.com/apache/incubator-metron/pull/325
METRON-512 up default junit to 4.12
"Global default JUnit version is 4.4. It would be nice to update this to
4.12 and expose newer JUnit featues. Existing pom files s
Github user ottobackwards commented on the issue:
https://github.com/apache/incubator-metron/pull/329
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
Github user ottobackwards closed the pull request at:
https://github.com/apache/incubator-metron/pull/329
---
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
GitHub user ottobackwards reopened a pull request:
https://github.com/apache/incubator-metron/pull/329
METRON-148 compress logs with logrotate
Add compress configuration for logrotate.d/ configurations
Tested using the logrotate -f /etc/logrotate.d/metron-* commands
You
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 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 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
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 https
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 https
1 - 100 of 310 matches
Mail list logo