[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-04-03 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r178870634 --- Diff: metron-platform/metron-elasticsearch/src/test/java/org/apache/metron/elasticsearch/integration/components/ElasticSearchComponent.java --- @@ -2

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-04-03 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r178923287 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/utils/ElasticsearchUtils.java --- @@ -107,32 +113,94 @@ public

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-04-04 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r179060961 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/utils/ElasticsearchUtils.java --- @@ -107,32 +113,94 @@ publ

[GitHub] metron pull request #978: METRON-1507 Added support for vagrant cachier

2018-04-04 Thread simonellistonball
GitHub user simonellistonball opened a pull request: https://github.com/apache/metron/pull/978 METRON-1507 Added support for vagrant cachier ## Contributor Comments This is a Vagrant config change to enable the use of vagrant-cachier to reduce build times for the full dev platfo

[GitHub] metron pull request #978: METRON-1507 Added support for vagrant cachier

2018-04-04 Thread simonellistonball
Github user simonellistonball closed the pull request at: https://github.com/apache/metron/pull/978 ---

[GitHub] metron pull request #979: METRON-1507 Added support for vagrant cachier

2018-04-04 Thread simonellistonball
GitHub user simonellistonball opened a pull request: https://github.com/apache/metron/pull/979 METRON-1507 Added support for vagrant cachier ## Contributor Comments This is a Vagrant config change to enable the use of vagrant-cachier to reduce build times for the full dev platfo

[GitHub] metron pull request #980: METRON-1509 Docs - metron-deployment/other-example...

2018-04-04 Thread as22323
GitHub user as22323 opened a pull request: https://github.com/apache/metron/pull/980 METRON-1509 Docs - metron-deployment/other-examples - Adding 0.4.1 AWS EC2 Single Node/Vagrant Method ## Contributor Comments [Please place any comments here. A description of the problem/enha

[GitHub] metron pull request #873: METRON-1367 Stellar should have some instrumentati...

2018-04-04 Thread ottobackwards
Github user ottobackwards closed the pull request at: https://github.com/apache/metron/pull/873 ---

[GitHub] metron pull request #856: METRON-1339 Stellar Shell functionality to verify ...

2018-04-04 Thread ottobackwards
Github user ottobackwards closed the pull request at: https://github.com/apache/metron/pull/856 ---

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-04-05 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r179502932 --- Diff: metron-interface/metron-rest/src/main/scripts/metron-rest.sh --- @@ -36,6 +36,7 @@ METRON_SYSCONFIG="${METRON_SYSCONFIG:-/etc/default/metron}"

[GitHub] metron pull request #982: METRON-1511 Unable to Serialize Profiler Configura...

2018-04-05 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/982 METRON-1511 Unable to Serialize Profiler Configuration In the course of some other work I was doing, I found that the Profiler configuration objects `ProfileConfig` and `ProfilerConfig` can be de

[GitHub] metron pull request #983: METRON-1512 Refactor CONFIG_GET/CONFIG_PUT to Use ...

2018-04-05 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/983 METRON-1512 Refactor CONFIG_GET/CONFIG_PUT to Use ZKConfigurationsCache NOTE: This depends on #982 and should not be merged until #982. Currently the `ConfigurationFunctions` like `CONFIG

[GitHub] metron pull request #979: METRON-1507 Added support for vagrant cachier

2018-04-05 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/979#discussion_r179567772 --- Diff: metron-deployment/ansible/roles/ambari_common/tasks/ambari-repo-centos.yml --- @@ -18,5 +18,13 @@ - name: Setup Ambari repo on CentOS

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-05 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/984 METRON-1499 Enable Configuration of Unified Enrichment Topology via Ambari We recently introduced an alternative enrichment topology; the Unified enrichment topology. This is an alternative to t

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-09 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/984#discussion_r180150453 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-enrichment-env.xml --- @@

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-09 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/984#discussion_r180154515 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-enrichment-env.xml --- @

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-09 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/metron/pull/985 METRON-1515: Errors loading stellar functions currently bomb the entire topology, they should be recoverable ## Contributor Comments If a bad stellar function gets on the classpath, the entire

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-09 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/984#discussion_r180166554 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-enrichment-env.xml --- @

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-09 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/984#discussion_r180167513 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-enrichment-env.xml ---

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-04-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/946 ---

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-10 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180576221 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-10 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180576511 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266,2

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-10 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180577592 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266,2

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-10 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180578439 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266,2

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-10 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180583454 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266

[GitHub] metron pull request #988: METRON-1445: Update performance tuning guide with ...

2018-04-10 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/metron/pull/988 METRON-1445: Update performance tuning guide with more explicit parameter instructions ## Contributor Comments https://issues.apache.org/jira/browse/METRON-1445 The current perfor

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-11 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/984#discussion_r18074 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-enrichment-env.xml --- @@

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-11 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180748583 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266,2

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-11 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180751165 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266,2

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-11 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180753753 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-11 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180755523 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266,2

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-11 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180760561 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-11 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/985#discussion_r180770770 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/dsl/functions/resolver/ClasspathFunctionResolver.java --- @@ -254,18 +266,2

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-11 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/984#discussion_r180814327 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/configuration/metron-enrichment-env.xml --- @

[GitHub] metron pull request #991: METRON-1521: JSONMapParser is no longer serializab...

2018-04-11 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/metron/pull/991 METRON-1521: JSONMapParser is no longer serializable ## Contributor Comments Currently the JSONMapParser does not function because an anonymous inner class (TypeRef) is not serializable.

[GitHub] metron pull request #979: METRON-1507 Added support for vagrant cachier

2018-04-12 Thread simonellistonball
Github user simonellistonball closed the pull request at: https://github.com/apache/metron/pull/979 ---

[GitHub] metron pull request #993: METRON-1507 Add configuration in development vagra...

2018-04-12 Thread simonellistonball
GitHub user simonellistonball opened a pull request: https://github.com/apache/metron/pull/993 METRON-1507 Add configuration in development vagrant files to support vagrant cac… ## Contributor Comments This is a Vagrant config change to enable the use of vagrant-cachier to re

[GitHub] metron pull request #991: METRON-1521: JSONMapParser is no longer serializab...

2018-04-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/991 ---

[GitHub] metron pull request #993: METRON-1507 Add configuration in development vagra...

2018-04-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/993 ---

[GitHub] metron pull request #863: METRON-1347: Indexing Topology should fail tuples ...

2018-04-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/863 ---

[GitHub] metron pull request #985: METRON-1515: Errors loading stellar functions curr...

2018-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/985 ---

[GitHub] metron pull request #984: METRON-1499 Enable Configuration of Unified Enrich...

2018-04-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/984 ---

[GitHub] metron pull request #988: METRON-1445: Update performance tuning guide with ...

2018-04-17 Thread nickwallen
Github user nickwallen commented on a diff in the pull request: https://github.com/apache/metron/pull/988#discussion_r182054125 --- Diff: metron-platform/metron-common/README.md --- @@ -400,3 +401,34 @@ __Side Effects__ Calling the mark() method multiple times simply resets the

[GitHub] metron pull request #988: METRON-1445: Update performance tuning guide with ...

2018-04-17 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/988#discussion_r182152983 --- Diff: metron-platform/metron-common/README.md --- @@ -400,3 +401,34 @@ __Side Effects__ Calling the mark() method multiple times simply resets the s

[GitHub] metron pull request #974: METRON-1502: Upgrade Doxia plugin to 1.8

2018-04-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/974 ---

[GitHub] metron pull request #988: METRON-1445: Update performance tuning guide with ...

2018-04-17 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/988#discussion_r182168873 --- Diff: metron-platform/Performance-tuning-guide.md --- @@ -111,15 +138,146 @@ __spout-config.json__ } ``` -These are the spout recom

[GitHub] metron pull request #988: METRON-1445: Update performance tuning guide with ...

2018-04-17 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/988#discussion_r182170422 --- Diff: metron-platform/Performance-tuning-guide.md --- @@ -111,15 +138,146 @@ __spout-config.json__ } ``` -These are the spout recomme

[GitHub] metron pull request #988: METRON-1445: Update performance tuning guide with ...

2018-04-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/988 ---

[GitHub] metron pull request #983: METRON-1512 Refactor CONFIG_GET/CONFIG_PUT to Use ...

2018-04-17 Thread nickwallen
Github user nickwallen closed the pull request at: https://github.com/apache/metron/pull/983 ---

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-17 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/995 METRON-1526: Location field types cause DocValuesField appear more than once error ## Contributor Comments Solr automatically expands location type fields internally when a document is indexe

[GitHub] metron pull request #996: METRON-1528: Fix missing file in metron.spec

2018-04-17 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/metron/pull/996 METRON-1528: Fix missing file in metron.spec ## Contributor Comments https://issues.apache.org/jira/browse/METRON-1528 I missed the spec file addition for cluster_info.py in https

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-18 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r182277467 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/dao/SolrUpdateDaoTest.java --- @@ -132,4 +134,19 @@ public void batchUpdateSh

[GitHub] metron pull request #996: METRON-1528: Fix missing file in metron.spec

2018-04-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/996 ---

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-18 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r182472485 --- Diff: metron-platform/metron-solr/src/main/java/org/apache/metron/solr/dao/SolrUpdateDao.java --- @@ -92,9 +93,17 @@ public void batchUpdate(Map> upda

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-18 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r182472751 --- Diff: metron-platform/metron-solr/src/main/java/org/apache/metron/solr/dao/SolrUpdateDao.java --- @@ -92,9 +93,17 @@ public void batchUpdate(Map> upda

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-18 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r182477621 --- Diff: metron-platform/metron-solr/src/main/java/org/apache/metron/solr/dao/SolrUpdateDao.java --- @@ -92,9 +93,17 @@ public void batchUpdate(Map> upd

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-18 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r182480647 --- Diff: metron-platform/metron-solr/src/main/java/org/apache/metron/solr/dao/SolrUpdateDao.java --- @@ -92,9 +93,17 @@ public void batchUpdate(Map> upd

[GitHub] metron pull request #982: METRON-1511 Unable to Serialize Profiler Configura...

2018-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/982 ---

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-24 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r183930130 --- Diff: metron-platform/metron-solr/README.md --- @@ -95,3 +95,17 @@ As of now, we have mapped out the Schemas in `src/main/config/schema`. Ambari

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184065416 --- Diff: metron-platform/metron-solr/README.md --- @@ -95,3 +95,17 @@ As of now, we have mapped out the Schemas in `src/main/config/schema`. Ambari w

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184145151 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java --- @@ -20,15 +20,21 @@ import j

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184141881 --- Diff: metron-platform/metron-indexing/src/test/java/org/apache/metron/indexing/dao/UpdateIntegrationTest.java --- @@ -46,6 +49,7 @@ private stat

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184142751 --- Diff: metron-platform/metron-solr/README.md --- @@ -95,3 +95,19 @@ As of now, we have mapped out the Schemas in `src/main/config/schema`. Ambari wi

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184143875 --- Diff: metron-platform/metron-solr/README.md --- @@ -95,3 +95,19 @@ As of now, we have mapped out the Schemas in `src/main/config/schema`. Ambari wi

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184151697 --- Diff: metron-platform/metron-solr/src/test/java/org/apache/metron/solr/integration/SolrUpdateIntegrationTest.java --- @@ -20,15 +20,21 @@ import

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184158654 --- Diff: metron-platform/metron-solr/README.md --- @@ -95,3 +95,19 @@ As of now, we have mapped out the Schemas in `src/main/config/schema`. Ambari w

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184159098 --- Diff: metron-platform/metron-indexing/src/test/java/org/apache/metron/indexing/dao/UpdateIntegrationTest.java --- @@ -46,6 +49,7 @@ private sta

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-25 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/995#discussion_r184159071 --- Diff: metron-platform/metron-solr/README.md --- @@ -95,3 +95,19 @@ As of now, we have mapped out the Schemas in `src/main/config/schema`. Ambari w

[GitHub] metron pull request #995: METRON-1526: Location field types cause DocValuesF...

2018-04-26 Thread merrimanr
Github user merrimanr closed the pull request at: https://github.com/apache/metron/pull/995 ---

[GitHub] metron pull request #439: METRON-571 add stellar external functions feature ...

2018-04-30 Thread devopsec
Github user devopsec closed the pull request at: https://github.com/apache/metron/pull/439 ---

[GitHub] metron pull request #1008: METRON-1545: Upgrade Spring and Spring Boot

2018-05-02 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/1008 METRON-1545: Upgrade Spring and Spring Boot ## Contributor Comments This PR upgrades Spring to version [5.0.5.RELEASE](https://docs.spring.io/spring/docs/5.0.5.RELEASE/spring-framework-refere

[GitHub] metron pull request #1008: METRON-1545: Upgrade Spring and Spring Boot

2018-05-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1008#discussion_r185860632 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/WebSecurityConfig.java --- @@ -87,13 +91,18 @@ public void configureJ

[GitHub] metron pull request #1008: METRON-1545: Upgrade Spring and Spring Boot

2018-05-03 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1008#discussion_r185883201 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/WebSecurityConfig.java --- @@ -87,13 +91,18 @@ public void configureJd

[GitHub] metron pull request #1008: METRON-1545: Upgrade Spring and Spring Boot

2018-05-03 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/1008#discussion_r185894824 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/WebSecurityConfig.java --- @@ -87,13 +91,18 @@ public void configureJ

[GitHub] metron pull request #1008: METRON-1545: Upgrade Spring and Spring Boot

2018-05-03 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1008#discussion_r185898854 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/WebSecurityConfig.java --- @@ -87,13 +91,18 @@ public void configureJd

[GitHub] metron pull request #1008: METRON-1545: Upgrade Spring and Spring Boot

2018-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1008 ---

[GitHub] metron pull request #1009: 1549: Add empty object test to WriterBoltIntegrat...

2018-05-08 Thread mmiklavc
GitHub user mmiklavc opened a pull request: https://github.com/apache/metron/pull/1009 1549: Add empty object test to WriterBoltIntegrationTest implementation ## Contributor Comments https://issues.apache.org/jira/browse/METRON-1549 Adding a new integration test cas

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-11 Thread sardell
GitHub user sardell opened a pull request: https://github.com/apache/metron/pull/1010 METRON-1548: Remove hardcoded source:type from Alerts UI ## Contributor Comments Instead of 'source:type' being hard coded as a string in the Alerts UI, we now use a global-config service to re

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-14 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1010#discussion_r188086075 --- Diff: metron-interface/metron-alerts/src/app/service/global-config.service.ts --- @@ -0,0 +1,88 @@ +/** + * Licensed to the Apache Software Fo

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-15 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1010#discussion_r188243982 --- Diff: metron-interface/metron-alerts/src/app/service/global-config.service.ts --- @@ -0,0 +1,88 @@ +/** + * Licensed to the Apache Software Foun

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-15 Thread sardell
Github user sardell closed the pull request at: https://github.com/apache/metron/pull/1010 ---

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-15 Thread sardell
GitHub user sardell reopened a pull request: https://github.com/apache/metron/pull/1010 METRON-1548: Remove hardcoded source:type from Alerts UI ## Contributor Comments Instead of 'source:type' being hard coded as a string in the Alerts UI, we now use a global-config service to

[GitHub] metron pull request #1016: METRON-1564: Full dev kafka has offsets.topic.rep...

2018-05-15 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/1016 METRON-1564: Full dev kafka has offsets.topic.replication.factor set to 3 instead of 1 ## Contributor Comments Adds settings `offsets.topic.replication.factor` to 1 in single_node_vm.yml

[GitHub] metron pull request #1016: METRON-1564: Full dev kafka has offsets.topic.rep...

2018-05-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1016 ---

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-17 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1010#discussion_r189060226 --- Diff: metron-interface/metron-alerts/src/app/alerts/alerts-list/table-view/table-view.component.ts --- @@ -77,12 +80,24 @@ export class TableViewComp

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-17 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1010#discussion_r189065371 --- Diff: metron-interface/metron-alerts/src/app/alerts/alerts-list/alerts-list.component.ts --- @@ -174,6 +177,13 @@ export class AlertsListComponent im

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-17 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/1010#discussion_r189066511 --- Diff: metron-interface/metron-alerts/src/app/service/elasticsearch-localstorage-impl.ts --- @@ -47,6 +51,18 @@ export class ElasticSearchLocalstorage

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-18 Thread sardell
Github user sardell commented on a diff in the pull request: https://github.com/apache/metron/pull/1010#discussion_r189261776 --- Diff: metron-interface/metron-alerts/src/app/alerts/alerts-list/table-view/table-view.component.ts --- @@ -77,12 +80,24 @@ export class TableViewCompon

[GitHub] metron pull request #1010: METRON-1548: Remove hardcoded source:type from Al...

2018-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1010 ---

[GitHub] metron pull request #1032: METRON-1578 - EC2 10 Node Deployment - Reverting ...

2018-05-25 Thread as22323
GitHub user as22323 opened a pull request: https://github.com/apache/metron/pull/1032 METRON-1578 - EC2 10 Node Deployment - Reverting to using control_path with %C for Mac ## Contributor Comments A description of the problem: This PR is to revert to the previous control_pat

[GitHub] metron pull request #1033: METRON-1579: Stellar should return the expression...

2018-05-25 Thread cestella
GitHub user cestella opened a pull request: https://github.com/apache/metron/pull/1033 METRON-1579: Stellar should return the expression that failed in the exception ## Contributor Comments There are situations where we are not including the expression in the exception. Also, i

[GitHub] metron pull request #1034: METRON-1580 Release candidate check script requir...

2018-05-25 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/1034 METRON-1580 Release candidate check script requires Bro Plugin Not all Metron releases will include a new version of the Bro Plugin. The release candidate script currently requires it though.

[GitHub] metron pull request #1034: METRON-1580 Release candidate check script requir...

2018-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/1034 ---

[GitHub] metron pull request #1033: METRON-1579: Stellar should return the expression...

2018-05-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1033#discussion_r191080770 --- Diff: metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/adapters/stellar/StellarAdapter.java --- @@ -92,6 +96,19 @@ pu

[GitHub] metron pull request #1033: METRON-1579: Stellar should return the expression...

2018-05-27 Thread ottobackwards
Github user ottobackwards commented on a diff in the pull request: https://github.com/apache/metron/pull/1033#discussion_r191080813 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/BaseStellarProcessor.java --- @@ -143,7 +143,11 @@ public T pa

[GitHub] metron pull request #1035: METRON-1581: kill the profiler topology immediate...

2018-05-28 Thread MohanDV
GitHub user MohanDV opened a pull request: https://github.com/apache/metron/pull/1035 METRON-1581: kill the profiler topology immediately before restarting the topology if it is in 'KILLED' state for longer time ## Contributor Comments Restarting profiler topology fails storm ki

[GitHub] metron pull request #1033: METRON-1579: Stellar should return the expression...

2018-05-29 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1033#discussion_r191513381 --- Diff: metron-platform/metron-enrichment/src/main/java/org/apache/metron/enrichment/adapters/stellar/StellarAdapter.java --- @@ -92,6 +96,19 @@ public

[GitHub] metron pull request #1033: METRON-1579: Stellar should return the expression...

2018-05-29 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1033#discussion_r191513907 --- Diff: metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/BaseStellarProcessor.java --- @@ -143,7 +143,11 @@ public T parse(f

[GitHub] metron pull request #1036: METRON-1584 Indexing Topology Crashes with Invali...

2018-05-29 Thread nickwallen
GitHub user nickwallen opened a pull request: https://github.com/apache/metron/pull/1036 METRON-1584 Indexing Topology Crashes with Invalid Message If the indexing topology receives a message containing invalid JSON, the topology will crash. The topology needs to handle these types

[GitHub] metron pull request #1033: METRON-1579: Stellar should return the expression...

2018-05-29 Thread cestella
Github user cestella closed the pull request at: https://github.com/apache/metron/pull/1033 ---

<    1   2   3   4   5   6   7   8   9   10   >