[GitHub] metron pull request #949: METRON-1471: Migrate shuffle connections to local ...

2018-03-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/949 ---

[GitHub] metron pull request #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/metron/pull/957 ---

[GitHub] metron pull request #945: METRON-1464: Convert schemas to be compatible with...

2018-03-08 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/metron/pull/945 ---

[GitHub] metron issue #958: METRON-1483: Create a tool to monitor performance of the ...

2018-03-08 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/958 # Testing Plan: We presume * `ZOOKEEPER` is an environment variable set to the zk quorum (e.g. `node1:2181`) * `BROKER` is an environment variable set to the broker (e.g. `node1:6667`)

[GitHub] metron issue #949: METRON-1471: Migrate shuffle connections to local or shuf...

2018-03-08 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/949 +1 thanks! ---

[GitHub] metron issue #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/957 +1, I'm able to spin it up and query. Good job! ---

[GitHub] metron issue #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/957 Don't leave an extra square brace. Sigh. My bad ---

[GitHub] metron issue #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/957 Error code is just a 400, btw. ---

[GitHub] metron issue #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/957 I'm unable to get a fairly basic query working. A matching doc exists, based on querying solr directly. I assume I'm doing something obviously wrong here? ``` { "fields": [

[GitHub] metron pull request #949: METRON-1471: Migrate shuffle connections to local ...

2018-03-08 Thread cestella
GitHub user cestella reopened a pull request: https://github.com/apache/metron/pull/949 METRON-1471: Migrate shuffle connections to local or shuffle ## Contributor Comments Currently, we use shuffle groupings when we do not want to group by field. We should, instead, use local

[GitHub] metron pull request #949: METRON-1471: Migrate shuffle connections to local ...

2018-03-08 Thread cestella
Github user cestella closed the pull request at: https://github.com/apache/metron/pull/949 ---

[GitHub] metron pull request #958: METRON-1483: In performance evaluation, generating...

2018-03-08 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/metron/pull/958 METRON-1483: In performance evaluation, generating synthetic load and monitoring the write throughput of our kafka-to-kafka topologies has required a lot of custom scripting. We should have a tool tha

[GitHub] metron issue #945: METRON-1464: Convert schemas to be compatible with Solr 5...

2018-03-08 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/945 Ok, I'm cool with it. +1 by inspection; great work. ---

[GitHub] metron issue #949: METRON-1471: Migrate shuffle connections to local or shuf...

2018-03-08 Thread cestella
Github user cestella commented on the issue: https://github.com/apache/metron/pull/949 @merrimanr Ah, I had forgotten to merge in master yesterday afternoon. There should be no more shuffles in any of the topologies. ---

[GitHub] metron issue #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/957 yeah, don't worry about it, it's not important, just a bit odd. ---

[GitHub] metron issue #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread merrimanr
Github user merrimanr commented on the issue: https://github.com/apache/metron/pull/957 I've been maintaining this branch for a while and have been merging in other branches before they were committed to master. I think that's why you see all these commits. I merged in feature/METRO

[GitHub] metron issue #941: METRON-1355: Convert metron-elasticsearch to new infrastr...

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/941 No, that can wait for the follow-on ticket. Could you add a quick one-sentence blurb about the ElasticsearchTestUtils in the README so things are easier for anyone to test later (with the knowled

[GitHub] metron issue #957: METRON-1482: Update REST to work with Solr

2018-03-08 Thread justinleet
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/957 There's a bunch of @cestella commits in here. Do you need to merge feature/METRON-1416-upgrade-solr into this PR? ---