Review Request 48819: As part of atlas startup, "/usr/lib/ambari-logsearch-solr-client/solrCloudCli.sh" script is executed by ambari, which fails with "java.io.FileNotFoundException"

2016-06-16 Thread Tom Beerbower
Thanks, Tom Beerbower

Re: Review Request 48743: Remove {{atlas_conf_dir}} from HADOOP_CLASSPATH in hive-env.

2016-06-16 Thread Tom Beerbower
folder. - Tom --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48743/#review138038 --- On June 15, 2016, 6:13 p.m., Tom

Re: Review Request 48743: Remove {{atlas_conf_dir}} from HADOOP_CLASSPATH in hive-env.

2016-06-16 Thread Tom Beerbower
> On June 16, 2016, 6:11 p.m., John Speidel wrote: > > Curious, how does the Hive hook work now that the Atlas config isn't in > > it's classpath? > > Tom Beerbower wrote: > Ambari now writes the atlas configuration file into the Hive conf folder. Thanks

Re: Review Request 48788: With Atlas HA enabled, atlas instance fails to come up with "Unable to find IDs matching any local host and port binding among id1"

2016-06-16 Thread Tom Beerbower
iew138020 --- On June 16, 2016, 12:23 p.m., Tom Beerbower wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Review Request 48788: With Atlas HA enabled, atlas instance fails to come up with "Unable to find IDs matching any local host and port binding among id1"

2016-06-16 Thread Tom Beerbower
://reviews.apache.org/r/48788/diff/ Testing --- Manual test Atlas install and HA. Verify configuration. mvn clean test Thanks, Tom Beerbower

Review Request 48743: Remove {{atlas_conf_dir}} from HADOOP_CLASSPATH in hive-env.

2016-06-15 Thread Tom Beerbower
/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-env.xml 0c9aeaf Diff: https://reviews.apache.org/r/48743/diff/ Testing --- Manual test. Verify configuration. mvn clean test Thanks, Tom Beerbower

Review Request 48710: atlas.authentication.principal should be set to atlas/_h...@example.com

2016-06-14 Thread Tom Beerbower
-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/kerberos.json 78ad6ea Diff: https://reviews.apache.org/r/48710/diff/ Testing --- Manual test install Atlas, set Kerberos, verify Atlas configuration. Thanks, Tom Beerbower

Review Request 48707: Atlas Server Web UI alert

2016-06-14 Thread Tom Beerbower
t; without requiring authentication. Diffs - ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/alerts.json e65b5c1 Diff: https://reviews.apache.org/r/48707/diff/ Testing --- Manual test install Atlas. Verify alert not triggered. mvn clean test Thanks, Tom Beerbower

Re: Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-14 Thread Tom Beerbower
d not be present in any existing clusters. - Tom --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48655/#review137373 ------- On June 13, 2016, 5:51 p.m., Tom Beerbower wrote: > > -

Review Request 48655: ATLAS conf dir needs to be present in all ATLAS hook deployed hosts

2016-06-13 Thread Tom Beerbower
dependant service restart suggestion from Ambari. mvn clean test Thanks, Tom Beerbower

Re: Review Request 48489: Atlas Integration : Required changes for atlas-application.properties

2016-06-09 Thread Tom Beerbower
configuration enable kerberos and verify configuration Thanks, Tom Beerbower

Review Request 48489: Atlas Integration : Required changes for atlas-application.properties

2016-06-09 Thread Tom Beerbower
Thanks, Tom Beerbower

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

2016-06-01 Thread Tom Beerbower
- > > (Updated June 1, 2016, 4:58 p.m.) > > > Review request for Ambari, Robert Nettleton, Sumit Mohanty, and Tom Beerbower. > > > Bugs: AMBARI-16961 > https://issues.apache.org/jira/browse/AMBARI-16961 > > > Repository: ambari > > > Description

Re: Review Request 47984: Make storm.topology.submission.notifier.plugin.class property optional

2016-05-31 Thread Tom Beerbower
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47984/#review135635 --- Ship it! Ship It! - Tom Beerbower On May 28, 2016, 5:15 a.m

Re: Review Request 47967: Atlas server start failed after Ambari upgrade due to missing solrCloudCli.sh script

2016-05-27 Thread Tom Beerbower
for Solr and that the LogSearch Solr is actually installed. if type == 'server' and params.search_backend_solr and params.has_logsearch_solr: - Tom --- This is an automatically generated e-mail. To reply, visit

Review Request 47967: Atlas server start failed after Ambari upgrade due to missing solrCloudCli.sh script

2016-05-27 Thread Tom Beerbower
.5/configs/default.json 0077280 Diff: https://reviews.apache.org/r/47967/diff/ Testing --- Manual test Adjust unit tests mvn clean test Thanks, Tom Beerbower

Re: Review Request 47785: Ambari install of Atlas should use external HBase and Logsearch SOLR

2016-05-25 Thread Tom Beerbower
onent_directory}/conf") Do, I need to add the same logic? - Tom --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47785/#review134876 ------- On May 25, 2016, 10:06 p.m., Tom Beerbower wrote: > > -

Review Request 47785: Ambari install of Atlas should use external HBase and Logsearch SOLR

2016-05-24 Thread Tom Beerbower
/test_atlas_server.py 9d0f00c ambari-server/src/test/python/stacks/2.5/configs/default.json e4a97f6 Diff: https://reviews.apache.org/r/47785/diff/ Testing --- Manual test of Atlas install. Updated unit tests. mvn clean test Thanks, Tom Beerbower

Re: Review Request 47773: one very careless mistake for ambari agent cancel logic

2016-05-24 Thread Tom Beerbower
it? Could you please complete the "testing done" section of the review? - Tom Beerbower On May 24, 2016, 1:13 p.m., chen jian wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revi

Re: Review Request 47626: Atlas HA configuration Property Changed

2016-05-20 Thread Tom Beerbower
--- manual check Atlas HA - verify configs mvn clean test Thanks, Tom Beerbower

Review Request 47626: Atlas HA configuration Property Changed

2016-05-19 Thread Tom Beerbower
/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/application-properties.xml 53c1369 Diff: https://reviews.apache.org/r/47626/diff/ Testing --- manual check Atlas HA - verify configs mvn clean test Thanks, Tom Beerbower

Review Request 47545: Rename atlas lineage configurations

2016-05-18 Thread Tom Beerbower
/47545/diff/ Testing --- manual test Atlas install mvn clean test Thanks, Tom Beerbower

Re: Review Request 47009: Falcon server start fails

2016-05-11 Thread Tom Beerbower
/setup_atlas_falcon.py 4b7c0c3 Diff: https://reviews.apache.org/r/47009/diff/ Testing --- mvn clean test Thanks, Tom Beerbower

Re: Review Request 47256: Atlas Integration : Atlas fails to come up with solr as indexing search when the zookeeper quorum has more than one host

2016-05-11 Thread Tom Beerbower
ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py 422b1c9 ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py e9ddc25 Diff: https://reviews.apache.org/r/47256/diff/ Testing --- Updated unit tests. mvn clean test Thanks, Tom Beerbower

Re: Review Request 47009: Falcon server start fails

2016-05-10 Thread Tom Beerbower
h. Please use it Great! Thanks Venkat. - Tom --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47009/#review132337 --- On May 5, 2016, 12:07 p

Re: Review Request 47009: Falcon server start fails

2016-05-10 Thread Tom Beerbower
sit: https://reviews.apache.org/r/47009/#review132337 ------- On May 5, 2016, 12:07 p.m., Tom Beerbower wrote: > > --- > This is an automatically generated e-mail. To re

Review Request 47009: Falcon server start fails

2016-05-05 Thread Tom Beerbower
--- mvn clean test Thanks, Tom Beerbower

Review Request 46903: Atlas Integration : Atlas start failed after removing old version of the atlas and installing new one

2016-05-02 Thread Tom Beerbower
/configs/default.json 5188730 Diff: https://reviews.apache.org/r/46903/diff/ Testing --- Manual test Atlas install. Updated unit tests. mvn clean test. Thanks, Tom Beerbower

Re: Review Request 46740: Atlas Integration : set atlas.cluster.name in hive-site

2016-04-27 Thread Tom Beerbower
/2.3/common/test_stack_advisor.py 617f73d Diff: https://reviews.apache.org/r/46740/diff/ Testing --- New stack advisor unit test. mvn clean test Thanks, Tom Beerbower

Review Request 46360: Atlas Integration :Use stack advisor to suggest correct integration related configs

2016-04-18 Thread Tom Beerbower
/common/test_stack_advisor.py 6c9fd46 Diff: https://reviews.apache.org/r/46360/diff/ Testing --- Added unit tests for new stack advisor methods. mvn clean test Thanks, Tom Beerbower

Re: Review Request 45937: Support Atlas / Storm integration

2016-04-08 Thread Tom Beerbower
ripts/params_linux.py e9e8ce5 ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/package/scripts/storm.py d2fafa6 ambari-server/src/main/resources/stacks/HDP/2.3/services/STORM/configuration/storm-env.xml 2bb0862 Diff: https://reviews.apache.org/r/45937/diff/ Testing ---

Review Request 45896: Atlas Integration : Support Atlas HA

2016-04-07 Thread Tom Beerbower
test. mvn clean test new unit tests added Thanks, Tom Beerbower

Review Request 45218: Atlas Integration : Rename Atlas Configurations (2.5 stack definition)

2016-03-23 Thread Tom Beerbower
hon/stacks/2.6/ATLAS/test_atlas_server.py 8e51ea0 ambari-server/src/test/python/stacks/2.6/configs/default.json 2e1bc68 Diff: https://reviews.apache.org/r/45218/diff/ Testing --- Manual test install Atlas (HDP 2.5 stack). Verify configuration. mvn clean test all pass Thanks, Tom Beerbower

Review Request 44878: Atlas Integration : Rename Atlas Configurations

2016-03-15 Thread Tom Beerbower
las (HDP 2.6 stack). Verify configuration. mvn clean test all pass Thanks, Tom Beerbower