Re: Review Request 49253: LDAP sync: force to use uid and cn in patterns to check a member is a dn or not

2016-06-27 Thread Oliver Szabo
green in apache report Thanks, Oliver Szabo

Re: Review Request 49253: LDAP sync: force to use uid and cn in patterns to check a member is a dn or not

2016-06-27 Thread Oliver Szabo
-------- On June 27, 2016, 11:59 a.m., Oliver Szabo wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49253/ > -

Re: Review Request 49253: LDAP sync: force to use uid and cn in patterns to check a member is a dn or not

2016-06-27 Thread Oliver Szabo
/diff/ Testing --- All green in apache report Thanks, Oliver Szabo

Re: Review Request 49253: LDAP sync: force to use uid and cn in patterns to check a member is a dn or not

2016-06-27 Thread Oliver Szabo
/diff/ Testing --- All green in apache report Thanks, Oliver Szabo

Review Request 49255: Upgrade Solr version to 5.5.2 and use apache archive repo

2016-06-27 Thread Oliver Szabo
/pom.xml c664ffb ambari-logsearch/pom.xml 2fd1591f Diff: https://reviews.apache.org/r/49255/diff/ Testing --- testing done. Thanks, Oliver Szabo

Review Request 49253: LDAP sync: force to use uid and cn in patterns to check a member is a dn or not

2016-06-27 Thread Oliver Szabo
/security/ldap/AmbariLdapDataPopulator.java 53ff16d ambari-server/src/test/java/org/apache/ambari/server/security/ldap/AmbariLdapDataPopulatorTest.java 2265eec Diff: https://reviews.apache.org/r/49253/diff/ Testing --- All green in apache report Thanks, Oliver Szabo

Re: Review Request 49238: Fix erroneous input configs

2016-06-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49238/#review139500 --- Ship it! Ship It! - Oliver Szabo On June 26, 2016, 1:35 p.m

Review Request 49237: Make solr-client log writeabe for others in Hadoop Services group

2016-06-26 Thread Oliver Szabo
/ATLAS/test_metadata_server.py ad531de ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py fe6dd31 Diff: https://reviews.apache.org/r/49237/diff/ Testing --- Total run:1073 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 49235: AMBARI-17390: Need to create field "text" in history collection

2016-06-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49235/#review139498 --- Ship it! Ship It! - Oliver Szabo On June 26, 2016, 3:05 a.m

Re: Review Request 49233: Remove superfluous Logfeeder codes

2016-06-26 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49233/#review139497 --- Ship it! Ship It! - Oliver Szabo On June 25, 2016, 11:49

Re: Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
0613770 Diff: https://reviews.apache.org/r/49195/diff/ Testing --- Total run:1072 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 49203: Add missing components to LogSearch (part II)

2016-06-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49203/#review139372 --- Ship it! Ship It! - Oliver Szabo On June 24, 2016, 2:48 p.m

Re: Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
/test_logfeeder.py 0613770 Diff: https://reviews.apache.org/r/49195/diff/ Testing --- Total run:1072 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
/49195/diff/ Testing --- Total run:1072 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Review Request 49195: Add missing components to Log Search (part I)

2016-06-24 Thread Oliver Szabo
Thanks, Oliver Szabo

Re: Review Request 49029: When requesting a Kerberos Descriptor via the REST API, 'when' clauses should optionally be processed

2016-06-23 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49029/#review139269 --- Ship it! Ship It! - Oliver Szabo On June 22, 2016, 5:57 p.m

Re: Review Request 49121: AMBARI-17386. Misc fixes for Atlas, write correct conf files for server and client, use PLAINTEXTSASL, and remove atlas from HADOOP_CLASSPATH in Hive

2016-06-23 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49121/#review139268 --- Ship it! Ship It! - Oliver Szabo On June 22, 2016, 11:44

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-23 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49036/#review139258 --- Ship it! Ship It! - Oliver Szabo On June 23, 2016, 2:37 p.m

Re: Review Request 49036: Allow https prtotocol for Log Search

2016-06-21 Thread Oliver Szabo
----- > > (Updated June 21, 2016, 6:03 p.m.) > > > Review request for Ambari, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. > > > Bugs: AMBARI-17348 > https://issues.apache.org/jira/browse/AMBARI-17348 > > > Repos

Re: Review Request 49033: Clear /security.json config on solr znode when kerberos is disabled.

2016-06-21 Thread Oliver Szabo
/test_solr.py 0474aed Diff: https://reviews.apache.org/r/49033/diff/ Testing --- Total run:1073 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 49033: Clear /security.json config on solr znode when kerberos is disabled.

2016-06-21 Thread Oliver Szabo
Diff: https://reviews.apache.org/r/49033/diff/ Testing --- Total run:1073 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 49033: Clear /security.json config on solr znode when kerberos is disabled.

2016-06-21 Thread Oliver Szabo
ython string will recognize that. - Oliver --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49033/#review138878 ------- On

Review Request 49033: Clear /security.json config on solr znode when kerberos is disabled.

2016-06-21 Thread Oliver Szabo
errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 48741: LogSearch Solr kerberos support

2016-06-21 Thread Oliver Szabo
:1061 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 48741: LogSearch Solr kerberos support

2016-06-21 Thread Oliver Szabo
test output: Total run:1061 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 48793: Log Level filter not applied before Log Search Starts at first

2016-06-21 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48793/#review138816 --- Ship it! Ship It! - Oliver Szabo On June 20, 2016, 4:15 p.m

Re: Review Request 48741: LogSearch Solr kerberos support

2016-06-21 Thread Oliver Szabo
/ Testing --- ambari server python unit test output: Total run:1061 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 48970: Follow up NiFi log changes in the LogFeeder config

2016-06-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48970/#review138712 --- Ship it! Ship It! - Oliver Szabo On June 20, 2016, 11:19

Re: Review Request 48866: Service check for logsearch server

2016-06-20 Thread Oliver Szabo
-server/src/test/python/stacks/2.4/configs/default.json 10e4660 Diff: https://reviews.apache.org/r/48866/diff/ Testing --- testing done. Thanks, Oliver Szabo

Re: Review Request 48953: Open Advanced logsearch-admin-json upon install

2016-06-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48953/#review138609 --- Ship it! Ship It! - Oliver Szabo On June 20, 2016, 2:35 p.m

Re: Review Request 48866: Service check for logsearch server

2016-06-20 Thread Oliver Szabo
Diff: https://reviews.apache.org/r/48866/diff/ Testing --- testing done. Thanks, Oliver Szabo

Re: Review Request 48866: Service check for logsearch server

2016-06-20 Thread Oliver Szabo
: https://reviews.apache.org/r/48866/diff/ Testing --- testing done. Thanks, Oliver Szabo

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-20 Thread Oliver Szabo
. Thanks, Oliver Szabo

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-20 Thread Oliver Szabo
://reviews.apache.org/r/48854/diff/ Testing --- ambari server python tests passed. Thanks, Oliver Szabo

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-20 Thread Oliver Szabo
Diff: https://reviews.apache.org/r/48854/diff/ Testing --- ambari server python tests passed. Thanks, Oliver Szabo

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-20 Thread Oliver Szabo
-------- On June 17, 2016, 6:38 p.m., Oliver Szabo wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48854/ > --

Re: Review Request 48903: AMBARI-17307: LogFeeder output to Solr doesn't handle empty buffer queue properly

2016-06-18 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48903/#review138411 --- Ship it! Ship It! - Oliver Szabo On June 18, 2016, 6:59 a.m

Re: Review Request 48866: Service check for logsearch server

2016-06-18 Thread Oliver Szabo
/role_command_order.json 4e6c6cd Diff: https://reviews.apache.org/r/48866/diff/ Testing --- testing done. Thanks, Oliver Szabo

Re: Review Request 48866: Service check for logsearch server

2016-06-18 Thread Oliver Szabo
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48866/#review138331 ------- On June 17, 2016, 6:50 p.m., Oliver Szabo wrote: > > --- > This

Review Request 48866: Service check for logsearch server

2016-06-17 Thread Oliver Szabo
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/service_check.py PRE-CREATION Diff: https://reviews.apache.org/r/48866/diff/ Testing --- testing done. Thanks, Oliver Szabo

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-17 Thread Oliver Szabo
://reviews.apache.org/r/48854/diff/ Testing --- ambari server python tests passed. Thanks, Oliver Szabo

Re: Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-17 Thread Oliver Szabo
/test_solr.py 7cbbfc4 Diff: https://reviews.apache.org/r/48854/diff/ Testing --- ambari server python tests passed. Thanks, Oliver Szabo

Re: Review Request 48727: Log Search default log levels can not be altered

2016-06-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48727/#review138225 --- Ship it! Ship It! - Oliver Szabo On June 16, 2016, 5:09 p.m

Review Request 48854: Logsearch: retry logic for znode creation during Solr startup

2016-06-17 Thread Oliver Szabo
server python tests passed. Thanks, Oliver Szabo

Re: Review Request 48741: LogSearch Solr kerberos support

2016-06-16 Thread Oliver Szabo
Diff: https://reviews.apache.org/r/48741/diff/ Testing --- ambari server python unit test output: Total run:1061 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 48741: LogSearch Solr kerberos support

2016-06-15 Thread Oliver Szabo
have write permission. > > > > I have also seen cases, where if we don't protect zookeeper path, then > > if knit'ed user accesses zookeeper, then it will fail. > > Oliver Szabo wrote: > can it be done in different patch? just because im not sur

Re: Review Request 48741: LogSearch Solr kerberos support

2016-06-15 Thread Oliver Szabo
not maybe the users should setup this manually with zookeeper cli anyway. - Oliver --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48741/#review137798 --------

Re: Review Request 48741: LogSearch Solr kerberos support

2016-06-15 Thread Oliver Szabo
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48741/#review137798 ------- On June 15, 2016, 5:37 p.m., Oliver Szabo wrote: > >

Re: Review Request 48589: Fix HA enabled logic in the alerts

2016-06-15 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48589/#review137800 --- Ship it! Ship It! - Oliver Szabo On June 14, 2016, 2:15 p.m

Review Request 48741: LogSearch Solr kerberos support

2016-06-15 Thread Oliver Szabo
Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 48650: Add recently added logs to HadoopServiceConfig.json too

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48650/#review137321 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 4:28 p.m

Re: Review Request 48642: NPE in ambari-server.out when cluster with kerberos is installed

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48642/#review137308 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 3:04 p.m

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48485/#review137297 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 10:24

Re: Review Request 48490: Last button in the log search pagination panel does not take the user to the last page and few more fixes

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48490/#review137275 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 6:36 a.m

Re: Review Request 48622: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-13 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48622/#review137274 --- Ship it! Ship It! - Oliver Szabo On June 13, 2016, 9:28 a.m

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-12 Thread Oliver Szabo
1015593 ambari-web/app/data/HDP2/site_properties.js 794da25 Diff: https://reviews.apache.org/r/48348/diff/ Testing --- FT: tested locally with 4 node cluster with umask 0027. Thanks, Oliver Szabo

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-12 Thread Oliver Szabo
://reviews.apache.org/r/48348/diff/ Testing --- FT: tested locally with 4 node cluster with umask 0027. Thanks, Oliver Szabo

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-11 Thread Oliver Szabo
/HDP2/site_properties.js 794da25 Diff: https://reviews.apache.org/r/48348/diff/ Testing --- FT: tested locally with 4 node cluster with umask 0027. Thanks, Oliver Szabo

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

2016-06-10 Thread Oliver Szabo
> On June 9, 2016, 6:14 p.m., Oliver Szabo wrote: > > ambari-logsearch/ambari-logsearch-portal/src/main/configsets/hadoop_logs/conf/solrconfig.xml.j2, > > line 661 > > <https://reviews.apache.org/r/48485/diff/1/?file=1412245#file1412245line661> > > > >

Re: Review Request 48490: Last button in the log search pagination panel does not take the user to the last page and few more fixes

2016-06-10 Thread Oliver Szabo
/default.properties (line 48) <https://reviews.apache.org/r/48490/#comment202114> use logsearch.solr.* prefix here - Oliver Szabo On June 9, 2016, 12:51 p.m., Dharmesh Makwana wrote: > > --- > This is an automatically gener

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-10 Thread Oliver Szabo
--- FT: tested locally with 4 node cluster with umask 0027. Thanks, Oliver Szabo

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-10 Thread Oliver Szabo
/data/HDP2/site_properties.js 794da25 Diff: https://reviews.apache.org/r/48348/diff/ Testing --- FT: tested locally with 4 node cluster with umask 0027. Thanks, Oliver Szabo

Re: Review Request 48501: AMBARI-17147: OSError: [Errno 1] Operation not permitted with non-root ambari agent user

2016-06-09 Thread Oliver Szabo
ted e-mail. To reply, visit: https://reviews.apache.org/r/48501/#review136892 --- On June 9, 2016, 6:56 p.m., Oliver Szabo wrote: > > --- > This is an automatically generated e-m

Review Request 48501: AMBARI-17147: OSError: [Errno 1] Operation not permitted with non-root ambari agent user

2016-06-09 Thread Oliver Szabo
/common-services/RANGER/0.4.0/package/scripts/setup_ranger_xml.py 30f9527 Diff: https://reviews.apache.org/r/48501/diff/ Testing --- Total run:1055 Total errors:0 Total failures:0 FT: tested on 4 node cluster Thanks, Oliver Szabo

Re: Review Request 48486: AMBARI-17136: If Solr is down or not ready, then LogFeeder to should retry

2016-06-09 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48486/#review136868 --- Ship it! Ship It! - Oliver Szabo On June 9, 2016, 10:47 a.m

Re: Review Request 48485: AMBARI-17044 Optimize LogSearch Solr for cache setting and default values

2016-06-09 Thread Oliver Szabo
/hadoop_logs/conf/solrconfig.xml.j2 (line 661) <https://reviews.apache.org/r/48485/#comment201948> can you add these changes to the stack definition too (there are some solrconfig.xml too)? or if you want to let this be configurable on ambari I can add it as a property - Oliver

Re: Review Request 48413: Fix misnamed Zookeeper connect strings

2016-06-09 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48413/#review136865 --- Ship it! Ship It! - Oliver Szabo On June 8, 2016, 2:14 p.m

Re: Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder with umask 0027

2016-06-08 Thread Oliver Szabo
1015593 ambari-web/app/data/HDP2/site_properties.js 794da25 Diff: https://reviews.apache.org/r/48348/diff/ Testing (updated) --- FT: tested locally with 4 node cluster with umask 0027. Thanks, Oliver Szabo

Re: Review Request 48405: [Log Search portal ]Date range on comparison tab are different from date range of selected logs for comparison

2016-06-08 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48405/#review136621 --- Ship it! Ship It! - Oliver Szabo On June 8, 2016, 10:27 a.m

Re: Review Request 48325: Unit tests failing because of the order of hash sets/maps

2016-06-08 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48325/#review136618 --- Ship it! Ship It! - Oliver Szabo On June 7, 2016, 1:02 p.m

Re: Review Request 48338: LogSearch https support.

2016-06-07 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48338/#review136487 --- Ship it! Ship It! - Oliver Szabo On June 7, 2016, 1:25 p.m

Review Request 48348: AMBARI-17089: HDFS logs not picked by log feeder on a newly installed cluster with log search

2016-06-07 Thread Oliver Szabo
od defined error, but it is not related with my change. logsearch related tests passed. FT: tested locally with 4 node cluster. Thanks, Oliver Szabo

Re: Review Request 48338: LogSearch https support.

2016-06-07 Thread Oliver Szabo
eview request for Ambari, Alejandro Fernandez, Andrew Onischuk, Don Bosco > Durai, Jaimin Jetly, Oliver Szabo, Robert Nettleton, Sandor Magyari, Sumit > Mohanty, and Sebastian Toader. > > > Bugs: AMBARI-17092 > https://issues.apache.org/jira/browse/AMBARI-17092 > &g

Re: Review Request 48332: Log search capability for Nifi

2016-06-07 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48332/#review136451 --- Ship it! Ship It! - Oliver Szabo On June 7, 2016, 10:05 a.m

Re: Review Request 48265: Comparison tab : Allow the comparison only when minimum 2 tabs are selected and minor fixes

2016-06-06 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48265/#review136235 --- Ship it! Ship It! - Oliver Szabo On June 6, 2016, 10:42 a.m

Re: Review Request 48265: Comparison tab : Allow the comparison only when minimum 2 tabs are selected and minor fixes

2016-06-06 Thread Oliver Szabo
che.org/r/48265/#comment201260> same as above - Oliver Szabo On June 6, 2016, 8:39 a.m., Dharmesh Makwana wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Review Request 48173: Fix logfeeder filter name of logsearch

2016-06-03 Thread Oliver Szabo
/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-properties.xml eaa1106 Diff: https://reviews.apache.org/r/48173/diff/ Testing --- FT: with 4 node cluster Thanks, Oliver Szabo

Re: Review Request 48169: Ensure smokeuser HDFS folder exists before running MR, YARN, PIG, OOZIE service checks

2016-06-02 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48169/#review135957 --- Ship it! Ship It! - Oliver Szabo On June 2, 2016, 2:38 p.m

Review Request 48173: Fix logfeeder filter name of logsearch

2016-06-02 Thread Oliver Szabo
: https://reviews.apache.org/r/48173/diff/ Testing --- FT: with 4 node cluster Thanks, Oliver Szabo

Re: Review Request 48044: Provide context for hdp-select failures during ambari component install

2016-06-02 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48044/#review135919 --- Ship it! Ship It! - Oliver Szabo On May 30, 2016, 2:34 p.m

Re: Review Request 48134: Remove logsearch-solr-client dependency from Atlas

2016-06-01 Thread Oliver Szabo
fresh ATLAS... so, i opened this review request mostly for only review. Thanks, Oliver Szabo

Re: Review Request 48147: Fix metric sink + rename misnamed zk-connect-string variables.

2016-06-01 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48147/#review135877 --- Ship it! Ship It! - Oliver Szabo On June 1, 2016, 7 p.m

Review Request 48134: Remove logsearch-solr-client dependency from Atlas

2016-06-01 Thread Oliver Szabo
anyway, so this patch should be applied with a fresh ATLAS... so, i opened this review request mostly for only review. Thanks, Oliver Szabo

Re: Review Request 48124: Host checkboxes getting unselected when user clicks on search host button and few minor issues

2016-06-01 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48124/#review135793 --- Ship it! Ship It! - Oliver Szabo On June 1, 2016, 12:13 p.m

Re: Review Request 47586: Log Feeder Global exclusion default filter configuration is not effective in the log search indexing.

2016-06-01 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47586/#review135775 --- Ship it! Ship It! - Oliver Szabo On June 1, 2016, 6:20 a.m

Review Request 48035: Upgrade Log Search SOLR to 5.5.1

2016-05-30 Thread Oliver Szabo
/48035/diff/ Testing --- In progress.. Thanks, Oliver Szabo

Re: Review Request 47985: Fix for AMBARI-16939, upgrade from 2.2.0 to 2.4.0 on Ubuntu 14.04 / MySQL.

2016-05-29 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47985/#review135428 --- Ship it! Ship It! - Oliver Szabo On May 27, 2016, 8:54 p.m

Re: Review Request 47933: Add conditional constraints for Kerberos identities to control when they are created

2016-05-27 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47933/#review135303 --- Ship it! Ship It! - Oliver Szabo On May 27, 2016, 7:58 p.m

Review Request 47976: LDAP sync cannot handle if the member attribute value is not DN or id

2016-05-27 Thread Oliver Szabo
iff: https://reviews.apache.org/r/47976/diff/ Testing --- testing is in progress... Thanks, Oliver Szabo

Re: Review Request 47947: AMBARI-16922 Remove LogSearch dependency for Ranger

2016-05-27 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47947/#review135227 --- Ship it! Ship It! - Oliver Szabo On May 27, 2016, 10:07 a.m

Re: Review Request 47886: Log Search Server start failed with KeeperException

2016-05-26 Thread Oliver Szabo
f06f58a ambari-server/src/test/python/stacks/2.4/LOGSEARCH/test_logsearch.py f1e5efd ambari-server/src/test/python/stacks/2.4/configs/default.json 7379082 Diff: https://reviews.apache.org/r/47886/diff/ Testing --- Total run:1042 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Review Request 47886: Log Search Server start failed with KeeperException

2016-05-26 Thread Oliver Szabo
/47886/diff/ Testing --- Total run:1042 Total errors:0 Total failures:0 Thanks, Oliver Szabo

Re: Review Request 47726: Blueprint Export does not replace hive.llap.zk.sm.connectionString

2016-05-24 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47726/#review134546 --- Ship it! Ship It! - Oliver Szabo On May 24, 2016, 8:55 a.m

Re: Review Request 47452: Do not use implicit routing by default in solr-client

2016-05-23 Thread Oliver Szabo
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47452/#review134474 ------- On May 17, 2016, 1:29 p.m., Oliver Szabo wrote: > > -

Re: Review Request 47452: Do not use implicit routing by default in solr-client

2016-05-23 Thread Oliver Szabo
/apache/ambari/logsearch/solr/commands/CreateCollectionCommand.java (line 53) <https://reviews.apache.org/r/47452/#comment199322> It's like that you described. but you cam switch that off, by default its used. - Oliver Szabo On May 17, 2016, 1:29 p.m., Oliver

Re: Review Request 47596: HiveServer interactive - incorrect default memory value

2016-05-20 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47596/#review134148 --- Ship it! Ship It! - Oliver Szabo On May 20, 2016, 8:01 a.m

Re: Review Request 47590: Zero valued bars are thicker than bars with value in thousands in the Log search Log Level histograms

2016-05-19 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47590/#review133995 --- Ship it! - Oliver Szabo On May 19, 2016, 3:20 p.m., Dharmesh

Re: Review Request 47452: Do not use implicit routing by default in solr-client

2016-05-17 Thread Oliver Szabo
, Oliver Szabo, Robert Nettleton, and Sumit Mohanty. Changes --- fixed compile error Bugs: AMBARI-16701 https://issues.apache.org/jira/browse/AMBARI-16701 Repository: ambari Description --- By default ambari should not use implicit routing for creating collections Diffs

Review Request 47452: Do not use implicit routing by default in solr-client

2016-05-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47452/ --- Review request for Ambari, Don Bosco Durai, Oliver Szabo, Robert Nettleton, and

Re: Review Request 47397: Consistent logsearch/logfeeder property names

2016-05-17 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47397/#review133526 --- Ship it! Ship It! - Oliver Szabo On May 17, 2016, 12:28 p.m

<    2   3   4   5   6   7   8   9   >