ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Harp" <michael.h...@teradata.com>
Subject Re: Review Request 17215: Add Existing Postgres Database option for Hive Database during Ambari cluster install
Date Fri, 28 Feb 2014 21:48:49 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17215/
-----------------------------------------------------------

(Updated Feb. 28, 2014, 9:48 p.m.)


Review request for Ambari, Mahadev Konar and Nate Cole.


Changes
-------

Added in 2.0 and 1.3 stack support.
My only concern is that /usr/share/java/postgresql-jdbc.jar need to exist and needs to be
the correct version for the installed postgres database. Should we assume this when user selects
"Existing Postgres Database"?


Bugs: AMBARI-4205
    https://issues.apache.org/jira/browse/AMBARI-4205


Repository: ambari


Description
-------

Add Existing Postgres Database option for Hive Database.


Diffs (updated)
-----

  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/package/scripts/hive.py
30aec45 
  ambari-server/src/main/resources/stacks/HDP/1.3.2/services/HIVE/package/scripts/params.py
c429555 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/package/scripts/hive.py
01fd2f9 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/package/scripts/hive_service.py
e81f04e 
  ambari-server/src/main/resources/stacks/HDP/2.0.6/services/HIVE/package/scripts/params.py
9961212 
  ambari-web/app/controllers/main/service/info/configs.js 35139a2 
  ambari-web/app/controllers/wizard/step8_controller.js 473cbb3 
  ambari-web/app/data/HDP2/global_properties.js d4c5103 
  ambari-web/app/data/global_properties.js b8432c3 
  ambari-web/app/views/wizard/controls_view.js ec88cd8 

Diff: https://reviews.apache.org/r/17215/diff/


Testing (updated)
-------

All unit test pass.
Deployed cluster with 1.3.3, 2.0.6 and 2.1.1


Thanks,

Michael Harp


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message