Re: Review Request 64083: Log Search: Adding new dynamic field for service logs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64083/#review191888 --- Ship it! Ship It! - Krisztian Kasa On Nov. 27, 2017, 2:54

Re: Review Request 64059: Initial implementation of Schedulable document deletion & archiving for Infra Solr

unit tests * manual test using docker-compose Thanks, Krisztian Kasa

Re: Review Request 64059: Initial implementation of Schedulable document deletion & archiving for Infra Solr

bari-logsearch/docker/test-config/logfeeder/logfeeder.properties fb7ddf2845 Diff: https://reviews.apache.org/r/64059/diff/2/ Changes: https://reviews.apache.org/r/64059/diff/1-2/ Testing --- * unit tests * manual test using docker-compose Thanks, Krisztian Kasa

Review Request 64059: Initial implementation of Schedulable document deletion & archiving for Infra Solr

eder/logfeeder.properties fb7ddf2845 Diff: https://reviews.apache.org/r/64059/diff/1/ Testing --- * unit tests * manual test using docker-compose Thanks, Krisztian Kasa

Re: Review Request 64156: Log Search: backend should not return with 302 redirect if authentication fails

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64156/#review192117 --- Ship it! Ship It! - Krisztian Kasa On Nov. 29, 2017, 12:44

Re: Review Request 64091: Log Feeder: handle multiple '=' (split char) in key/value filter

/apache/ambari/logfeeder/filter/FilterKeyValue.java Line 140 (original), 140 (patched) <https://reviews.apache.org/r/64091/#comment269993> Why do we truncate the logmessage? Information can be lost. - Krisztian Kasa On Nov. 27, 2017, 8:54 p.m., Oliver Szabo

Re: Review Request 64206: Upgrade Ambari Infra to use Solr 5.5.5 for Ambari 2.6.1

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64206/#review192280 --- Ship it! Ship It! - Krisztian Kasa On Nov. 30, 2017, 11:51

Re: Review Request 64553: Log Feeder refactor: integrate with spring boot

pache/ambari/logfeeder/metrics/StatsLogger.java Lines 64 (patched) <https://reviews.apache.org/r/64553/#comment272239> Wouldn't be better to halt the entire application in case of an Error? So Errors would be rethrown but Excep

Review Request 64607: Ambari Infra Manager: solr data exporting jobs and integration test environment

* manual testing using docker containers and ide Thanks, Krisztian Kasa

Re: Review Request 64553: Log Feeder refactor: integrate with spring boot

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64553/#review193683 --- Ship it! Ship It! - Krisztian Kasa On Dec. 13, 2017, 3:24

Re: Review Request 64607: Ambari Infra Manager: solr data exporting jobs and integration test environment

, Krisztian Kasa

Re: Review Request 64660: Rafactor: Package Log Search and Log Feeder into jars + cleanup start scripts

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64660/#review194039 --- Ship it! Ship It! - Krisztian Kasa On Dec. 16, 2017, 1:27

Re: Review Request 63793: Log Feeder: cleanup checkpoint files periodically

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63793/#review190940 --- waitInterval -> waitIntervalMin - Krisztian Kasa On Nov.

Re: Review Request 63793: Log Feeder: cleanup checkpoint files periodically

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63793/#review190941 --- Ship it! Ship It! - Krisztian Kasa On Nov. 14, 2017, 12:50

Re: Review Request 63832: Reference Log Feeder IDE/maven dev setup in README.md

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63832/#review191057 --- Ship it! Ship It! - Krisztian Kasa On Nov. 15, 2017, 11:35

Re: Review Request 63525: Log Search server should be able to run locally with dockerized dev env by default

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63525/#review190024 --- Ship it! Ship It! - Krisztian Kasa On Nov. 2, 2017, 9:35

Re: Review Request 63525: Log Search server should be able to run locally with dockerized dev env by default

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63525/#review190025 --- Ship it! Ship It! - Krisztian Kasa On Nov. 2, 2017, 9:35

Re: Review Request 63696: Exclude test scope jars from Log Search / Log Feeder rpm/deb packages

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63696/#review190574 --- Ship it! Ship It! - Krisztian Kasa On Nov. 9, 2017, 12:53

Re: Review Request 64412: Log Search IT: Expose the right port for Selenium in docker-compose file

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64412/#review193110 --- Ship it! Ship It! - Krisztian Kasa On Dec. 7, 2017, 1:24

Re: Review Request 64513: Update the default storm log search configuration (use wildcards)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64513/#review193502 --- Ship it! Ship It! - Krisztian Kasa On Dec. 11, 2017, 11:47

Re: Review Request 64349: LogFeeder: filter objects for wildcard input paths need to be cloned

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64349/#review192987 --- Ship it! Ship It! - Krisztian Kasa On Dec. 5, 2017, 8:50

Re: Review Request 63393: Log Search: Add docker compose file for Log Search server and Log Feeder

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63393/#review189543 --- Ship it! Ship It! - Krisztian Kasa On Oct. 29, 2017, 10:19

Review Request 63416: Remove python & npm install from Log Search docker container

file 70e80b65e9 ambari-logsearch/docker/docker-compose.yml a6eeb0df6f ambari-logsearch/docker/logsearch-docker.sh b44e7ee43b Diff: https://reviews.apache.org/r/63416/diff/1/ Testing --- manully with docker-compose Thanks, Krisztian Kasa

Re: Review Request 63323: Get rid of deprecated Solr usages (infra + logsearch)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63323/#review189287 --- Ship it! Ship It! - Krisztian Kasa On Oct. 26, 2017, 9:24

Re: Review Request 63265: Log Feeder: allow applying grok filter on default fields (like type, path, host etc.)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63265/#review189175 --- Ship it! Ship It! - Krisztian Kasa On Oct. 24, 2017, 10:30

Re: Review Request 63429: Log Search: add descriptions about how to use Log Search environment with docker compose

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63429/#review189695 --- Ship it! Ship It! - Krisztian Kasa On Oct. 30, 2017, 9:04

Re: Review Request 63328: Log Search: Add docker-compose file for creating ZooKeeper containers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63328/#review189424 --- Ship it! Ship It! - Krisztian Kasa On Oct. 27, 2017, 9:54

Re: Review Request 63360: Log Search: Add docker compose file for Solr

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63360/#review189451 --- Ship it! Ship It! - Krisztian Kasa On Oct. 27, 2017, 1:19

Re: Review Request 63362: Fix Log Search configuration dependencies

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63362/#review189452 --- Ship it! Ship It! - Krisztian Kasa On Oct. 27, 2017, 4:28

Review Request 64976: Ambari Infra Manager: hdfs upload support

using docker * integration test using vagrant, ambari and ide Thanks, Krisztian Kasa

Re: Review Request 64852: Ambari Infra Manager: deleting documents

/SolrQueryBuilderTest.java e9513dccd9 Diff: https://reviews.apache.org/r/64852/diff/3/ Changes: https://reviews.apache.org/r/64852/diff/2-3/ Testing --- * unit tests, * integration tests using docker containers * manual testing using docker containers and ide Thanks, Krisztian Kasa

Re: Review Request 64852: Ambari Infra Manager: deleting documents

tests using docker containers * manual testing using docker containers and ide Thanks, Krisztian Kasa

Review Request 64850: Fixing interface and file renaming issues

Diff: https://reviews.apache.org/r/64850/diff/1/ Testing --- maven build unit and integration tests Thanks, Krisztian Kasa

Re: Review Request 64850: Fixing interface and file renaming issues

https://reviews.apache.org/r/64850/diff/2/ Changes: https://reviews.apache.org/r/64850/diff/1-2/ Testing --- maven build unit and integration tests Thanks, Krisztian Kasa

Review Request 64852: Ambari Infra Manager: deleting documents

Diff: https://reviews.apache.org/r/64852/diff/1/ Testing --- * unit tests, * integration tests using docker containers * manual testing using docker containers and ide Thanks, Krisztian Kasa