+1 (binding)
- Verified Keys
- Verified mvn clean install
- Ran full dev and performed several smoke tests including:
- tested several REST endpoints
- verified data in ES
- tested Management UI
- verified Storm topologies in Storm UI
- verified data in
Hey all, we need more votes!
So far we have 6 +1’s (including mine) and no 0’s or -1’s.
BUT, only two are binding, ie from PMC members.
Rules require 3 or more PMC member +1’s. Could one more
member please step up and vote?
Thanks!
--Matt
On 6/29/17, 3:00 PM, "James Sirota"
Github user ottobackwards closed the pull request at:
https://github.com/apache/metron/pull/627
---
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 feature
My branch before your changes
Ran for 49 min 59 sec
My branch after
Ran for 26 min 51 sec
You guys are my hero, great job!
Best,
Mike
On Fri, Jun 30, 2017 at 8:11 AM, Justin Leet wrote:
> METRON-1004 was made to address several issues we were having with the
> Travis
METRON-1004 was made to address several issues we were having with the
Travis builds. Specifically that the builds went on too long.
After the merge into master, builds should be more consistent, stable, and
take about 25-30 minutes.
Please merge master into PRs if you are experiencing issues
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/586
+1. I'm good with the most recent changes, pending 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
Github user JonZeolla commented on the issue:
https://github.com/apache/metron/pull/586
Should be good to go now, pending 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
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/586
@JonZeolla 1004 is in, but there's been a few changes in, so I'd ask that
you double check that things still line up as expected. I'm going to look back
through the changes this morning on here
Github user asfgit closed the pull request at:
https://github.com/apache/metron/pull/624
---
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 feature is
Github user merrimanr commented on the issue:
https://github.com/apache/metron/pull/624
+1. Really nice job guys. Looking forward to working builds again!
---
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 merrimanr commented on a diff in the pull request:
https://github.com/apache/metron/pull/624#discussion_r125037580
--- Diff:
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/KafkaControllerIntegrationTest.java
---
@@ -61,6 +62,7 @@
Github user justinleet closed the pull request at:
https://github.com/apache/metron/pull/624
---
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 feature is
GitHub user justinleet reopened a pull request:
https://github.com/apache/metron/pull/624
METRON-1004: Travis CI - Job Exceeded Maximum Time Limit
## Contributor Comments
This PR cleans up and speeds up various issues we've seen on Travis.
- This should avoid the
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/624
I am +1 on these changes.
---
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 justinleet closed the pull request at:
https://github.com/apache/metron/pull/624
---
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 feature is
GitHub user justinleet reopened a pull request:
https://github.com/apache/metron/pull/624
METRON-1004: Travis CI - Job Exceeded Maximum Time Limit
## Contributor Comments
This PR cleans up and speeds up various issues we've seen on Travis.
- This should avoid the
GitHub user justinleet reopened a pull request:
https://github.com/apache/metron/pull/624
METRON-1004: Travis CI - Job Exceeded Maximum Time Limit
## Contributor Comments
This PR cleans up and speeds up various issues we've seen on Travis.
- This should avoid the
Github user justinleet closed the pull request at:
https://github.com/apache/metron/pull/624
---
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 feature is
Github user justinleet commented on a diff in the pull request:
https://github.com/apache/metron/pull/624#discussion_r125009044
--- Diff:
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/KafkaControllerIntegrationTest.java
---
@@ -148,6 +150,15 @@
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/624#discussion_r125007939
--- Diff:
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/KafkaControllerIntegrationTest.java
---
@@ -61,6 +62,7 @@
Github user justinleet commented on a diff in the pull request:
https://github.com/apache/metron/pull/624#discussion_r125007493
--- Diff:
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/KafkaControllerIntegrationTest.java
---
@@ -61,6 +62,7 @@
Github user iraghumitra commented on the issue:
https://github.com/apache/metron/pull/620
@merrimanr sounds good I will map the field's as they are today. Can you
take another pass at it and let me know if it looks good.
---
If your project is set up for it, you can reply to this
GitHub user justinleet reopened a pull request:
https://github.com/apache/metron/pull/624
[DO NOT MERGE] METRON-1004: Travis CI - Job Exceeded Maximum Time Limit
## Contributor Comments
This PR cleans up and speeds up various issues we've seen on Travis.
- This should
Github user justinleet closed the pull request at:
https://github.com/apache/metron/pull/624
---
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 feature is
Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/624#discussion_r124998536
--- Diff:
metron-interface/metron-rest/src/test/java/org/apache/metron/rest/controller/KafkaControllerIntegrationTest.java
---
@@ -61,6 +62,7 @@
GitHub user justinleet reopened a pull request:
https://github.com/apache/metron/pull/624
[DO NOT MERGE] METRON-1004: Travis CI - Job Exceeded Maximum Time Limit
## Contributor Comments
This PR cleans up and speeds up various issues we've seen on Travis.
- This should
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/624
NC
---
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 user justinleet closed the pull request at:
https://github.com/apache/metron/pull/624
---
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 feature is
28 matches
Mail list logo