> On Jan. 23, 2014, 10:37 a.m., Nate Cole wrote:
> > This issue is marked for completion for 1.5.0 - but there are puppet code 
> > changes here.  1.5.0 will utilize python instead of puppet and you may want 
> > to implement that instead.
> 
> Michael Harp wrote:
>     Ah ok, was not clear to me that puppet was out for the 1.5 branch. Is 
> there a JIRA/DOC that has more detail on the switch from puppet to python for 
> framework management?

Documentation is underway, so it's very sparse so far.  You can look in the 
HDP/1.3.3/hive.py (also hive_service.py, params.py) to see the similarities 
with the puppet code.  It should be pretty straightforward to make the 
equivalent changes.


- Nate


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


On Jan. 22, 2014, 6:35 p.m., Michael Harp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17215/
> -----------------------------------------------------------
> 
> (Updated Jan. 22, 2014, 6:35 p.m.)
> 
> 
> Review request for Ambari, Mahadev Konar and Nate Cole.
> 
> 
> Bugs: AMBARI-4205
>     https://issues.apache.org/jira/browse/AMBARI-4205
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Add Existing Postgres Database option for Hive Database.
> 
> 
> Diffs
> -----
> 
>   ambari-agent/src/main/puppet/modules/hdp-hive/manifests/init.pp d7a58c1 
>   ambari-agent/src/main/puppet/modules/hdp-hive/manifests/jdbc-connector.pp 
> 8bd9302 
>   ambari-agent/src/main/puppet/modules/hdp-hive/manifests/params.pp 65f321e 
>   ambari-web/app/controllers/main/service/info/configs.js b235de2 
>   ambari-web/app/controllers/wizard/step8_controller.js 6beabdd 
>   ambari-web/app/data/HDP2/global_properties.js a7ec956 
>   ambari-web/app/data/global_properties.js cd6b26a 
>   ambari-web/app/views/wizard/controls_view.js b4333f5 
> 
> Diff: https://reviews.apache.org/r/17215/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Michael Harp
> 
>

Reply via email to