---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45566/#review126669
---
Ship it!
Ship It!
- Matt
On April 1, 2016, 2:39 p.m., Lav Jain wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45566/
> ---
>
> (Updated April 1, 2016, 2:39 p.m.)
>
>
> Review request for Ambari, Andriy Babiichuk, Alexander Denissov, Alexandr
> Antonenko, Andrii Tkach, bhuvnesh chaudhary, Goutam Tadi, jun aoki, Matt,
> Newton Alex, Oleksandr Diachenko, and Srimanth Gunturi.
>
>
> Bugs: AMBARI-15657
> https://issues.apache.org/jira/browse/AMBARI-15657
>
>
> Repository: ambari
>
>
> Description
> ---
>
> User can add multiple space delimited directories, but after installation, it
> shows a comma in between. however, only first directory goes into effect,
> that too with a comma in the end.
> ```
> [pivotal@ip-10-32-36-213 etc]$ cat hawq-site.xml
>
> hawq_master_directory
> /data/hawq/master,/data/hawq/master2
>
>
>
> hawq_segment_directory
> /data/hawq/segment,/data/hawq/segment2
>
> [pivotal@ip-10-32-36-213 etc]$ ls -l /data/hawq
> drwxr-xr-x 3 root root 4096 Mar 12 01:00 master,
> drwxr-xr-x 3 root root 4096 Mar 12 01:00 segment,
> ```
>
>
> Diffs
> -
>
>
> ambari-server/src/main/resources/common-services/HAWQ/2.0.0/configuration/hawq-site.xml
> ec7275f
>
> ambari-server/src/main/resources/common-services/HAWQ/2.0.0/themes/theme.json
> a23afe6
> ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py
> 741011c
> ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py
> d415b6f
> ambari-web/app/messages.js ea06bf3
> ambari-web/app/mixins/common/serverValidator.js 99a5921
> ambari-web/app/models/configs/objects/service_config_property.js a437c98
> ambari-web/app/styles/application.less a6b79ce
>
> ambari-web/app/templates/common/modal_popups/config_recommendation_popup.hbs
> f58e086
> ambari-web/app/views/common/configs/service_config_layout_tab_view.js
> 694e883
>
> Diff: https://reviews.apache.org/r/45566/diff/
>
>
> Testing
> ---
>
> Manually tested the UI.
>
> Unit test added for stack advisor.
>
>
> Thanks,
>
> Lav Jain
>
>