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


Ship it!




Ship It!

- Oliver Szabo


On May 12, 2016, 11:06 a.m., Dharmesh Makwana wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47300/
> -----------------------------------------------------------
> 
> (Updated May 12, 2016, 11:06 a.m.)
> 
> 
> Review request for Ambari, Don Bosco Durai, Jaimin Jetly, Oliver Szabo, and 
> Sumit Mohanty.
> 
> 
> Bugs: AMBARI-16632
>     https://issues.apache.org/jira/browse/AMBARI-16632
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Patch contains
> 1) Added few steps more in "Take a tour" option.
> 2) When url contains "host_name" and "component_name" params than separate 
> tab should be open for that component.
> 3) Comparison tab clean-up, default pageSize changed to 25 and css changes.
> 4) In histogram graph, time gap is shown above the graph.
> 
> 
> Diffs
> -----
> 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/utils/Intro.js
>  6abccf6 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/utils/Utils.js
>  5d55689 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/utils/ViewUtils.js
>  ba21b65 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/common/DatePickerLayout.js
>  0849820 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/dashboard/MainLayoutView.js
>  09bbacc 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/graphs/GraphLayoutView.js
>  805b507 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/tabs/ComparisonView.js
>  8ad0c4b 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/tabs/HierarchyTabLayoutView.js
>  cb4228c 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/tabs/LogFileView.js
>  2d19d78 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/troubleshoot/TroubleShootLayoutView.js
>  e1bc90a 
>   ambari-logsearch/ambari-logsearch-portal/src/main/webapp/styles/style.css 
> 05fd491 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/templates/audit/AuditTabLayoutView_tmpl.html
>  2f0a596 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/templates/graphs/GraphLayoutView_tmpl.html
>  fb4ee3c 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/templates/tabs/HierarchyTabLayoutView_tmpl.html
>  4564562 
>   
> ambari-logsearch/ambari-logsearch-portal/src/main/webapp/templates/troubleshoot/TroubleShootLayoutView_tmpl.html
>  b3eeaf5 
> 
> Diff: https://reviews.apache.org/r/47300/diff/
> 
> 
> Testing
> -------
> 
> Tested in local Ambari environment.
> 
> 
> Thanks,
> 
> Dharmesh Makwana
> 
>

Reply via email to